From bf8f4ff665be84ba15cb2920fd9ae7c7832ff2f9 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Wed, 12 Aug 2026 16:11:01 -0500 Subject: [PATCH] chore(test): prune stale sim test regexes and stub files The v0.2.0 handle-API migration gutted 14 sim test files to 4-line stubs; the Makefile/CI regexes kept matching zero tests and passed vacuously. DST and seed-sweep suites live in go-libfossil now. - make test / CI sim job run the tests that actually exist - drop dead sim, sim-full, test-interop targets and ghost dst PHONYs - delete the stub files (originals in git history) --- .github/workflows/test.yml | 8 ++------ Makefile | 35 ++++++++--------------------------- sim/buggify_test.go | 4 ---- sim/checkout_test.go | 4 ---- sim/diff_interop_test.go | 4 ---- sim/equivalence_test.go | 4 ---- sim/finfo_interop_test.go | 4 ---- sim/interop_test.go | 4 ---- sim/otel_test.go | 4 ---- sim/proxy_test.go | 4 ---- sim/rebuild_test.go | 4 ---- sim/schedule_test.go | 4 ---- sim/serve_test.go | 4 ---- sim/shun_test.go | 4 ---- sim/sim_test.go | 4 ---- sim/uv_test.go | 4 ---- 16 files changed, 10 insertions(+), 89 deletions(-) delete mode 100644 sim/buggify_test.go delete mode 100644 sim/checkout_test.go delete mode 100644 sim/diff_interop_test.go delete mode 100644 sim/equivalence_test.go delete mode 100644 sim/finfo_interop_test.go delete mode 100644 sim/interop_test.go delete mode 100644 sim/otel_test.go delete mode 100644 sim/proxy_test.go delete mode 100644 sim/rebuild_test.go delete mode 100644 sim/schedule_test.go delete mode 100644 sim/serve_test.go delete mode 100644 sim/shun_test.go delete mode 100644 sim/sim_test.go delete mode 100644 sim/uv_test.go diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 10441a13..0a60a695 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -39,12 +39,8 @@ jobs: go-version-file: go.mod - name: Install Fossil run: sudo apt-get update && sudo apt-get install -y fossil - - name: Sim unit tests - run: cd sim && go test . -run 'TestFaultProxy|TestGenerateSchedule|TestBuggify' -count=1 - - name: Sim serve tests - run: cd sim && go test . -run 'TestServeHTTP|TestLeafToLeaf|TestAgentServe|TestHubLeafE2E|TestAgentNew_|TestHubNATSFossilSync_' -count=1 -timeout=120s - - name: Interop (short) - run: cd sim && go test . -run 'TestInterop' -count=1 -short -timeout=60s + - name: Sim integration tests + run: cd sim && go test . -run 'TestHubLeafE2E|TestAgentNew_|TestHubNATSFossilSync_' -count=1 -timeout=120s build: name: Build diff --git a/Makefile b/Makefile index a548aa36..473018dd 100644 --- a/Makefile +++ b/Makefile @@ -1,4 +1,4 @@ -.PHONY: build test clean leaf bridge edgesync iroh-sidecar wasm-wasi wasm-browser wasm dst dst-full dst-hostile dst-drivers sim sim-full setup-hooks setup test-interop test-iroh update-libfossil auto-release auto-release-dry +.PHONY: build test clean leaf bridge edgesync iroh-sidecar wasm-wasi wasm-browser wasm setup-hooks setup test-iroh update-libfossil auto-release auto-release-dry # --- Build --- @@ -34,7 +34,8 @@ clean: rm -rf bin/ # --- Test (what CI runs) --- -# Unit tests run in parallel across modules; sim/dst run sequentially after. +# Unit tests run in parallel across modules; sim integration tests run after. +# Iroh tests need the sidecar binary — run them via test-iroh. test: @pids=""; fail=0; \ @@ -42,10 +43,7 @@ test: (cd bridge && go test ./... -short -count=1) & pids="$$pids $$!"; \ for pid in $$pids; do wait $$pid || fail=1; done; \ if [ $$fail -ne 0 ]; then echo "FAIL: unit tests"; exit 1; fi - (cd sim && go test . -run 'TestFaultProxy|TestGenerateSchedule|TestBuggify' -count=1) - (cd sim && go test . -run 'TestServeHTTP|TestLeafToLeaf|TestAgentServe|TestHubLeafE2E|TestAgentNew_|TestHubNATSFossilSync_' -count=1 -timeout=120s) - (cd sim && go test . -run 'TestInterop' -count=1 -short -timeout=60s) - (cd sim && go test . -run 'TestSimulation' -sim.seed=1 -count=1 -timeout=120s) + (cd sim && go test . -run 'TestHubLeafE2E|TestAgentNew_|TestHubNATSFossilSync_' -count=1 -timeout=120s) vet: go vet ./... @@ -65,27 +63,10 @@ setup-hooks: @echo "Pre-commit hook installed. Runs ~5s of tests before each commit." @echo "Skip with: git commit --no-verify" -# --- Sim (Integration Simulation) — run locally, requires fossil --- -# Note: DST tests now live in libfossil (github.com/danmestas/libfossil/dst) - -# Quick: 1 seed, normal severity -sim: - cd sim && go test . -run TestSimulation -sim.seed=1 -v -timeout=120s - -# Full: 16 seeds × 3 severities -sim-full: - @echo "=== Sim full (16 seeds × 3 severities) ===" - @for severity in normal adversarial hostile; do \ - for seed in 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16; do \ - echo " seed=$$seed severity=$$severity ..."; \ - (cd sim && go test . -run TestSimulation -sim.seed=$$seed -sim.severity=$$severity -timeout=120s) || true; \ - done; \ - done - @echo "=== Sim full done ===" - -# Fossil interop: Tier 1 + Tier 2 (requires fossil binary, Tier 2 samples 5K+2K blobs) -test-interop: - cd sim && go test -buildvcs=false . -run TestInterop -timeout=10m -v +# --- Sim (Integration Simulation) --- +# DST and the seed-sweep simulation/interop suites live in go-libfossil +# (github.com/danmestas/go-libfossil): the old sim/dst regex targets here +# ran against tests that were removed in the v0.2.0 handle-API migration. # Iroh P2P: build sidecar then run iroh unit + integration tests test-iroh: iroh-sidecar diff --git a/sim/buggify_test.go b/sim/buggify_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/buggify_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/checkout_test.go b/sim/checkout_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/checkout_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/diff_interop_test.go b/sim/diff_interop_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/diff_interop_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/equivalence_test.go b/sim/equivalence_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/equivalence_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/finfo_interop_test.go b/sim/finfo_interop_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/finfo_interop_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/interop_test.go b/sim/interop_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/interop_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/otel_test.go b/sim/otel_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/otel_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/proxy_test.go b/sim/proxy_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/proxy_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/rebuild_test.go b/sim/rebuild_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/rebuild_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/schedule_test.go b/sim/schedule_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/schedule_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/serve_test.go b/sim/serve_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/serve_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/shun_test.go b/sim/shun_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/shun_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/sim_test.go b/sim/sim_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/sim_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim diff --git a/sim/uv_test.go b/sim/uv_test.go deleted file mode 100644 index 7bfd8e99..00000000 --- a/sim/uv_test.go +++ /dev/null @@ -1,4 +0,0 @@ -// TODO(v0.2.0): This test file needs migration to the libfossil handle API. -// Tests are skipped pending migration of internal type dependencies. -// Original file backed up in git history. -package sim