Source Facebook Marketing #5190 - fixing unit tests

airbytehq · Nov 1, 2021 · 49df07c · 49df07c
1 parent 6a66d7b
commit 49df07c
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 39 deletions.
diff --git a/airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/utils/compare.py b/airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/utils/compare.py
@@ -81,10 +81,10 @@ class ListWithHashMixin(HashMixin, list):
 
 def make_hashable(obj, exclude_fields: List[str] = None) -> str:
     """
-   Simplify comparison of nested dicts/lists
-   :param obj value for comparison
-   :param exclude_fields if value is Mapping, some fields can be excluded
-   """
+    Simplify comparison of nested dicts/lists
+    :param obj value for comparison
+    :param exclude_fields if value is Mapping, some fields can be excluded
+    """
     if isinstance(obj, Mapping):
         # If value is Mapping, some fields can be excluded
         exclude_fields = exclude_fields or []

diff --git a/airbyte-integrations/bases/source-acceptance-test/unit_tests/test_utils.py b/airbyte-integrations/bases/source-acceptance-test/unit_tests/test_utils.py
@@ -154,7 +154,9 @@ def test_compare_two_records_nested_with_different_orders(obj1, obj2, is_same):
 
 def test_exclude_fields():
     """Test that check ignoring fields"""
-    data = [sorted_data(), ]
+    data = [
+        sorted_data(),
+    ]
     ignored_fields = [
         "organization_id",
     ]

diff --git a/...onnectors/source-google-workspace-admin-reports/integration_tests/configured_catalog.json b/...onnectors/source-google-workspace-admin-reports/integration_tests/configured_catalog.json
@@ -61,4 +61,4 @@
       "destination_sync_mode": "append"
     }
   ]
-}
+}
diff --git a/...ns/connectors/source-google-workspace-admin-reports/integration_tests/invalid_config.json b/...ns/connectors/source-google-workspace-admin-reports/integration_tests/invalid_config.json
@@ -1,5 +1,5 @@
 {
-    "credentials_json": "{\n  \"type\": \"service_account\"}\n",
-    "email": "test_email",
-    "lookback": 0
+  "credentials_json": "{\n  \"type\": \"service_account\"}\n",
+  "email": "test_email",
+  "lookback": 0
 }
diff --git a/...-google-workspace-admin-reports/source_google_workspace_admin_reports/schemas/logins.json b/...-google-workspace-admin-reports/source_google_workspace_admin_reports/schemas/logins.json
@@ -21,12 +21,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "time",
-        "uniqueQualifier",
-        "applicationName",
-        "customerId"
-      ]
+      "required": ["time", "uniqueQualifier", "applicationName", "customerId"]
     },
     "etag": {
       "type": "string"
@@ -41,9 +36,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "email"
-      ]
+      "required": ["email"]
     },
     "ipAddress": {
       "type": "string"

diff --git a/...-google-workspace-admin-reports/source_google_workspace_admin_reports/schemas/mobile.json b/...-google-workspace-admin-reports/source_google_workspace_admin_reports/schemas/mobile.json
@@ -21,12 +21,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "time",
-        "uniqueQualifier",
-        "applicationName",
-        "customerId"
-      ]
+      "required": ["time", "uniqueQualifier", "applicationName", "customerId"]
     },
     "etag": {
       "type": "string"
@@ -44,11 +39,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "callerType",
-        "email",
-        "profileId"
-      ]
+      "required": ["callerType", "email", "profileId"]
     },
     "events": {
       "type": "array",

diff --git a/...e-workspace-admin-reports/source_google_workspace_admin_reports/schemas/oauth_tokens.json b/...e-workspace-admin-reports/source_google_workspace_admin_reports/schemas/oauth_tokens.json
@@ -21,12 +21,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "time",
-        "uniqueQualifier",
-        "applicationName",
-        "customerId"
-      ]
+      "required": ["time", "uniqueQualifier", "applicationName", "customerId"]
     },
     "etag": {
       "type": "string"
@@ -41,10 +36,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "email",
-        "profileId"
-      ]
+      "required": ["email", "profileId"]
     },
     "ipAddress": {
       "type": "string"