Build test (#34)

KaushikiAnand · sonamtenzin2 · web-flow · commit 074c4f230487 · 2025-06-24T10:06:35.000+05:30
* workflow updation

* workflow update

* IND-2585 workflow updated with unit test coverage &amp; linting

* fix for handler test

* errcheck corrected as done in previous code

* resolved comment of using t.Fatalf

* break added to newly added errors

* disables

* reordering of workflow

* resolved comments

* test fix

---------

Co-authored-by: Sonam Tenzin &lt;sonam.tenzin@hashicorp.com&gt;
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -12,11 +12,29 @@ jobs:
       matrix:
         go: [ '1.21', '1.20', '1.19', '1.18']
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
+      - name: Checkout Code
+        uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
 
-      - uses: actions/setup-go@93397bea11091df50f3d7e59dc26a7711a8bcfbe # v4.1.0
+      - name: Setup Go
+        uses: actions/setup-go@93397bea11091df50f3d7e59dc26a7711a8bcfbe # v4.1.0
         with:
           go-version: ${{ matrix.go }}
+      
+      - name: Build Go
+        run: go build ./...
 
-      - name: test
-        run: go test -race . -v
+      - name: Run golangci-lint
+        uses: golangci/golangci-lint-action@08e2f20817b15149a52b5b3ebe7de50aff2ba8c5
+
+      - name: run test and generate coverage report
+        run: go test -race ./... -v -coverprofile=coverage.out
+
+      - name: Upload coverage report 
+        uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808
+        with: 
+          path: coverage.out
+          name: Coverage-report-${{matrix.go}}
+
+      - name: Display coverage report
+        run: go tool cover -func=coverage.out
+  
diff --git a/errors_test.go b/errors_test.go
@@ -47,7 +47,9 @@ func TestMarshalErrorsWritesTheExpectedPayload(t *testing.T) {
 			var writer io.Writer = buffer
 
 			_ = MarshalErrors(writer, testRow.In)
-			json.Unmarshal(buffer.Bytes(), &output)
+			if err := json.Unmarshal(buffer.Bytes(), &output); err != nil {
+				t.Fatalf("failed to unmarshal: %v", err)
+			}
 
 			if !reflect.DeepEqual(output, testRow.Out) {
 				t.Fatalf("Expected: \n%#v \nto equal: \n%#v", output, testRow.Out)
diff --git a/examples/handler_test.go b/examples/handler_test.go
@@ -12,7 +12,9 @@ import (
 func TestExampleHandler_post(t *testing.T) {
 	blog := fixtureBlogCreate(1)
 	requestBody := bytes.NewBuffer(nil)
-	jsonapi.MarshalOnePayloadEmbedded(requestBody, blog)
+	if err := jsonapi.MarshalOnePayloadEmbedded(requestBody, blog); err != nil {
+		t.Fatal(err)
+	}
 
 	r, err := http.NewRequest(http.MethodPost, "/blogs?id=1", requestBody)
 	if err != nil {
@@ -36,7 +38,9 @@ func TestExampleHandler_put(t *testing.T) {
 		fixtureBlogCreate(3),
 	}
 	requestBody := bytes.NewBuffer(nil)
-	jsonapi.MarshalPayload(requestBody, blogs)
+	if err := jsonapi.MarshalPayload(requestBody, blogs); err != nil {
+		t.Fatal(err)
+	}
 
 	r, err := http.NewRequest(http.MethodPut, "/blogs", requestBody)
 	if err != nil {
@@ -102,7 +106,7 @@ func TestHttpErrorWhenHeaderDoesNotMatch(t *testing.T) {
 }
 
 func TestHttpErrorWhenMethodDoesNotMatch(t *testing.T) {
-	r, err := http.NewRequest(http.MethodPatch, "/blogs", nil)
+	r, err := http.NewRequest(http.MethodOptions, "/blogs", nil)
 	if err != nil {
 		t.Fatal(err)
 	}
diff --git a/models_test.go b/models_test.go
@@ -246,7 +246,6 @@ type Video struct {
 
 type OneOfMedia struct {
 	Image       *Image
-	random      int
 	Video       *Video
 	RandomStuff *string
 }
diff --git a/request.go b/request.go
@@ -453,8 +453,8 @@ func unmarshalNode(data *Node, model reflect.Value, included *map[string]*Node)
 
 				buf := bytes.NewBuffer(nil)
 
-				json.NewEncoder(buf).Encode(data.Relationships[args[1]])
-				json.NewDecoder(buf).Decode(relationship)
+				json.NewEncoder(buf).Encode(data.Relationships[args[1]]) //nolint:errcheck
+				json.NewDecoder(buf).Decode(relationship)                //nolint:errcheck
 
 				data := relationship.Data
 
@@ -483,7 +483,7 @@ func unmarshalNode(data *Node, model reflect.Value, included *map[string]*Node)
 
 				buf := bytes.NewBuffer(nil)
 				relDataStr := data.Relationships[args[1]]
-				json.NewEncoder(buf).Encode(relDataStr)
+				json.NewEncoder(buf).Encode(relDataStr) //nolint:errcheck
 
 				isExplicitNull := false
 				relationshipDecodeErr := json.NewDecoder(buf).Decode(relationship)
diff --git a/request_test.go b/request_test.go
@@ -249,7 +249,6 @@ func TestUnmarshalToStructWithPointerAttr_BadType_Struct(t *testing.T) {
 
 func TestUnmarshalToStructWithPointerAttr_BadType_IntSlice(t *testing.T) {
 	out := new(WithPointer)
-	type FooStruct struct{ A, B int }
 	in := map[string]interface{}{
 		"name": []int{4, 5}, // This is the wrong type.
 	}
@@ -405,7 +404,9 @@ func TestUnmarshalNullableRelationshipsNonNullValue(t *testing.T) {
 	}
 
 	outBuf := bytes.NewBuffer(nil)
-	json.NewEncoder(outBuf).Encode(payload)
+	if err := json.NewEncoder(outBuf).Encode(payload); err != nil {
+		t.Fatal(err)
+	}
 
 	out := new(WithNullableAttrs)
 
@@ -442,7 +443,9 @@ func TestUnmarshalNullableRelationshipsExplicitNullValue(t *testing.T) {
 	}
 
 	outBuf := bytes.NewBuffer(nil)
-	json.NewEncoder(outBuf).Encode(payload)
+	if err := json.NewEncoder(outBuf).Encode(payload); err != nil {
+		t.Fatal(err)
+	}
 
 	out := new(WithNullableAttrs)
 
@@ -467,7 +470,9 @@ func TestUnmarshalNullableRelationshipsNonExistentValue(t *testing.T) {
 	}
 
 	outBuf := bytes.NewBuffer(nil)
-	json.NewEncoder(outBuf).Encode(payload)
+	if err := json.NewEncoder(outBuf).Encode(payload); err != nil {
+		t.Fatal(err)
+	}
 
 	out := new(WithNullableAttrs)
 
@@ -490,7 +495,9 @@ func TestUnmarshalNullableRelationshipsNoRelationships(t *testing.T) {
 	}
 
 	outBuf := bytes.NewBuffer(nil)
-	json.NewEncoder(outBuf).Encode(payload)
+	if err := json.NewEncoder(outBuf).Encode(payload); err != nil {
+		t.Fatal(err)
+	}
 
 	out := new(WithNullableAttrs)
 
@@ -1028,7 +1035,7 @@ func Test_choiceStructMapping(t *testing.T) {
 			t.Errorf("expected \"images\" to be the first field, but got %d", imageField.FieldNum)
 		}
 		videoField, ok := result["videos"]
-		if !ok || videoField.FieldNum != 2 {
+		if !ok || videoField.FieldNum != 1 {
 			t.Errorf("expected \"videos\" to be the third field, but got %d", videoField.FieldNum)
 		}
 	}
@@ -1120,7 +1127,10 @@ func TestUnmarshalNestedRelationships(t *testing.T) {
 }
 
 func TestUnmarshalRelationshipsSerializedEmbedded(t *testing.T) {
-	out := sampleSerializedEmbeddedTestModel()
+	out, err := sampleSerializedEmbeddedTestModel()
+	if err != nil {
+		t.Fatal(err)
+	}
 
 	if out.CurrentPost == nil {
 		t.Fatalf("Current post was not materialized")
@@ -1169,7 +1179,10 @@ func TestUnmarshalNestedRelationshipsEmbedded(t *testing.T) {
 }
 
 func TestUnmarshalRelationshipsSideloaded(t *testing.T) {
-	payload := samplePayloadWithSideloaded()
+	payload, err := samplePayloadWithSideloaded()
+	if err != nil {
+		t.Fatal(err)
+	}
 	out := new(Blog)
 
 	if err := UnmarshalPayload(payload, out); err != nil {
@@ -1190,7 +1203,10 @@ func TestUnmarshalRelationshipsSideloaded(t *testing.T) {
 }
 
 func TestUnmarshalNestedRelationshipsSideloaded(t *testing.T) {
-	payload := samplePayloadWithSideloaded()
+	payload, err := samplePayloadWithSideloaded()
+	if err != nil {
+		t.Fatal(err)
+	}
 	out := new(Blog)
 
 	if err := UnmarshalPayload(payload, out); err != nil {
@@ -1621,7 +1637,7 @@ func samplePayload() io.Reader {
 	}
 
 	out := bytes.NewBuffer(nil)
-	json.NewEncoder(out).Encode(payload)
+	json.NewEncoder(out).Encode(payload) //nolint:errcheck
 
 	return out
 }
@@ -1639,7 +1655,7 @@ func samplePayloadWithID() io.Reader {
 	}
 
 	out := bytes.NewBuffer(nil)
-	json.NewEncoder(out).Encode(payload)
+	json.NewEncoder(out).Encode(payload) //nolint:errcheck
 
 	return out
 }
@@ -1654,7 +1670,7 @@ func samplePayloadWithBadTypes(m map[string]interface{}) io.Reader {
 	}
 
 	out := bytes.NewBuffer(nil)
-	json.NewEncoder(out).Encode(payload)
+	json.NewEncoder(out).Encode(payload) //nolint:errcheck
 
 	return out
 }
@@ -1669,7 +1685,7 @@ func sampleWithPointerPayload(m map[string]interface{}) io.Reader {
 	}
 
 	out := bytes.NewBuffer(nil)
-	json.NewEncoder(out).Encode(payload)
+	json.NewEncoder(out).Encode(payload) //nolint:errcheck
 
 	return out
 }
@@ -1684,7 +1700,7 @@ func samplePayloadWithNullableAttrs(m map[string]interface{}) io.Reader {
 	}
 
 	out := bytes.NewBuffer(nil)
-	json.NewEncoder(out).Encode(payload)
+	json.NewEncoder(out).Encode(payload) //nolint:errcheck
 
 	return out
 }
@@ -1757,23 +1773,29 @@ func testModel() *Blog {
 	}
 }
 
-func samplePayloadWithSideloaded() io.Reader {
+func samplePayloadWithSideloaded() (io.Reader, error) {
 	testModel := testModel()
 
 	out := bytes.NewBuffer(nil)
-	MarshalPayload(out, testModel)
+	if err := MarshalPayload(out, testModel); err != nil {
+		return nil, err
+	}
 
-	return out
+	return out, nil
 }
 
-func sampleSerializedEmbeddedTestModel() *Blog {
+func sampleSerializedEmbeddedTestModel() (*Blog, error) {
 	out := bytes.NewBuffer(nil)
-	MarshalOnePayloadEmbedded(out, testModel())
+	if err := MarshalOnePayloadEmbedded(out, testModel()); err != nil {
+		return nil, err
+	}
 
 	blog := new(Blog)
-	UnmarshalPayload(out, blog)
+	if err := UnmarshalPayload(out, blog); err != nil {
+		return nil, err
+	}
 
-	return blog
+	return blog, nil
 }
 
 func TestUnmarshalNestedStructPtr(t *testing.T) {
diff --git a/response_test.go b/response_test.go
@@ -18,7 +18,9 @@ func TestMarshalPayload(t *testing.T) {
 
 	// One
 	out1 := bytes.NewBuffer(nil)
-	MarshalPayload(out1, book)
+	if err := MarshalPayload(out1, book); err != nil {
+		t.Fatal(err)
+	}
 
 	if err := json.Unmarshal(out1.Bytes(), &jsonData); err != nil {
 		t.Fatal(err)
@@ -29,7 +31,9 @@ func TestMarshalPayload(t *testing.T) {
 
 	// Many
 	out2 := bytes.NewBuffer(nil)
-	MarshalPayload(out2, books)
+	if err := MarshalPayload(out2, books); err != nil {
+		t.Fatal(err)
+	}
 
 	if err := json.Unmarshal(out2.Bytes(), &jsonData); err != nil {
 		t.Fatal(err)
@@ -936,7 +940,9 @@ func TestMarshal_Times(t *testing.T) {
 			}
 			// Use the standard JSON library to traverse the genereated JSON payload.
 			data := map[string]interface{}{}
-			json.Unmarshal(out.Bytes(), &data)
+			if err := json.Unmarshal(out.Bytes(), &data); err != nil {
+				t.Fatal(err)
+			}
 			if tc.verification != nil {
 				if err := tc.verification(data); err != nil {
 					t.Fatal(err)
@@ -1017,7 +1023,9 @@ func TestNullableRelationship(t *testing.T) {
 
 			// Use the standard JSON library to traverse the genereated JSON payload.
 			data := map[string]interface{}{}
-			json.Unmarshal(out.Bytes(), &data)
+			if err := json.Unmarshal(out.Bytes(), &data); err != nil {
+				t.Fatal(err)
+			}
 			if tc.verification != nil {
 				if err := tc.verification(data); err != nil {
 					t.Fatal(err)
@@ -1114,7 +1122,9 @@ func TestNullableAttr_Time(t *testing.T) {
 			}
 			// Use the standard JSON library to traverse the genereated JSON payload.
 			data := map[string]interface{}{}
-			json.Unmarshal(out.Bytes(), &data)
+			if err := json.Unmarshal(out.Bytes(), &data); err != nil {
+				t.Fatal(err)
+			}
 			if tc.verification != nil {
 				if err := tc.verification(data); err != nil {
 					t.Fatal(err)
@@ -1184,7 +1194,9 @@ func TestNullableAttr_Bool(t *testing.T) {
 			}
 			// Use the standard JSON library to traverse the genereated JSON payload.
 			data := map[string]interface{}{}
-			json.Unmarshal(out.Bytes(), &data)
+			if err := json.Unmarshal(out.Bytes(), &data); err != nil {
+				t.Fatal(err)
+			}
 			if tc.verification != nil {
 				if err := tc.verification(data); err != nil {
 					t.Fatal(err)
diff --git a/runtime.go b/runtime.go
@@ -77,7 +77,7 @@ func (r *Runtime) UnmarshalPayload(reader io.Reader, model interface{}) error {
 
 // UnmarshalManyPayload has docs in request.go for UnmarshalManyPayload.
 func (r *Runtime) UnmarshalManyPayload(reader io.Reader, kind reflect.Type) (elems []interface{}, err error) {
-	r.instrumentCall(UnmarshalStart, UnmarshalStop, func() error {
+	r.instrumentCall(UnmarshalStart, UnmarshalStop, func() error { //nolint:errcheck
 		elems, err = UnmarshalManyPayload(reader, kind)
 		return err
 	})

Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,6 @@ type Video struct {`
`246`	`246`
`247`	`247`	`type OneOfMedia struct {`
`248`	`248`	`Image *Image`
`249`		`- random int`
`250`	`249`	`Video *Video`
`251`	`250`	`RandomStuff *string`
`252`	`251`	`}`
Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,6 @@ func TestUnmarshalToStructWithPointerAttr_BadType_Struct(t *testing.T) {`
`249`	`249`
`250`	`250`	`func TestUnmarshalToStructWithPointerAttr_BadType_IntSlice(t *testing.T) {`
`251`	`251`	`out := new(WithPointer)`
`252`		`- type FooStruct struct{ A, B int }`
`253`	`252`	`in := map[string]interface{}{`
`254`	`253`	`"name": []int{4, 5}, // This is the wrong type.`
`255`	`254`	`}`
`@@ -405,7 +404,9 @@ func TestUnmarshalNullableRelationshipsNonNullValue(t *testing.T) {`
`405`	`404`	`}`
`406`	`405`
`407`	`406`	`outBuf := bytes.NewBuffer(nil)`
`408`		`- json.NewEncoder(outBuf).Encode(payload)`
	`407`	`+ if err := json.NewEncoder(outBuf).Encode(payload); err != nil {`
	`408`	`+ t.Fatal(err)`
	`409`	`+ }`
`409`	`410`
`410`	`411`	`out := new(WithNullableAttrs)`
`411`	`412`
`@@ -442,7 +443,9 @@ func TestUnmarshalNullableRelationshipsExplicitNullValue(t *testing.T) {`
`442`	`443`	`}`
`443`	`444`
`444`	`445`	`outBuf := bytes.NewBuffer(nil)`
`445`		`- json.NewEncoder(outBuf).Encode(payload)`
	`446`	`+ if err := json.NewEncoder(outBuf).Encode(payload); err != nil {`
	`447`	`+ t.Fatal(err)`
	`448`	`+ }`
`446`	`449`
`447`	`450`	`out := new(WithNullableAttrs)`
`448`	`451`
`@@ -467,7 +470,9 @@ func TestUnmarshalNullableRelationshipsNonExistentValue(t *testing.T) {`
`467`	`470`	`}`
`468`	`471`
`469`	`472`	`outBuf := bytes.NewBuffer(nil)`
`470`		`- json.NewEncoder(outBuf).Encode(payload)`
	`473`	`+ if err := json.NewEncoder(outBuf).Encode(payload); err != nil {`
	`474`	`+ t.Fatal(err)`
	`475`	`+ }`
`471`	`476`
`472`	`477`	`out := new(WithNullableAttrs)`
`473`	`478`
`@@ -490,7 +495,9 @@ func TestUnmarshalNullableRelationshipsNoRelationships(t *testing.T) {`
`490`	`495`	`}`
`491`	`496`
`492`	`497`	`outBuf := bytes.NewBuffer(nil)`
`493`		`- json.NewEncoder(outBuf).Encode(payload)`
	`498`	`+ if err := json.NewEncoder(outBuf).Encode(payload); err != nil {`
	`499`	`+ t.Fatal(err)`
	`500`	`+ }`
`494`	`501`
`495`	`502`	`out := new(WithNullableAttrs)`
`496`	`503`
`@@ -1028,7 +1035,7 @@ func Test_choiceStructMapping(t *testing.T) {`
`1028`	`1035`	`t.Errorf("expected \"images\" to be the first field, but got %d", imageField.FieldNum)`
`1029`	`1036`	`}`
`1030`	`1037`	`videoField, ok := result["videos"]`
`1031`		`- if !ok \|\| videoField.FieldNum != 2 {`
	`1038`	`+ if !ok \|\| videoField.FieldNum != 1 {`
`1032`	`1039`	`t.Errorf("expected \"videos\" to be the third field, but got %d", videoField.FieldNum)`
`1033`	`1040`	`}`
`1034`	`1041`	`}`
`@@ -1120,7 +1127,10 @@ func TestUnmarshalNestedRelationships(t *testing.T) {`
`1120`	`1127`	`}`
`1121`	`1128`
`1122`	`1129`	`func TestUnmarshalRelationshipsSerializedEmbedded(t *testing.T) {`
`1123`		`- out := sampleSerializedEmbeddedTestModel()`
	`1130`	`+ out, err := sampleSerializedEmbeddedTestModel()`
	`1131`	`+ if err != nil {`
	`1132`	`+ t.Fatal(err)`
	`1133`	`+ }`
`1124`	`1134`
`1125`	`1135`	`if out.CurrentPost == nil {`
`1126`	`1136`	`t.Fatalf("Current post was not materialized")`
`@@ -1169,7 +1179,10 @@ func TestUnmarshalNestedRelationshipsEmbedded(t *testing.T) {`
`1169`	`1179`	`}`
`1170`	`1180`
`1171`	`1181`	`func TestUnmarshalRelationshipsSideloaded(t *testing.T) {`
`1172`		`- payload := samplePayloadWithSideloaded()`
	`1182`	`+ payload, err := samplePayloadWithSideloaded()`
	`1183`	`+ if err != nil {`
	`1184`	`+ t.Fatal(err)`
	`1185`	`+ }`
`1173`	`1186`	`out := new(Blog)`
`1174`	`1187`
`1175`	`1188`	`if err := UnmarshalPayload(payload, out); err != nil {`
`@@ -1190,7 +1203,10 @@ func TestUnmarshalRelationshipsSideloaded(t *testing.T) {`
`1190`	`1203`	`}`
`1191`	`1204`
`1192`	`1205`	`func TestUnmarshalNestedRelationshipsSideloaded(t *testing.T) {`
`1193`		`- payload := samplePayloadWithSideloaded()`
	`1206`	`+ payload, err := samplePayloadWithSideloaded()`
	`1207`	`+ if err != nil {`
	`1208`	`+ t.Fatal(err)`
	`1209`	`+ }`
`1194`	`1210`	`out := new(Blog)`
`1195`	`1211`
`1196`	`1212`	`if err := UnmarshalPayload(payload, out); err != nil {`
`@@ -1621,7 +1637,7 @@ func samplePayload() io.Reader {`
`1621`	`1637`	`}`
`1622`	`1638`
`1623`	`1639`	`out := bytes.NewBuffer(nil)`
`1624`		`- json.NewEncoder(out).Encode(payload)`
	`1640`	`+ json.NewEncoder(out).Encode(payload) //nolint:errcheck`
`1625`	`1641`
`1626`	`1642`	`return out`
`1627`	`1643`	`}`
`@@ -1639,7 +1655,7 @@ func samplePayloadWithID() io.Reader {`
`1639`	`1655`	`}`
`1640`	`1656`
`1641`	`1657`	`out := bytes.NewBuffer(nil)`
`1642`		`- json.NewEncoder(out).Encode(payload)`
	`1658`	`+ json.NewEncoder(out).Encode(payload) //nolint:errcheck`
`1643`	`1659`
`1644`	`1660`	`return out`
`1645`	`1661`	`}`
`@@ -1654,7 +1670,7 @@ func samplePayloadWithBadTypes(m map[string]interface{}) io.Reader {`
`1654`	`1670`	`}`
`1655`	`1671`
`1656`	`1672`	`out := bytes.NewBuffer(nil)`
`1657`		`- json.NewEncoder(out).Encode(payload)`
	`1673`	`+ json.NewEncoder(out).Encode(payload) //nolint:errcheck`
`1658`	`1674`
`1659`	`1675`	`return out`
`1660`	`1676`	`}`
`@@ -1669,7 +1685,7 @@ func sampleWithPointerPayload(m map[string]interface{}) io.Reader {`
`1669`	`1685`	`}`
`1670`	`1686`
`1671`	`1687`	`out := bytes.NewBuffer(nil)`
`1672`		`- json.NewEncoder(out).Encode(payload)`
	`1688`	`+ json.NewEncoder(out).Encode(payload) //nolint:errcheck`
`1673`	`1689`
`1674`	`1690`	`return out`
`1675`	`1691`	`}`
`@@ -1684,7 +1700,7 @@ func samplePayloadWithNullableAttrs(m map[string]interface{}) io.Reader {`
`1684`	`1700`	`}`
`1685`	`1701`
`1686`	`1702`	`out := bytes.NewBuffer(nil)`
`1687`		`- json.NewEncoder(out).Encode(payload)`
	`1703`	`+ json.NewEncoder(out).Encode(payload) //nolint:errcheck`
`1688`	`1704`
`1689`	`1705`	`return out`
`1690`	`1706`	`}`
`@@ -1757,23 +1773,29 @@ func testModel() *Blog {`
`1757`	`1773`	`}`
`1758`	`1774`	`}`
`1759`	`1775`
`1760`		`-func samplePayloadWithSideloaded() io.Reader {`
	`1776`	`+func samplePayloadWithSideloaded() (io.Reader, error) {`
`1761`	`1777`	`testModel := testModel()`
`1762`	`1778`
`1763`	`1779`	`out := bytes.NewBuffer(nil)`
`1764`		`- MarshalPayload(out, testModel)`
	`1780`	`+ if err := MarshalPayload(out, testModel); err != nil {`
	`1781`	`+ return nil, err`
	`1782`	`+ }`
`1765`	`1783`
`1766`		`- return out`
	`1784`	`+ return out, nil`
`1767`	`1785`	`}`
`1768`	`1786`
`1769`		`-func sampleSerializedEmbeddedTestModel() *Blog {`
	`1787`	`+func sampleSerializedEmbeddedTestModel() (*Blog, error) {`
`1770`	`1788`	`out := bytes.NewBuffer(nil)`
`1771`		`- MarshalOnePayloadEmbedded(out, testModel())`
	`1789`	`+ if err := MarshalOnePayloadEmbedded(out, testModel()); err != nil {`
	`1790`	`+ return nil, err`
	`1791`	`+ }`
`1772`	`1792`
`1773`	`1793`	`blog := new(Blog)`
`1774`		`- UnmarshalPayload(out, blog)`
	`1794`	`+ if err := UnmarshalPayload(out, blog); err != nil {`
	`1795`	`+ return nil, err`
	`1796`	`+ }`
`1775`	`1797`
`1776`		`- return blog`
	`1798`	`+ return blog, nil`
`1777`	`1799`	`}`
`1778`	`1800`
`1779`	`1801`	`func TestUnmarshalNestedStructPtr(t *testing.T) {`