test(lens): fix distinct changes test

raveclassic · May 24, 2022 · 446247e · 446247e
1 parent 0b0047f
commit 446247e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 22 deletions.
diff --git a/packages/core/src/property.spec.ts b/packages/core/src/property.spec.ts
@@ -206,28 +206,6 @@ describe('combine', () => {
 		a.set(undefined)
 		expect(cb).toHaveBeenCalledTimes(1)
 	})
-	it('does not emit if combined property has not change value', () => {
-		const a = newAtom({ foo: 1, bar: '1' })
-		const foo = combine(a, (a) => a.foo)
-		const bar = combine(a, (a) => a.bar)
-
-		const fooObserver = {
-			next: jest.fn(),
-		}
-		const barObserver = {
-			next: jest.fn(),
-		}
-		foo.subscribe(fooObserver)
-		bar.subscribe(barObserver)
-
-		a.modify(p => ({ ...p, foo: 2 }))
-		expect(fooObserver.next).toHaveBeenCalledTimes(1)
-		expect(barObserver.next).toHaveBeenCalledTimes(0)
-
-		a.modify(p => ({ ...p, bar: '2' }))
-		expect(barObserver.next).toHaveBeenCalledTimes(1)
-		expect(fooObserver.next).toHaveBeenCalledTimes(1)
-	})
 })
 
 describe('flatten', () => {

diff --git a/packages/lens/src/lensed-atom.spec.ts b/packages/lens/src/lensed-atom.spec.ts
@@ -121,6 +121,9 @@ describe('view', () => {
 		}
 		foo.subscribe(fooObserver)
 		bar.subscribe(barObserver)
+		// imitate consumer to warm up the cache
+		foo.get()
+		bar.get()
 
 		foo.set(2)
 		expect(fooObserver.next).toHaveBeenCalledTimes(1)