Browse Source

chore(reactivity): correct the description of the test case (#4135)

Pick 4 years ago
parent
commit
35cc7b0d31
1 changed files with 4 additions and 2 deletions
  1. 4 2
      packages/reactivity/__tests__/effect.spec.ts

+ 4 - 2
packages/reactivity/__tests__/effect.spec.ts

@@ -850,7 +850,7 @@ describe('reactivity/effect', () => {
     expect(dummy).toBe(1)
   })
 
-  it('should not be trigger when the value and the old value both are NaN', () => {
+  it('should not be triggered when the value and the old value both are NaN', () => {
     const obj = reactive({
       foo: NaN
     })
@@ -884,13 +884,14 @@ describe('reactivity/effect', () => {
     expect(record).toBeUndefined()
   })
 
-  it('should trigger once effect when set the equal proxy', () => {
+  it('should not be triggered when set with the same proxy', () => {
     const obj = reactive({ foo: 1 })
     const observed: any = reactive({ obj })
     const fnSpy = jest.fn(() => observed.obj)
 
     effect(fnSpy)
 
+    expect(fnSpy).toHaveBeenCalledTimes(1)
     observed.obj = obj
     expect(fnSpy).toHaveBeenCalledTimes(1)
 
@@ -900,6 +901,7 @@ describe('reactivity/effect', () => {
 
     effect(fnSpy2)
 
+    expect(fnSpy2).toHaveBeenCalledTimes(1)
     observed2.obj2 = obj2
     expect(fnSpy2).toHaveBeenCalledTimes(1)
   })