Use testCases instead of dataList

internal/builds/register_test.go

@@ -53,10 +53,13 @@ func TestRegisterHandlerInvalidJsonPayload(t *testing.T) {
 }
 
 func TestRegisterHandlerMissingData(t *testing.T) {
-	dataList := []string{`{"token":"token"}`, `{"last_update":"data"}`}
+	testCases := []string{
+		`{"token":"token"}`,
+		`{"last_update":"data"}`,
+	}
 
-	for _, data := range dataList {
-		expectHandler(t, data, "application/json", http.StatusOK,
+	for _, testCase := range testCases {
+		expectHandler(t, testCase, "application/json", http.StatusOK,
 			"fails on argument validation and proxies request to upstream")
 	}
 }

internal/helper/helpers_test.go

@@ -68,7 +68,7 @@ func TestReadRequestBodyLimit(t *testing.T) {
 	rw := httptest.NewRecorder()
 	req, _ := http.NewRequest("POST", "/test", bytes.NewBuffer(data))
 
-	result, err := ReadRequestBody(rw, req, 2)
+	_, err := ReadRequestBody(rw, req, 2)
 	assert.Error(t, err)
 }