fix up

Author: easyCZ

components/usage/pkg/db/types.go

@@ -39,41 +39,34 @@ func (n *VarcharTime) Scan(value interface{}) error {
 
 	switch s := value.(type) {
 	case []uint8:
-		// Null value - empty string mean value is not set
-		if len(s) == 0 {
-			n.valid = false
-			return nil
-		}
-
-		parsed, err := iso8601.ParseString(string(s))
-		if err != nil {
-			return fmt.Errorf("failed to parse %v into ISO8601: %w", string(s), err)
-		}
-
-		if parsed.IsZero() {
-			n.valid = false
-			return nil
-		}
+		return n.parseString(string(s))
+	case string:
+		return n.parseString(s)
+	}
+	return fmt.Errorf("unknown scan value for VarcharTime with value: %v", value)
+}
 
-		n.valid = true
-		n.t = parsed.UTC()
+func (n *VarcharTime) parseString(s string) error {
+	// Null value - empty string mean value is not set
+	if len(s) == 0 {
+		n.valid = false
 		return nil
-	case string:
-		if len(s) == 0 {
-			n.valid = false
-			return nil
-		}
+	}
 
-		parsed, err := iso8601.ParseString(s)
-		if err != nil {
-			return fmt.Errorf("failed to parse %v into ISO8601: %w", s, err)
-		}
+	parsed, err := iso8601.ParseString(s)
+	if err != nil {
+		return fmt.Errorf("failed to parse %v into ISO8601: %w", s, err)
+	}
 
-		n.valid = true
-		n.t = parsed.UTC()
+	if parsed.UTC().IsZero() {
+		n.t = time.Time{}.UTC()
+		n.valid = false
 		return nil
 	}
-	return fmt.Errorf("unknown scan value for VarcharTime with value: %v", value)
+
+	n.valid = true
+	n.t = parsed.UTC()
+	return nil
 }
 
 func (n VarcharTime) Time() time.Time {

components/usage/pkg/db/types_test.go

@@ -6,6 +6,7 @@ package db
 
 import (
 	"github.com/stretchr/testify/require"
+	"gorm.io/gorm"
 	"testing"
 	"time"
 )
@@ -64,15 +65,6 @@ func TestVarcharTime_Scan(t *testing.T) {
 				},
 				Error: false,
 			},
-		}, {
-			Name:  "zero timestamp is invalid",
-			Input: "",
-			Expected: Expectation{
-				Time: VarcharTime{
-					valid: false,
-				},
-				Error: false,
-			},
 		},
 	} {
 		t.Run(scenario.Name, func(t *testing.T) {
@@ -86,3 +78,43 @@ func TestVarcharTime_Scan(t *testing.T) {
 		})
 	}
 }
+
+func TestVarcharTime_SerializeAndDeserialize(t *testing.T) {
+	// Custom table to be able to exercise serialization easily, independent of other models
+	type VarcharModel struct {
+		ID   int         `gorm:"primaryKey"`
+		Time VarcharTime `gorm:"column:time;type:varchar(255);"`
+	}
+
+	conn := ConnectForTests(t)
+	require.NoError(t, conn.AutoMigrate(&VarcharModel{}))
+
+	conn.Session(&gorm.Session{AllowGlobalUpdate: true}).Delete(&VarcharModel{})
+
+	for _, scenario := range []struct {
+		Description string
+		Input       VarcharModel
+		Expected    VarcharModel
+	}{
+		{
+			Description: "empty value for VarcharTime",
+			Input: VarcharModel{
+				ID:   1,
+				Time: VarcharTime{},
+			},
+			Expected: VarcharModel{
+				ID:   1,
+				Time: VarcharTime{},
+			},
+		},
+	} {
+		tx := conn.Create(scenario.Input)
+		require.NoError(t, tx.Error)
+
+		var read VarcharModel
+		tx = conn.First(&read, scenario.Input.ID)
+		require.NoError(t, tx.Error)
+
+		require.Equal(t, scenario.Expected, read)
+	}
+}