test: provide basic Dandiset metadata through fixture

So that the metadata can be
used amount different tests

Author: candleindark

dandischema/tests/test_models.py

@@ -41,6 +41,53 @@
 _INSTANCE_CONFIG = get_instance_config()
 
 
+@pytest.fixture
+def base_dandiset_metadata() -> Dict[str, Any]:
+    """
+    Fixture providing basic Dandiset metadata for constructing a `Dandiset` instance.
+
+    Returns:
+    Dict[str, Any]
+        A dictionary containing basic Dandiset metadata without `doi`, `datePublished`,
+        and `publishedBy`, suitable for constructing a `Dandiset` instance but not a
+        `PublishedDandiset` instance.
+
+    Note:
+        This metadata is returned by a fixture to ensure that each test receives a fresh
+        copy of the metadata dictionary.
+    """
+
+    return {
+        "identifier": f"{INSTANCE_NAME}:999999",
+        "id": f"{INSTANCE_NAME}:999999/draft",
+        "version": "1.0.0",
+        "name": "testing dataset",
+        "description": "testing",
+        "contributor": [
+            {
+                "name": "last name, first name",
+                "email": "someone@dandiarchive.org",
+                "roleName": [RoleType("dcite:ContactPerson")],
+                "schemaKey": "Person",
+            }
+        ],
+        "license": [LicenseType("spdx:CC-BY-4.0")],
+        "citation": "Last, first (2021). Test citation.",
+        "assetsSummary": {
+            "numberOfBytes": 0,
+            "numberOfFiles": 0,
+            "dataStandard": [{"name": "NWB"}],
+            "approach": [{"name": "electrophysiology"}],
+            "measurementTechnique": [{"name": "two-photon microscopy technique"}],
+            "species": [{"name": "Human"}],
+        },
+        "manifestLocation": [
+            "https://api.dandiarchive.org/api/dandisets/999999/versions/draft/assets/"
+        ],
+        "url": "https://dandiarchive.org/dandiset/999999/draft",
+    }
+
+
 @pytest.mark.parametrize(
     ("y_type", "anys_value"),
     [
@@ -403,46 +450,15 @@ def test_autogenerated_titles() -> None:
 
 
 @skipif_no_doi_prefix
-def test_dandimeta_1() -> None:
+def test_dandimeta_1(base_dandiset_metadata) -> None:
     """checking basic metadata for publishing"""
 
     assert DOI_PREFIX is not None
 
-    # metadata without doi, datePublished and publishedBy
-    meta_dict: Dict[str, Any] = {
-        "identifier": f"{INSTANCE_NAME}:999999",
-        "id": f"{INSTANCE_NAME}:999999/draft",
-        "version": "1.0.0",
-        "name": "testing dataset",
-        "description": "testing",
-        "contributor": [
-            {
-                "name": "last name, first name",
-                "email": "someone@dandiarchive.org",
-                "roleName": [RoleType("dcite:ContactPerson")],
-                "schemaKey": "Person",
-            }
-        ],
-        "license": [LicenseType("spdx:CC-BY-4.0")],
-        "citation": "Last, first (2021). Test citation.",
-        "assetsSummary": {
-            "numberOfBytes": 0,
-            "numberOfFiles": 0,
-            "dataStandard": [{"name": "NWB"}],
-            "approach": [{"name": "electrophysiology"}],
-            "measurementTechnique": [{"name": "two-photon microscopy technique"}],
-            "species": [{"name": "Human"}],
-        },
-        "manifestLocation": [
-            "https://api.dandiarchive.org/api/dandisets/999999/versions/draft/assets/"
-        ],
-        "url": "https://dandiarchive.org/dandiset/999999/draft",
-    }
-
     # should work for Dandiset but PublishedDandiset should raise an error
-    Dandiset(**meta_dict)
+    Dandiset(**base_dandiset_metadata)
     with pytest.raises(ValidationError) as exc:
-        PublishedDandiset(**meta_dict)
+        PublishedDandiset(**base_dandiset_metadata)
 
     ErrDetail = namedtuple("ErrDetail", ["type", "msg"])
 
@@ -490,21 +506,23 @@ def test_dandimeta_1() -> None:
 
     # after adding basic meta required to publish: doi, datePublished, publishedBy, assetsSummary,
     # so PublishedDandiset should work
-    meta_dict["url"] = "https://dandiarchive.org/dandiset/999999/0.0.0"
-    meta_dict["id"] = f"{INSTANCE_NAME}:999999/0.0.0"
-    meta_dict["version"] = "0.0.0"
-    meta_dict.update(
+    base_dandiset_metadata["url"] = "https://dandiarchive.org/dandiset/999999/0.0.0"
+    base_dandiset_metadata["id"] = f"{INSTANCE_NAME}:999999/0.0.0"
+    base_dandiset_metadata["version"] = "0.0.0"
+    base_dandiset_metadata.update(
         basic_publishmeta(INSTANCE_NAME, dandi_id="999999", prefix=DOI_PREFIX)
     )
-    meta_dict["assetsSummary"].update(**{"numberOfBytes": 1, "numberOfFiles": 1})
+    base_dandiset_metadata["assetsSummary"].update(
+        **{"numberOfBytes": 1, "numberOfFiles": 1}
+    )
 
     # Test that releaseNotes is optional (can be omitted)
-    dandiset_without_notes = PublishedDandiset(**meta_dict)
+    dandiset_without_notes = PublishedDandiset(**base_dandiset_metadata)
     assert dandiset_without_notes.releaseNotes is None
 
     # Test that releaseNotes can be set to a string value
-    meta_dict["releaseNotes"] = "Releasing during testing"
-    dandiset_with_notes = PublishedDandiset(**meta_dict)
+    base_dandiset_metadata["releaseNotes"] = "Releasing during testing"
+    dandiset_with_notes = PublishedDandiset(**base_dandiset_metadata)
     assert dandiset_with_notes.releaseNotes == "Releasing during testing"
 
     # Test that releaseNotes appears in model_dump