From 2024bb0f1a1de224d1040c101f4bee9841436061 Mon Sep 17 00:00:00 2001 From: shankar0123 Date: Mon, 27 Apr 2026 21:52:31 +0000 Subject: [PATCH] Bundle N.A/B-extended CI follow-up #2: 4th QF1002 hit at line 102 in TestDigicert_GetOrderStatus_PendingProcessingDeniedUnknown MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit CI flagged one more QF1002 hit at digicert_failure_test.go:102:5 that I missed in the prior fix (only got the three at 32/51/70). Same fix: 'switch { case r.URL.Path == "/user/me" }' → 'switch r.URL.Path { case "/user/me" }'. The remaining switches in this file (lines 126, 149) mix r.URL.Path == "x" with strings.Contains(r.URL.Path, "..."), which can't be expressed as tagged switches — staticcheck correctly does not flag those (same shape as the sectigo switches that pass clean). Verification: go test -short -count=1 ./internal/connector/issuer/ digicert/... PASS in 0.6s. Bundle: N.AB-ci-fix-2 --- internal/connector/issuer/digicert/digicert_failure_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/connector/issuer/digicert/digicert_failure_test.go b/internal/connector/issuer/digicert/digicert_failure_test.go index 9ee545d..6412c12 100644 --- a/internal/connector/issuer/digicert/digicert_failure_test.go +++ b/internal/connector/issuer/digicert/digicert_failure_test.go @@ -99,8 +99,8 @@ func TestDigicert_GetOrderStatus_PendingProcessingDeniedUnknown(t *testing.T) { for _, tc := range cases { t.Run(tc.name, func(t *testing.T) { srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { - switch { - case r.URL.Path == "/user/me": + switch r.URL.Path { + case "/user/me": w.WriteHeader(http.StatusOK) _, _ = w.Write([]byte(`{"id":1}`)) default: