]> git.ipfire.org Git - thirdparty/vuejs/core.git/commitdiff
test: fix reactivity tests
authorEvan You <yyx990803@gmail.com>
Tue, 20 Aug 2019 13:58:10 +0000 (09:58 -0400)
committerEvan You <yyx990803@gmail.com>
Tue, 20 Aug 2019 13:58:10 +0000 (09:58 -0400)
14 files changed:
packages/reactivity/README.md
packages/reactivity/__tests__/collections/Map.spec.ts
packages/reactivity/__tests__/collections/Set.spec.ts
packages/reactivity/__tests__/collections/WeakMap.spec.ts
packages/reactivity/__tests__/collections/WeakSet.spec.ts
packages/reactivity/__tests__/computed.spec.ts
packages/reactivity/__tests__/effect.spec.ts
packages/reactivity/__tests__/immutable.spec.ts [moved from packages/reactivity/__tests__/immutableState.spec.ts with 97% similarity]
packages/reactivity/__tests__/reactive.spec.ts
packages/reactivity/__tests__/ref.spec.ts
packages/reactivity/src/reactive.ts
packages/runtime-core/package.json
packages/runtime-core/src/component.ts
packages/runtime-dom/src/index.ts

index eec656ac1a65a92396dab16eb5455128b6d6f627..d633eb6f3f764ccdaab6b11b73ee839cb983224f 100644 (file)
@@ -1,4 +1,4 @@
-# @vue/observer
+# @vue/reactivity
 
 ## Usage Note
 
