fixes

slackapi · Aug 27, 2024 · 36e15ef · 36e15ef
1 parent 8328f8a
commit 36e15ef
Show file tree

Hide file tree

Showing 4 changed files with 59 additions and 47 deletions.
diff --git a/scripts/src/import_map/update_test.ts b/scripts/src/import_map/update_test.ts
@@ -1,5 +1,5 @@
 import { isHttpError } from "@std/http/http-errors";
-import { mf } from "../../src/dev_deps.ts";
+import { mf } from "../../../src/dev_deps.ts";
 import { assertEquals, assertRejects } from "@std/assert";
 import { afterEach, beforeAll } from "@std/testing/bdd";
 import { apiDepsIn } from "./update.ts";

diff --git a/src/typed-method-types/workflows/triggers/tests/event_test.ts b/src/typed-method-types/workflows/triggers/tests/event_test.ts
@@ -201,11 +201,13 @@ Deno.test("Mock call for event", async (t) => {
             },
           });
           assertEquals(res.ok, true);
-          assertObjectMatch(res.trigger, event_response.trigger);
-          assertEquals(
-            res.trigger?.event_type,
-            event_response.trigger.event_type,
-          );
+          if (res.ok) {
+            assertObjectMatch(res.trigger, event_response.trigger);
+            assertEquals(
+              res.trigger?.event_type,
+              event_response.trigger.event_type,
+            );
+          }
 
           mf.reset();
         },
@@ -239,11 +241,13 @@ Deno.test("Mock call for event", async (t) => {
           },
         });
         assertEquals(res.ok, true);
-        assertObjectMatch(res.trigger, event_response.trigger);
-        assertEquals(
-          res.trigger?.event_type,
-          event_response.trigger.event_type,
-        );
+        if (res.ok) {
+          assertObjectMatch(res.trigger, event_response.trigger);
+          assertEquals(
+            res.trigger?.event_type,
+            event_response.trigger.event_type,
+          );
+        }
 
         mf.reset();
       },

diff --git a/src/typed-method-types/workflows/triggers/tests/scheduled_test.ts b/src/typed-method-types/workflows/triggers/tests/scheduled_test.ts
@@ -233,19 +233,21 @@ Deno.test("Mock call for schedule", async (t) => {
             },
           });
           assertEquals(res.ok, true);
-          assertObjectMatch(res.trigger, scheduled_response.trigger);
-          assertEquals(
-            res.trigger?.schedule,
-            scheduled_response.trigger.schedule,
-          );
-          assertEquals(
-            res.trigger?.schedule?.start_time,
-            scheduled_response.trigger.schedule.start_time,
-          );
-          assertEquals(
-            res.trigger?.schedule?.timezone,
-            scheduled_response.trigger.schedule.timezone,
-          );
+          if (res.ok) {
+            assertObjectMatch(res.trigger, scheduled_response.trigger);
+            assertEquals(
+              res.trigger?.schedule,
+              scheduled_response.trigger.schedule,
+            );
+            assertEquals(
+              res.trigger?.schedule?.start_time,
+              scheduled_response.trigger.schedule.start_time,
+            );
+            assertEquals(
+              res.trigger?.schedule?.timezone,
+              scheduled_response.trigger.schedule.timezone,
+            );
+          }
 
           mf.reset();
         },
@@ -287,19 +289,21 @@ Deno.test("Mock call for schedule", async (t) => {
           },
         });
         assertEquals(res.ok, true);
-        assertObjectMatch(res.trigger, scheduled_response.trigger);
-        assertEquals(
-          res.trigger?.schedule,
-          scheduled_response.trigger.schedule,
-        );
-        assertEquals(
-          res.trigger?.schedule?.start_time,
-          scheduled_response.trigger.schedule.start_time,
-        );
-        assertEquals(
-          res.trigger?.schedule?.timezone,
-          scheduled_response.trigger.schedule.timezone,
-        );
+        if (res.ok) {
+          assertObjectMatch(res.trigger, scheduled_response.trigger);
+          assertEquals(
+            res.trigger?.schedule,
+            scheduled_response.trigger.schedule,
+          );
+          assertEquals(
+            res.trigger?.schedule?.start_time,
+            scheduled_response.trigger.schedule.start_time,
+          );
+          assertEquals(
+            res.trigger?.schedule?.timezone,
+            scheduled_response.trigger.schedule.timezone,
+          );
+        }
 
         mf.reset();
       },

diff --git a/src/typed-method-types/workflows/triggers/tests/webhook_test.ts b/src/typed-method-types/workflows/triggers/tests/webhook_test.ts
@@ -82,11 +82,13 @@ Deno.test("Mock call for webhook", async (t) => {
             },
           });
           assertEquals(res.ok, true);
-          assertObjectMatch(res.trigger, webhook_response.trigger);
-          assertEquals(
-            res.trigger?.webhook_url,
-            webhook_response.trigger.webhook_url,
-          );
+          if (res.ok) {
+            assertObjectMatch(res.trigger, webhook_response.trigger);
+            assertEquals(
+              res.trigger?.webhook_url,
+              webhook_response.trigger.webhook_url,
+            );
+          }
           mf.reset();
         },
       );
@@ -118,11 +120,13 @@ Deno.test("Mock call for webhook", async (t) => {
           },
         });
         assertEquals(res.ok, true);
-        assertObjectMatch(res.trigger, webhook_response.trigger);
-        assertEquals(
-          res.trigger?.webhook_url,
-          webhook_response.trigger.webhook_url,
-        );
+        if (res.ok) {
+          assertObjectMatch(res.trigger, webhook_response.trigger);
+          assertEquals(
+            res.trigger?.webhook_url,
+            webhook_response.trigger.webhook_url,
+          );
+        }
         mf.reset();
       },
     );