diff options
author | Ben Sturmfels <ben@sturm.com.au> | 2021-03-05 23:12:19 +1100 |
---|---|---|
committer | Ben Sturmfels <ben@sturm.com.au> | 2021-03-05 23:12:19 +1100 |
commit | dec47c7102cf0aa3a4debf002928db8e460c0d71 (patch) | |
tree | 47631fc15c7af172aa699506adf3d76d3a71976c /mediagoblin/tests/test_edit.py | |
parent | 5f3a782fef4855e10b7259624a14d8afb0f7be93 (diff) | |
download | mediagoblin-dec47c7102cf0aa3a4debf002928db8e460c0d71.tar.lz mediagoblin-dec47c7102cf0aa3a4debf002928db8e460c0d71.tar.xz mediagoblin-dec47c7102cf0aa3a4debf002928db8e460c0d71.zip |
Apply `pyupgrade --py3-plus` to remove Python 2 compatibility code.
Diffstat (limited to 'mediagoblin/tests/test_edit.py')
-rw-r--r-- | mediagoblin/tests/test_edit.py | 50 |
1 files changed, 25 insertions, 25 deletions
diff --git a/mediagoblin/tests/test_edit.py b/mediagoblin/tests/test_edit.py index 5c109be6..c2cee048 100644 --- a/mediagoblin/tests/test_edit.py +++ b/mediagoblin/tests/test_edit.py @@ -25,12 +25,12 @@ from mediagoblin import auth from mediagoblin.tools import template, mail -class TestUserEdit(object): +class TestUserEdit: def setup(self): # set up new user - self.user_password = u'toast' + self.user_password = 'toast' self.user = fixture_add_user(password = self.user_password, - privileges=[u'active']) + privileges=['active']) def login(self, test_app): test_app.post( @@ -44,19 +44,19 @@ class TestUserEdit(object): self.login(test_app) # Make sure user exists - assert LocalUser.query.filter(LocalUser.username==u'chris').first() + assert LocalUser.query.filter(LocalUser.username=='chris').first() res = test_app.post('/edit/account/delete/', {'confirmed': 'y'}) # Make sure user has been deleted - assert LocalUser.query.filter(LocalUser.username==u'chris').first() == None + assert LocalUser.query.filter(LocalUser.username=='chris').first() == None #TODO: make sure all corresponding items comments etc have been # deleted too. Perhaps in submission test? #Restore user at end of test self.user = fixture_add_user(password = self.user_password, - privileges=[u'active']) + privileges=['active']) self.login(test_app) @@ -67,8 +67,8 @@ class TestUserEdit(object): # Test if legacy profile editing URL redirects correctly res = test_app.post( '/edit/profile/', { - 'bio': u'I love toast!', - 'url': u'http://dustycloud.org/'}, expect_errors=True) + 'bio': 'I love toast!', + 'url': 'http://dustycloud.org/'}, expect_errors=True) # Should redirect to /u/chris/edit/ assert res.status_int == 302 @@ -76,20 +76,20 @@ class TestUserEdit(object): res = test_app.post( '/u/chris/edit/', { - 'bio': u'I love toast!', - 'url': u'http://dustycloud.org/'}) + 'bio': 'I love toast!', + 'url': 'http://dustycloud.org/'}) - test_user = LocalUser.query.filter(LocalUser.username==u'chris').first() - assert test_user.bio == u'I love toast!' - assert test_user.url == u'http://dustycloud.org/' + test_user = LocalUser.query.filter(LocalUser.username=='chris').first() + assert test_user.bio == 'I love toast!' + assert test_user.url == 'http://dustycloud.org/' # change a different user than the logged in (should fail with 403) - fixture_add_user(username=u"foo", - privileges=[u'active']) + fixture_add_user(username="foo", + privileges=['active']) res = test_app.post( '/u/foo/edit/', { - 'bio': u'I love toast!', - 'url': u'http://dustycloud.org/'}, expect_errors=True) + 'bio': 'I love toast!', + 'url': 'http://dustycloud.org/'}, expect_errors=True) assert res.status_int == 403 # test changing the bio and the URL inproperly @@ -107,9 +107,9 @@ class TestUserEdit(object): form = context['form'] assert form.bio.errors == [ - u'Field must be between 0 and 500 characters long.'] + 'Field must be between 0 and 500 characters long.'] assert form.url.errors == [ - u'This address contains errors'] + 'This address contains errors'] def test_email_change(self, test_app): self.login(test_app) @@ -125,7 +125,7 @@ class TestUserEdit(object): context = template.TEMPLATE_TEST_CONTEXT[ 'mediagoblin/edit/change_email.html'] assert context['form'].new_email.errors == [ - u'Sorry, a user with that email address already exists.'] + 'Sorry, a user with that email address already exists.'] # Test successful email change template.clear_test_template_context() @@ -147,7 +147,7 @@ class TestUserEdit(object): assert email_context['verification_url'].encode('ascii') in message.get_payload(decode=True) path = urlparse.urlsplit(email_context['verification_url'])[2] - assert path == u'/edit/verify_email/' + assert path == '/edit/verify_email/' ## Try verifying with bs verification key, shouldn't work template.clear_test_template_context() @@ -169,7 +169,7 @@ class TestUserEdit(object): # Verify email activation works template.clear_test_template_context() get_params = urlparse.urlsplit(email_context['verification_url'])[3] - res = test_app.get('%s?%s' % (path, get_params)) + res = test_app.get('{}?{}'.format(path, get_params)) res.follow() # New email saved? @@ -181,10 +181,10 @@ class TestMetaDataEdit: @pytest.fixture(autouse=True) def setup(self, test_app): # set up new user - self.user_password = u'toast' + self.user_password = 'toast' self.user = fixture_add_user( password = self.user_password, - privileges=[u'active',u'admin'] + privileges=['active','admin'] ) self.test_app = test_app @@ -209,7 +209,7 @@ class TestMetaDataEdit: @pytest.mark.skipif(six.PY2, reason='Breaks in Python 2 but seems non-critical') def test_edit_metadata(self, test_app): media_entry = fixture_media_entry(uploader=self.user.id, - state=u'processed') + state='processed') media_slug = "/u/{username}/m/{media_id}/metadata/".format( username = str(self.user.username), media_id = str(media_entry.id)) |