diff --git a/Makefile b/Makefile
index 727e2d2e7e49d7c806c912898bb2a83f053d57bd..7e6418b328ff3302460bba1af7179e6d59250102 100644
--- a/Makefile
+++ b/Makefile
@@ -54,7 +54,7 @@ fmt:
 
 lint:
 	@for package in $(shell go list ./... | grep -v '/vendor/' | grep -v '/api'); do \
-      golint -set_exit_status $$package; \
+      golint -set_exit_status $$package $$i || exit 1; \
 	done
 
 server/templates_default.go: $(wildcard web/templates/**)
diff --git a/server/handlers_test.go b/server/handlers_test.go
index d9f91888380aef9910c0633d3974cc16b92e9e2d..233af2795c3442cf95a665d5d8d58f067969fbee 100644
--- a/server/handlers_test.go
+++ b/server/handlers_test.go
@@ -12,7 +12,7 @@ func TestHandleHealth(t *testing.T) {
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 
-	httpServer, server := newTestServer(t, ctx, nil)
+	httpServer, server := newTestServer(ctx, t, nil)
 	defer httpServer.Close()
 
 	rr := httptest.NewRecorder()
diff --git a/server/server_test.go b/server/server_test.go
index 62fe38e61d09b94e563243414a90e2e9ef8928b7..009a8dc2feef5b9c7e92ebd1f95fe68c913da31b 100644
--- a/server/server_test.go
+++ b/server/server_test.go
@@ -69,7 +69,7 @@ FDWV28nTP9sqbtsmU8Tem2jzMvZ7C/Q0AuDoKELFUpux8shm8wfIhyaPnXUGZoAZ
 Np4vUwMSYV5mopESLWOg3loBxKyLGFtgGKVCjGiQvy6zISQ4fQo=
 -----END RSA PRIVATE KEY-----`)
 
-func newTestServer(t *testing.T, ctx context.Context, updateConfig func(c *Config)) (*httptest.Server, *Server) {
+func newTestServer(ctx context.Context, t *testing.T, updateConfig func(c *Config)) (*httptest.Server, *Server) {
 	var server *Server
 	s := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		server.ServeHTTP(w, r)
@@ -101,14 +101,14 @@ func newTestServer(t *testing.T, ctx context.Context, updateConfig func(c *Confi
 func TestNewTestServer(t *testing.T) {
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
-	newTestServer(t, ctx, nil)
+	newTestServer(ctx, t, nil)
 }
 
 func TestDiscovery(t *testing.T) {
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 
-	httpServer, _ := newTestServer(t, ctx, func(c *Config) {
+	httpServer, _ := newTestServer(ctx, t, func(c *Config) {
 		c.Issuer = c.Issuer + "/non-root-path"
 	})
 	defer httpServer.Close()
@@ -257,7 +257,7 @@ func TestOAuth2CodeFlow(t *testing.T) {
 			ctx, cancel := context.WithCancel(context.Background())
 			defer cancel()
 
-			httpServer, s := newTestServer(t, ctx, func(c *Config) {
+			httpServer, s := newTestServer(ctx, t, func(c *Config) {
 				c.Issuer = c.Issuer + "/non-root-path"
 			})
 			defer httpServer.Close()
@@ -370,7 +370,7 @@ func TestOAuth2ImplicitFlow(t *testing.T) {
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 
-	httpServer, s := newTestServer(t, ctx, func(c *Config) {
+	httpServer, s := newTestServer(ctx, t, func(c *Config) {
 		// Enable support for the implicit flow.
 		c.SupportedResponseTypes = []string{"code", "token"}
 	})
@@ -500,7 +500,7 @@ func TestCrossClientScopes(t *testing.T) {
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 
-	httpServer, s := newTestServer(t, ctx, func(c *Config) {
+	httpServer, s := newTestServer(ctx, t, func(c *Config) {
 		c.Issuer = c.Issuer + "/non-root-path"
 	})
 	defer httpServer.Close()