diff --git a/users/serializers.py b/users/serializers.py index 3b557749d..2e1f0339e 100644 --- a/users/serializers.py +++ b/users/serializers.py @@ -41,11 +41,8 @@ USERNAME_ALREADY_EXISTS_MSG = ( "A user already exists with this username. Please try a different one." ) -EMAIL_CONFLICT_MSG = ( - "This email is already associated with an existing account") -RETIRED_EMAIL_MSG = ( - "This email is associated to a retired account." -) +EMAIL_CONFLICT_MSG = "This email is already associated with an existing account" +RETIRED_EMAIL_MSG = "This email is associated to a retired account." OPENEDX_VALIDATION_MSGS_MAP = { "It looks like this username is already taken": USERNAME_ALREADY_EXISTS_MSG, @@ -245,9 +242,7 @@ def validate(self, data): {"username": "This field is required."} ) if not data.get("email"): - raise serializers.ValidationError( - {"email": "This field is required."} - ) + raise serializers.ValidationError({"email": "This field is required."}) username = data.get("username") email = data.get("email") diff --git a/users/serializers_test.py b/users/serializers_test.py index b9a8c28b3..37cd8b069 100644 --- a/users/serializers_test.py +++ b/users/serializers_test.py @@ -267,7 +267,9 @@ def test_username_validation_connection_exception( UserSerializer should raise a RequestsConnectionError or HTTPError if the connection to OpenEdx fails. The serializer should raise a validation error. """ - mocker.patch("openedx.api.validate_username_email_with_edx", side_effect=exception_raised) + mocker.patch( + "openedx.api.validate_username_email_with_edx", side_effect=exception_raised + ) serializer = UserSerializer( data={