def test_suppress_revocation_for_zero_dollar_refund(self):
"""
Verify that the function does not require use of fulfillment modules to mark lines in a refund
corresponding to a total credit of $0 as complete.
"""
refund = RefundFactory(status=REFUND.PAYMENT_REFUNDED)
refund.total_credit_excl_tax = 0
refund.save()
self.assertTrue(revoke_fulfillment_for_refund(refund))
self.assertEqual(refund.status, REFUND.PAYMENT_REFUNDED)
self.assertEqual(set([line.status for line in refund.lines.all()]), {REFUND_LINE.COMPLETE})
def test_suppress_revocation_for_zero_dollar_refund(self):
"""
Verify that the function does not require use of fulfillment modules to mark lines in a refund
corresponding to a total credit of $0 as complete.
"""
refund = RefundFactory(status=REFUND.PAYMENT_REFUNDED)
refund.total_credit_excl_tax = 0
refund.save()
self.assertTrue(revoke_fulfillment_for_refund(refund))
self.assertEqual(refund.status, REFUND.PAYMENT_REFUNDED)
self.assertEqual(set([line.status for line in refund.lines.all()]),
{REFUND_LINE.COMPLETE})
class RefundProcessViewTests(ThrottlingMixin, TestCase):
def setUp(self):
super(RefundProcessViewTests, self).setUp()
self.user = self.create_user(is_staff=True)
self.client.login(username=self.user.username, password=self.password)
self.refund = RefundFactory(user=self.user)
def put(self, action):
data = '{{"action": "{}"}}'.format(action)
path = reverse('api:v2:refunds:process', kwargs={'pk': self.refund.id})
return self.client.put(path, data, JSON_CONTENT_TYPE)
def test_staff_only(self):
""" The view should only be accessible to staff users. """
user = self.create_user(is_staff=False)
self.client.login(username=user.username, password=self.password)
response = self.put('approve')
self.assertEqual(response.status_code, 403)
def test_invalid_action(self):
""" If the action is neither approve nor deny, the view should return HTTP 400. """
response = self.put('reject')
self.assertEqual(response.status_code, 400)
@ddt.data('approve', 'deny')
def test_success(self, action):
""" If the action succeeds, the view should return HTTP 200 and the serialized Refund. """
with mock.patch('ecommerce.extensions.refund.models.Refund.{}'.format(action), mock.Mock(return_value=True)):
response = self.put(action)
self.assertEqual(response.status_code, 200)
self.assertEqual(response.data, RefundSerializer(self.refund).data)
@mock.patch('ecommerce.extensions.refund.models.Refund._revoke_lines')
@mock.patch('ecommerce.extensions.refund.models.Refund._issue_credit')
def test_success_approve_payment_only(self, mock_issue_credit, mock_revoke_lines):
""" Verify the endpoint supports approving the refund, and issuing credit without revoking fulfillment. """
mock_issue_credit.return_value = None
with mock.patch('ecommerce.extensions.refund.models.logger') as patched_log:
response = self.put('approve_payment_only')
self.assertEqual(response.status_code, 200)
self.refund.refresh_from_db()
self.assertEqual(response.data['status'], self.refund.status)
self.assertEqual(response.data['status'], 'Complete')
patched_log.info.assert_called_with('Skipping the revocation step for refund [%d].', self.refund.id)
self.assertFalse(mock_revoke_lines.called)
@ddt.data(
('approve', 'approve'),
('approve', 'approve_payment_only'),
('deny', 'deny')
)
@ddt.unpack
def test_failure(self, action, decision):
""" If the action fails, the view should return HTTP 500 and the serialized Refund. """
with mock.patch('ecommerce.extensions.refund.models.Refund.{}'.format(action), mock.Mock(return_value=False)):
response = self.put(decision)
self.assertEqual(response.status_code, 500)
self.assertEqual(response.data, RefundSerializer(self.refund).data)
@ddt.data(
('approve', REFUND.COMPLETE),
('approve_payment_only', REFUND.COMPLETE),
('deny', REFUND.DENIED)
)
@ddt.unpack
def test_subsequent_approval(self, action, _status):
""" Verify the endpoint supports reprocessing a previously-processed refund. """
self.refund.status = _status
self.refund.save()
response = self.put(action)
self.assertEqual(response.status_code, 200)