diff --git a/openedx/core/djangoapps/credit/api/provider.py b/openedx/core/djangoapps/credit/api/provider.py index 4260149cca..8e2bf81d21 100644 --- a/openedx/core/djangoapps/credit/api/provider.py +++ b/openedx/core/djangoapps/credit/api/provider.py @@ -274,11 +274,7 @@ def create_credit_request(course_key, provider_id, username): "user_username": user.username, "user_email": user.email, "user_full_name": user.profile.name, - "user_mailing_address": ( - user.profile.mailing_address - if user.profile.mailing_address is not None - else "" - ), + "user_mailing_address": "", "user_country": ( user.profile.country.code if user.profile.country.code is not None diff --git a/openedx/core/djangoapps/credit/tests/test_api.py b/openedx/core/djangoapps/credit/tests/test_api.py index 64c7d8136f..11ea560876 100644 --- a/openedx/core/djangoapps/credit/tests/test_api.py +++ b/openedx/core/djangoapps/credit/tests/test_api.py @@ -639,7 +639,8 @@ class CreditProviderIntegrationApiTests(CreditApiTestBase): for key in self.USER_INFO.keys(): param_key = 'user_{key}'.format(key=key) self.assertIn(param_key, parameters) - self.assertEqual(parameters[param_key], self.USER_INFO[key]) + expected = '' if key == 'mailing_address' else self.USER_INFO[key] + self.assertEqual(parameters[param_key], expected) def test_create_credit_request_grade_length(self): """ Verify the length of the final grade is limited to seven (7) characters total. @@ -656,6 +657,11 @@ class CreditProviderIntegrationApiTests(CreditApiTestBase): request = api.create_credit_request(self.course_key, self.PROVIDER_ID, self.USER_INFO['username']) self.assertEqual(request['parameters']['final_grade'], u'0.33333') + def test_create_credit_request_address_empty(self): + """ Verify the mailing address is always empty. """ + request = api.create_credit_request(self.course_key, self.PROVIDER_ID, self.user.username) + self.assertEqual(request['parameters']['user_mailing_address'], '') + def test_credit_request_disable_integration(self): CreditProvider.objects.all().update(enable_integration=False)