delete test cases with genargs

Author: bung87

src/sqlbuilderpkg/delete.nim

@@ -30,14 +30,7 @@ proc sqlDelete*(table: string, where: varargs[string], args: ArgsContainer.query
 
   var res = "DELETE FROM " & table
   if where.len > 0:
-    res.add " WHERE "
-    for i, d in where:
-      if i > 0:
-        res.add(" AND ")
-      if args[i].isNull:
-        res.add(d & " = NULL")
-      else:
-        res.add(d & " = ?")
+    res.add(sqlWhere(where))
   result = sql(res)
 
 

tests/delete/test_delete.nim

@@ -67,4 +67,32 @@ suite "delete - macro":
     check querycompare(test, sql("DELETE FROM my-table WHERE name IS NULL AND age = ?"))
 
 
+suite "delete - genArgs":
+
+  test "sqlDelete with genArgs":
+
+    var a = genArgs("123", dbNullVal)
+
+    var test = sqlDelete("tasksQQ", ["id =", "status IS"], a.query)
+
+    check querycompare(test, sql("""DELETE FROM tasksQQ WHERE id = ? AND status IS ?"""))
+
+
+
+    a = genArgs("123", dbNullVal, dbNullVal)
+
+    test = sqlDelete("tasksQQ", ["id =", "status IS NOT", "phase IS"], a.query)
+
+    check querycompare(test, sql("""DELETE FROM tasksQQ WHERE id = ? AND status IS NOT ? AND phase IS ?"""))
+
+
+
+  test "sqlDelete with genArgsSetNull":
+
+    var a = genArgsSetNull("123", "", "")
+
+    var test = sqlDelete("tasksQQ",  ["id =", "status IS NOT", "phase IS"],  a.query)
+
+    check querycompare(test, sql("""DELETE FROM tasksQQ WHERE id = ? AND status IS NOT ? AND phase IS ?"""))
+