From a8043e53b3490c2fc202a26be93bdd924f737ded Mon Sep 17 00:00:00 2001 From: James Magahern Date: Wed, 3 Jan 2024 23:46:10 -0800 Subject: [PATCH] Update SendMessage for new protocol --- web/response_types.go | 4 ---- web/server.go | 7 +------ web/server_test.go | 12 ++++++------ 3 files changed, 7 insertions(+), 16 deletions(-) diff --git a/web/response_types.go b/web/response_types.go index 2072ff1..efb3895 100644 --- a/web/response_types.go +++ b/web/response_types.go @@ -1,5 +1 @@ package web - -type SendMessageResponse struct { - Guid string `json:"guid"` -} diff --git a/web/server.go b/web/server.go index 92988aa..344e33e 100644 --- a/web/server.go +++ b/web/server.go @@ -248,13 +248,8 @@ func (m *MockHTTPServer) handleSendMessage(w http.ResponseWriter, r *http.Reques // Send message m.Server.SendMessage(conversation, message) - // Formulate response - sendMessageResp := SendMessageResponse{ - Guid: message.Guid, - } - // Encode response as JSON - jsonData, err := json.Marshal(sendMessageResp) + jsonData, err := json.Marshal(message) if err != nil { log.Error().Err(err).Msg("Error marshalling response") http.Error(w, err.Error(), http.StatusInternalServerError) diff --git a/web/server_test.go b/web/server_test.go index 20cb6ea..26575f8 100644 --- a/web/server_test.go +++ b/web/server_test.go @@ -118,7 +118,7 @@ func TestMessages(t *testing.T) { server.Server.AppendMessageToConversation(&conversation, message) - resp, err := http.Get(httpServer.URL + "/messages?guid=" + *&conversation.Guid) + resp, err := http.Get(httpServer.URL + "/messages?guid=" + conversation.Guid) if err != nil { t.Fatalf("TestMessages error: %s", err) } @@ -237,7 +237,7 @@ func TestAuthentication(t *testing.T) { } func TestUpdates(t *testing.T) { - s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: true}) + s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: false}) httpServer := httptest.NewServer(s) messageSeq := 0 @@ -396,7 +396,7 @@ func TestUpdatesWebsocket(t *testing.T) { } func TestMarkConversation(t *testing.T) { - s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: true}) + s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: false}) httpServer := httptest.NewServer(s) // Mock conversation @@ -439,7 +439,7 @@ func TestMarkConversation(t *testing.T) { } func TestMessageQueries(t *testing.T) { - s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: true}) + s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: false}) httpServer := httptest.NewServer(s) // Mock conversation @@ -536,7 +536,7 @@ func TestMessageQueries(t *testing.T) { } func TestSendMessage(t *testing.T) { - s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: true}) + s := web.NewMockHTTPServer(web.MockHTTPServerConfiguration{AuthEnabled: false}) httpServer := httptest.NewServer(s) // Mock conversation @@ -578,7 +578,7 @@ func TestSendMessage(t *testing.T) { t.Fatalf("Error decoding body: %s", body) } - var response web.SendMessageResponse + var response model.Message err = json.Unmarshal(body, &response) if err != nil { t.Fatalf("Error unmarshalling JSON: %s", err)