comment out unneeded test that returns different status in dev / prod (add todo)

This commit is contained in:
henrygd
2026-04-05 16:14:28 -04:00
parent a68e02ca84
commit 5ab82183fa

View File

@@ -630,13 +630,17 @@ func TestApiRoutesAuthentication(t *testing.T) {
"systems": []string{system.Id}, "systems": []string{system.Id},
}), }),
}, },
{ // this works but diff behavior on prod vs dev.
Name: "GET /update - shouldn't exist without CHECK_UPDATES env var", // dev returns 502; prod returns 200 with static html page 404
Method: http.MethodGet, // TODO: align dev and prod behavior and re-enable this test
URL: "/api/beszel/update", // {
ExpectedStatus: 502, // Name: "GET /update - shouldn't exist without CHECK_UPDATES env var",
TestAppFactory: testAppFactory, // Method: http.MethodGet,
}, // URL: "/api/beszel/update",
// NotExpectedContent: []string{"v:", "\"v\":"},
// ExpectedStatus: 502,
// TestAppFactory: testAppFactory,
// },
} }
for _, scenario := range scenarios { for _, scenario := range scenarios {