what am I doing?

Author: WilliamBergamin

.github/workflows/deno.yml

@@ -2,9 +2,9 @@ name: Deno Format, Lint and Unit Tests
 
 on:
   push:
-    branches: [ main ]
+    branches: [main]
   pull_request:
-    branches: [ main ]
+    branches: [main]
 
 jobs:
   test:
@@ -13,7 +13,10 @@ jobs:
       matrix:
         # we test on both most recent stable version of deno (v1.x) as well as
         # the version of deno used by Run on Slack (as noted in https://api.slack.com/slackcli/metadata.json)
-        deno-version: [v1.x, v1.45.4]
+        deno-version:
+          - v1.x
+          - v1.46.2
+          - v2.x
     steps:
       - name: Setup repo
         uses: actions/checkout@v4
@@ -28,7 +31,7 @@ jobs:
       - name: Upload coverage to CodeCov
         uses: codecov/codecov-action@v5
         with:
-          file: ./lcov.info
+          files: ./lcov.info
           token: ${{ secrets.CODECOV_TOKEN }}
 
   health-score:

scripts/src/import_map/update_test.ts

@@ -1,28 +1,15 @@
 import { isHttpError } from "@std/http/http-errors";
-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";
+import { stubFetch } from "../../../src/utils_test.ts";
 
 const depsTsMock =
   `export { SlackAPI } from "https://deno.land/x/deno_slack_api@2.1.0/mod.ts";
    export type {SlackAPIClient, Trigger} from "https://deno.land/x/deno_slack_api@2.2.0/types.ts";`;
 
-beforeAll(() => {
-  mf.install();
-});
-
-afterEach(() => {
-  mf.reset();
-});
-
 Deno.test("apiDepsIn should return a list of the api module urls used by a module", async () => {
-  mf.mock("GET@/x/deno_slack_sdk@x.x.x/deps.ts", (req: Request) => {
-    assertEquals(
-      req.url,
-      "https://deno.land/x/deno_slack_sdk@x.x.x/deps.ts?source,file",
-    );
-    return new Response(depsTsMock);
+  using _fetchStub = stubFetch(new Response(depsTsMock), {
+    url: "https://deno.land/x/deno_slack_sdk@x.x.x/deps.ts?source,file",
   });
 
   const apiDeps = await apiDepsIn(
@@ -39,8 +26,8 @@ Deno.test("apiDepsIn should return a list of the api module urls used by a modul
 });
 
 Deno.test("apiDepsIn should throw http error on response not ok", async () => {
-  mf.mock("GET@/x/deno_slack_sdk@x.x.x/deps.ts", () => {
-    return new Response("error", { status: 500 });
+  using _fetchStub = stubFetch(new Response("error", { status: 500 }), {
+    url: "https://deno.land/x/deno_slack_sdk@x.x.x/deps.ts",
   });
 
   const error = await assertRejects(() =>

src/api_test.ts

@@ -7,6 +7,7 @@ import {
 } from "@std/assert";
 import { SlackAPI } from "./mod.ts";
 import { HttpError } from "@std/http/http-errors";
+import { stubFetch } from "./utils_test.ts";
 
 Deno.test("SlackAPI class", async (t) => {
   mf.install(); // mock out calls to `fetch`
@@ -24,10 +25,14 @@ Deno.test("SlackAPI class", async (t) => {
 
       await t.step("apiCall method", async (t) => {
         await t.step("should call the default API URL", async () => {
-          mf.mock("POST@/api/chat.postMessage", (req: Request) => {
-            assertEquals(req.url, "https://slack.com/api/chat.postMessage");
-            assertExists(req.headers.has("user-agent"));
-            return new Response('{"ok":true}');
+          // mf.mock("POST@/api/chat.postMessage", (req: Request) => {
+          //   assertEquals(req.url, "https://slack.com/api/chat.postMessage");
+          //   assertExists(req.headers.has("user-agent"));
+          //   return new Response('{"ok":true}');
+          // });
+          using _fetchStub = stubFetch(new Response('{"ok":true}'), {
+            url: "https://slack.com/api/chat.postMessage",
+            init: { headers: { "user-agent": "hello" } },
           });
 
           await client.apiCall("chat.postMessage", {});