fix: performance enhancements (#637/#638)

Uzlopak · web-flow · commit 9843e38561de · 2023-11-21T19:16:40.000-05:00
diff --git a/test/log.test.ts b/test/log.test.ts
@@ -1,25 +1,55 @@
-import { Octokit } from "../src";
-
 describe("octokit.log", () => {
-  it("has .debug(), .info(), .warn(), and .error() functions", () => {
-    const octokit = new Octokit();
-    expect(typeof octokit.log.debug).toBe("function");
-    expect(typeof octokit.log.info).toBe("function");
-    expect(typeof octokit.log.warn).toBe("function");
-    expect(typeof octokit.log.error).toBe("function");
-  });
+  it(".debug() and .info() are no-ops by default", async () => {
+    const calls: String[] = [];
+
+    const debug = jest
+      .spyOn(console, "debug")
+      .mockImplementation(() => calls.push("debug"));
+    const info = jest
+      .spyOn(console, "info")
+      .mockImplementation(() => calls.push("info"));
+    const warn = jest
+      .spyOn(console, "warn")
+      .mockImplementation(() => calls.push("warn"));
+    const error = jest
+      .spyOn(console, "error")
+      .mockImplementation(() => calls.push("error"));
+    const Octokit = (await import("../src")).Octokit;
 
-  it(".debug() and .info() are no-ops by default", () => {
     const octokit = new Octokit();
 
+    octokit.log.debug("foo");
+    octokit.log.info("bar");
+    octokit.log.warn("baz");
+    octokit.log.error("daz");
+
     expect(octokit.log.debug.name).toBe("noop");
     expect(octokit.log.info.name).toBe("noop");
 
-    octokit.log.debug("foo");
-    octokit.log.info("bar");
+    expect(debug).toHaveBeenCalledTimes(0);
+    expect(info).toHaveBeenCalledTimes(0);
+    expect(warn).toHaveBeenCalledTimes(1);
+    expect(error).toHaveBeenCalledTimes(1);
+    expect(calls).toStrictEqual(["warn", "error"]);
+
+    debug.mockRestore();
+    info.mockRestore();
+    warn.mockRestore();
+    error.mockRestore();
+  });
+
+  it("has .debug(), .info(), .warn(), and .error() functions", async () => {
+    const Octokit = (await import("../src")).Octokit;
+
+    const octokit = new Octokit();
+    expect(typeof octokit.log.debug).toBe("function");
+    expect(typeof octokit.log.info).toBe("function");
+    expect(typeof octokit.log.warn).toBe("function");
+    expect(typeof octokit.log.error).toBe("function");
   });
 
-  it("all .log.*() methods can be overwritten", () => {
+  it("all .log.*() methods can be overwritten", async () => {
+    const Octokit = (await import("../src")).Octokit;
     const calls: String[] = [];
 
     const octokit = new Octokit({
