Merge pull request #7 from thiagokokada/add-validation-error

request_types: add ValidationError

request.go

@@ -137,7 +137,7 @@ func parseResponse(raw RawResponse) (response []Response, err error) {
 func validateResponse(params []string, response []Response) ([]Response, error) {
 	// Empty response, something went terrible wrong
 	if len(response) == 0 {
-		return []Response{""}, errors.New("empty response")
+		return []Response{""}, ValidationError("empty response")
 	}
 
 	want := len(params)
@@ -147,18 +147,22 @@ func validateResponse(params []string, response []Response) ([]Response, error)
 	}
 	// we have a different number of requests and responses
 	if want != len(response) {
-		return response, fmt.Errorf(
+		return response, ValidationError(fmt.Sprintf(
 			"want responses: %d, got: %d, responses: %v",
 			want,
 			len(response),
 			response,
-		)
+		))
 	}
 
 	// validate that all responses are ok
 	for i, r := range response {
 		if r != "ok" {
-			return response, fmt.Errorf("non-ok response from param: %s, response: %s", params[i], r)
+			return response, ValidationError(fmt.Sprintf(
+				"non-ok response from param: %s, response: %s",
+				params[i],
+				r,
+			))
 		}
 	}
 

request_test.go

@@ -201,6 +201,8 @@ func TestValidateResponse(t *testing.T) {
 			assert.DeepEqual(t, response, tt.want)
 			if tt.wantErr {
 				assert.Error(t, err)
+				_, ok := err.(ValidationError)
+				assert.True(t, ok)
 			} else {
 				assert.NoError(t, err)
 			}

request_types.go

@@ -19,6 +19,15 @@ type RequestClient struct {
 	conn *net.UnixAddr
 }
 
+// ValidationError is used to return errors from response validation. In some
+// cases you may want to ignore those errors, in this case you type check this
+// kind of error and ignore it.
+type ValidationError string
+
+func (v ValidationError) Error() string {
+	return string(v)
+}
+
 // Unmarshal structs for requests.
 // Try to keep struct fields in the same order as the output for `hyprctl -j`
 // for sanity.