Rename to parsing failed as it's an error string

Author: ashmckenzie

internal/command/personalaccesstoken/personalaccesstoken_test.go

@@ -139,7 +139,7 @@ func TestExecute(t *testing.T) {
 				GitlabKeyId: "badresponse",
 				SshArgs:     []string{cmdname, "newtoken", "read_api,read_repository"},
 			},
-			expectedError: "Parsing failed",
+			expectedError: "parsing failed",
 		},
 		{
 			desc: "when API returns an error",

internal/command/twofactorrecover/twofactorrecover_test.go

@@ -85,7 +85,7 @@ func TestExecute(t *testing.T) {
 			desc:           "With bad response",
 			arguments:      &commandargs.Shell{GitlabKeyId: "-1"},
 			answer:         "yes\n",
-			expectedOutput: question + errorHeader + "Parsing failed\n",
+			expectedOutput: question + errorHeader + "parsing failed\n",
 		},
 		{
 			desc:           "With API returns an error",

internal/command/twofactorverify/twofactorverify_test.go

@@ -126,7 +126,7 @@ func TestExecute(t *testing.T) {
 		{
 			desc:           "With bad response",
 			arguments:      &commandargs.Shell{GitlabKeyId: "-1"},
-			expectedOutput: errorHeader + "Parsing failed\n",
+			expectedOutput: errorHeader + "parsing failed\n",
 		},
 		{
 			desc:           "With API returns an error",

internal/gitlabnet/accessverifier/client_test.go

@@ -174,7 +174,7 @@ func TestErrorResponses(t *testing.T) {
 		{
 			desc:          "A response with bad JSON",
 			fakeID:        "3",
-			expectedError: "Parsing failed",
+			expectedError: "parsing failed",
 		},
 		{
 			desc:          "An error response without message",

internal/gitlabnet/authorizedcerts/client_test.go

@@ -70,7 +70,7 @@ func TestGetByKeyErrorResponses(t *testing.T) {
 		{
 			desc:          "A response with bad JSON",
 			key:           "broken-json",
-			expectedError: "Parsing failed",
+			expectedError: "parsing failed",
 		},
 		{
 			desc:          "A forbidden (403) response without message",

internal/gitlabnet/authorizedkeys/client_test.go

@@ -70,7 +70,7 @@ func TestGetByKeyErrorResponses(t *testing.T) {
 		{
 			desc:          "A response with bad JSON",
 			key:           "broken-json",
-			expectedError: "Parsing failed",
+			expectedError: "parsing failed",
 		},
 		{
 			desc:          "A forbidden (403) response without message",

internal/gitlabnet/client.go

@@ -11,10 +11,6 @@ import (
 	"gitlab.com/gitlab-org/gitlab-shell/v14/internal/config"
 )
 
-var (
-	ParsingError = fmt.Errorf("Parsing failed")
-)
-
 func GetClient(config *config.Config) (*client.GitlabNetClient, error) {
 	httpClient, err := config.HTTPClient()
 	if err != nil {
@@ -30,7 +26,7 @@ func GetClient(config *config.Config) (*client.GitlabNetClient, error) {
 
 func ParseJSON(hr *http.Response, response interface{}) error {
 	if err := json.NewDecoder(hr.Body).Decode(response); err != nil {
-		return ParsingError
+		return fmt.Errorf("parsing failed")
 	}
 
 	return nil

internal/gitlabnet/discover/client_test.go

@@ -120,7 +120,7 @@ func TestErrorResponses(t *testing.T) {
 		{
 			desc:          "A response with bad JSON",
 			fakeUsername:  "broken_json",
-			expectedError: "Parsing failed",
+			expectedError: "parsing failed",
 		},
 		{
 			desc:          "An error response without message",

internal/gitlabnet/lfsauthenticate/client_test.go

@@ -64,7 +64,7 @@ func TestFailedRequests(t *testing.T) {
 		{
 			desc:           "With bad response",
 			args:           &commandargs.Shell{GitlabKeyId: "-1", CommandType: commandargs.LfsAuthenticate, SshArgs: []string{"git-lfs-authenticate", repo, "download"}},
-			expectedOutput: "Parsing failed",
+			expectedOutput: "parsing failed",
 		},
 		{
 			desc:           "With API returns an error",

internal/gitlabnet/personalaccesstoken/client_test.go

@@ -140,7 +140,7 @@ func TestErrorResponses(t *testing.T) {
 		{
 			desc:          "A response with bad JSON",
 			fakeID:        "3",
-			expectedError: "Parsing failed",
+			expectedError: "parsing failed",
 		},
 		{
 			desc:          "An error response without message",