@playfast/reform 1.2.0 → 1.2.1

package/src/event/event.fromSource.test.ts

@@ -1,13 +1,13 @@
 import { expect, it } from '@effect/vitest'
-import { Duration, Effect, Exit, Layer, Option, Schema as S, Scope } from 'effect'
+import { Effect, Exit, Layer, Option, Schema as S, Scope } from 'effect'
 import { Engine, Event, Reducer, State } from '../index'
+import { flush } from '../testkit/flight.testkit'
 
 // `Event.liveFromSource` turns a Source *transition* into an ordinary event, so
 // a rule can be triggered by a value moving rather than by a UI dispatch. Every
 // case observes through a reducer — the framework's own writer — so what the
 // log holds is exactly what the bus carried, with no spy in the middle.
 
-const tick = Effect.sleep(Duration.millis(1))
 
 // One string carrying both ends of a transition: `none->0` for the initial
 // emission (nothing preceded it), `0->1` for a change.
@@ -47,17 +47,17 @@ it.live('emits nothing for the value present at construction (changes only)', ()
     const level = yield* Level.store
     const log = yield* Log.store
 
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual([])
 
     level.set(1)
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['0->1'])
 
     // Store equality is the framework's, not this adapter's: re-setting the same
     // value is not a change, so nothing notifies and nothing is emitted.
     level.set(1)
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['0->1'])
   }).pipe(Effect.provide(TestLayer))
 })
@@ -81,13 +81,13 @@ it.live('emitInitial reports the construction value once, with no previous', ()
     const level = yield* Level.store
     const log = yield* Log.store
 
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['none->2'])
 
     // The initial emission seeds `previous`: the next change reports the
     // construction value as its left-hand side, not another `none`.
     level.set(1)
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['none->2', '2->1'])
   }).pipe(Effect.provide(TestLayer))
 })
@@ -121,11 +121,11 @@ it.live('a payload of Option.none() suppresses a non-qualifying transition', ()
     const log = yield* Log.store
 
     phase.set('busy')
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual([])
 
     phase.set('ready')
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['busy'])
   }).pipe(Effect.provide(TestLayer))
 })
@@ -153,7 +153,7 @@ it.live('closing the scope ends the emissions, including one already in flight',
     const log = yield* Log.store
 
     level.set(1)
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['0->1'])
 
     // A write that already handed the listener to the notification scheduler,
@@ -162,11 +162,11 @@ it.live('closing the scope ends the emissions, including one already in flight',
       Effect.sync(() => level.set(2)),
       Scope.close(scope, Exit.void),
     )
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['0->1'])
 
     level.set(0)
-    yield* tick
+    yield* flush()
     expect(log.get()).toEqual(['0->1'])
   }).pipe(Effect.provide(TestLayer))
 })