feat: bind console.log always to the same value (#111)

* feat: bind console.log always to the same value

* vitest: clearmocks true

Author: JoshuaKGoldberg

src/greet.test.ts

@@ -1,12 +1,14 @@
 import { describe, expect, it, vi } from "vitest";
 
+const logger = vi.spyOn(console, "log");
+
 import { greet } from "./greet.js";
 
 const message = "Yay, testing!";
 
 describe("greet", () => {
 	it("logs to the console once when message is provided as a string", () => {
-		const logger = vi.spyOn(console, "log").mockImplementation(() => undefined);
+		logger.mockImplementation(() => undefined);
 
 		greet(message);
 
@@ -15,7 +17,7 @@ describe("greet", () => {
 	});
 
 	it("logs to the console once when message is provided as an object", () => {
-		const logger = vi.spyOn(console, "log").mockImplementation(() => undefined);
+		logger.mockImplementation(() => undefined);
 
 		greet({ message });
 

src/greet.ts

@@ -1,8 +1,10 @@
 import { GreetOptions } from "./types.js";
 
+const consoleLogBound = console.log.bind(console);
+
 export function greet(options: GreetOptions | string) {
 	const {
-		logger = console.log.bind(console),
+		logger = consoleLogBound,
 		message,
 		times = 1,
 	} = typeof options === "string" ? { message: options } : options;

vitest.config.ts

@@ -2,6 +2,7 @@ import { defineConfig } from "vitest/config";
 
 export default defineConfig({
 	test: {
+		clearMocks: true,
 		coverage: {
 			all: true,
 			include: ["src"],