| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646 |
- import { h, nextTick, nodeOps, render, serializeInner } from '@vue/runtime-test'
- import {
- type DebuggerEvent,
- ITERATE_KEY,
- TrackOpTypes,
- TriggerOpTypes,
- type WritableComputedRef,
- computed,
- effect,
- isReadonly,
- reactive,
- ref,
- shallowRef,
- toRaw,
- } from '../src'
- import { DirtyLevels } from '../src/constants'
- import { COMPUTED_SIDE_EFFECT_WARN } from '../src/computed'
- describe('reactivity/computed', () => {
- it('should return updated value', () => {
- const value = reactive<{ foo?: number }>({})
- const cValue = computed(() => value.foo)
- expect(cValue.value).toBe(undefined)
- value.foo = 1
- expect(cValue.value).toBe(1)
- })
- it('should compute lazily', () => {
- const value = reactive<{ foo?: number }>({})
- const getter = vi.fn(() => value.foo)
- const cValue = computed(getter)
- // lazy
- expect(getter).not.toHaveBeenCalled()
- expect(cValue.value).toBe(undefined)
- expect(getter).toHaveBeenCalledTimes(1)
- // should not compute again
- cValue.value
- expect(getter).toHaveBeenCalledTimes(1)
- // should not compute until needed
- value.foo = 1
- expect(getter).toHaveBeenCalledTimes(1)
- // now it should compute
- expect(cValue.value).toBe(1)
- expect(getter).toHaveBeenCalledTimes(2)
- // should not compute again
- cValue.value
- expect(getter).toHaveBeenCalledTimes(2)
- })
- it('should trigger effect', () => {
- const value = reactive<{ foo?: number }>({})
- const cValue = computed(() => value.foo)
- let dummy
- effect(() => {
- dummy = cValue.value
- })
- expect(dummy).toBe(undefined)
- value.foo = 1
- expect(dummy).toBe(1)
- })
- it('should work when chained', () => {
- const value = reactive({ foo: 0 })
- const c1 = computed(() => value.foo)
- const c2 = computed(() => c1.value + 1)
- expect(c2.value).toBe(1)
- expect(c1.value).toBe(0)
- value.foo++
- expect(c2.value).toBe(2)
- expect(c1.value).toBe(1)
- })
- it('should trigger effect when chained', () => {
- const value = reactive({ foo: 0 })
- const getter1 = vi.fn(() => value.foo)
- const getter2 = vi.fn(() => {
- return c1.value + 1
- })
- const c1 = computed(getter1)
- const c2 = computed(getter2)
- let dummy
- effect(() => {
- dummy = c2.value
- })
- expect(dummy).toBe(1)
- expect(getter1).toHaveBeenCalledTimes(1)
- expect(getter2).toHaveBeenCalledTimes(1)
- value.foo++
- expect(dummy).toBe(2)
- // should not result in duplicate calls
- expect(getter1).toHaveBeenCalledTimes(2)
- expect(getter2).toHaveBeenCalledTimes(2)
- })
- it('should trigger effect when chained (mixed invocations)', () => {
- const value = reactive({ foo: 0 })
- const getter1 = vi.fn(() => value.foo)
- const getter2 = vi.fn(() => {
- return c1.value + 1
- })
- const c1 = computed(getter1)
- const c2 = computed(getter2)
- let dummy
- effect(() => {
- dummy = c1.value + c2.value
- })
- expect(dummy).toBe(1)
- expect(getter1).toHaveBeenCalledTimes(1)
- expect(getter2).toHaveBeenCalledTimes(1)
- value.foo++
- expect(dummy).toBe(3)
- // should not result in duplicate calls
- expect(getter1).toHaveBeenCalledTimes(2)
- expect(getter2).toHaveBeenCalledTimes(2)
- })
- it('should no longer update when stopped', () => {
- const value = reactive<{ foo?: number }>({})
- const cValue = computed(() => value.foo)
- let dummy
- effect(() => {
- dummy = cValue.value
- })
- expect(dummy).toBe(undefined)
- value.foo = 1
- expect(dummy).toBe(1)
- cValue.effect.stop()
- value.foo = 2
- expect(dummy).toBe(1)
- })
- it('should support setter', () => {
- const n = ref(1)
- const plusOne = computed({
- get: () => n.value + 1,
- set: val => {
- n.value = val - 1
- },
- })
- expect(plusOne.value).toBe(2)
- n.value++
- expect(plusOne.value).toBe(3)
- plusOne.value = 0
- expect(n.value).toBe(-1)
- })
- it('should trigger effect w/ setter', () => {
- const n = ref(1)
- const plusOne = computed({
- get: () => n.value + 1,
- set: val => {
- n.value = val - 1
- },
- })
- let dummy
- effect(() => {
- dummy = n.value
- })
- expect(dummy).toBe(1)
- plusOne.value = 0
- expect(dummy).toBe(-1)
- })
- // #5720
- it('should invalidate before non-computed effects', () => {
- let plusOneValues: number[] = []
- const n = ref(0)
- const plusOne = computed(() => n.value + 1)
- effect(() => {
- n.value
- plusOneValues.push(plusOne.value)
- })
- // access plusOne, causing it to be non-dirty
- plusOne.value
- // mutate n
- n.value++
- // on the 2nd run, plusOne.value should have already updated.
- expect(plusOneValues).toMatchObject([1, 2])
- })
- it('should warn if trying to set a readonly computed', () => {
- const n = ref(1)
- const plusOne = computed(() => n.value + 1)
- ;(plusOne as WritableComputedRef<number>).value++ // Type cast to prevent TS from preventing the error
- expect(
- 'Write operation failed: computed value is readonly',
- ).toHaveBeenWarnedLast()
- })
- it('should be readonly', () => {
- let a = { a: 1 }
- const x = computed(() => a)
- expect(isReadonly(x)).toBe(true)
- expect(isReadonly(x.value)).toBe(false)
- expect(isReadonly(x.value.a)).toBe(false)
- const z = computed<typeof a>({
- get() {
- return a
- },
- set(v) {
- a = v
- },
- })
- expect(isReadonly(z)).toBe(false)
- expect(isReadonly(z.value.a)).toBe(false)
- })
- it('should expose value when stopped', () => {
- const x = computed(() => 1)
- x.effect.stop()
- expect(x.value).toBe(1)
- })
- it('debug: onTrack', () => {
- let events: DebuggerEvent[] = []
- const onTrack = vi.fn((e: DebuggerEvent) => {
- events.push(e)
- })
- const obj = reactive({ foo: 1, bar: 2 })
- const c = computed(() => (obj.foo, 'bar' in obj, Object.keys(obj)), {
- onTrack,
- })
- expect(c.value).toEqual(['foo', 'bar'])
- expect(onTrack).toHaveBeenCalledTimes(3)
- expect(events).toEqual([
- {
- effect: c.effect,
- target: toRaw(obj),
- type: TrackOpTypes.GET,
- key: 'foo',
- },
- {
- effect: c.effect,
- target: toRaw(obj),
- type: TrackOpTypes.HAS,
- key: 'bar',
- },
- {
- effect: c.effect,
- target: toRaw(obj),
- type: TrackOpTypes.ITERATE,
- key: ITERATE_KEY,
- },
- ])
- })
- it('debug: onTrigger (reactive)', () => {
- let events: DebuggerEvent[] = []
- const onTrigger = vi.fn((e: DebuggerEvent) => {
- events.push(e)
- })
- const obj = reactive<{ foo?: number }>({ foo: 1 })
- const c = computed(() => obj.foo, { onTrigger })
- // computed won't trigger compute until accessed
- c.value
- obj.foo!++
- expect(c.value).toBe(2)
- expect(onTrigger).toHaveBeenCalledTimes(1)
- expect(events[0]).toEqual({
- effect: c.effect,
- target: toRaw(obj),
- type: TriggerOpTypes.SET,
- key: 'foo',
- oldValue: 1,
- newValue: 2,
- })
- delete obj.foo
- expect(c.value).toBeUndefined()
- expect(onTrigger).toHaveBeenCalledTimes(2)
- expect(events[1]).toEqual({
- effect: c.effect,
- target: toRaw(obj),
- type: TriggerOpTypes.DELETE,
- key: 'foo',
- oldValue: 2,
- })
- })
- // https://github.com/vuejs/core/pull/5912#issuecomment-1497596875
- it('should query deps dirty sequentially', () => {
- const cSpy = vi.fn()
- const a = ref<null | { v: number }>({
- v: 1,
- })
- const b = computed(() => {
- return a.value
- })
- const c = computed(() => {
- cSpy()
- return b.value?.v
- })
- const d = computed(() => {
- if (b.value) {
- return c.value
- }
- return 0
- })
- d.value
- a.value!.v = 2
- a.value = null
- d.value
- expect(cSpy).toHaveBeenCalledTimes(1)
- })
- // https://github.com/vuejs/core/pull/5912#issuecomment-1738257692
- it('chained computed dirty reallocation after querying dirty', () => {
- let _msg: string | undefined
- const items = ref<number[]>()
- const isLoaded = computed(() => {
- return !!items.value
- })
- const msg = computed(() => {
- if (isLoaded.value) {
- return 'The items are loaded'
- } else {
- return 'The items are not loaded'
- }
- })
- effect(() => {
- _msg = msg.value
- })
- items.value = [1, 2, 3]
- items.value = [1, 2, 3]
- items.value = undefined
- expect(_msg).toBe('The items are not loaded')
- })
- it('chained computed dirty reallocation after trigger computed getter', () => {
- let _msg: string | undefined
- const items = ref<number[]>()
- const isLoaded = computed(() => {
- return !!items.value
- })
- const msg = computed(() => {
- if (isLoaded.value) {
- return 'The items are loaded'
- } else {
- return 'The items are not loaded'
- }
- })
- _msg = msg.value
- items.value = [1, 2, 3]
- isLoaded.value // <- trigger computed getter
- _msg = msg.value
- items.value = undefined
- _msg = msg.value
- expect(_msg).toBe('The items are not loaded')
- })
- // https://github.com/vuejs/core/pull/5912#issuecomment-1739159832
- it('deps order should be consistent with the last time get value', () => {
- const cSpy = vi.fn()
- const a = ref(0)
- const b = computed(() => {
- return a.value % 3 !== 0
- })
- const c = computed(() => {
- cSpy()
- if (a.value % 3 === 2) {
- return 'expensive'
- }
- return 'cheap'
- })
- const d = computed(() => {
- return a.value % 3 === 2
- })
- const e = computed(() => {
- if (b.value) {
- if (d.value) {
- return 'Avoiding expensive calculation'
- }
- }
- return c.value
- })
- e.value
- a.value++
- e.value
- expect(e.effect.deps.length).toBe(3)
- expect(e.effect.deps.indexOf((b as any).dep)).toBe(0)
- expect(e.effect.deps.indexOf((d as any).dep)).toBe(1)
- expect(e.effect.deps.indexOf((c as any).dep)).toBe(2)
- expect(cSpy).toHaveBeenCalledTimes(2)
- a.value++
- e.value
- expect(cSpy).toHaveBeenCalledTimes(2)
- })
- it('should trigger by the second computed that maybe dirty', () => {
- const cSpy = vi.fn()
- const src1 = ref(0)
- const src2 = ref(0)
- const c1 = computed(() => src1.value)
- const c2 = computed(() => (src1.value % 2) + src2.value)
- const c3 = computed(() => {
- cSpy()
- c1.value
- c2.value
- })
- c3.value
- src1.value = 2
- c3.value
- expect(cSpy).toHaveBeenCalledTimes(2)
- src2.value = 1
- c3.value
- expect(cSpy).toHaveBeenCalledTimes(3)
- })
- it('should trigger the second effect', () => {
- const fnSpy = vi.fn()
- const v = ref(1)
- const c = computed(() => v.value)
- effect(() => {
- c.value
- })
- effect(() => {
- c.value
- fnSpy()
- })
- expect(fnSpy).toBeCalledTimes(1)
- v.value = 2
- expect(fnSpy).toBeCalledTimes(2)
- })
- it('should chained recurse effects clear dirty after trigger', () => {
- const v = ref(1)
- const c1 = computed(() => v.value)
- const c2 = computed(() => c1.value)
- c1.effect.allowRecurse = true
- c2.effect.allowRecurse = true
- c2.value
- expect(c1.effect._dirtyLevel).toBe(DirtyLevels.NotDirty)
- expect(c2.effect._dirtyLevel).toBe(DirtyLevels.NotDirty)
- })
- it('should chained computeds dirtyLevel update with first computed effect', () => {
- const v = ref(0)
- const c1 = computed(() => {
- if (v.value === 0) {
- v.value = 1
- }
- return v.value
- })
- const c2 = computed(() => c1.value)
- const c3 = computed(() => c2.value)
- c3.value
- expect(c1.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
- expect(c2.effect._dirtyLevel).toBe(
- DirtyLevels.MaybeDirty_ComputedSideEffect,
- )
- expect(c3.effect._dirtyLevel).toBe(
- DirtyLevels.MaybeDirty_ComputedSideEffect,
- )
- expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
- })
- it('should work when chained(ref+computed)', () => {
- const v = ref(0)
- const c1 = computed(() => {
- if (v.value === 0) {
- v.value = 1
- }
- return 'foo'
- })
- const c2 = computed(() => v.value + c1.value)
- expect(c2.value).toBe('0foo')
- expect(c2.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
- expect(c2.value).toBe('1foo')
- expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
- })
- it('should trigger effect even computed already dirty', () => {
- const fnSpy = vi.fn()
- const v = ref(0)
- const c1 = computed(() => {
- if (v.value === 0) {
- v.value = 1
- }
- return 'foo'
- })
- const c2 = computed(() => v.value + c1.value)
- effect(() => {
- fnSpy()
- c2.value
- })
- expect(fnSpy).toBeCalledTimes(1)
- expect(c1.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
- expect(c2.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
- v.value = 2
- expect(fnSpy).toBeCalledTimes(2)
- expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
- })
- // #10185
- it('should not override queried MaybeDirty result', () => {
- class Item {
- v = ref(0)
- }
- const v1 = shallowRef()
- const v2 = ref(false)
- const c1 = computed(() => {
- let c = v1.value
- if (!v1.value) {
- c = new Item()
- v1.value = c
- }
- return c.v.value
- })
- const c2 = computed(() => {
- if (!v2.value) return 'no'
- return c1.value ? 'yes' : 'no'
- })
- const c3 = computed(() => c2.value)
- c3.value
- v2.value = true
- expect(c2.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
- expect(c3.effect._dirtyLevel).toBe(DirtyLevels.MaybeDirty)
- c3.value
- expect(c1.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
- expect(c2.effect._dirtyLevel).toBe(
- DirtyLevels.MaybeDirty_ComputedSideEffect,
- )
- expect(c3.effect._dirtyLevel).toBe(
- DirtyLevels.MaybeDirty_ComputedSideEffect,
- )
- v1.value.v.value = 999
- expect(c1.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
- expect(c2.effect._dirtyLevel).toBe(DirtyLevels.MaybeDirty)
- expect(c3.effect._dirtyLevel).toBe(DirtyLevels.MaybeDirty)
- expect(c3.value).toBe('yes')
- expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
- })
- it('should be not dirty after deps mutate (mutate deps in computed)', async () => {
- const state = reactive<any>({})
- const consumer = computed(() => {
- if (!('a' in state)) state.a = 1
- return state.a
- })
- const Comp = {
- setup: () => {
- nextTick().then(() => {
- state.a = 2
- })
- return () => consumer.value
- },
- }
- const root = nodeOps.createElement('div')
- render(h(Comp), root)
- await nextTick()
- await nextTick()
- expect(serializeInner(root)).toBe(`2`)
- expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
- })
- it('should not trigger effect scheduler by recurse computed effect', async () => {
- const v = ref('Hello')
- const c = computed(() => {
- v.value += ' World'
- return v.value
- })
- const Comp = {
- setup: () => {
- return () => c.value
- },
- }
- const root = nodeOps.createElement('div')
- render(h(Comp), root)
- await nextTick()
- expect(serializeInner(root)).toBe('Hello World')
- v.value += ' World'
- await nextTick()
- expect(serializeInner(root)).toBe('Hello World World World World')
- expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
- })
- it('debug: onTrigger (ref)', () => {
- let events: DebuggerEvent[] = []
- const onTrigger = vi.fn((e: DebuggerEvent) => {
- events.push(e)
- })
- const obj = ref(1)
- const c = computed(() => obj.value, { onTrigger })
- // computed won't trigger compute until accessed
- c.value
- obj.value++
- expect(c.value).toBe(2)
- expect(onTrigger).toHaveBeenCalledTimes(1)
- expect(events[0]).toEqual({
- effect: c.effect,
- target: toRaw(obj),
- type: TriggerOpTypes.SET,
- key: 'value',
- oldValue: 1,
- newValue: 2,
- })
- })
- })
|