Merge pull request #2970 from metabase/fix-ci-test-failure

Fix CI Test failures

src/metabase/email.clj

@@ -43,7 +43,8 @@
   [& {:keys [subject recipients message-type message]}]
   {:pre [(string? subject)
          (sequential? recipients)
-         (every? u/is-email? recipients)
+         (or (every? u/is-email? recipients)
+             (log/error "recipients contains an invalid email:" recipients))
          (contains? #{:text :html :attachments} message-type)
          (if (= message-type :attachments) (sequential? message) (string? message))]}
   (try

test/metabase/api/session_test.clj

@@ -241,6 +241,7 @@
 
 ;; test that a user that doesn't exist with the *same* domain as the auto-create accounts domain means a new user gets created
 (expect
-  (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"]
     (u/prog1 (is-session? (google-auth-fetch-or-create-user! "Rasta" "Toucan" "rasta@sf-toucannery.com"))
       (db/cascade-delete! User :email "rasta@sf-toucannery.com"))))                                       ; clean up after ourselves