feat(appstore): update ConsumptionRequest to use pointer for ConsumptionPercentage

.github/workflows/unit_test.yml

@@ -11,13 +11,15 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        go: [ '1.22', '1.23', '1.24' ]
+        go: [ '1.24', '1.25' ]
     name: Go ${{ matrix.go }} test
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
+
       - name: Setup go
-        uses: actions/setup-go@v4
+        uses: actions/setup-go@v6
         with:
           go-version: ${{ matrix.go }}
+
       - name: test
         run: go test -v -race ./...

appstore/api/model.go

@@ -93,7 +93,7 @@ const (
 // ConsumptionRequestBody https://developer.apple.com/documentation/appstoreserverapi/consumptionrequest
 type ConsumptionRequest struct {
 	CustomerConsented     bool             `json:"customerConsented"`
-	ConsumptionPercentage int32            `json:"consumptionPercentage"`
+	ConsumptionPercentage *int32           `json:"consumptionPercentage,omitempty"`
 	DeliveryStatus        DeliveryStatus   `json:"deliveryStatus"`
 	RefundPreference      RefundPreference `json:"refundPreference"`
 	SampleContentProvided bool             `json:"sampleContentProvided"`

appstore/api/model_test.go

@@ -0,0 +1,77 @@
+package api
+
+import (
+	"encoding/json"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestConsumptionRequest_JSONSerialization(t *testing.T) {
+	t.Run("should omit ConsumptionPercentage when nil", func(t *testing.T) {
+		t.Parallel()
+		req := ConsumptionRequest{
+			CustomerConsented:     true,
+			ConsumptionPercentage: nil,
+			DeliveryStatus:        DELIVERED,
+			RefundPreference:      DECLINE,
+			SampleContentProvided: false,
+		}
+
+		data, err := json.Marshal(req)
+		assert.NoError(t, err)
+
+		var result map[string]interface{}
+		err = json.Unmarshal(data, &result)
+		assert.NoError(t, err)
+
+		_, exists := result["consumptionPercentage"]
+		assert.False(t, exists, "consumptionPercentage should be omitted")
+	})
+
+	t.Run("should include ConsumptionPercentage when set to 0", func(t *testing.T) {
+		t.Parallel()
+		zero := int32(0)
+		req := ConsumptionRequest{
+			CustomerConsented:     true,
+			ConsumptionPercentage: &zero,
+			DeliveryStatus:        DELIVERED,
+			RefundPreference:      GRANT_FULL,
+			SampleContentProvided: true,
+		}
+
+		data, err := json.Marshal(req)
+		assert.NoError(t, err)
+
+		var result map[string]interface{}
+		err = json.Unmarshal(data, &result)
+		assert.NoError(t, err)
+
+		val, exists := result["consumptionPercentage"]
+		assert.True(t, exists)
+		assert.Equal(t, float64(0), val)
+	})
+
+	t.Run("should include ConsumptionPercentage when set to non-zero value", func(t *testing.T) {
+		t.Parallel()
+		fifty := int32(50)
+		req := ConsumptionRequest{
+			CustomerConsented:     true,
+			ConsumptionPercentage: &fifty,
+			DeliveryStatus:        DELIVERED,
+			RefundPreference:      GRANT_PRORATED,
+			SampleContentProvided: true,
+		}
+
+		data, err := json.Marshal(req)
+		assert.NoError(t, err)
+
+		var result map[string]interface{}
+		err = json.Unmarshal(data, &result)
+		assert.NoError(t, err)
+
+		val, exists := result["consumptionPercentage"]
+		assert.True(t, exists)
+		assert.Equal(t, float64(50), val)
+	})
+}