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

fix: pipeline_user_details serializer error on mfe context response (#32146)

Browse Source 
* fix: pipeline_user_details serializer error on mfe context response

* fix: failed tests for mfe context API

---------

Co-authored-by: Syed Sajjad  Hussain Shah <syed.sajjad@H7FKF7K6XD.local>
Co-authored-by: Shahbaz Shabbir <shbzshbr@gmail.com>
This commit is contained in:
Syed Sajjad Hussain Shah
2023-04-27 18:56:58 +05:00
committed by GitHub
parent b1f8b9339a
commit 6af95b46e7
2 changed files with 16 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
openedx/core/djangoapps/user_authn/api/tests/test_views.py
View File

@@ -42,8 +42,7 @@ class MFEContextViewTest(ThirdPartyAuthTestMixin, APITestCase):
hostname = socket.gethostname()
ip_address = socket.gethostbyname(hostname)
self.country_code = country_code_from_ip(ip_address)
self.pipeline_user_details = {'username': None, 'email': None, 'name': None,
'firstName': None, 'lastName': None}
self.pipeline_user_details = {}
# Several third party auth providers are created for these tests:
self.configure_google_provider(enabled=True, visible=True)
@@ -97,7 +96,15 @@ class MFEContextViewTest(ThirdPartyAuthTestMixin, APITestCase):
"""
if add_user_details:
self.pipeline_user_details.update({'email': 'test@test.com'})
self.pipeline_user_details.update(
{
'username': None,
'email': 'test@test.com',
'name': None,
'firstName': None,
'lastName': None
}
)
return {
'contextData': {

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

@@ -53,7 +53,12 @@ class ContextDataSerializer(serializers.Serializer):
autoSubmitRegForm = serializers.BooleanField(default=False)
syncLearnerProfileData = serializers.BooleanField(default=False)
countryCode = serializers.CharField(allow_null=True)
pipelineUserDetails = PipelineUserDetailsSerializer(source='pipeline_user_details', allow_null=True)
pipelineUserDetails = serializers.SerializerMethodField()
def get_pipelineUserDetails(self, obj):
if obj.get('pipeline_user_details'):
return PipelineUserDetailsSerializer(obj.get('pipeline_user_details')).data
return {}
class MFEContextSerializer(serializers.Serializer):