adding migration tests back

Dashlane · Jul 20, 2023 · c39fa3f · c39fa3f
1 parent bdf1115
commit c39fa3f
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 37 deletions.
diff --git a/tests/dbt.duckdb b/tests/dbt.duckdb
diff --git a/tests/test_properties.py b/tests/test_properties.py
@@ -149,43 +149,43 @@ def migrate_wrapper(self, resource_type, *args, **kwargs):
             except FileNotFoundError:
                 continue
 
-    # def test_partial_migrate(self):
-    #     """
-    #     Test the partial migration of structure from one property file
-    #     for multiple resources to one property file per resource
-    #
-    #     :return: None
-    #     """
-    #     self.migrate_wrapper(
-    #         'model',
-    #         'migration_leftover_comment.yml',
-    #         analyses__revenue_by_daily_cohort=False,
-    #         data__items=False,
-    #         models__marts__core__customers=True,
-    #         models__marts__core__orders=True,
-    #         models__migration_leftover_comment=True,
-    #         snapshots__items_snapshot=False,
-    #     )
-    #
-    # def test_full_migrate(self):
-    #     """
-    #     Test the full migration of structure from one property file for
-    #     multiple resources to one property file per resource
-    #
-    #     :return: None
-    #     """
-    #     self.migrate_wrapper(
-    #         None,
-    #         'migration_leftover_comment.yml',
-    #         'migration_no_leftover_comment.yml',
-    #         analyses__revenue_by_daily_cohort=True,
-    #         data__items=True,
-    #         models__marts__core__customers=True,
-    #         models__marts__core__orders=True,
-    #         models__migration_leftover_comment=True,
-    #         models__migration_no_leftover_comment=False,
-    #         snapshots__items_snapshot=True,
-    #     )
+    def test_partial_migrate(self):
+        """
+        Test the partial migration of structure from one property file
+        for multiple resources to one property file per resource
+
+        :return: None
+        """
+        self.migrate_wrapper(
+            'model',
+            'migration_leftover_comment.yml',
+            analyses__revenue_by_daily_cohort=False,
+            data__items=False,
+            models__marts__core__customers=True,
+            models__marts__core__orders=True,
+            models__migration_leftover_comment=True,
+            snapshots__items_snapshot=False,
+        )
+
+    def test_full_migrate(self):
+        """
+        Test the full migration of structure from one property file for
+        multiple resources to one property file per resource
+
+        :return: None
+        """
+        self.migrate_wrapper(
+            None,
+            'migration_leftover_comment.yml',
+            'migration_no_leftover_comment.yml',
+            analyses__revenue_by_daily_cohort=True,
+            data__items=True,
+            models__marts__core__customers=True,
+            models__marts__core__orders=True,
+            models__migration_leftover_comment=True,
+            models__migration_no_leftover_comment=False,
+            snapshots__items_snapshot=True,
+        )
 
     def test_multiline(self):
         self.edit_update_compare(