make save private, library is no longer a datamodel

Author: braingram

src/stpipe/library.py

@@ -568,7 +568,7 @@ def _load_member(self, index):
 
         return model
 
-    def save(self, path, **kwargs):
+    def _save(self, path, **kwargs):
         """
         .. warning:: This save is NOT used by Step/Pipeline. This is
                      intentional as the Step/Pipeline has special requirements.

tests/test_library.py

@@ -737,7 +737,7 @@ def assign_code(model, i):
     codes = list(example_library.map_function(assign_code))
 
     output_path = tmp_path / "tmp_output"
-    asn_path = example_library.save(output_path)
+    asn_path = example_library._save(output_path)
 
     if as_str:
         dst = str(asn_path)
@@ -770,13 +770,13 @@ def test_ledger():
     assert len(ledger) == 0
 
 
-def test_library_is_not_a_datamodel():
+def test_library_datamodel_relationship():
     """
     Smoke test to make sure the relationship between
     AbstractModelLibrary and AbstractDataModel doesn't
     change.
     """
-    assert issubclass(AbstractModelLibrary, AbstractDataModel)
+    assert not issubclass(AbstractModelLibrary, AbstractDataModel)
 
 
 def test_library_is_not_sequence():