Merge pull request #1559 from KimMachineGun/timepointer

fix format function to handle pointer of time.Time

Author: SpencerTorres

bind.go

@@ -288,6 +288,11 @@ func format(tz *time.Location, scale TimeUnit, v any) (string, error) {
 		return quote(v), nil
 	case time.Time:
 		return formatTime(tz, scale, v)
+	case *time.Time:
+		if v == nil {
+			return "NULL", nil
+		}
+		return formatTime(tz, scale, *v)
 	case bool:
 		if v {
 			return "1", nil

bind_test.go

@@ -254,7 +254,15 @@ func TestFormatTime(t *testing.T) {
 			val, _ = format(tz, Seconds, t1)
 			assert.Equal(t, "toDateTime('2022-01-12 15:00:00', 'UTC')", val)
 		}
+
+		// test with pointer to time.Time
+		val, _ = format(t1.Location(), Seconds, &t1)
+		assert.Equal(t, "toDateTime('2022-01-12 15:00:00')", val)
 	}
+
+	// test with nil pointer to time.Time
+	val, _ := format(time.UTC, Seconds, (*time.Time)(nil))
+	assert.Equal(t, "NULL", val)
 }
 
 func TestFormatScaledTime(t *testing.T) {