denozord/telemt-api
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Improve URL assertion logic in reverse proxy tests. Updated assertSameURL function in reverse_test.go to handle nil URLs and enhance comparison logic, ensuring more accurate URL mismatch reporting and better test reliability.
Publish telemt-api gateway Docker image / test (push) Successful in 29s
Details
Publish telemt-api gateway Docker image / build-and-push (push) Successful in 1m25s
Details

Browse Source
This commit is contained in:
Denozordec
2026-03-30 00:36:50 +07:00
parent 0dfe14b347
commit 89638d29bb
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/proxy/reverse_test.go
+10 -2
View File
@@ -133,7 +133,15 @@ func TestBuildUpstreamURLRoundTrip(t *testing.T) {
func assertSameURL(t *testing.T, got, want *url.URL) {
t.Helper()
if got.Scheme != want.Scheme || got.Host != want.Host || got.Path != want.Path || got.RawQuery != want.RawQuery {
t.Fatalf("URL mismatch\ngot %q\nwant %q", got.String(), want.String())
if got == nil || want == nil {
t.Fatalf("nil URL: got=%v want=%v", got, want)
}
// JoinPath vs url.Parse can differ in Path vs RawPath while String() is identical.
if got.String() == want.String() {
return
}
if got.Scheme == want.Scheme && got.Host == want.Host && got.RawQuery == want.RawQuery && got.EscapedPath() == want.EscapedPath() {
return
}
t.Fatalf("URL mismatch\ngot %q\nwant %q", got.String(), want.String())
}