fix effect vitest layer (#4382)

Effect-TS · Feb 1, 2025 · e374d94 · e374d94
1 parent fd92e69
commit e374d94
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 2 deletions.
diff --git a/.changeset/six-ghosts-pull.md b/.changeset/six-ghosts-pull.md
@@ -0,0 +1,5 @@
+---
+"@effect/vitest": patch
+---
+
+Fix @effect.vitest layer method not running tests in some cases
diff --git a/packages/vitest/src/internal.ts b/packages/vitest/src/internal.ts
@@ -227,7 +227,7 @@ export const layer = <R, E>(layer_: Layer.Layer<R, E>, options?: {
     return args[0](makeIt(V.it))
   }
 
-  return V.describe(args[0], (it) => {
+  return V.describe(args[0], () => {
     V.beforeAll(
       () => runPromise()(Effect.asVoid(runtimeEffect)),
       options?.timeout ? Duration.toMillis(options.timeout) : undefined
@@ -236,7 +236,7 @@ export const layer = <R, E>(layer_: Layer.Layer<R, E>, options?: {
       () => runPromise()(Scope.close(scope, Exit.void)),
       options?.timeout ? Duration.toMillis(options.timeout) : undefined
     )
-    return args[1](makeIt(it))
+    return args[1](makeIt(V.it))
   })
 }
 

diff --git a/packages/vitest/test/index.test.ts b/packages/vitest/test/index.test.ts
@@ -182,6 +182,21 @@ describe("layer", () => {
         yield* Fiber.join(fiber)
       }))
   })
+
+  layer(Foo.Live)("with a name", (it) => {
+    describe("with a nested describe", () => {
+      it.effect("adds context", () =>
+        Effect.gen(function*() {
+          const foo = yield* Foo
+          expect(foo).toEqual("foo")
+        }))
+    })
+    it.effect("adds context", () =>
+      Effect.gen(function*() {
+        const foo = yield* Foo
+        expect(foo).toEqual("foo")
+      }))
+  })
 })
 
 // property testing