Merge pull request #758 from simvue-io/dev

2.0.0 release

Author: kzscisoft

simvue/api/objects/base.py

@@ -479,6 +479,9 @@ def delete(
         if self._offline:
             return {"id": self._identifier}
 
+        if not self._identifier:
+            raise RuntimeError(f"Object of type '{self._label}' has no identifier.")
+
         if not self.url:
             raise RuntimeError(f"Identifier for instance of {self._label} Unknown")
         _response = sv_delete(url=f"{self.url}", headers=self._headers, params=kwargs)

simvue/run.py

@@ -1360,6 +1360,7 @@ def save_object(
                 allow_pickling=allow_pickle,
                 storage=self._storage_id,
                 metadata=metadata,
+                offline=self._user_config.run.mode == "offline",
             )
             _artifact.attach_to_run(self.id, category)
         except (ValueError, RuntimeError) as e:

simvue/utilities.py

@@ -92,6 +92,13 @@ def parse_validation_response(
 
     out: list[list[str]] = []
 
+    if isinstance(issues, str):
+        return tabulate.tabulate(
+            ["Unknown", "N/A", issues],
+            headers=["Type", "Location", "Message"],
+            tablefmt="fancy_grid",
+        )
+
     for issue in issues:
         obj_type: str = issue["type"]
         location: list[str] = issue["loc"]

tests/conftest.py

@@ -71,6 +71,9 @@ def create_test_run(request) -> typing.Generator[typing.Tuple[sv_run.Run, dict],
 
 @pytest.fixture
 def create_test_run_offline(mocker: pytest_mock.MockerFixture, request, monkeypatch: pytest.MonkeyPatch) -> typing.Generator[typing.Tuple[sv_run.Run, dict], None, None]:
+    def testing_exit(status: int) -> None:
+        raise SystemExit(status)
+    mocker.patch("os._exit", testing_exit)
     with tempfile.TemporaryDirectory() as temp_d:
         monkeypatch.setenv("SIMVUE_OFFLINE_DIRECTORY", temp_d)
         with sv_run.Run("offline") as run:
@@ -79,7 +82,10 @@ def create_test_run_offline(mocker: pytest_mock.MockerFixture, request, monkeypa
 
 
 @pytest.fixture
-def create_plain_run(request) -> typing.Generator[typing.Tuple[sv_run.Run, dict], None, None]:
+def create_plain_run(request, mocker: pytest_mock.MockFixture) -> typing.Generator[typing.Tuple[sv_run.Run, dict], None, None]:
+    def testing_exit(status: int) -> None:
+        raise SystemExit(status)
+    mocker.patch("os._exit", testing_exit)
     with sv_run.Run() as run:
         yield run, setup_test_run(run, False, request)
     clear_out_files()
@@ -102,7 +108,10 @@ def create_plain_run_offline(mocker: pytest_mock.MockerFixture, request, monkeyp
 
 
 @pytest.fixture
-def create_run_object() -> sv_api_obj.Run:
+def create_run_object(mocker: pytest_mock.MockFixture) -> sv_api_obj.Run:
+    def testing_exit(status: int) -> None:
+        raise SystemExit(status)
+    mocker.patch("os._exit", testing_exit)
     _fix_use_id: str = str(uuid.uuid4()).split('-', 1)[0]
     _folder = sv_api_obj.Folder.new(path=f"/simvue_unit_testing/{_fix_use_id}")
     _folder.commit()
@@ -114,22 +123,24 @@ def create_run_object() -> sv_api_obj.Run:
 
 def setup_test_run(run: sv_run.Run, create_objects: bool, request: pytest.FixtureRequest, created_only: bool=False):
     fix_use_id: str = str(uuid.uuid4()).split('-', 1)[0]
+    _test_name: str = request.node.name.replace("[", "_").replace("]", "")
     TEST_DATA = {
         "event_contains": "sent event",
         "metadata": {
             "test_engine": "pytest",
-            "test_identifier": fix_use_id
+            "test_identifier": f"{_test_name}_{fix_use_id}"
         },
         "folder": f"/simvue_unit_testing/{fix_use_id}",
-        "tags": ["simvue_client_unit_tests", request.node.name.replace("[", "_").replace("]", "")]
+        "tags": ["simvue_client_unit_tests", _test_name]
     }
 
     if os.environ.get("CI"):
         TEST_DATA["tags"].append("ci")
 
     run.config(suppress_errors=False)
+    run._heartbeat_interval = 1
     run.init(
-        name=f"test_run_{TEST_DATA['metadata']['test_identifier']}_{uuid.uuid4()}",
+        name=TEST_DATA['metadata']['test_identifier'],
         tags=TEST_DATA["tags"],
         folder=TEST_DATA["folder"],
         visibility="tenant" if os.environ.get("CI") else None,

tests/functional/test_client.py

@@ -7,6 +7,7 @@
 import glob
 import pathlib
 import time
+import pytest_mock
 import tempfile
 import simvue.client as svc
 from simvue.exception import ObjectNotFoundError
@@ -383,19 +384,14 @@ def test_alert_deletion() -> None:
 
 
 @pytest.mark.client
-def test_abort_run() -> None:
+def test_abort_run(create_plain_run: tuple[sv_run.Run, dict]) -> None:
+    run, run_data = create_plain_run
     _uuid = f"{uuid.uuid4()}".split("-")[0]
-    _folder = sv_api_obj.Folder.new(path=f"/simvue_unit_testing/{_uuid}")
-    _run = sv_api_obj.Run.new(folder=f"/simvue_unit_testing/{_uuid}")
-    _run.status = "running"
-    _folder.commit()
-    _run.commit()
+    run.update_tags([f"delete_me_{_uuid}"])
     time.sleep(1)
     _client = svc.Client()
-    _client.abort_run(_run.id, reason="Test abort")
+    _client.abort_run(run.id, reason="Test abort")
     time.sleep(1)
-    assert _run.abort_trigger
-    _run.delete()
-    _folder.delete(recursive=True, delete_runs=True, runs_only=False)
+    assert run._status == "terminated"