Merge pull request #77 from ThiagoPanini/2.0.x

Finishing unit tests for everything but no the main script
ThiagoPanini · May 1, 2023 · 3ddbdb3 · 3ddbdb3
2 parents 7a57d0e + 3d480d9
commit 3ddbdb3
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 27 deletions.
diff --git a/app/tests/test_main.py b/app/tests/test_main.py
diff --git a/docs/mkdocs/demos/learning-mode.md b/docs/mkdocs/demos/learning-mode.md
@@ -132,6 +132,11 @@ The only thing that is required when calling terraglue on learning mode is the s
       # Setting up output variables
       job_output_bucket_name = "datadelivery-sot-data-${data.aws_caller_identity.current.account_id}-${data.aws_region.current.name}"
       job_output_database = "db_datadelivery_sot"
+
+      # Ensuring terraglue will be deployed only after datadelivery
+      depends_on = [
+        module.datadelivery
+      ]
     }
     ```
 

diff --git a/pytest.ini b/pytest.ini
@@ -13,4 +13,3 @@ markers =
     transform_payments: Unit test for transform_payments() function from transformers.py auxiliar module
     transform_reviews: Unit test for transform_reviews() function from transformers.py auxiliar module
     transform_sot: Unit test for transform_sot() function from transformers.py auxiliar module
-    main: Unit test for the main Spark application to be deployed as a Glue job in AWS