MicroTime/Time: remove pointer receivers and redundant IsZero() method

Co-authored-by: David Timm <dtimm@pivotal.io>

staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/micro_time.go

@@ -57,50 +57,24 @@ func NowMicro() MicroTime {
 	return MicroTime{time.Now()}
 }
 
-// IsZero returns true if the value is nil or time is zero.
-func (t *MicroTime) IsZero() bool {
-	if t == nil {
-		return true
-	}
-	return t.Time.IsZero()
-}
-
 // Before reports whether the time instant t is before u.
-func (t *MicroTime) Before(u *MicroTime) bool {
+func (t MicroTime) Before(u MicroTime) bool {
-	if t != nil && u != nil {
+	return t.Time.Before(u.Time)
-		return t.Time.Before(u.Time)
-	}
-	return false
 }
 
 // Equal reports whether the time instant t is equal to u.
-func (t *MicroTime) Equal(u *MicroTime) bool {
+func (t MicroTime) Equal(u MicroTime) bool {
-	if t == nil && u == nil {
+	return t.Time.Equal(u.Time)
-		return true
-	}
-	if t != nil && u != nil {
-		return t.Time.Equal(u.Time)
-	}
-	return false
 }
 
 // BeforeTime reports whether the time instant t is before second-lever precision u.
-func (t *MicroTime) BeforeTime(u *Time) bool {
+func (t MicroTime) BeforeTime(u Time) bool {
-	if t != nil && u != nil {
+	return t.Time.Before(u.Time)
-		return t.Time.Before(u.Time)
-	}
-	return false
 }
 
 // EqualTime reports whether the time instant t is equal to second-lever precision u.
-func (t *MicroTime) EqualTime(u *Time) bool {
+func (t MicroTime) EqualTime(u Time) bool {
-	if t == nil && u == nil {
+	return t.Time.Equal(u.Time)
-		return true
-	}
-	if t != nil && u != nil {
-		return t.Time.Equal(u.Time)
-	}
-	return false
 }
 
 // UnixMicro returns the local time corresponding to the given Unix time

staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/micro_time_test.go

@@ -140,115 +140,73 @@ func TestMicroTimeProto(t *testing.T) {
 
 func TestMicroTimeEqual(t *testing.T) {
 	t1 := NewMicroTime(time.Now())
-	cases := []struct {
+	t2 := NewMicroTime(time.Now().Add(time.Second))
-		name   string
-		x      *MicroTime
-		y      *MicroTime
-		result bool
-	}{
-		{"nil =? nil", nil, nil, true},
-		{"!nil =? !nil", &t1, &t1, true},
-		{"nil =? !nil", nil, &t1, false},
-		{"!nil =? nil", &t1, nil, false},
-	}
 
-	for _, c := range cases {
+	if !t1.Equal(t1) {
-		t.Run(c.name, func(t *testing.T) {
+		t.Errorf("Failed equality test for '%v', '%v': t1 should equal t1", t1, t1)
-			result := c.x.Equal(c.y)
+	}
-			if result != c.result {
+	if t1.Equal(t2) {
-				t.Errorf("Failed equality test for '%v', '%v': expected %+v, got %+v", c.x, c.y, c.result, result)
+		t.Errorf("Failed equality test for '%v', '%v': t1 should not equal t2", t1, t2)
-			}
-		})
 	}
 }
 
 func TestMicroTimeEqualTime(t *testing.T) {
 	t1 := NewMicroTime(time.Now())
 	t2 := NewTime(t1.Time)
-	cases := []struct {
+	t3 := NewTime(time.Now().Add(time.Second))
-		name   string
-		x      *MicroTime
-		y      *Time
-		result bool
-	}{
-		{"nil =? nil", nil, nil, true},
-		{"!nil =? !nil", &t1, &t2, true},
-		{"nil =? !nil", nil, &t2, false},
-		{"!nil =? nil", &t1, nil, false},
-	}
 
-	for _, c := range cases {
+	if !t1.EqualTime(t2) {
-		t.Run(c.name, func(t *testing.T) {
+		t.Errorf("Failed equality test for '%v', '%v': t1 should equal t2", t1, t1)
-			result := c.x.EqualTime(c.y)
+	}
-			if result != c.result {
+	if t1.EqualTime(t3) {
-				t.Errorf("Failed equality test for '%v', '%v': expected %+v, got %+v", c.x, c.y, c.result, result)
+		t.Errorf("Failed equality test for '%v', '%v': t1 should not equal t3", t1, t2)
-			}
-		})
 	}
 }
 
 func TestMicroTimeBefore(t *testing.T) {
-	t1 := NewMicroTime(time.Now())
+	tBefore := NewMicroTime(time.Now())
+	tAfter := NewMicroTime(tBefore.Time.Add(time.Second))
 	cases := []struct {
-		name string
+		name   string
-		x    *MicroTime
+		x      MicroTime
-		y    *MicroTime
+		y      MicroTime
+		result bool
 	}{
-		{"nil <? nil", nil, nil},
+		{"tBefore <? tBefore", tBefore, tBefore, false},
-		{"!nil <? !nil", &t1, &t1},
+		{"tBefore <? tAfter", tBefore, tAfter, true},
-		{"nil <? !nil", nil, &t1},
+		{"tAfter <? tBefore", tAfter, tBefore, false},
-		{"!nil <? nil", &t1, nil},
 	}
 
 	for _, c := range cases {
 		t.Run(c.name, func(t *testing.T) {
 			result := c.x.Before(c.y)
-			if result {
+			if result != c.result {
-				t.Errorf("Failed before test for '%v', '%v': expected false, got %+v", c.x, c.y, result)
+				t.Errorf("Failed before test for '%v', '%v': expected %+v, got %+v", c.x, c.y, c.result, result)
 			}
 		})
 	}
 }
 func TestMicroTimeBeforeTime(t *testing.T) {
-	t1 := NewMicroTime(time.Now())
+	tNow := NewMicroTime(time.Now())
-	t2 := NewTime(t1.Time)
+	tAfter := NewTime(tNow.Time.Add(time.Second))
+	tBefore := NewTime(tNow.Time.Add(-time.Second))
+	tTimeNow := NewTime(tNow.Time)
 	cases := []struct {
-		name string
+		name   string
-		x    *MicroTime
+		x      MicroTime
-		y    *Time
+		y      Time
+		result bool
 	}{
-		{"nil <? nil", nil, nil},
+		{"tNow <? tAfter", tNow, tAfter, true},
-		{"!nil <? !nil", &t1, &t2},
+		{"tNow <? tBefore", tNow, tBefore, false},
-		{"nil <? !nil", nil, &t2},
+		{"tNow <? tTimeNow", tNow, tTimeNow, false},
-		{"!nil <? nil", &t1, nil},
 	}
 
 	for _, c := range cases {
 		t.Run(c.name, func(t *testing.T) {
 			result := c.x.BeforeTime(c.y)
-			if result {
-				t.Errorf("Failed before test for '%v', '%v': expected false, got %+v", c.x, c.y, result)
-			}
-		})
-	}
-}
-
-func TestMicroTimeIsZero(t *testing.T) {
-	t1 := NewMicroTime(time.Now())
-	cases := []struct {
-		name   string
-		x      *MicroTime
-		result bool
-	}{
-		{"nil =? 0", nil, true},
-		{"!nil =? 0", &t1, false},
-	}
-
-	for _, c := range cases {
-		t.Run(c.name, func(t *testing.T) {
-			result := c.x.IsZero()
 			if result != c.result {
-				t.Errorf("Failed equality test for '%v': expected %+v, got %+v", c.x, c.result, result)
+				t.Errorf("Failed before test for '%v', '%v': expected %+v, got %+v", c.x, c.y, c.result, result)
 			}
 		})
 	}

staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/time.go

@@ -57,31 +57,14 @@ func Now() Time {
 	return Time{time.Now()}
 }
 
-// IsZero returns true if the value is nil or time is zero.
-func (t *Time) IsZero() bool {
-	if t == nil {
-		return true
-	}
-	return t.Time.IsZero()
-}
-
 // Before reports whether the time instant t is before u.
-func (t *Time) Before(u *Time) bool {
+func (t Time) Before(u Time) bool {
-	if t != nil && u != nil {
+	return t.Time.Before(u.Time)
-		return t.Time.Before(u.Time)
-	}
-	return false
 }
 
 // Equal reports whether the time instant t is equal to u.
-func (t *Time) Equal(u *Time) bool {
+func (t Time) Equal(u Time) bool {
-	if t == nil && u == nil {
+	return t.Time.Equal(u.Time)
-		return true
-	}
-	if t != nil && u != nil {
-		return t.Time.Equal(u.Time)
-	}
-	return false
 }
 
 // Unix returns the local time corresponding to the given Unix time

staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/time_test.go

@@ -174,67 +174,35 @@ func TestTimeProto(t *testing.T) {
 
 func TestTimeEqual(t *testing.T) {
 	t1 := NewTime(time.Now())
-	cases := []struct {
+	t2 := NewTime(time.Now().Add(time.Second))
-		name   string
-		x      *Time
-		y      *Time
-		result bool
-	}{
-		{"nil =? nil", nil, nil, true},
-		{"!nil =? !nil", &t1, &t1, true},
-		{"nil =? !nil", nil, &t1, false},
-		{"!nil =? nil", &t1, nil, false},
-	}
 
-	for _, c := range cases {
+	if !t1.Equal(t1) {
-		t.Run(c.name, func(t *testing.T) {
+		t.Errorf("Failed equality test for '%v', '%v': t1 should equal t1", t1, t1)
-			result := c.x.Equal(c.y)
+	}
-			if result != c.result {
+	if t1.Equal(t2) {
-				t.Errorf("Failed equality test for '%v', '%v': expected %+v, got %+v", c.x, c.y, c.result, result)
+		t.Errorf("Failed equality test for '%v', '%v': t1 should not equal t2", t1, t2)
-			}
-		})
 	}
 }
 
 func TestTimeBefore(t *testing.T) {
-	t1 := NewTime(time.Now())
+	tBefore := NewTime(time.Now())
+	tAfter := NewTime(tBefore.Time.Add(time.Second))
 	cases := []struct {
-		name string
+		name   string
-		x    *Time
+		x      Time
-		y    *Time
+		y      Time
+		result bool
 	}{
-		{"nil <? nil", nil, nil},
+		{"tBefore <? tBefore", tBefore, tBefore, false},
-		{"!nil <? !nil", &t1, &t1},
+		{"tBefore <? tAfter", tBefore, tAfter, true},
-		{"nil <? !nil", nil, &t1},
+		{"tAfter <? tBefore", tAfter, tBefore, false},
-		{"!nil <? nil", &t1, nil},
 	}
 
 	for _, c := range cases {
 		t.Run(c.name, func(t *testing.T) {
 			result := c.x.Before(c.y)
-			if result {
-				t.Errorf("Failed equality test for '%v', '%v': expected false, got %+v", c.x, c.y, result)
-			}
-		})
-	}
-}
-
-func TestTimeIsZero(t *testing.T) {
-	t1 := NewTime(time.Now())
-	cases := []struct {
-		name   string
-		x      *Time
-		result bool
-	}{
-		{"nil =? 0", nil, true},
-		{"!nil =? 0", &t1, false},
-	}
-
-	for _, c := range cases {
-		t.Run(c.name, func(t *testing.T) {
-			result := c.x.IsZero()
 			if result != c.result {
-				t.Errorf("Failed equality test for '%v': expected %+v, got %+v", c.x, c.result, result)
+				t.Errorf("Failed before test for '%v', '%v': expected %+v, got %+v", c.x, c.y, c.result, result)
 			}
 		})
 	}

staging/src/k8s.io/apimachinery/pkg/test/apis_meta_v1_unstructed_unstructure_test.go

@@ -207,11 +207,11 @@ func TestUnstructuredGetters(t *testing.T) {
 		t.Errorf("GetSelfLink() = %s, want %s", got, want)
 	}
 
-	if got, want := unstruct.GetCreationTimestamp(), metav1.Date(2009, time.November, 10, 23, 0, 0, 0, time.UTC); !got.Equal(&want) {
+	if got, want := unstruct.GetCreationTimestamp(), metav1.Date(2009, time.November, 10, 23, 0, 0, 0, time.UTC); !got.Equal(want) {
 		t.Errorf("GetCreationTimestamp() = %s, want %s", got, want)
 	}
 
-	if got, want := unstruct.GetDeletionTimestamp(), metav1.Date(2010, time.November, 10, 23, 0, 0, 0, time.UTC); got == nil || !got.Equal(&want) {
+	if got, want := unstruct.GetDeletionTimestamp(), metav1.Date(2010, time.November, 10, 23, 0, 0, 0, time.UTC); got == nil || !got.Equal(want) {
 		t.Errorf("GetDeletionTimestamp() = %s, want %s", got, want)
 	}