Merge pull request #1937 from codysoyland/empty-query-response

Return empty result set when query empty. Fixes #1840

handler.go

@@ -45,18 +45,19 @@ type QueryResponse struct {
 
 // MarshalJSON marshals QueryResponse into a JSON-encoded byte slice
 func (resp *QueryResponse) MarshalJSON() ([]byte, error) {
-	var output struct {
-		Results        []interface{}    `json:"results,omitempty"`
-		ColumnAttrSets []*ColumnAttrSet `json:"columnAttrs,omitempty"`
-		Err            string           `json:"error,omitempty"`
-	}
-	output.Results = resp.Results
-	output.ColumnAttrSets = resp.ColumnAttrSets
-
 	if resp.Err != nil {
-		output.Err = resp.Err.Error()
+		return json.Marshal(struct {
+			Err string `json:"error"`
+		}{Err: resp.Err.Error()})
 	}
-	return json.Marshal(output)
+
+	return json.Marshal(struct {
+		Results        []interface{}    `json:"results"`
+		ColumnAttrSets []*ColumnAttrSet `json:"columnAttrs,omitempty"`
+	}{
+		Results:        resp.Results,
+		ColumnAttrSets: resp.ColumnAttrSets,
+	})
 }
 
 type Handler interface {

server/handler_test.go

@@ -445,6 +445,14 @@ func TestHandler_Endpoints(t *testing.T) {
 		}
 	})
 
+	t.Run("Query empty", func(t *testing.T) {
+		w := httptest.NewRecorder()
+		h.ServeHTTP(w, test.MustNewHTTPRequest("POST", "/index/i0/query", strings.NewReader("")))
+		if body := w.Body.String(); body != `{"results":[]}`+"\n" {
+			t.Fatalf("unexpected body: %q", body)
+		}
+	})
+
 	t.Run("Method not allowed", func(t *testing.T) {
 		w := httptest.NewRecorder()
 		h.ServeHTTP(w, test.MustNewHTTPRequest("GET", "/index/i0/query", nil))