From da79fdfe51b5f755bed5119740d11716b404763f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Cam=20Sa=C3=BCl?= <cammsaul@gmail.com> Date: Wed, 13 Jul 2016 16:46:54 -0700 Subject: [PATCH] Test fix :wrench: --- test/metabase/api/session_test.clj | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/test/metabase/api/session_test.clj b/test/metabase/api/session_test.clj index 6ee4336b4c4..722ede3721a 100644 --- a/test/metabase/api/session_test.clj +++ b/test/metabase/api/session_test.clj @@ -211,7 +211,8 @@ ;; should totally work if the email domains match up (expect {:first_name "Rasta", :last_name "Toucan", :email "rasta@sf-toucannery.com"} - (with-temporary-setting-values [google-auth-auto-create-accounts-domain "sf-toucannery.com"] + (with-temporary-setting-values [google-auth-auto-create-accounts-domain "sf-toucannery.com" + admin-email "rasta@toucans.com"] (select-keys (u/prog1 (google-auth-create-new-user! "Rasta" "Toucan" "rasta@sf-toucannery.com") (db/cascade-delete! User :id (:id <>))) ; make sure we clean up after ourselves ! [:first_name :last_name :email]))) @@ -234,7 +235,8 @@ ;; test that a user that doesn't exist with a *different* domain than the auto-create accounts domain gets an exception (expect clojure.lang.ExceptionInfo - (with-temporary-setting-values [google-auth-auto-create-accounts-domain nil] + (with-temporary-setting-values [google-auth-auto-create-accounts-domain nil + admin-email "rasta@toucans.com"] (google-auth-fetch-or-create-user! "Rasta" "Can" "rasta@sf-toucannery.com"))) ;; test that a user that doesn't exist with the *same* domain as the auto-create accounts domain means a new user gets created -- GitLab