From 2ba1fc0061d027ff34548e8a260df61b6cd4941f Mon Sep 17 00:00:00 2001 From: Sven van Ginkel Date: Mon, 19 Oct 2026 18:19:19 +0200 Subject: [PATCH] test(hub): re-enable /update route test (#2651) Authenticate the request and expect 404 so the test checks that the route is not registered without CHECK_UPDATES. Tests run with the no_ui tag, so the status is stable. --- internal/hub/api_test.go | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/internal/hub/api_test.go b/internal/hub/api_test.go index f3e869a88..5ef4237d9 100644 --- a/internal/hub/api_test.go +++ b/internal/hub/api_test.go @@ -796,17 +796,17 @@ func TestApiRoutesAuthentication(t *testing.T) { "systems": []string{system.Id}, }), }, - // this works but diff behavior on prod vs dev. - // dev returns 502; prod returns 200 with static html page 404 - // TODO: align dev and prod behavior and re-enable this test - // { - // Name: "GET /update - shouldn't exist without CHECK_UPDATES env var", - // Method: http.MethodGet, - // URL: "/api/beszel/update", - // NotExpectedContent: []string{"v:", "\"v\":"}, - // ExpectedStatus: 502, - // TestAppFactory: testAppFactory, - // }, + { + Name: "GET /update - shouldn't exist without CHECK_UPDATES env var", + Method: http.MethodGet, + URL: "/api/beszel/update", + Headers: map[string]string{ + "Authorization": userToken, + }, + NotExpectedContent: []string{"v:", "\"v\":"}, + ExpectedStatus: 404, + TestAppFactory: testAppFactory, + }, } for _, scenario := range scenarios {