bugfix/updatedAt in Marketing is not pointer type

bold-commerce · Jan 31, 2024 · fccafb8 · fccafb8
1 parent aab0f61
commit fccafb8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/customer.go b/customer.go
@@ -91,16 +91,16 @@ type CustomerSearchOptions struct {
 }
 
 type EmailMarketingConsent struct {
-	State            string    `json:"state"`
-	OptInLevel       string    `json:"opt_in_level"`
-	ConsentUpdatedAt time.Time `json:"consent_updated_at"`
+	State            string     `json:"state"`
+	OptInLevel       string     `json:"opt_in_level"`
+	ConsentUpdatedAt *time.Time `json:"consent_updated_at"`
 }
 
 type SMSMarketingConsent struct {
-	State                string    `json:"state"`
-	OptInLevel           string    `json:"opt_in_level"`
-	ConsentUpdatedAt     time.Time `json:"consent_updated_at"`
-	ConsentCollectedFrom string    `json:"consent_collected_from"`
+	State                string     `json:"state"`
+	OptInLevel           string     `json:"opt_in_level"`
+	ConsentUpdatedAt     *time.Time `json:"consent_updated_at"`
+	ConsentCollectedFrom string     `json:"consent_collected_from"`
 }
 
 // List customers

diff --git a/customer_test.go b/customer_test.go
@@ -230,13 +230,13 @@ func TestCustomerGet(t *testing.T) {
 	emailMarketingConsent1 := EmailMarketingConsent{
 		State:            "not_subscribed",
 		OptInLevel:       "single_opt_in",
-		ConsentUpdatedAt: updatedAt,
+		ConsentUpdatedAt: &updatedAt,
 	}
 
 	smsMarketingConsent1 := SMSMarketingConsent{
 		State:                "not_subscribed",
 		OptInLevel:           "single_opt_in",
-		ConsentUpdatedAt:     updatedAt,
+		ConsentUpdatedAt:     &updatedAt,
 		ConsentCollectedFrom: "OTHER",
 	}
 
@@ -375,7 +375,7 @@ func TestCustomerGet(t *testing.T) {
 	if customer.EmailMarketingConsent == nil {
 		t.Errorf("Customer.EmailMarketingConsent is nil, expected not nil")
 	} else {
-		if !customer.EmailMarketingConsent.ConsentUpdatedAt.Equal(expectation.EmailMarketingConsent.ConsentUpdatedAt) {
+		if !customer.EmailMarketingConsent.ConsentUpdatedAt.Equal(*expectation.EmailMarketingConsent.ConsentUpdatedAt) {
 			t.Errorf("Customer.EmailMarketingConsent.ConsentUpdatedAt returned %+v, expected %+v", customer.EmailMarketingConsent.ConsentUpdatedAt, expectation.EmailMarketingConsent.ConsentUpdatedAt)
 		}
 		if customer.EmailMarketingConsent.State != expectation.EmailMarketingConsent.State {
@@ -388,7 +388,7 @@ func TestCustomerGet(t *testing.T) {
 	if customer.SMSMarketingConsent == nil {
 		t.Errorf("Customer.SMSMarketingConsent is nil, expected not nil")
 	} else {
-		if !customer.SMSMarketingConsent.ConsentUpdatedAt.Equal(expectation.SMSMarketingConsent.ConsentUpdatedAt) {
+		if !customer.SMSMarketingConsent.ConsentUpdatedAt.Equal(*expectation.SMSMarketingConsent.ConsentUpdatedAt) {
 			t.Errorf("Customer.SMSMarketingConsent.ConsentUpdatedAt returned %+v, expected %+v", customer.SMSMarketingConsent.ConsentUpdatedAt, expectation.SMSMarketingConsent.ConsentUpdatedAt)
 		}
 		if customer.SMSMarketingConsent.State != expectation.SMSMarketingConsent.State {