about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFrederick Muriuki Muriithi2022-11-14 04:30:52 +0300
committerFrederick Muriuki Muriithi2022-11-14 04:30:52 +0300
commit0454f5372bf70249d9a6380c6059818576beec34 (patch)
treeaa6e500f642c0af2ffb600117340352f574ab326
parent2a29b82504c6381f4a6121df9e2bfbcab4a6fb7e (diff)
downloadgenenetwork3-0454f5372bf70249d9a6380c6059818576beec34.tar.gz
Migrations: Add generic_role_privileges table
Add table `generic_role_privileges` table to link the generic roles to the
privileges they provide.

* migrations/auth/20221114_01_n8gsF-create-generic-role-privileges-table.py:
  new migration
* tests/unit/auth/test_create_table_migrations.py: test new migration
* tests/unit/auth/test_migrations_indexes.py: test new migration
-rw-r--r--migrations/auth/20221114_01_n8gsF-create-generic-role-privileges-table.py33
-rw-r--r--tests/unit/auth/test_create_table_migrations.py4
-rw-r--r--tests/unit/auth/test_migrations_indexes.py5
3 files changed, 40 insertions, 2 deletions
diff --git a/migrations/auth/20221114_01_n8gsF-create-generic-role-privileges-table.py b/migrations/auth/20221114_01_n8gsF-create-generic-role-privileges-table.py
new file mode 100644
index 0000000..7102a57
--- /dev/null
+++ b/migrations/auth/20221114_01_n8gsF-create-generic-role-privileges-table.py
@@ -0,0 +1,33 @@
+"""
+Create 'generic_role_privileges' table
+
+This table links the generic_roles to the privileges they provide
+"""
+
+from yoyo import step
+
+__depends__ = {'20221113_01_7M0hv-enumerate-initial-privileges'}
+
+steps = [
+    step(
+        """
+        CREATE TABLE IF NOT EXISTS generic_role_privileges(
+            generic_role_id TEXT NOT NULL,
+            privilege_id TEXT NOT NULL,
+            PRIMARY KEY(generic_role_id, privilege_id),
+            FOREIGN KEY(generic_role_id) REFERENCES generic_roles(role_id),
+            FOREIGN KEY(privilege_id) REFERENCES privileges(privilege_id)
+        ) WITHOUT ROWID
+        """,
+        "DROP TABLE IF EXISTS generic_role_privileges"),
+    step(
+        """
+        CREATE INDEX IF NOT EXISTS
+            idx_tbl_generic_role_privileges_cols_generic_role_id
+        ON generic_role_privileges(generic_role_id)
+        """,
+        """
+        DROP INDEX IF EXISTS
+            idx_tbl_generic_role_privileges_cols_generic_role_id
+        """)
+]
diff --git a/tests/unit/auth/test_create_table_migrations.py b/tests/unit/auth/test_create_table_migrations.py
index fbb9c0f..ce02f62 100644
--- a/tests/unit/auth/test_create_table_migrations.py
+++ b/tests/unit/auth/test_create_table_migrations.py
@@ -21,7 +21,9 @@ migrations_and_tables = (
     ("20221110_04_6PRFQ-create-genotype-resources-table.py", "genotype_resources"),
     ("20221110_05_BaNtL-create-roles-table.py", "roles"),
     ("20221110_06_Pq2kT-create-generic-roles-table.py", "generic_roles"),
-    ("20221110_07_7WGa1-create-role-privileges-table.py", "role_privileges"))
+    ("20221110_07_7WGa1-create-role-privileges-table.py", "role_privileges"),
+    ("20221114_01_n8gsF-create-generic-role-privileges-table.py",
+     "generic_role_privileges"))
 
 @pytest.mark.unit_test
 @pytest.mark.parametrize("migration_file,the_table", migrations_and_tables)
diff --git a/tests/unit/auth/test_migrations_indexes.py b/tests/unit/auth/test_migrations_indexes.py
index eed0272..87a624f 100644
--- a/tests/unit/auth/test_migrations_indexes.py
+++ b/tests/unit/auth/test_migrations_indexes.py
@@ -18,7 +18,10 @@ migrations_tables_and_indexes = (
     ("20221110_05_BaNtL-create-roles-table.py", "roles",
      "idx_tbl_roles_cols_group_id"),
     ("20221110_07_7WGa1-create-role-privileges-table.py", "role_privileges",
-     "idx_tbl_role_privileges_cols_role_id"))
+     "idx_tbl_role_privileges_cols_role_id"),
+    ("20221114_01_n8gsF-create-generic-role-privileges-table.py",
+     "generic_role_privileges",
+     "idx_tbl_generic_role_privileges_cols_generic_role_id"))
 
 @pytest.mark.unit_test
 @pytest.mark.parametrize(