fix tests, specs to match 400 error on bad filtering requests

linksmart · Jun 12, 2020 · 55984eb · 55984eb
1 parent 273b834
commit 55984eb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/apidoc/openapi-spec.yml b/apidoc/openapi-spec.yml
@@ -51,6 +51,8 @@ paths:
               examples:
                 ThingDescriptionList:
                   $ref: '#/components/examples/ThingDescriptionList'
+        '400':
+          $ref: '#/components/responses/RespBadRequest'
         '401':
           $ref: '#/components/responses/RespUnauthorized'
         '403':

diff --git a/catalog/http_test.go b/catalog/http_test.go
@@ -468,8 +468,8 @@ func TestGetAll(t *testing.T) {
 		if err != nil {
 			t.Fatalf("Error reading response body: %s", err)
 		}
-		if res.StatusCode != http.StatusInternalServerError {
-			t.Fatalf("Expected status %v, got: %d. Response body:\n%s", http.StatusInternalServerError, res.StatusCode, b)
+		if res.StatusCode != http.StatusBadRequest {
+			t.Fatalf("Expected status %v, got: %d. Response body:\n%s", http.StatusBadRequest, res.StatusCode, b)
 		}
 	})
 
@@ -484,8 +484,8 @@ func TestGetAll(t *testing.T) {
 		if err != nil {
 			t.Fatalf("Error reading response body: %s", err)
 		}
-		if res.StatusCode != http.StatusInternalServerError {
-			t.Fatalf("Expected status %v, got: %d. Response body:\n%s", http.StatusInternalServerError, res.StatusCode, b)
+		if res.StatusCode != http.StatusBadRequest {
+			t.Fatalf("Expected status %v, got: %d. Response body:\n%s", http.StatusBadRequest, res.StatusCode, b)
 		}
 	})