fix(ci): allow more http codes in tests (#473)

Crow-Control · web-flow · commit 202cf8a5130f · 2025-11-22T12:30:18.000+01:00
diff --git a/apps/balfolk-ics/container_test.go b/apps/balfolk-ics/container_test.go
@@ -26,7 +26,7 @@ func Test(t *testing.T) {
 		testcontainers.WithExposedPorts("8000/tcp"),
 		testcontainers.WithWaitStrategy(
 			wait.ForHTTP("/balfolk.ics").WithPort("8000/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/bazarr/container_test.go b/apps/bazarr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("6767/tcp"),
 			wait.ForHTTP("/").WithPort("6767/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/caddy/container_test.go b/apps/caddy/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8080/tcp"),
 			wait.ForHTTP("/").WithPort("8080/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/code-server/container_test.go b/apps/code-server/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8443/tcp"),
 			// wait.ForHTTP("'").WithPort("8443/tcp").WithStatusCodeMatcher(func(status int) bool {
-			// 	return status == 200
+			// 	return status >= 200 && status < 400
 			// }),
 		),
 	)
diff --git a/apps/duplicacy/container_test.go b/apps/duplicacy/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("3875/tcp"),
 			wait.ForHTTP("/").WithPort("3875/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/duplicati/container_test.go b/apps/duplicati/container_test.go
@@ -30,7 +30,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8200/tcp"),
 			wait.ForHTTP("/signin.html").WithPort("8200/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/emby/container_test.go b/apps/emby/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8096/tcp"),
 			wait.ForHTTP("/").WithPort("8096/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/esphome/container_test.go b/apps/esphome/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("6052/tcp"),
 			wait.ForHTTP("/").WithPort("6052/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/faster-whisper/container_test.go b/apps/faster-whisper/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("10300/tcp"),
 			// wait.ForHTTP("'").WithPort("/10300tcp").WithStatusCodeMatcher(func(status int) bool {
-			// 	return status == 200
+			// 	return status >= 200 && status < 400
 			// }),
 		),
 	)
diff --git a/apps/foldingathome/container_test.go b/apps/foldingathome/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("7396/tcp"),
 			wait.ForHTTP("'").WithPort("7396/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/home-assistant/container_test.go b/apps/home-assistant/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8123/tcp"),
 			wait.ForHTTP("/").WithPort("8123/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/it-tools/container_test.go b/apps/it-tools/container_test.go
@@ -26,7 +26,7 @@ func Test(t *testing.T) {
 		testcontainers.WithExposedPorts("8080/tcp"), // replace with actual port if different
 		testcontainers.WithWaitStrategy(
 			wait.ForHTTP("/").WithPort("8080/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/jellyfin/container_test.go b/apps/jellyfin/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8096/tcp"),
 			// wait.ForHTTP("'").WithPort("8096/tcp").WithStatusCodeMatcher(func(status int) bool {
-			// 	return status == 200
+			// 	return status >= 200 && status < 400
 			// }),
 		),
 	)
diff --git a/apps/jellyseer/container_test.go b/apps/jellyseer/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("5055/tcp"),
 			wait.ForHTTP("/").WithPort("5055/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/kavita/container_test.go b/apps/kavita/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("5000/tcp"),
 			wait.ForHTTP("'").WithPort("5000/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/lidarr/container_test.go b/apps/lidarr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8686/tcp"),
 			wait.ForHTTP("/").WithPort("8686/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/nginx/container_test.go b/apps/nginx/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8080/tcp"),
 			wait.ForHTTP("/").WithPort("8080/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/nzbget/container_test.go b/apps/nzbget/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("6789/tcp"),
 			wait.ForHTTP("/").WithPort("6789/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/nzbhydra2/container_test.go b/apps/nzbhydra2/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("5076/tcp"),
 			wait.ForHTTP("/").WithPort("5076/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/ombi/container_test.go b/apps/ombi/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("3579/tcp"),
 			wait.ForHTTP("'").WithPort("3579/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/openvscode-server/container_test.go b/apps/openvscode-server/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("3000/tcp"),
 			wait.ForHTTP("/?tkn").WithPort("3000/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/overseerr/container_test.go b/apps/overseerr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("5055/tcp"),
 			wait.ForHTTP("/").WithPort("5055/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/piper/container_test.go b/apps/piper/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("10200/tcp"),
 			// wait.ForHTTP("'").WithPort("10200/tcp").WithStatusCodeMatcher(func(status int) bool {
-			// 	return status == 200
+			// 	return status >= 200 && status < 400
 			// }),
 		),
 	)
diff --git a/apps/plex/container_test.go b/apps/plex/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("32400/tcp"),
 			wait.ForHTTP("/web/index.html").WithPort("32400/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/prowlarr/container_test.go b/apps/prowlarr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("9696/tcp"),
 			wait.ForHTTP("/").WithPort("9696/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/pyload-ng/container_test.go b/apps/pyload-ng/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8000/tcp"),
 			wait.ForHTTP("/").WithPort("8000/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/qbittorrent/container_test.go b/apps/qbittorrent/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8080/tcp"),
 			wait.ForHTTP("/").WithPort("8080/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/qui/container_test.go b/apps/qui/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("7476/tcp"),
 			wait.ForHTTP("/").WithPort("7476/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/radarr/container_test.go b/apps/radarr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("7878/tcp"),
 			wait.ForHTTP("/").WithPort("7878/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/readarr/container_test.go b/apps/readarr/container_test.go
@@ -26,7 +26,7 @@ func Test(t *testing.T) {
 		testcontainers.WithExposedPorts("8787/tcp"),
 		testcontainers.WithWaitStrategy(
 			wait.ForHTTP("/").WithPort("8787/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/requestrr/container_test.go b/apps/requestrr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("4545/tcp"),
 			wait.ForHTTP("/").WithPort("4545/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/resilio-sync/container_test.go b/apps/resilio-sync/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8888/tcp"),
 			// wait.ForHTTP("/gui'").WithPort("8888/tcp").WithStatusCodeMatcher(func(status int) bool {
-			// 	return status == 200
+			// 	return status >= 200 && status < 400
 			// }),
 		),
 	)
diff --git a/apps/sabnzbd/container_test.go b/apps/sabnzbd/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8080/tcp"),
 			wait.ForHTTP("/sabnzbd").WithPort("8080/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/seerr/container_test.go b/apps/seerr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("5055/tcp"),
 			wait.ForHTTP("/").WithPort("5055/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/smartctl-exporter/container_test.go b/apps/smartctl-exporter/container_test.go
@@ -26,7 +26,7 @@ func Test(t *testing.T) {
 		testcontainers.WithExposedPorts("9633/tcp"),
 		testcontainers.WithWaitStrategy(
 			wait.ForHTTP("/metrics").WithPort("9633/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/sonarr/container_test.go b/apps/sonarr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8989/tcp"),
 			wait.ForHTTP("/").WithPort("8989/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/stash/container_test.go b/apps/stash/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("9999/tcp"),
 			wait.ForHTTP("/").WithPort("9999/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/tautulli/container_test.go b/apps/tautulli/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8181/tcp"),
 			wait.ForHTTP("/").WithPort("8181/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/theme-park/container_test.go b/apps/theme-park/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("8080/tcp"),
 			wait.ForHTTP("/").WithPort("8080/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/webhook/container_test.go b/apps/webhook/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("9000/tcp"),
 			wait.ForHTTP("/").WithPort("9000/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/whisparr/container_test.go b/apps/whisparr/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("6969/tcp"),
 			wait.ForHTTP("/").WithPort("6969/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)
diff --git a/apps/wud/container_test.go b/apps/wud/container_test.go
@@ -27,7 +27,7 @@ func Test(t *testing.T) {
 		testcontainers.WithWaitStrategy(
 			wait.ForListeningPort("3000/tcp"),
 			wait.ForHTTP("/").WithPort("3000/tcp").WithStatusCodeMatcher(func(status int) bool {
-				return status == 200
+				return status >= 200 && status < 400
 			}),
 		),
 	)

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ func Test(t *testing.T) {`
`26`	`26`	`testcontainers.WithExposedPorts("8000/tcp"),`
`27`	`27`	`testcontainers.WithWaitStrategy(`
`28`	`28`	`wait.ForHTTP("/balfolk.ics").WithPort("8000/tcp").WithStatusCodeMatcher(func(status int) bool {`
`29`		`- return status == 200`
	`29`	`+ return status >= 200 && status < 400`
`30`	`30`	`}),`
`31`	`31`	`),`
`32`	`32`	`)`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ func Test(t *testing.T) {`
`27`	`27`	`testcontainers.WithWaitStrategy(`
`28`	`28`	`wait.ForListeningPort("6767/tcp"),`
`29`	`29`	`wait.ForHTTP("/").WithPort("6767/tcp").WithStatusCodeMatcher(func(status int) bool {`
`30`		`- return status == 200`
	`30`	`+ return status >= 200 && status < 400`
`31`	`31`	`}),`
`32`	`32`	`),`
`33`	`33`	`)`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ func Test(t *testing.T) {`
`30`	`30`	`testcontainers.WithWaitStrategy(`
`31`	`31`	`wait.ForListeningPort("8200/tcp"),`
`32`	`32`	`wait.ForHTTP("/signin.html").WithPort("8200/tcp").WithStatusCodeMatcher(func(status int) bool {`
`33`		`- return status == 200`
	`33`	`+ return status >= 200 && status < 400`
`34`	`34`	`}),`
`35`	`35`	`),`
`36`	`36`	`)`