Adding ability to import dataset asset with no 'targets' field like filter divider

src/preset_cli/cli/superset/sync/native/command.py

@@ -230,7 +230,6 @@ def native(  # pylint: disable=too-many-locals, too-many-arguments, too-many-bra
     url = URL(ctx.obj["INSTANCE"])
     client = SupersetClient(url, auth)
     root = Path(directory)
-
     base_url = URL(external_url_prefix) if external_url_prefix else None
 
     # collecting existing database UUIDs so we know if we're creating or updating
@@ -382,7 +381,7 @@ def get_dataset_filter_uuids(config: AssetConfig) -> Set[str]:
     """
     dataset_uuids = set()
     for filter_config in config["metadata"].get("native_filter_configuration", []):
-        for target in filter_config["targets"]:
+        for target in filter_config.get("targets", {}):
             if uuid := target.get("datasetUuid"):
                 if uuid not in dataset_uuids:
                     dataset_uuids.add(uuid)

tests/cli/superset/sync/native/command_test.py

@@ -1,6 +1,7 @@
 """
 Tests for the native import command.
 """
+
 # pylint: disable=redefined-outer-name, invalid-name, too-many-lines
 
 import json
@@ -896,6 +897,27 @@ def test_native_split(  # pylint: disable=too-many-locals
         },
         "uuid": "6",
     }
+
+    dashboard_with_filter_divider_config = {
+        "dashboard_title": "Some dashboard",
+        "is_managed_externally": False,
+        "position": {},
+        "metadata": {
+            "native_filter_configuration": [
+                {
+                    "type": "NATIVE_FILTER",
+                    "targets": [
+                        {
+                            "column": "some_column",
+                            "datasetUuid": "5",
+                        },
+                    ],
+                },
+                {"type": "DIVIDER", "targets": {}},
+            ],
+        },
+        "uuid": "7",
+    }
     fs.create_file(
         root / "databases/gsheets.yaml",
         contents=yaml.dump(database_config),
@@ -928,6 +950,10 @@ def test_native_split(  # pylint: disable=too-many-locals
         root / "dashboards/dashboard_deleted_dataset.yaml",
         contents=yaml.dump(dashboard_deleted_dataset),
     )
+    fs.create_file(
+        root / "dashboards/dashboard_with_filter_divider_config.yaml",
+        contents=yaml.dump(dashboard_with_filter_divider_config),
+    )
 
     SupersetClient = mocker.patch(
         "preset_cli.cli.superset.sync.native.command.SupersetClient",
@@ -1023,7 +1049,21 @@ def test_native_split(  # pylint: disable=too-many-locals
                 client,
                 False,
             ),
+            mock.call(
+                {
+                    "bundle/dashboards/dashboard_with_filter_divider_config.yaml": yaml.dump(
+                        dashboard_with_filter_divider_config,
+                    ),
+                    "bundle/datasets/gsheets/filter_test.yaml": yaml.dump(
+                        dataset_filter_config,
+                    ),
+                    "bundle/databases/gsheets.yaml": yaml.dump(database_config),
+                },
+                client,
+                False,
+            ),
         ],
+        any_order=True,
     )