throw error

Author: 38elements

Diff for: packages/test-utils/src/vue-wrapper.js

@@ -1,6 +1,7 @@
 // @flow
 
 import Wrapper from './wrapper'
+import { throwError } from 'shared/util'
 import { setWatchersToSync } from './set-watchers-to-sync'
 import { orderWatchers } from './order-watchers'
 
@@ -11,17 +12,17 @@ export default class VueWrapper extends Wrapper implements BaseWrapper {
     // $FlowIgnore : issue with defineProperty
     Object.defineProperty(this, 'vnode', {
       get: () => vm._vnode,
-      set: () => {}
+      set: () => throwError(`VueWrapper.vnode is read-only`)
     })
     // $FlowIgnore
     Object.defineProperty(this, 'element', {
       get: () => vm.$el,
-      set: () => {}
+      set: () => throwError(`VueWrapper.element is read-only`)
     })
     // $FlowIgnore
     Object.defineProperty(this, 'vm', {
       get: () => vm,
-      set: () => {}
+      set: () => throwError(`VueWrapper.vm is read-only`)
     })
     if (options.sync) {
       setWatchersToSync(vm)

Diff for: packages/test-utils/src/wrapper-array.js

@@ -13,12 +13,12 @@ export default class WrapperArray implements BaseWrapper {
     // $FlowIgnore
     Object.defineProperty(this, 'wrappers', {
       get: () => wrappers,
-      set: () => {}
+      set: () => throwError(`WrapperArray.wrappers is read-only`)
     })
     // $FlowIgnore
     Object.defineProperty(this, 'length', {
       get: () => length,
-      set: () => {}
+      set: () => throwError(`WrapperArray.length is read-only`)
     })
   }
 

Diff for: packages/test-utils/src/wrapper.js

@@ -24,6 +24,7 @@ import { orderWatchers } from './order-watchers'
 
 export default class Wrapper implements BaseWrapper {
   +vnode: VNode | null;
+  _vnode: VNode | null;
   +vm: Component | null;
   _emitted: { [name: string]: Array<Array<any>> };
   _emittedByOrder: Array<{ name: string, args: Array<any> }>;
@@ -39,27 +40,28 @@ export default class Wrapper implements BaseWrapper {
     const element = node instanceof Element ? node : node.elm
     // Prevent redefine by VueWrapper
     if (this.constructor.name === 'Wrapper') {
+      this._vnode = vnode
       // $FlowIgnore
       Object.defineProperty(this, 'vnode', {
-        get: () => vnode,
-        set: () => {}
+        get: () => this._vnode,
+        set: () => throwError(`Wrapper.vnode is read-only`)
       })
       // $FlowIgnore
       Object.defineProperty(this, 'element', {
         get: () => element,
-        set: () => {}
+        set: () => throwError(`Wrapper.element is read-only`)
       })
       // $FlowIgnore
       Object.defineProperty(this, 'vm', {
         get: () => undefined,
-        set: () => {}
+        set: () => throwError(`Wrapper.vm is read-only`)
       })
     }
     const frozenOptions = Object.freeze(options)
     // $FlowIgnore
     Object.defineProperty(this, 'options', {
       get: () => frozenOptions,
-      set: () => {}
+      set: () => throwError(`${this.constructor.name}.options is read-only`)
     })
     if (
       this.vnode &&
@@ -399,7 +401,6 @@ export default class Wrapper implements BaseWrapper {
     }
 
     return !!(
-      this.element &&
       this.element.getAttribute &&
       this.element.matches(selector)
     )
@@ -667,7 +668,7 @@ export default class Wrapper implements BaseWrapper {
     })
 
     // $FlowIgnore : Problem with possibly null this.vm
-    this.vnode = this.vm._vnode
+    this._vnode = this.vm._vnode
     orderWatchers(this.vm || this.vnode.context.$root)
     Vue.config.silent = originalConfig
   }

Diff for: test/specs/vuewrapper.js renamed to test/specs/vue-wrapper.spec.js

@@ -5,9 +5,10 @@ describeWithShallowAndMount('VueWrapper', mountingMethod => {
     it(`has the ${property} property which is read-only`, () => {
       const wrapper = mountingMethod({ template: '<div><p></p></div>' })
       expect(wrapper.constructor.name).to.equal('VueWrapper')
-      const originalProperty = wrapper[property]
-      wrapper[property] = 'foo'
-      expect(wrapper[property]).to.equal(originalProperty)
+      const message = `[vue-test-utils]: VueWrapper.${property} is read-only`
+      expect(() => { wrapper[property] = 'foo' })
+        .to.throw()
+        .with.property('message', message)
     })
   })
 })

Diff for: test/specs/wrapper-array.spec.js

@@ -10,6 +10,16 @@ describeWithShallowAndMount('WrapperArray', mountingMethod => {
     return wrappers ? new wrapperArray.constructor(wrappers) : wrapperArray
   }
 
+  ['wrappers', 'length'].forEach(property => {
+    it(`has the ${property} property which is read-only`, () => {
+      const wrapperArray = getWrapperArray()
+      const message = `[vue-test-utils]: WrapperArray.${property} is read-only`
+      expect(() => { wrapperArray[property] = 'foo' })
+        .to.throw()
+        .with.property('message', message)
+    })
+  })
+
   it('returns class with length equal to length of wrappers passed in constructor', () => {
     const wrapperArray = getWrapperArray()
     expect(wrapperArray.length).to.equal(3)

Diff for: test/specs/wrapper.spec.js

@@ -6,9 +6,10 @@ describeWithShallowAndMount('Wrapper', mountingMethod => {
       const wrapper = mountingMethod({ template: '<div><p></p></div>' })
         .find('p')
       expect(wrapper.constructor.name).to.equal('Wrapper')
-      const originalProperty = wrapper[property]
-      wrapper[property] = 'foo'
-      expect(wrapper[property]).to.equal(originalProperty)
+      const message = `[vue-test-utils]: Wrapper.${property} is read-only`
+      expect(() => { wrapper[property] = 'foo' })
+        .to.throw()
+        .with.property('message', message)
     })
   })
 })

Diff for: test/specs/wrapper/attributes.spec.js

@@ -15,11 +15,4 @@ describeWithShallowAndMount('attributes', mountingMethod => {
     const wrapper = mountingMethod(compiled)
     expect(wrapper.attributes()).to.eql({})
   })
-
-  it('returns empty object if wrapper element is null', () => {
-    const compiled = compileToFunctions('<div />')
-    const wrapper = mountingMethod(compiled)
-    wrapper.element = null
-    expect(wrapper.attributes()).to.eql({})
-  })
 })

Diff for: test/specs/wrapper/is.spec.js

@@ -28,12 +28,6 @@ describeWithShallowAndMount('is', mountingMethod => {
     expect(wrapper.is('#div')).to.equal(true)
   })
 
-  it('returns false if wrapper does not contain element', () => {
-    const wrapper = mountingMethod(ComponentWithChild)
-    wrapper.element = null
-    expect(wrapper.is('a')).to.equal(false)
-  })
-
   it('returns true if root node matches Vue Component selector', () => {
     const wrapper = mountingMethod(ComponentWithChild)
     const component = wrapper.findAll(Component).at(0)