Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

updated endpoint for verifications to include receipt id (#25999)

Browse Source
This commit is contained in:
alangsto
2021-01-07 08:34:03 -05:00
committed by GitHub
parent c6497d8879
commit 40cf6ba413
2 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
openedx/core/djangoapps/user_api/serializers.py
View File

@@ -107,6 +107,7 @@ class IDVerificationDetailsSerializer(serializers.Serializer):
expiration_datetime = serializers.DateTimeField()
message = serializers.SerializerMethodField()
updated_at = serializers.DateTimeField()
receipt_id = serializers.SerializerMethodField()
def get_type(self, obj):
if isinstance(obj, SoftwareSecurePhotoVerification):
@@ -123,3 +124,9 @@ class IDVerificationDetailsSerializer(serializers.Serializer):
return obj.reason
else:
return ''
def get_receipt_id(self, obj):
if isinstance(obj, SoftwareSecurePhotoVerification):
return obj.receipt_id
else:
return None

9
openedx/core/djangoapps/user_api/verification_api/tests/test_views.py
View File

@@ -132,7 +132,8 @@ class VerificationsDetailsViewTests(VerificationStatusViewTestsMixin, TestCase):
'status': self.photo_verification.status,
'expiration_datetime': '{}Z'.format(kwargs.get('expected_expires').isoformat()),
'message': '',
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat())
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat()),
'receipt_id': self.photo_verification.receipt_id,
}]
def test_multiple_verification_types(self):
@@ -157,6 +158,7 @@ class VerificationsDetailsViewTests(VerificationStatusViewTestsMixin, TestCase):
'expiration_datetime': '{}Z'.format(expected_expires.isoformat()),
'message': self.photo_verification.error_msg,
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat()),
'receipt_id': self.photo_verification.receipt_id
},
{
'type': 'SSO',
@@ -164,6 +166,7 @@ class VerificationsDetailsViewTests(VerificationStatusViewTestsMixin, TestCase):
'expiration_datetime': '{}Z'.format(expected_expires.isoformat()),
'message': '',
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat()),
'receipt_id': None,
},
{
'type': 'Manual',
@@ -171,6 +174,7 @@ class VerificationsDetailsViewTests(VerificationStatusViewTestsMixin, TestCase):
'expiration_datetime': '{}Z'.format(expected_expires.isoformat()),
'message': self.manual_verification.reason,
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat()),
'receipt_id': None,
},
]
self.assertEqual(json.loads(response.content.decode('utf-8')), expected)
@@ -194,6 +198,7 @@ class VerificationsDetailsViewTests(VerificationStatusViewTestsMixin, TestCase):
'expiration_datetime': '{}Z'.format(expected_expires.isoformat()),
'message': self.photo_verification.error_msg,
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat()),
'receipt_id': self.photo_verification.receipt_id,
},
{
'type': 'Software Secure',
@@ -201,6 +206,7 @@ class VerificationsDetailsViewTests(VerificationStatusViewTestsMixin, TestCase):
'expiration_datetime': '{}Z'.format(expected_expires.isoformat()),
'message': second_ss_photo_verification.error_msg,
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat()),
'receipt_id': second_ss_photo_verification.receipt_id,
},
{
'type': 'SSO',
@@ -208,6 +214,7 @@ class VerificationsDetailsViewTests(VerificationStatusViewTestsMixin, TestCase):
'expiration_datetime': '{}Z'.format(expected_expires.isoformat()),
'message': '',
'updated_at': '{}Z'.format(self.CREATED_AT.isoformat()),
'receipt_id': None,
},
]
self.assertEqual(json.loads(response.content.decode('utf-8')), expected)