Skip to content

Commit 86032c1

Browse files
committed
test: Remove tests that rely on edx-name-affirmation service being installed
The service is specific to 2U and should not be installed by default. When we try to patch objects from that library, that can only be done if the original object is importable. So those tests don't make sense to have in the base system which you should be able to run without the edx-name-affirmation library.
1 parent c2df150 commit 86032c1

2 files changed

Lines changed: 3 additions & 15 deletions

File tree

openedx/core/djangoapps/user_api/accounts/tests/test_api.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,8 @@ def test_validate_name_change_same_name(self):
395395
updated_meta = user_profile.get_meta()
396396
self.assertEqual(meta, updated_meta)
397397

398-
@patch('edx_name_affirmation.name_change_validator.NameChangeValidator.validate', Mock(return_value=False))
398+
@patch('openedx.core.djangoapps.user_api.accounts.api._does_name_change_require_verification',
399+
Mock(return_value=True))
399400
@patch('openedx.core.djangoapps.user_api.accounts.api.get_certificates_for_user',
400401
Mock(return_value=[{'status': CertificateStatuses.downloadable}]))
401402
@patch('openedx.core.djangoapps.user_api.accounts.api.get_verified_enrollments',
@@ -414,8 +415,6 @@ def test_name_update_requires_idv(self):
414415
account_settings = get_account_settings(self.default_request)[0]
415416
assert account_settings['name'] != 'New Name'
416417

417-
@patch('edx_name_affirmation.name_change_validator.NameChangeValidator', Mock())
418-
@patch('edx_name_affirmation.name_change_validator.NameChangeValidator.validate', Mock(return_value=True))
419418
@ddt.data(
420419
(True, False),
421420
(False, True),

openedx/features/name_affirmation_api/tests/test_utils.py

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
import ddt
77
from edx_django_utils.plugins import PluginError
88

9-
from openedx.features.name_affirmation_api.utils import is_name_affirmation_installed, get_name_affirmation_service
9+
from openedx.features.name_affirmation_api.utils import is_name_affirmation_installed
1010

1111

1212
@ddt.ddt
@@ -23,14 +23,3 @@ def test_name_affirmation_not_installed(self, mock_manager):
2323
mock_manager.side_effect = PluginError('No such plugin')
2424
with self.assertRaises(PluginError):
2525
self.assertFalse(is_name_affirmation_installed())
26-
27-
@patch('edx_name_affirmation.services.NameAffirmationService')
28-
@ddt.data(True, False)
29-
def test_get_name_affirmation_service(self, name_affirmation_installed, mock_service):
30-
with patch('openedx.features.name_affirmation_api.utils.is_name_affirmation_installed',
31-
return_value=name_affirmation_installed):
32-
name_affirmation_service = get_name_affirmation_service()
33-
if name_affirmation_installed:
34-
self.assertEqual(name_affirmation_service, mock_service())
35-
else:
36-
self.assertIsNone(name_affirmation_service)

0 commit comments

Comments
 (0)