From: Maisha Tremblay <65948611+coderwei99@users.noreply.github.com>
Date: Mon, 2 Sep 2024 09:08:19 +0000 (+0800)
Subject: test(reactive): use vitest fn instead of counting manually (#11746)
X-Git-Tag: v3.5.0~16
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=3de55565214807c7d4b9b374d00e6f7d560bb3ce;p=thirdparty%2Fvuejs%2Fcore.git

test(reactive): use vitest fn instead of counting manually (#11746)
---

diff --git a/packages/reactivity/__tests__/ref.spec.ts b/packages/reactivity/__tests__/ref.spec.ts
index 56863471ad..7976a5373b 100644
--- a/packages/reactivity/__tests__/ref.spec.ts
+++ b/packages/reactivity/__tests__/ref.spec.ts
@@ -46,26 +46,22 @@ describe('reactivity/ref', () => {
   it('ref wrapped in reactive should not track internal _value access', () => {
     const a = ref(1)
     const b = reactive(a)
-    let calls = 0
     let dummy
-
-    effect(() => {
-      calls++
+    const fn = vi.fn(() => {
       dummy = b.value // this will observe both b.value and a.value access
     })
-    expect(calls).toBe(1)
+    effect(fn)
+    expect(fn).toHaveBeenCalledTimes(1)
     expect(dummy).toBe(1)
 
     // mutating a.value should only trigger effect once
-    calls = 0
     a.value = 3
-    expect(calls).toBe(1)
+    expect(fn).toHaveBeenCalledTimes(2)
     expect(dummy).toBe(3)
 
     // mutating b.value should trigger the effect twice. (once for a.value change and once for b.value change)
-    calls = 0
     b.value = 5
-    expect(calls).toBe(2)
+    expect(fn).toHaveBeenCalledTimes(4)
     expect(dummy).toBe(5)
   })