Merge pull request #22088 from edx/BOM-958

Updated the social link creation to return social links sorted - BOM-958
This commit is contained in:
Aarif
2019-10-21 16:40:51 +05:00
committed by GitHub
2 changed files with 8 additions and 7 deletions

View File

@@ -4,6 +4,7 @@ Mixins for fields.
"""
from __future__ import absolute_import
import six
from bok_choy.promise import EmptyPromise
from common.test.acceptance.tests.helpers import get_selected_option_text, select_option_by_text
@@ -69,9 +70,9 @@ class FieldsMixin(object):
Return the title of a field.
"""
self.wait_for_field(field_id)
query = self.q(css=u'.u-field-{} .u-field-title'.format(field_id))
return query.text[0] if query.present else None
# pylint: disable=unicode-format-string
query = self.q(css=six.u('.u-field-{} .u-field-title').format(field_id))
return sorted(query.text)[0] if query.present else None
def message_for_field(self, field_id):
"""

View File

@@ -407,11 +407,11 @@ class AccountSettingsPageTest(AccountSettingsTestMixin, AcceptanceTest):
Test behaviour of one of the social media links field.
"""
self._test_text_field(
u'social_links',
u'Twitter Link',
six.u('social_links'),
six.u('Facebook Link'),
self.social_link,
u'www.google.com/invalidlink',
[u'https://www.twitter.com/edX', self.social_link],
six.u('www.google.com/invalidlink)'),
[six.u('https://www.twitter.com/edX'), self.social_link],
)
def test_linked_accounts(self):