Merge pull request #52 from metabase/fix-get-inserted-id-with-no-resu…

…lts-returned

Fix get-inserted-id if no results are returned

src/toucan/db.clj

@@ -525,7 +525,8 @@
 (defn get-inserted-id
   "Get the ID of a row inserted by `jdbc/db-do-prepared-return-keys`."
   [insert-result]
-  (some insert-result inserted-id-keys))
+  (when insert-result
+    (some insert-result inserted-id-keys)))
 
 (defn simple-insert-many!
   "Do a simple JDBC `insert!` of multiple objects into the database.

test/toucan/db_test.clj

@@ -301,6 +301,11 @@
  (test/with-clean-db
    (db/insert! User {:first-name "Grass" :last-name #sql/call [:upper "Hopper"]})))
 
+;; get-inserted-id shouldn't fail if nothing is returned for some reason
+(expect
+ nil
+ (db/get-inserted-id nil))
+
 ;; Test select-one
 (expect
   #toucan.test_models.user.UserInstance{:id 1, :first-name "Cam", :last-name "Saul"}