darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-02-06 03:13:42 -05:00
Code Issues Packages Projects Releases Wiki Activity

Make users.name unique.

Browse Source
This commit is contained in:
Rebecca Law
2015-11-26 11:53:49 +00:00
parent 7669437bf5
commit 9e27ef964a
4 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/models.py
View File

@@ -16,7 +16,7 @@ class Users(db.Model):
__tablename__ = 'users'
id = db.Column(db.Integer, primary_key=True)
name = db.Column(db.String, nullable=False, index=True)
name = db.Column(db.String, nullable=False, index=True, unique=True)
email_address = db.Column(db.String(255), nullable=False, index=True)
password = db.Column(db.String, index=False, unique=False, nullable=False)
mobile_number = db.Column(db.String, index=False, unique=False, nullable=False)

2
migrations/versions/create_users.py → migrations/versions/10_create_users.py
View File

@@ -21,7 +21,7 @@ def upgrade():
op.create_table('users',
sa.Column('id', sa.Integer, primary_key=True),
sa.Column('name', sa.String, nullable=False),
sa.Column('name', sa.String, nullable=False, unique=True),
sa.Column('email_address', sa.String(length=255), nullable=False),
sa.Column('password', sa.String, nullable=False),
sa.Column('mobile_number', sa.String, nullable=False),

2
tests/app/main/dao/test_get_all_users.py
View File

@@ -17,7 +17,7 @@ def test_get_all_users_returns_all_users(notifications_admin, notifications_admi
mobile_number='+441234123412',
created_at=datetime.now(),
role_id=1)
user3 = Users(name='test two',
user3 = Users(name='test three',
password='some2ndpassword',
email_address='test2@get_all.gov.uk',
mobile_number='+441234123412',

2
tests/app/main/dao/test_users_dao.py
View File

@@ -21,7 +21,7 @@ def test_insert_user_should_add_user(notifications_admin, notifications_admin_db
def test_insert_user_with_role_that_does_not_exist_fails(notifications_admin, notifications_admin_db):
user = Users(name='test insert',
user = Users(name='role does not exist',
password='somepassword',
email_address='test@insert.gov.uk',
mobile_number='+441234123412',