removed logging

Moonsong-Labs · Nov 22, 2023 · 7224ae4 · 7224ae4
1 parent 3ef4cb1
commit 7224ae4
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 13 deletions.
diff --git a/packages/cli/src/cmds/entrypoint.ts b/packages/cli/src/cmds/entrypoint.ts
@@ -177,6 +177,7 @@ const cliStart = Effect.gen(function* (_) {
     case "test": {
       yield* _(
         testEffect(args["envName"], { testNamePattern: args["GrepTest"] }).pipe(
+          Effect.tap(() => Effect.sync(() => console.log("✅  All Tests Passed"))),
           Effect.catchTag("TestsFailedError", (error) => {
             failedTests = error.fails;
             return Effect.succeed(

diff --git a/packages/cli/src/cmds/runTests.ts b/packages/cli/src/cmds/runTests.ts
@@ -51,9 +51,7 @@ export const testEffect = (envName: string, additionalArgs?: object) => {
       Effect.sync(() => vitest!.state.getFiles().filter((file) => file.result!.state === "fail"))
     );
 
-    if (failed.length === 0) {
-      yield* _(Effect.succeed(() => console.log("✅ All tests passed")));
-    } else {
+    if (failed.length !== 0) {
       yield* _(new Err.TestsFailedError({ fails: failed.length }));
     }
   });

diff --git a/packages/cli/src/internal/localNode.ts b/packages/cli/src/internal/localNode.ts
@@ -54,7 +54,6 @@ export const launchNodeEffect = (cmd: string, args: string[]) =>
         }
       }
     }
-    console.log("this is runningNode.pid", runningNode.pid);
     return { pid: runningNode.pid, kill: runningNode.kill } satisfies LaunchedNode;
   });
 

diff --git a/packages/cli/src/lib/globalContextEffect.ts b/packages/cli/src/lib/globalContextEffect.ts
@@ -327,8 +327,6 @@ export class MoonwallContext {
     const promises = nodes.map(async ({ cmd, args, launch }) => {
       if (launch) {
         const result = await Effect.runPromise(launchNodeEffect(cmd, args));
-        console.log("result pushed");
-        console.log(result);
         this.nodes.push(result);
       } else {
         return Promise.resolve();

diff --git a/packages/cli/src/lib/runnerContext.ts b/packages/cli/src/lib/runnerContext.ts
@@ -83,7 +83,6 @@ export function describeSuite<T extends FoundationType>({
   let ctx: any;
 
   beforeAll(async function () {
-    console.log("this is before runner");
     const effect = Effect.gen(function* (_) {
       const globalConfig = yield* _(
         Effect.tryPromise({
@@ -104,12 +103,9 @@ export function describeSuite<T extends FoundationType>({
     });
 
     ctx = await Effect.runPromise(effect);
-    console.log("Nodes in this instance are:");
-    console.log(ctx.nodes);
   });
 
   afterAll(async function () {
-    console.log("This is after all runner");
     const effect = ctx.destroyEffect().pipe(
       Effect.timeoutFail({
         duration: "10 seconds",
@@ -118,8 +114,6 @@ export function describeSuite<T extends FoundationType>({
     );
 
     await Effect.runPromise(effect);
-    console.log("Nodes leftover are:");
-    console.log(ctx.nodes);
     ctx = null;
   });
 

diff --git a/test/suites/basic/test_basic.ts b/test/suites/basic/test_basic.ts
@@ -16,7 +16,7 @@ describeSuite({
       id: "T01",
       title: "This is a bool test case",
       test: function () {
-        expect(true).to.be.true;
+        expect(false).to.be.true;
       },
     });
-Original file line number
+Diff line change
@@ Expand Up @@
             }
           }
         }
-        console.log("this is runningNode.pid", runningNode.pid);
         return { pid: runningNode.pid, kill: runningNode.kill } satisfies LaunchedNode;
       });
@@ Expand Down @@