From 8f09c1dc30d24d5f2c1873fb0681ca77ae168675 Mon Sep 17 00:00:00 2001 From: montehurd Date: Mon, 21 Sep 2026 13:51:48 -0700 Subject: [PATCH] Fix the handler test build The denylist tests call NewDebianHandler with three arguments, but the additional APT archives change gave it a fourth. Each passed CI on its own branch, and together they leave internal/handler without a test binary that compiles, so go vet and go test fail on main. --- internal/handler/denylist_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/handler/denylist_test.go b/internal/handler/denylist_test.go index 3bcb7e73..5034a403 100644 --- a/internal/handler/denylist_test.go +++ b/internal/handler/denylist_test.go @@ -172,7 +172,7 @@ func TestDenylistDownloadRoutes(t *testing.T) { {"npm", "/demo/-/demo-1.0.0.tgz", "demo-1.0.0.tgz", func(p *Proxy) http.Handler { return NewNPMHandler(p, "http://proxy.test", "").Routes() }}, {"pypi", "/packages/a/b/demo-1.0.0.tar.gz", "demo-1.0.0.tar.gz", func(p *Proxy) http.Handler { return NewPyPIHandler(p, "http://proxy.test").Routes() }}, {"cargo", "/crates/demo/1.0.0/download", "demo-1.0.0.crate", func(p *Proxy) http.Handler { return NewCargoHandler(p, "http://proxy.test", "", "").Routes() }}, - {"deb", "/pool/main/d/demo/demo_1.0.0_amd64.deb", "demo_1.0.0_amd64.deb", func(p *Proxy) http.Handler { return NewDebianHandler(p, "http://proxy.test", "").Routes() }}, + {"deb", "/pool/main/d/demo/demo_1.0.0_amd64.deb", "demo_1.0.0_amd64.deb", func(p *Proxy) http.Handler { return NewDebianHandler(p, "http://proxy.test", "", nil).Routes() }}, } { t.Run(tc.ecosystem, func(t *testing.T) { p, db, store, _ := setupTestProxy(t)