Merge pull request #17 from weni-ai/fix/zendesk-reopen

Remove returned ticket in case of reopening for zendesk
weni-ai · Jun 9, 2023 · 0f8a0a1 · 0f8a0a1
2 parents 486713e + 0c4e5dc
commit 0f8a0a1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/services/tickets/zendesk/service.go b/services/tickets/zendesk/service.go
@@ -118,7 +118,7 @@ func (s *service) Open(session flows.Session, topic *flows.Topic, body string, a
 			_, trace, _ := s.restClient.UpdateManyTickets(ids, statusOpen)
 			if trace != nil {
 				logHTTP(flows.NewHTTPLog(trace, flows.HTTPStatusFromCode, s.redactor))
-				return ticket, nil
+				return nil, nil
 			}
 		}
 

diff --git a/services/tickets/zendesk/service_test.go b/services/tickets/zendesk/service_test.go
@@ -145,9 +145,8 @@ func TestOpenAndForward(t *testing.T) {
 	test.AssertSnapshot(t, "forward_message", logger.Logs[0].Request)
 
 	logger = &flows.HTTPLogger{}
-	ticket1, err := svc.Open(session, defaultTopic, fieldTicket1, nil, logger.Log)
+	_, err = svc.Open(session, defaultTopic, fieldTicket1, nil, logger.Log)
 	assert.NoError(t, err)
-	assert.Equal(t, fieldTicket1, ticket1.Body())
 	assert.Equal(t, 1, len(logger.Logs))
 	test.AssertSnapshot(t, "open", logger.Logs[0].Request)
 }