test for setting dynamic non-existent path

Author: yyx990803

src/api/data.js

@@ -44,12 +44,6 @@ exports.$set = function (exp, val) {
  */
 
 exports.$add = function (key, val) {
-  _.warn(
-    'You are dynamically adding a property "' + key + '" to ' +
-    'a vm instance. Consider pre-initializing the property ' +
-    'with the "data" option for more reliable reactivity ' +
-    'and better performance.'
-  )
   this._data.$add(key, val)
 }
 

src/parsers/path.js

@@ -200,6 +200,7 @@ function parsePath (path) {
     action()
 
     if (mode === 'afterPath') {
+      keys.raw = path
       return keys
     }
   }
@@ -298,14 +299,25 @@ exports.set = function (obj, path, val) {
       if (!_.isObject(obj)) {
         obj = {}
         last.$add(key, obj)
+        warnNonExistent(path)
       }
     } else {
       if (key in obj) {
         obj[key] = val
       } else {
         obj.$add(key, val)
+        warnNonExistent(path)
       }
     }
   }
   return true
+}
+
+function warnNonExistent (path) {
+  _.warn(
+    'You are setting a non-existent path "' + path.raw + '" ' +
+    'on a vm instance. Consider pre-initializing the property ' +
+    'with the "data" option for more reliable reactivity ' +
+    'and better performance.'
+  )
 }

test/unit/specs/parsers/path_spec.js

@@ -44,6 +44,7 @@ describe('Path Parser', function () {
     assertPath('foo["baz"]', ['foo', 'baz'])
     assertPath('foo["b\\"az"]', ['foo', 'b"az'])
     assertPath("foo['b\\'az']", ['foo', "b'az"])
+    assertPath('a[b][c]', ['a', '*b', '*c'])
   })
 
   it('handle invalid paths', function () {
@@ -64,6 +65,7 @@ describe('Path Parser', function () {
     assertInvalidPath('  42   ')
     assertInvalidPath('foo["bar]')
     assertInvalidPath("foo['bar]")
+    assertInvalidPath('foo[bar + boo]')
   })
 
   it('caching', function () {
@@ -104,6 +106,16 @@ describe('Path Parser', function () {
     expect(target.a.b.c).toBe(123)
   })
 
+  it('set dynamic non-existent', function () {
+    var target = {
+      key: 'what',
+      obj: {}
+    }
+    var res = Path.set(target, 'obj[key]', 123)
+    expect(res).toBe(true)
+    expect(target.obj.what).toBe(123)
+  })
+
   it('set on prototype chain', function () {
     var parent = { a: {} }
     var target = Object.create(parent)