diff options
author | feistel <6742251-feistel@users.noreply.gitlab.com> | 2021-07-25 16:24:03 +0300 |
---|---|---|
committer | feistel <6742251-feistel@users.noreply.gitlab.com> | 2021-09-09 14:06:29 +0300 |
commit | d08da349a24cb0ec322eae287fcc27ce8730bba9 (patch) | |
tree | 0b64732f5c98725a9ab60e8e7e118d455877c788 /internal/auth | |
parent | 6b9b03db88911c9939d5fc1d92d4b15912cc7665 (diff) |
test: update source mock to use mockgen
Diffstat (limited to 'internal/auth')
-rw-r--r-- | internal/auth/auth_test.go | 40 |
1 files changed, 33 insertions, 7 deletions
diff --git a/internal/auth/auth_test.go b/internal/auth/auth_test.go index 77ea0b7a..ff95183c 100644 --- a/internal/auth/auth_test.go +++ b/internal/auth/auth_test.go @@ -10,11 +10,12 @@ import ( "strings" "testing" + "github.com/golang/mock/gomock" "github.com/gorilla/sessions" "github.com/stretchr/testify/require" + "gitlab.com/gitlab-org/gitlab-pages/internal/mocks" "gitlab.com/gitlab-org/gitlab-pages/internal/request" - "gitlab.com/gitlab-org/gitlab-pages/internal/source" ) func createTestAuth(t *testing.T, internalServer string, publicServer string) *Auth { @@ -82,7 +83,11 @@ func TestTryAuthenticate(t *testing.T) { reqURL.Scheme = request.SchemeHTTPS r := &http.Request{URL: reqURL} - require.Equal(t, false, auth.TryAuthenticate(result, r, source.NewMockSource())) + mockCtrl := gomock.NewController(t) + defer mockCtrl.Finish() + + mockSource := mocks.NewMockSource(mockCtrl) + require.Equal(t, false, auth.TryAuthenticate(result, r, mockSource)) } func TestTryAuthenticateWithError(t *testing.T) { @@ -95,7 +100,11 @@ func TestTryAuthenticateWithError(t *testing.T) { reqURL.Scheme = request.SchemeHTTPS r := &http.Request{URL: reqURL} - require.Equal(t, true, auth.TryAuthenticate(result, r, source.NewMockSource())) + mockCtrl := gomock.NewController(t) + defer mockCtrl.Finish() + + mockSource := mocks.NewMockSource(mockCtrl) + require.Equal(t, true, auth.TryAuthenticate(result, r, mockSource)) require.Equal(t, http.StatusUnauthorized, result.Code) } @@ -114,7 +123,11 @@ func TestTryAuthenticateWithCodeButInvalidState(t *testing.T) { session.Values["state"] = "state" session.Save(r, result) - require.Equal(t, true, auth.TryAuthenticate(result, r, source.NewMockSource())) + mockCtrl := gomock.NewController(t) + defer mockCtrl.Finish() + + mockSource := mocks.NewMockSource(mockCtrl) + require.Equal(t, true, auth.TryAuthenticate(result, r, mockSource)) require.Equal(t, http.StatusUnauthorized, result.Code) } @@ -136,7 +149,11 @@ func TestTryAuthenticateRemoveTokenFromRedirect(t *testing.T) { session.Values["proxy_auth_domain"] = "https://domain.com" session.Save(r, result) - require.Equal(t, true, auth.TryAuthenticate(result, r, source.NewMockSource())) + mockCtrl := gomock.NewController(t) + defer mockCtrl.Finish() + + mockSource := mocks.NewMockSource(mockCtrl) + require.Equal(t, true, auth.TryAuthenticate(result, r, mockSource)) require.Equal(t, http.StatusFound, result.Code) redirect, err := url.Parse(result.Header().Get("Location")) @@ -152,7 +169,11 @@ func TestTryAuthenticateWithDomainAndState(t *testing.T) { require.NoError(t, err) r := &http.Request{URL: reqURL} - require.Equal(t, true, auth.TryAuthenticate(result, r, source.NewMockSource())) + mockCtrl := gomock.NewController(t) + defer mockCtrl.Finish() + + mockSource := mocks.NewMockSource(mockCtrl) + require.Equal(t, true, auth.TryAuthenticate(result, r, mockSource)) require.Equal(t, http.StatusFound, result.Code) redirect, err := url.Parse(result.Header().Get("Location")) require.NoError(t, err) @@ -208,7 +229,12 @@ func testTryAuthenticateWithCodeAndState(t *testing.T, https bool) { }) result := httptest.NewRecorder() - require.Equal(t, true, auth.TryAuthenticate(result, r, source.NewMockSource())) + + mockCtrl := gomock.NewController(t) + defer mockCtrl.Finish() + + mockSource := mocks.NewMockSource(mockCtrl) + require.Equal(t, true, auth.TryAuthenticate(result, r, mockSource)) res := result.Result() defer res.Body.Close() |