From 1660fe025f040a48cd15d236b6ee1326fe343648 Mon Sep 17 00:00:00 2001 From: Virgil Date: Mon, 30 Mar 2026 12:20:10 +0000 Subject: [PATCH] test(ax): sync health and security coverage --- cmd/qa/cmd_health_test.go | 8 ++++++++ pkg/php/security_test.go | 5 ++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/cmd/qa/cmd_health_test.go b/cmd/qa/cmd_health_test.go index 4ee3429..81f40a8 100644 --- a/cmd/qa/cmd_health_test.go +++ b/cmd/qa/cmd_health_test.go @@ -71,8 +71,12 @@ esac assert.Equal(t, 1, payload.Summary.Passing) assert.Equal(t, 1, payload.Summary.Errors) assert.Equal(t, 2, payload.Summary.FilteredRepos) + assert.Len(t, payload.Summary.ByStatus, 6) assert.Equal(t, 1, payload.Summary.ByStatus["passing"]) assert.Equal(t, 1, payload.Summary.ByStatus["error"]) + assert.Equal(t, 0, payload.Summary.ByStatus["pending"]) + assert.Equal(t, 0, payload.Summary.ByStatus["disabled"]) + assert.Equal(t, 0, payload.Summary.ByStatus["no_ci"]) require.Len(t, payload.Repos, 2) assert.Equal(t, "error", payload.Repos[0].Status) assert.Equal(t, "beta", payload.Repos[0].Name) @@ -146,8 +150,12 @@ esac assert.Equal(t, 1, payload.Summary.Errors) assert.Equal(t, 1, payload.Summary.FilteredRepos) assert.True(t, payload.Summary.ProblemsOnly) + assert.Len(t, payload.Summary.ByStatus, 6) assert.Equal(t, 1, payload.Summary.ByStatus["passing"]) assert.Equal(t, 1, payload.Summary.ByStatus["error"]) + assert.Equal(t, 0, payload.Summary.ByStatus["pending"]) + assert.Equal(t, 0, payload.Summary.ByStatus["disabled"]) + assert.Equal(t, 0, payload.Summary.ByStatus["no_ci"]) require.Len(t, payload.Repos, 1) assert.Equal(t, "error", payload.Repos[0].Status) assert.Equal(t, "beta", payload.Repos[0].Name) diff --git a/pkg/php/security_test.go b/pkg/php/security_test.go index 94dbce9..e2b55be 100644 --- a/pkg/php/security_test.go +++ b/pkg/php/security_test.go @@ -297,5 +297,8 @@ func setupSecurityFixture(t *testing.T, dir string, envContent string) { require.NoError(t, os.WriteFile(composerBin, []byte("#!/bin/sh\ncat <<'JSON'\n{\"advisories\":{}}\nJSON\n"), 0o755)) oldPath := os.Getenv("PATH") - t.Setenv("PATH", dir+string(os.PathListSeparator)+oldPath) + require.NoError(t, os.Setenv("PATH", dir+string(os.PathListSeparator)+oldPath)) + t.Cleanup(func() { + require.NoError(t, os.Setenv("PATH", oldPath)) + }) }