further simplify

Author: Saba9

tests/e2e-local/test_cli_agent_commands.py

@@ -150,21 +150,6 @@ def test_summary(temp_dir):
         assert {"run", "status", "last_step", "num_logs", "config", "metric_value"} <= run_entry.keys()
 
 
-def test_list_runs_json_includes_status(temp_dir):
-    _seed(temp_dir)
-    r = _cli(["list", "runs", "--project", PROJECT, "--json"], temp_dir)
-    assert r.returncode == 0
-    data = json.loads(r.stdout)
-    assert "runs" in data
-    for entry in data["runs"]:
-        assert "name" in entry
-        assert "status" in entry
-    statuses = {e["name"]: e["status"] for e in data["runs"]}
-    assert statuses.get("run-lr0.01") == "finished"
-    assert statuses.get("run-lr0.1") == "finished"
-    assert statuses.get("run-lr1.0") == "finished"
-
-
 def test_best_error_cases(temp_dir):
     _seed(temp_dir)
     assert _cli(["best", "--project", "nope", "--metric", "loss", "--json"], temp_dir).returncode != 0

tests/e2e-local/test_run_status.py

@@ -32,22 +32,14 @@ def test_status_failed_not_overwritten_by_finish(temp_dir):
     )
 
 
-def test_api_run_status(temp_dir):
-    trackio.init(project="api_status", name="run1")
-    trackio.log({"loss": 0.5}, step=0)
-    trackio.finish()
-
-    run = trackio.Api().runs("api_status")[0]
-    assert run.status == "finished"
-
-
-def test_api_run_final_metrics(temp_dir):
-    trackio.init(project="final_metrics_test", name="run1")
+def test_api_run_access(temp_dir):
+    trackio.init(project="api_test", name="run1")
     trackio.log({"loss": 1.0, "acc": 0.5}, step=0)
     trackio.log({"loss": 0.5, "acc": 0.8}, step=1)
     trackio.finish()
 
-    run = trackio.Api().runs("final_metrics_test")[0]
+    run = trackio.Api().runs("api_test")[0]
+    assert run.status == "finished"
     fm = run.final_metrics
     assert abs(fm["loss"] - 0.5) < 1e-6
     assert abs(fm["acc"] - 0.8) < 1e-6

tests/e2e-local/test_watchers.py

@@ -24,15 +24,12 @@ def test_max_value_with_dedup():
     assert len(w.check(15.0, step=4)) == 1
 
 
-def test_min_value_with_dedup():
+def test_min_value_triggers():
     w = MetricWatcher("acc", min_value=0.5)
     assert len(w.check(0.8, step=0)) == 0
     alerts = w.check(0.3, step=1)
     assert len(alerts) == 1
     assert alerts[0]["data"]["reason"] == AlertReason.MIN_EXCEEDED
-    assert len(w.check(0.3, step=2)) == 0
-    w.check(0.8, step=3)
-    assert len(w.check(0.3, step=4)) == 1
 
 
 def test_spike_detection_with_dedup_and_reset():