test: silent logs

Author: pi0

test/fixture/server/tasks/db/migrate.ts

@@ -5,7 +5,6 @@ export default defineTask({
     description: "Run database migrations",
   },
   run() {
-    console.log("Running DB migration task...");
     return { result: "Success" };
   },
 });

test/fixture/server/tasks/test.ts

@@ -5,7 +5,6 @@ export default defineTask({
     description: "task to debug",
   },
   async run(taskEvent) {
-    console.log("test task", taskEvent);
     if (taskEvent.payload.wait) {
       await new Promise((resolve) => setTimeout(resolve, Number(taskEvent.payload.wait)));
     }

test/presets/bun.test.ts

@@ -11,13 +11,14 @@ describe.runIf(hasBun)("nitro:preset:bun", async () => {
   testNitro(ctx, async () => {
     const port = await getRandomPort();
     process.env.PORT = String(port);
-    execa("bun", [resolve(ctx.outDir, "server/index.mjs")], {
-      stdio: "inherit",
+    const p = execa("bun", [resolve(ctx.outDir, "server/index.mjs")], {
+      stdio: process.env.TEST_DEBUG ? "inherit" : "ignore",
+      reject: false,
     });
     ctx.server = {
       url: `http://127.0.0.1:${port}`,
-      close: () => {
-        // p.kill()
+      close: async () => {
+        p.kill();
       },
     } as any;
     await waitForPort(port);

test/presets/deno-server.test.ts

@@ -9,19 +9,19 @@ describe.runIf(hasDeno)("nitro:preset:deno-server", async () => {
   const ctx = await setupTest("deno-server");
   testNitro(ctx, async () => {
     const port = await getRandomPort();
-    execa("deno", ["task", "start"], {
+    const p = execa("deno", ["task", "start"], {
       cwd: ctx.outDir,
-      // stdio: "inherit",
-      stdio: "ignore",
+      stdio: process.env.TEST_DEBUG ? "inherit" : "ignore",
+      reject: false,
       env: {
         NITRO_PORT: String(port),
         NITRO_HOST: "127.0.0.1",
       },
     });
     ctx.server = {
       url: `http://127.0.0.1:${port}`,
-      close: () => {
-        // p.kill()
+      close: async () => {
+        p.kill();
       },
     } as any;
     await waitForPort(port, { delay: 1000, retries: 20, host: "127.0.0.1" });

test/unit/bump-version.test.ts

@@ -21,6 +21,7 @@ function mockRegistry(versions: string[]) {
 beforeEach(() => {
   vi.restoreAllMocks();
   fetchMock.mockReset();
+  vi.spyOn(console, "log").mockImplementation(() => {});
 });
 
 describe("fmtDate", async () => {

test/vite/app.test.ts

@@ -16,7 +16,7 @@ describe("vite:app", () => {
 
   beforeAll(async () => {
     process.chdir(rootDir);
-    server = await createServer({ root: rootDir });
+    server = await createServer({ root: rootDir, logLevel: "warn" });
     await server.listen("0" as unknown as number);
     const addr = server.httpServer?.address() as {
       port: number;

test/vite/baseurl-dotted-param.test.ts

@@ -14,7 +14,7 @@ describe("vite:baseURL dotted params", { sequential: true }, () => {
 
   beforeAll(async () => {
     process.chdir(rootDir);
-    server = await createServer({ root: rootDir });
+    server = await createServer({ root: rootDir, logLevel: "warn" });
     await server.listen("0" as unknown as number);
     const addr = server.httpServer?.address() as {
       port: number;

test/vite/hmr.test.ts

@@ -24,7 +24,7 @@ describe("vite:hmr", { sequential: true }, () => {
 
   beforeAll(async () => {
     process.chdir(rootDir);
-    server = await createServer({ root: rootDir });
+    server = await createServer({ root: rootDir, logLevel: "warn" });
 
     const originalSend = server.ws.send.bind(server.ws);
     server.ws.send = function (payload: any) {

test/vite/openapi.test.ts

@@ -13,7 +13,7 @@ describe("openapi", () => {
   const rootDir = fileURLToPath(new URL("./openapi-fixture", import.meta.url));
 
   beforeAll(async () => {
-    server = await createServer({ root: rootDir });
+    server = await createServer({ root: rootDir, logLevel: "warn" });
     await server.listen("0" as unknown as number);
     const addr = server.httpServer?.address() as {
       port: number;

test/vite/plugin-modules.test.ts

@@ -85,7 +85,8 @@ describe("vite: nitro modules from vite plugins", () => {
         }
       },
     };
-    consola.addReporter(reporter);
+    const originalReporters = [...consola.options.reporters];
+    consola.setReporters([reporter]);
 
     // Vite resolves its plugin list before running config hooks, so plugins added
     // from a `config` hook are ignored by Vite (but visible to Nitro's discovery).
@@ -96,7 +97,7 @@ describe("vite: nitro modules from vite plugins", () => {
       config: () => ({ plugins: [modulePlugin("injected")] }) as any,
     };
     await resolve([injector, modulePlugin("applied")]);
-    consola.removeReporter(reporter);
+    consola.setReporters(originalReporters);
 
     expect(installed).toEqual(["applied", "injected"]);
     expect(warnings).toHaveLength(1);