diff options
author | cpanato <ctadeu@gmail.com> | 2018-06-14 18:46:19 +0200 |
---|---|---|
committer | cpanato <ctadeu@gmail.com> | 2018-06-14 18:46:19 +0200 |
commit | 8883f487886f8ad6b1529372fa467aaa7aeed5ec (patch) | |
tree | 1e967b0621b1130cadc08cb47f63658f193f0932 /web/handlers_test.go | |
parent | f106417103b036e8c349531f25487e526252d084 (diff) | |
parent | 695c5d6bf82f5a5c58aa0a22b4911439f08a80fb (diff) | |
download | chat-8883f487886f8ad6b1529372fa467aaa7aeed5ec.tar.gz chat-8883f487886f8ad6b1529372fa467aaa7aeed5ec.tar.bz2 chat-8883f487886f8ad6b1529372fa467aaa7aeed5ec.zip |
Merge remote-tracking branch 'upstream/release-5.0' into release-5.0-merge-to-master-20180614-1
Diffstat (limited to 'web/handlers_test.go')
-rw-r--r-- | web/handlers_test.go | 58 |
1 files changed, 58 insertions, 0 deletions
diff --git a/web/handlers_test.go b/web/handlers_test.go new file mode 100644 index 000000000..b4c89e50f --- /dev/null +++ b/web/handlers_test.go @@ -0,0 +1,58 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + +package web + +import ( + "net/http" + "net/http/httptest" + "testing" + + "github.com/mattermost/mattermost-server/app" + "github.com/mattermost/mattermost-server/model" + "github.com/stretchr/testify/assert" +) + +func handlerForTest(c *Context, w http.ResponseWriter, r *http.Request) { + c.Err = model.NewAppError("loginWithSaml", "api.user.saml.not_available.app_error", nil, "", http.StatusFound) +} + +func TestHandlerServeHTTPErrors(t *testing.T) { + a, err := app.New(app.StoreOverride(testStore), app.DisableConfigWatch) + defer a.Shutdown() + + web := NewWeb(a, a.Srv.Router) + if err != nil { + panic(err) + } + handler := web.NewHandler(handlerForTest) + + var flagtests = []struct { + name string + url string + mobile bool + redirect bool + }{ + {"redirect on destkop non-api endpoint", "/login/sso/saml", false, true}, + {"not redirect on destkop api endpoint", "/api/v4/test", false, false}, + {"not redirect on mobile non-api endpoint", "/login/sso/saml", true, false}, + {"not redirect on mobile api endpoint", "/api/v4/test", true, false}, + } + + for _, tt := range flagtests { + t.Run(tt.name, func(t *testing.T) { + request := httptest.NewRequest("GET", tt.url, nil) + if tt.mobile { + request.Header.Add("X-Mobile-App", "mattermost") + } + response := httptest.NewRecorder() + handler.ServeHTTP(response, request) + + if tt.redirect { + assert.Contains(t, response.Body.String(), "/error?message=") + } else { + assert.NotContains(t, response.Body.String(), "/error?message=") + } + }) + } +} |