Fix OAuth redirect URL test

sreya · sreya · commit ef5fc1f10401 · 2024-09-10T01:07:15.000Z
Update the OAuth redirect URL test to exclude fragment

Previously, the test included a fragment in the constructed URL.
Removing the fragment ensures consistency with the actual URL format
used in the application.
diff --git a/coderd/httpmw/oauth2_test.go b/coderd/httpmw/oauth2_test.go
@@ -77,9 +77,8 @@ func TestOAuth2(t *testing.T) {
 			Host:     "some.bad.domain.com",
 			Path:     "/sadf/asdfasdf",
 			RawQuery: "foo=hello&bar=world",
-			Fragment: "my_section",
 		}
-		expectedValue := uri.Path + "?" + uri.RawQuery + "#" + uri.Fragment
+		expectedValue := uri.Path + "?" + uri.RawQuery
 		req := httptest.NewRequest("GET", "/?redirect="+url.QueryEscape(uri.String()), nil)
 		res := httptest.NewRecorder()
 		tp := newTestOAuth2Provider(t, oauth2.AccessTypeOffline)
