Handle DeepCopy of zero value versions

Author: wallrj

pkg/cassandra/version/version.go

@@ -76,6 +76,11 @@ func (v Version) MarshalJSON() ([]byte, error) {
 var _ json.Marshaler = &Version{}
 
 // DeepCopy returns a deep-copy of the Version value.
+// If the underlying semver is a nil pointer, assume that the zero value is being copied,
+// and return that.
 func (v Version) DeepCopy() Version {
+	if v.semver == nil {
+		return Version{}
+	}
 	return *New(v.String())
 }

pkg/cassandra/version/version_test.go

@@ -93,3 +93,18 @@ func TestUnmarshalJSON(t *testing.T) {
 		)
 	}
 }
+
+func TestDeepCopy(t *testing.T) {
+	t.Run(
+		"zero value",
+		func(t *testing.T) {
+			t.Log(version.Version{}.DeepCopy())
+		},
+	)
+	t.Run(
+		"validated version",
+		func(t *testing.T) {
+			t.Log(version.New("3.11.2").DeepCopy())
+		},
+	)
+}