@@ -9,10 +9,9 @@ This package is inlined into Global & Browser ESM builds of user-facing renderer
 The implementation of this module is inspired by the following prior art in the JavaScript ecosystem:
 
 - [Meteor Tracker](https://docs.meteor.com/api/tracker.html)
-- [nx-js/observer-util](https://github.com/nx-js/observer-util)
+- [nx-js/reactivity-util](https://github.com/nx-js/reactivity-util)
 - [salesforce/observable-membrane](https://github.com/salesforce/observable-membrane)
 
-
 ## Caveats
 
 - Built-in objects are not observed except for `Map`, `WeakMap`, `Set` and `WeakSet`.
index 14558270abde18d057aa6ed46b5346cc968c2d7e..bb5f4f6b2a45747870b39428cba27701cec76dd6 100644 (file)
@@ -1,6 +1,6 @@
 import { reactive, effect, toRaw, isReactive } from '../../src'
 
-describe('observer/collections', () => {
+describe('reactivity/collections', () => {
   describe('Map', () => {
     test('instanceof', () => {
       const original = new Map()
index 1fc4bfda582cd934f7c2012e210b814399387c3a..3efa18250ea5217f1ff34a4807b5489c304265ea 100644 (file)
@@ -1,6 +1,6 @@
 import { reactive, effect, isReactive, toRaw } from '../../src'
 
-describe('observer/collections', () => {
+describe('reactivity/collections', () => {
   describe('Set', () => {
     it('instanceof', () => {
       const original = new Set()
index 0ad7b93a26da1b5569e53cbcda2a95d62195a7aa..ddf7db597fd800f7e5b0d54c08ae9de7738a3b17 100644 (file)
@@ -1,6 +1,6 @@
 import { reactive, effect, toRaw, isReactive } from '../../src'
 
-describe('observer/collections', () => {
+describe('reactivity/collections', () => {
   describe('WeakMap', () => {
     test('instanceof', () => {
       const original = new WeakMap()
index e26d40294cba47ef3fbb56ded220586dfe21bfc6..3dc2cc838831643bce12a0eb924796a9406865a3 100644 (file)
@@ -1,6 +1,6 @@
 import { reactive, isReactive, effect, toRaw } from '../../src'
 
-describe('observer/collections', () => {
+describe('reactivity/collections', () => {
   describe('WeakSet', () => {
     it('instanceof', () => {
       const original = new Set()
index abde4e0848a5af6f0031b87cc757fe2ba94ca97a..585f7abb33950fec356a4878d345a3c81cd2b0ef 100644 (file)
@@ -1,6 +1,6 @@
 import { computed, reactive, effect, stop } from '../src'
 
-describe('observer/computed', () => {
+describe('reactivity/computed', () => {
   it('should return updated value', () => {
     const value: any = reactive({})
     const cValue = computed(() => value.foo)
index a99e9a40b4a084a75a7270ecdbc4e48fd1a01030..650eda9cebc656ac11fd78d68342dec33e859d93 100644 (file)
@@ -9,7 +9,7 @@ import {
 } from '../src/index'
 import { ITERATE_KEY } from '../src/effect'
 
-describe('observer/effect', () => {
+describe('reactivity/effect', () => {
   it('should run the passed function once (wrapped by a effect)', () => {
     const fnSpy = jest.fn(() => {})
     effect(fnSpy)
@@ -269,7 +269,7 @@ describe('observer/effect', () => {
 
   it('should not observe raw mutations', () => {
     let dummy
-    const obj: any = reactive()
+    const obj: any = reactive({})
     effect(() => (dummy = toRaw(obj).prop))
 
     expect(dummy).toBe(undefined)
@@ -279,7 +279,7 @@ describe('observer/effect', () => {
 
   it('should not be triggered by raw mutations', () => {
     let dummy
-    const obj: any = reactive()
+    const obj: any = reactive({})
     effect(() => (dummy = obj.prop))
 
     expect(dummy).toBe(undefined)
@@ -437,7 +437,7 @@ describe('observer/effect', () => {
 
   it('should not run multiple times for a single mutation', () => {
     let dummy
-    const obj: any = reactive()
+    const obj: any = reactive({})
     const fnSpy = jest.fn(() => {
       for (const key in obj) {
         dummy = obj[key]
similarity index 97%
rename from packages/reactivity/__tests__/immutableState.spec.ts
rename to packages/reactivity/__tests__/immutable.spec.ts
index e30d027a18fa2a4029f4209f0734c26615a0d72b..83d4de7547af41c0ce5433bae8944ad0c44af88c 100644 (file)
@@ -11,7 +11,7 @@ import {
   effect
 } from '../src'
 
-describe('observer/immutable', () => {
+describe('reactivity/immutable', () => {
   let warn: any
 
   beforeEach(() => {
@@ -341,16 +341,16 @@ describe('observer/immutable', () => {
     })
   })
 
-  test('calling observable on an immutable should return immutable', () => {
-    const a = immutable()
+  test('calling reactive on an immutable should return immutable', () => {
+    const a = immutable({})
     const b = reactive(a)
     expect(isImmutable(b)).toBe(true)
     // should point to same original
     expect(toRaw(a)).toBe(toRaw(b))
   })
 
-  test('calling immutable on an observable should return immutable', () => {
-    const a = reactive()
+  test('calling immutable on a reactive object should return immutable', () => {
+    const a = reactive({})
     const b = immutable(a)
     expect(isImmutable(b)).toBe(true)
     // should point to same original
index 0fe68e73592597ee2fc8b642c1d520f34e1ef79f..8888aaed10b215ac8b6cadf8b3650e34b2dcc994 100644 (file)
@@ -1,6 +1,6 @@
 import { reactive, isReactive, toRaw, markNonReactive } from '../src/reactive'
 
-describe('observer/observable', () => {
+describe('reactivity/reactive', () => {
   test('Object', () => {
     const original = { foo: 1 }
     const observed = reactive(original)
@@ -30,7 +30,7 @@ describe('observer/observable', () => {
     expect(Object.keys(observed)).toEqual(['0'])
   })
 
-  test('cloned observable Array should point to observed values', () => {
+  test('cloned reactive Array should point to observed values', () => {
     const original = [{ foo: 1 }]
     const observed = reactive(original)
     const clone = observed.slice()
@@ -39,7 +39,7 @@ describe('observer/observable', () => {
     expect(clone[0]).toBe(observed[0])
   })
 
-  test('nested observables', () => {
+  test('nested reactives', () => {
     const original = {
       nested: {
         foo: 1
@@ -70,9 +70,9 @@ describe('observer/observable', () => {
     const observed = reactive(original)
     // set
     const value = { baz: 3 }
-    const observableValue = reactive(value)
+    const reactiveValue = reactive(value)
     observed[0] = value
-    expect(observed[0]).toBe(observableValue)
+    expect(observed[0]).toBe(reactiveValue)
     expect(original[0]).toBe(value)
     // delete
     delete observed[0]
@@ -80,11 +80,11 @@ describe('observer/observable', () => {
     expect(original[0]).toBeUndefined()
     // mutating methods
     observed.push(value)
-    expect(observed[2]).toBe(observableValue)
+    expect(observed[2]).toBe(reactiveValue)
     expect(original[2]).toBe(value)
   })
 
-  test('setting a property with an unobserved value should wrap with observable', () => {
+  test('setting a property with an unobserved value should wrap with reactive', () => {
     const observed: any = reactive({})
     const raw = {}
     observed.foo = raw
@@ -123,14 +123,15 @@ describe('observer/observable', () => {
     expect(toRaw(original)).toBe(original)
   })
 
-  test('unobservable values', () => {
+  test('non-observable values', () => {
     const warn = jest.spyOn(console, 'warn')
     let lastMsg: string
     warn.mockImplementation(msg => {
       lastMsg = msg
     })
 
-    const getMsg = (value: any) => `value is not observable: ${String(value)}`
+    const getMsg = (value: any) =>
+      `value cannot be made reactive: ${String(value)}`
     const assertValue = (value: any) => {
       reactive(value)
       expect(lastMsg).toMatch(getMsg(value))
@@ -144,10 +145,8 @@ describe('observer/observable', () => {
     assertValue(false)
     // null
     assertValue(null)
-    // undefined should work because it returns empty object observable
-    lastMsg = ''
-    reactive(undefined)
-    expect(lastMsg).toBe('')
+    // undefined
+    assertValue(undefined)
     // symbol
     const s = Symbol()
     assertValue(s)
index e5cff502f958d4959c9d35b05bfaf49ebef6bc9f..2d26c933af53ad2770cefc69c76eef7ad01d2f3c 100644 (file)
@@ -1,6 +1,6 @@
 import { ref, effect, reactive } from '../src/index'
 
-describe('observer/value', () => {
+describe('reactivity/value', () => {
   it('should hold a value', () => {
     const a = ref(1)
     expect(a.value).toBe(1)
index e09882bba80eab87c4d6933abb0967ec878179b1..c88ac28bd7933da11b40a8ac97714e4cf7671eb6 100644 (file)
@@ -51,7 +51,7 @@ export const reactive = ((target: unknown): any => {
   if (immutableValues.has(target)) {
     return immutable(target)
   }
-  return createObservable(
+  return createReactiveObject(
     target,
     rawToObserved,
     observedToRaw,
@@ -66,7 +66,7 @@ export const immutable = ((target: unknown): any => {
   if (observedToRaw.has(target)) {
     target = observedToRaw.get(target)
   }
-  return createObservable(
+  return createReactiveObject(
     target,
     rawToImmutable,
     immutableToRaw,
@@ -75,7 +75,7 @@ export const immutable = ((target: unknown): any => {
   )
 }) as ObservableFactory
 
-function createObservable(
+function createReactiveObject(
   target: any,
   toProxy: WeakMap<any, any>,
   toRaw: WeakMap<any, any>,
@@ -84,7 +84,7 @@ function createObservable(
 ) {
   if (!isObject(target)) {
     if (__DEV__) {
-      console.warn(`value is not observable: ${String(target)}`)
+      console.warn(`value cannot be made reactive: ${String(target)}`)
     }
     return target
   }
index 81356f1ce80e70d5343b93f32ee0dcb0a2425655..14d2cb5c52425f703219c3c3ebbd13001c3ff741 100644 (file)
@@ -20,6 +20,6 @@
   },
   "homepage": "https://github.com/vuejs/vue/tree/dev/packages/runtime-core#readme",
   "dependencies": {
-    "@vue/observer": "3.0.0-alpha.1"
+    "@vue/reactivity": "3.0.0-alpha.1"
   }
 }
index 117f4b4760b7be67b53dd67e2c9ac550d3cfbdff..25c2f6b70449be013245c2d0720504698e02aa41 100644 (file)
@@ -242,7 +242,7 @@ export function setupStatefulComponent(instance: ComponentInstance) {
     } else {
       // setup returned bindings.
       // assuming a render function compiled from template is present.
-      instance.data = reactive(setupResult)
+      instance.data = reactive(setupResult || {})
       if (__DEV__ && !Component.render) {
         // TODO warn missing render fn
       }
index e687f1da77190343fd32d31975fd1134876ebdc9..019999ff1ae0642c0db5fd2b37930b5364ce50f6 100644 (file)
@@ -8,5 +8,5 @@ export const render = createRenderer({
 }) as (vnode: VNode | null, container: HTMLElement) => VNode
 
 // re-export everything from core
-// h, Component, observer API, nextTick, flags & types
+// h, Component, reactivity API, nextTick, flags & types
 export * from '@vue/runtime-core'