Skip to content

Commit 4268030

Browse files
authored
test(reactivity): 100% reactivity coverage (#1299)
1 parent 3fafc3b commit 4268030

File tree

3 files changed

+17
-0
lines changed

3 files changed

+17
-0
lines changed

packages/reactivity/__tests__/reactive.spec.ts

+9
Original file line numberDiff line numberDiff line change
@@ -177,4 +177,13 @@ describe('reactivity/reactive', () => {
177177
})
178178
expect(isReactive(obj.foo)).toBe(false)
179179
})
180+
181+
test('should not observe objects with __v_skip', () => {
182+
const original = {
183+
foo: 1,
184+
__v_skip: true
185+
}
186+
const observed = reactive(original)
187+
expect(isReactive(observed)).toBe(false)
188+
})
180189
})

packages/reactivity/__tests__/readonly.spec.ts

+2
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,7 @@ describe('reactivity/readonly', () => {
214214
const key2 = {}
215215
const original = new Collection([[key1, {}], [key2, {}]])
216216
const wrapped: any = readonly(original)
217+
expect(wrapped.size).toBe(2)
217218
for (const [key, value] of wrapped) {
218219
expect(isReadonly(key)).toBe(true)
219220
expect(isReadonly(value)).toBe(true)
@@ -267,6 +268,7 @@ describe('reactivity/readonly', () => {
267268
test('should retrieve readonly values on iteration', () => {
268269
const original = new Collection([{}, {}])
269270
const wrapped: any = readonly(original)
271+
expect(wrapped.size).toBe(2)
270272
for (const value of wrapped) {
271273
expect(isReadonly(value)).toBe(true)
272274
}

packages/reactivity/__tests__/ref.spec.ts

+6
Original file line numberDiff line numberDiff line change
@@ -277,6 +277,12 @@ describe('reactivity/ref', () => {
277277
expect(dummyY).toBe(5)
278278
})
279279

280+
test('toRefs pass a reactivity object', () => {
281+
console.warn = jest.fn()
282+
const obj = { x: 1 }
283+
toRefs(obj)
284+
expect(console.warn).toBeCalled()
285+
})
280286
test('customRef', () => {
281287
let value = 1
282288
let _trigger: () => void

0 commit comments

Comments
 (0)