diff --git a/anitya/db/migrations/versions/ebc827e80373_remove_unique_attribute_from_username.py b/anitya/db/migrations/versions/ebc827e80373_remove_unique_attribute_from_username.py index 44876c14..ae78b777 100644 --- a/anitya/db/migrations/versions/ebc827e80373_remove_unique_attribute_from_username.py +++ b/anitya/db/migrations/versions/ebc827e80373_remove_unique_attribute_from_username.py @@ -7,19 +7,20 @@ from alembic import op - # revision identifiers, used by Alembic. -revision = 'ebc827e80373' -down_revision = '8ba7d4c42044' +revision = "ebc827e80373" +down_revision = "8ba7d4c42044" def upgrade(): + """Alembic migration.""" # ### commands auto generated by Alembic - please adjust! ### - op.drop_constraint('ix_users_username', 'users', type_='unique') + op.drop_constraint("ix_users_username", "users", type_="unique") # ### end Alembic commands ### def downgrade(): + """Downgrade migration.""" # ### commands auto generated by Alembic - please adjust! ### - op.create_unique_constraint('ix_users_username', 'users', ['username']) + op.create_unique_constraint("ix_users_username", "users", ["username"]) # ### end Alembic commands ### diff --git a/anitya/tests/db/test_models.py b/anitya/tests/db/test_models.py index 0b9849de..ad7728a1 100644 --- a/anitya/tests/db/test_models.py +++ b/anitya/tests/db/test_models.py @@ -1128,6 +1128,21 @@ def test_is_admin_configured(self): self.assertTrue(user.is_admin) self.assertTrue(user.admin) + def test_is_admin_configured_email(self): + """Assert default value for admin flag.""" + user = models.User(email="user@fedoraproject.org", username="user", admin=False) + self.session.add(user) + self.session.commit() + + mock_dict = mock.patch.dict( + "anitya.config.config", {"ANITYA_WEB_ADMINS": [six.text_type(user.email)]} + ) + + with mock_dict: + self.assertFalse(user.admin) + self.assertTrue(user.is_admin) + self.assertTrue(user.admin) + def test_to_dict(self): """Assert the correct dictionary is returned.""" user = models.User(email="user@fedoraproject.org", username="user")