add IsZero to Time

Author: guregu

time.go

@@ -141,3 +141,9 @@ func (t Time) Ptr() *time.Time {
 	}
 	return &t.Time
 }
+
+// IsZero returns true for invalid Times, hopefully for future omitempty support.
+// A non-null Time with a zero value will not be considered zero.
+func (t Time) IsZero() bool {
+	return !t.Valid
+}

time_test.go

@@ -172,6 +172,23 @@ func TestTimeValueOrZero(t *testing.T) {
 	}
 }
 
+func TestTimeIsZero(t *testing.T) {
+	str := TimeFrom(timeValue)
+	if str.IsZero() {
+		t.Errorf("IsZero() should be false")
+	}
+
+	zero := TimeFrom(time.Time{})
+	if zero.IsZero() {
+		t.Errorf("IsZero() should be false")
+	}
+
+	null := TimeFromPtr(nil)
+	if !null.IsZero() {
+		t.Errorf("IsZero() should be true")
+	}
+}
+
 func assertTime(t *testing.T, ti Time, from string) {
 	if ti.Time != timeValue {
 		t.Errorf("bad %v time: %v ≠ %v\n", from, ti.Time, timeValue)

zero/time.go

@@ -143,3 +143,8 @@ func (t Time) Ptr() *time.Time {
 	}
 	return &t.Time
 }
+
+// IsZero returns true for null or zero Times, for potential future omitempty support.
+func (t Time) IsZero() bool {
+	return !t.Valid || t.Time.IsZero()
+}

zero/time_test.go

@@ -193,6 +193,23 @@ func TestTimeValue(t *testing.T) {
 	}
 }
 
+func TestTimeIsZero(t *testing.T) {
+	str := TimeFrom(timeValue)
+	if str.IsZero() {
+		t.Errorf("IsZero() should be false")
+	}
+
+	zero := TimeFrom(time.Time{})
+	if !zero.IsZero() {
+		t.Errorf("IsZero() should be true")
+	}
+
+	null := TimeFromPtr(nil)
+	if !null.IsZero() {
+		t.Errorf("IsZero() should be true")
+	}
+}
+
 func assertTime(t *testing.T, ti Time, from string) {
 	if ti.Time != timeValue {
 		t.Errorf("bad %v time: %v ≠ %v\n", from, ti.Time, timeValue)