Update documentation and remove users creation

Author: 99Lys

.github/scripts/create_dbt_test_users.sh

@@ -13,12 +13,7 @@ DREMIO_SOFTWARE_USERNAME=${DREMIO_SOFTWARE_USERNAME}
 DREMIO_SOFTWARE_PASSWORD=${DREMIO_SOFTWARE_PASSWORD}
 DREMIO_DATALAKE=dbt_test_source
 DREMIO_DATABASE=dbt_test
-DBT_TEST_USER_1=dbt_test_user_1
-DBT_TEST_USER_2=dbt_test_user_2
-DBT_TEST_USER_3=dbt_test_user_3
-DBT_TEST_ROLE_1=dbt_test_role_1
-DBT_TEST_ROLE_2=dbt_test_role_2
 DREMIO_EDITION=community
 EOF
 
-echo ".env file created successfully."
+echo ".env file created successfully."

.github/scripts/create_env_file.sh

@@ -93,9 +93,6 @@ jobs:
           pip install -r dev_requirements.txt
           pip install .
 
-      - name: Create dbt test users
-        run: bash .github/scripts/create_dbt_test_users.sh
-
       - name: Create dbt projects
         run: bash .github/scripts/create_dbt_projects.sh
 

.github/workflows/ci.yml

@@ -17,6 +17,8 @@ To run our tests, a test environment must be set up.
     DBT_TEST_USER_1=dbt_test_user_1
     DBT_TEST_USER_2=dbt_test_user_2
     DBT_TEST_USER_3=dbt_test_user_3
+    DBT_TEST_ROLE_1=dbt_test_role_1
+    DBT_TEST_ROLE_2=dbt_test_role_2
     ```
    For cloud version of Dremio:
     ```
@@ -29,6 +31,8 @@ To run our tests, a test environment must be set up.
     DBT_TEST_USER_1=dbt_test_user_1
     DBT_TEST_USER_2=dbt_test_user_2
     DBT_TEST_USER_3=dbt_test_user_3
+    DBT_TEST_ROLE_1=dbt_test_role_1
+    DBT_TEST_ROLE_2=dbt_test_role_2
     ```
 1. Create the three users listed above (dbt_test_user_1, dbt_test_user_2, dbt_test_user_3) in the Dremio instance.
 1. Create a bucket in Object storage with a name `dbtdremios3`

tests/README.md

@@ -127,13 +127,13 @@ def check_hooks(self, state, project, host, count=1):
 
 
 class TestPrePostModelHooksDremio(BaseTestPrePost):
-    # @pytest.fixture(scope="class")
-    # def unique_schema(self, request, prefix) -> str:
-    #     test_file = request.module.__name__
-    #     # We only want the last part of the name
-    #     test_file = test_file.split(".")[-1]
-    #     unique_schema = f"{BUCKET}.{prefix}_{test_file}"
-    #     return unique_schema
+    @pytest.fixture(scope="class")
+    def unique_schema(self, request, prefix) -> str:
+        test_file = request.module.__name__
+        # We only want the last part of the name
+        test_file = test_file.split(".")[-1]
+        unique_schema = f"{BUCKET}.{prefix}_{test_file}"
+        return unique_schema
 
     @pytest.fixture(scope="class")
     def project_config_update(self):
@@ -308,13 +308,13 @@ def models(self):
 class TestPrePostModelHooksOnSeedsPlusPrefixedDremio(
     TestPrePostModelHooksOnSeedsDremio
 ):
-    # @pytest.fixture(scope="class")
-    # def unique_schema(self, request, prefix) -> str:
-    #     test_file = request.module.__name__
-    #     # We only want the last part of the name
-    #     test_file = test_file.split(".")[-1]
-    #     unique_schema = f"{BUCKET}.{prefix}_{test_file}"
-    #     return unique_schema
+    @pytest.fixture(scope="class")
+    def unique_schema(self, request, prefix) -> str:
+        test_file = request.module.__name__
+        # We only want the last part of the name
+        test_file = test_file.split(".")[-1]
+        unique_schema = f"{BUCKET}.{prefix}_{test_file}"
+        return unique_schema
 
     @pytest.fixture(scope="class")
     def project_config_update(self):