Merge pull request #2142 from ulfryk/fix/uninformative-null-error-simple

Make null param error more informative (#809)

modules/core/src/main/scala/doobie/util/put.scala

@@ -56,7 +56,7 @@ sealed abstract class Put[A](
     ) {}
 
   def unsafeSetNonNullable(ps: PreparedStatement, n: Int, a: A): Unit =
-    if (a == null) sys.error("oops, null")
+    if (a == null) sys.error(s"Expected non-nullable param at $n. Use Option to describe nullable values.")
     else put.fi.apply(ps, n, (put.k(a)))
 
   def unsafeSetNullable(ps: PreparedStatement, n: Int, oa: Option[A]): Unit =
@@ -66,7 +66,7 @@ sealed abstract class Put[A](
     }
 
   def unsafeUpdateNonNullable(rs: ResultSet, n: Int, a: A): Unit =
-    if (a == null) sys.error("oops, null")
+    if (a == null) sys.error(s"Expected non-nullable param at $n. Use Option to describe nullable values.")
     else update.fi.apply(rs, n, (update.k(a)))
 
   def unsafeUpdateNullable(rs: ResultSet, n: Int, oa: Option[A]): Unit =

modules/core/src/test/scala/doobie/util/WriteSuite.scala

@@ -122,4 +122,27 @@ class WriteSuite extends munit.FunSuite with WriteSuitePlatform {
       .unsafeRunSync()
   }
 
+  test("Write should yield correct error when Some(null) inserted") {
+    interceptMessage[RuntimeException]("Expected non-nullable param at 2. Use Option to describe nullable values.") {
+      testNullPut(("a", Some(null)))
+    }
+  }
+
+  test("Write should yield correct error when null inserted into non-nullable field") {
+    interceptMessage[RuntimeException]("Expected non-nullable param at 1. Use Option to describe nullable values.") {
+      testNullPut((null, Some("b")))
+    }
+  }
+
+  private def testNullPut(input: (String, Option[String])): Int = {
+    import doobie.implicits.*
+
+    (for {
+      _ <- sql"create temp table t0 (a text, b text null)".update.run
+      n <- Update[(String, Option[String])]("insert into t0 (a, b) values (?, ?)").run(input)
+    } yield n)
+      .transact(xa)
+      .unsafeRunSync()
+  }
+
 }