Tested a few untested cases

tests/dataset/test_dataset.py

@@ -233,6 +233,26 @@ def test_save_and_load():
     assert newfile2.exists()
     assert newfile2.stat().st_size == len(buf)
 
+    # Test save existing dataset with anonymous distribution
+    newfile2.unlink(missing_ok=True)
+    save(ts, buf, dataset=SEMDATA.newimage2)
+    assert newfile2.exists()
+    assert newfile2.stat().st_size == len(buf)
+
+    # Test save new dataset with reference to existing distribution
+    newfile2.unlink(missing_ok=True)
+    save(
+        ts,
+        buf,
+        dataset={
+            "@id": SEMDATA.newimage3,
+            "title": "A dataset with no default distribution",
+            "distribution": SEMDATA.newdistr2,
+        },
+    )
+    assert newfile2.exists()
+    assert newfile2.stat().st_size == len(buf)
+
     # Test searching the triplestore
     SAMPLE = ts.namespaces["sample"]
     datasets = list_dataset_iris(ts)

tests/input/semdata.yaml

@@ -77,4 +77,3 @@ generators:
     generatorType: application/vnd.dlite-generate
     configuration:
       driver: hitachi
-      options: mode=w

tripper/dataset/dataset.py

@@ -173,7 +173,7 @@ def save(
             add(dataset, "distribution", distribution)
             triples.append((dataset["@id"], DCAT.distribution, newiri))
             save_distribution = True
-    elif isinstance(distribution, str):
+    if isinstance(distribution, str):
         distr = load_dict(ts, iri=distribution, use_sparql=use_sparql)
         if distr:
             distribution = distr