Fixing track for multi-value models. Fixes #682

Author: davidkpiano

src/utils/track.js

@@ -1,6 +1,7 @@
 import findKey from '../utils/find-key';
 import _get from '../utils/get';
 import iteratee from '../utils/iteratee';
+import isMulti from '../utils/is-multi';
 
 const defaultStrategy = {
   get: _get,
@@ -38,8 +39,12 @@ function createTrack(s = defaultStrategy) {
         fullPath += `.${subPath}`;
       });
 
+      if (isMulti(childModel) && predicates.length < childModelPaths.length) {
+        fullPath += '[]';
+      }
+
       return isPartial
-        ? [parentModel, fullPath].join('.')
+        ? `${parentModel}.${fullPath}`
         : fullPath;
     };
   };

test/tracking-spec.js

@@ -140,4 +140,25 @@ describe('tracking', () => {
       assert.equal(tracker(quizState), 'quiz.questions.0.choices.1');
     });
   });
+
+  describe('multiple levels of tracking with multi-value models', () => {
+    it('should ignore final set of brackets if following args mismatch count', () => {
+      const testState = {
+        foo: [
+          {
+            id: 1,
+            bar: [
+              {
+                id: 2,
+                multiple: ['a', 'b', 'c'],
+              },
+            ],
+          },
+        ],
+      };
+
+      const tracker = track('foo[].bar[].multiple[]', { id: 1 }, { id: 2 });
+      assert.equal(tracker(testState), 'foo.0.bar.0.multiple[]');
+    });
+  });
 });