fix(receive.go): return a concrete error type

Author: arschles

pkg/gitreceive/receive.go

@@ -23,6 +23,15 @@ import (
 //   -d "{\"receive_user\": \"$username\", \"receive_repo\": \"$app\", \"sha\": \"$sha\", \"fingerprint\": \"$fingerprint\", \"ssh_connection\": \"$SSH_CONNECTION\", \"ssh_original_command\": \"$SSH_ORIGINAL_COMMAND\"}" \
 //   --silent "http://$DEIS_WORKFLOW_SERVICE_HOST:$DEIS_WORKFLOW_SERVICE_PORT/v2/hooks/push" >/dev/null
 
+type errInvalidHTTPStatusCode struct {
+	expected int
+	actual   int
+}
+
+func (e errInvalidHTTPStatusCode) Error() string {
+	return fmt.Sprintf("expected status code %d, got %d", e.expected, e.actual)
+}
+
 func receive(conf *Config, newRev string) error {
 	urlStr := fmt.Sprintf("http://%s:%s/v2/hooks/push", conf.WorkflowHost, conf.WorkflowPort)
 	bodyMap := map[string]string{
@@ -42,13 +51,13 @@ func receive(conf *Config, newRev string) error {
 		return err
 	}
 
-	// TODO: inspect resp for status code
 	// TODO: use ctxhttp here (https://godoc.org/golang.org/x/net/context/ctxhttp)
-	// resp, err := http.Do(req)
-	_, err = http.DefaultClient.Do(req)
+	resp, err = http.DefaultClient.Do(req)
 	if err != nil {
 		return err
 	}
-	// TODO: inspect resp for status code
+	if resp.StatusCode != 201 {
+		return errInvalidHTTPStatusCode{expected: 201, actual: resp.StatusCode}
+	}
 	return nil
 }