Merge pull request #21 from kenshin579/feat/#20-not

[#20] not to caching even if the updatedAt has a default time value
kenshin579 · Jul 16, 2023 · c0e9e12 · c0e9e12
2 parents 4731aac + 3d44e21
commit c0e9e12
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/cache.go b/cache.go
@@ -331,7 +331,7 @@ func isMapEmpty(m map[string]any) bool {
 	for _, v := range m {
 		switch val := v.(type) {
 		case string:
-			if val != "" {
+			if val != "" && val != "0001-01-01T00:00:00Z" {
 				return false
 			}
 		case int:

diff --git a/cache_test.go b/cache_test.go
@@ -292,6 +292,7 @@ func Test_isAllFieldsEmpty(t *testing.T) {
 			City string `json:"city"`
 			Zip  int    `json:"zip"`
 		} `json:"address"`
+		UpdatedAt time.Time `json:"updatedAt"`
 	}
 
 	p1 := person{
@@ -323,6 +324,7 @@ func Test_isAllFieldsEmpty(t *testing.T) {
 	assert.False(t, isAllFieldsEmpty([]byte(`{"a":"","b":"b","c":0}`)))
 	assert.True(t, isAllFieldsEmpty([]byte(`{"a":"","b":"","c":0}`)))
 	assert.True(t, isAllFieldsEmpty([]byte(`{"a":"","b":"","c":0.0}`)))
+	assert.True(t, isAllFieldsEmpty([]byte(`{"a":"","b":"","c":0.0,"updatedAt":"0001-01-01T00:00:00Z"}`)))
 }
 
 func Test_isIncludePaths(t *testing.T) {