fix(proxy-state-tree): allow untracking paths in different order

Author: christianalfoni

packages/node_modules/proxy-state-tree/jest.config.js

@@ -1,5 +1,6 @@
 module.exports = {
   collectCoverage: true,
+  testEnvironment: 'node',
   collectCoverageFrom: ['src/**/*.{t,j}s?(x)', '!src/**/*.d.ts'],
   moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json'],
   transform: {

packages/node_modules/proxy-state-tree/src/index.test.ts

@@ -150,15 +150,6 @@ describe('ARRAYS', () => {
       const pathsA = tree.clearPathsTracking(trackIdA)
       expect(pathsA).toEqual(new Set(['foo', 'foo.0']))
     })
-    test('should throw when stopping outer nested tracking before inner', () => {
-      const tree = new ProxyStateTree({})
-
-      const trackIdA = tree.startPathsTracking()
-      tree.startPathsTracking()
-      expect(() => {
-        tree.clearPathsTracking(trackIdA)
-      }).toThrow()
-    })
     test('should correctly keep track of changing indexes', () => {
       expect.assertions(4)
       let listener

packages/node_modules/proxy-state-tree/src/index.ts

@@ -129,12 +129,11 @@ class ProxyStateTree {
     return this.paths.push(new Set()) - 1
   }
   clearPathsTracking(index: number) {
-    if (index !== this.paths.length - 1) {
-      throw new Error(
-        'Nested path tracking requires you to stop the nested path tracker before the outer'
-      )
+    const pathSet = this.paths.splice(index, 1, null)[0]
+
+    while (this.paths[this.paths.length - 1] === null) {
+      this.paths.pop()
     }
-    const pathSet = this.paths.pop()
 
     if (!this.paths.length) {
       this.status = STATUS.IDLE