Support Spanish status names passed back from Zendesk targets

Ilhasoft · Sep 6, 2021 · ac1060f · ac1060f
1 parent a827881
commit ac1060f
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 2 deletions.
diff --git a/services/tickets/zendesk/testdata/target.json b/services/tickets/zendesk/testdata/target.json
@@ -105,5 +105,28 @@
                 "count": 1
             }
         ]
+    },
+    {
+        "label": "ticket updated also when status is non-English",
+        "method": "POST",
+        "path": "/mr/tickets/types/zendesk/target/4ee6d4f3-f92b-439b-9718-8da90c05490b",
+        "headers": {
+            "Authorization": "Basic emVuZGVzazpzZXNhbWU="
+        },
+        "body": {
+            "event": "status_changed",
+            "id": 1234,
+            "status": "Abierto"
+        },
+        "status": 200,
+        "response": {
+            "status": "handled"
+        },
+        "db_assertions": [
+            {
+                "query": "select count(*) from tickets_ticket where status = 'O'",
+                "count": 1
+            }
+        ]
     }
 ]
diff --git a/services/tickets/zendesk/web.go b/services/tickets/zendesk/web.go
@@ -283,9 +283,9 @@ func handleTicketerTarget(ctx context.Context, rt *runtime.Runtime, r *http.Requ
 
 	if request.Event == "status_changed" {
 		switch strings.ToLower(request.Status) {
-		case statusSolved, statusClosed:
+		case statusSolved, statusClosed, "resuelto", "cerrado":
 			err = tickets.CloseTicket(ctx, rt, nil, ticket, false, l)
-		case statusOpen:
+		case statusOpen, "abierto":
 			err = tickets.ReopenTicket(ctx, rt, nil, ticket, false, l)
 		}