Skip to content

Commit 3aea4dd

Browse files
committed
test: Adding an observer spec for new feature to catch properties being set that already have been c
fix vuejs#8129
1 parent 6fecabe commit 3aea4dd

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

Diff for: test/unit/modules/observer/observer.spec.js

+15
Original file line numberDiff line numberDiff line change
@@ -316,6 +316,21 @@ describe('Observer', () => {
316316
}).then(done)
317317
})
318318

319+
it('Cannot enable reactivity on a property that is already defined', done => {
320+
const vm = new Vue({
321+
data: {
322+
person: {
323+
age: 32
324+
}
325+
}
326+
}).$mount()
327+
vm.person.job = 'Programmer'
328+
Vue.set(vm.person, 'job', 'Educator')
329+
waitForUpdate(() => {
330+
expect(`Cannot enable reactivity on a property that is already defined`).toHaveBeenWarned()
331+
}).end(done)
332+
})
333+
319334
it('warning set/delete on Vue instance root $data', done => {
320335
const data = { a: 1 }
321336
const vm = new Vue({

0 commit comments

Comments
 (0)