aboutsummaryrefslogtreecommitdiff
path: root/server/auth/google
diff options
context:
space:
mode:
authorNiall Sheridan <nsheridan@gmail.com>2017-01-15 21:50:38 +0000
committerNiall Sheridan <nsheridan@gmail.com>2017-01-15 22:43:41 +0000
commit17b17fc8bb690d1f6344e5af1c62b3b37166bc48 (patch)
tree8e50f8ea3f5a38002447021e7fdcacfa48c4c9cd /server/auth/google
parentd3d2d5384cbb53c91c321c552cc56b42c90437fa (diff)
Add more context to errors
Diffstat (limited to 'server/auth/google')
-rw-r--r--server/auth/google/google.go2
-rw-r--r--server/auth/google/google_test.go7
2 files changed, 4 insertions, 5 deletions
diff --git a/server/auth/google/google.go b/server/auth/google/google.go
index 08a4083..643ecfe 100644
--- a/server/auth/google/google.go
+++ b/server/auth/google/google.go
@@ -34,7 +34,7 @@ func New(c *config.Auth) (auth.Provider, error) {
uw[u] = true
}
if c.ProviderOpts["domain"] == "" && len(uw) == 0 {
- return nil, errors.New("google_opts domain and the users whitelist must not be both empty")
+ return nil, errors.New("either Google Apps domain or users whitelist must be specified")
}
return &Config{
diff --git a/server/auth/google/google_test.go b/server/auth/google/google_test.go
index b80c4bf..781cf6f 100644
--- a/server/auth/google/google_test.go
+++ b/server/auth/google/google_test.go
@@ -28,12 +28,11 @@ func TestNew(t *testing.T) {
}
func TestNewWithoutDomain(t *testing.T) {
- a := assert.New(t)
-
domain = ""
- _, err := newGoogle()
- a.EqualError(err, "google_opts domain and the users whitelist must not be both empty")
+ if _, err := newGoogle(); err == nil {
+ t.Error("creating a provider without a domain set should return an error")
+ }
domain = "example.com"
}