fix: correct http status codes and endpoint validation in convo tests

- add missing http.statuscreated header in conversation models test
- fix endpoint comparison logic for urls with query strings

Author: tharropoulos

typesense/conversation_models_test.go

@@ -79,6 +79,7 @@ func TestConversationModelsCreate(t *testing.T) {
 
 		data := jsonEncode(t, expectedData)
 		w.Header().Set("Content-Type", "application/json")
+		w.WriteHeader(http.StatusCreated)
 		w.Write(data)
 	})
 	defer server.Close()

typesense/utils_test.go

@@ -4,6 +4,7 @@ import (
 	"encoding/json"
 	"net/http"
 	"net/http/httptest"
+	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -15,8 +16,14 @@ func newTestServerAndClient(handler func(w http.ResponseWriter, r *http.Request)
 }
 
 func validateRequestMetadata(t *testing.T, r *http.Request, expectedEndpoint string, expectedMethod string) {
-	if r.RequestURI != expectedEndpoint {
-		t.Fatal("Invalid request endpoint!")
+	if strings.Contains(expectedEndpoint, "?") {
+		if r.URL.String() != expectedEndpoint {
+			t.Fatal("Invalid request endpoint!")
+		}
+	} else {
+		if r.URL.Path != expectedEndpoint {
+			t.Fatal("Invalid request endpoint!")
+		}
 	}
 	if r.Method != expectedMethod {
 		t.Fatal("Invalid HTTP method!")