From 17cf26cd829bd5fdaa4daba451ae8af43eb0ba85 Mon Sep 17 00:00:00 2001 From: celeste Date: Mon, 22 Jun 2026 21:23:12 -0700 Subject: [PATCH] test(modules): fix favicon_test checkMatchers arity after matchers-condition (#228) #189 added the condition param to checkMatchers; the favicon matcher test (#183) still called the 3-arg form, breaking the build once both landed. pass "" (default and-condition) to restore single-matcher semantics. --- internal/modules/favicon_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/modules/favicon_test.go b/internal/modules/favicon_test.go index aea4929..2fc9af5 100644 --- a/internal/modules/favicon_test.go +++ b/internal/modules/favicon_test.go @@ -142,7 +142,7 @@ func TestCheckMatcherFaviconNegative(t *testing.T) { signed := int64(fingerprint.FaviconHash(faviconFixture)) matchers := []Matcher{{Type: "favicon", Hash: []int64{signed}, Negative: true}} resp := fakeResponse(t, 200, nil) - if checkMatchers(matchers, resp, string(faviconFixture)) { + if checkMatchers(matchers, "", resp, string(faviconFixture)) { t.Error("negative favicon matcher should not match its own hash") } }