✨ Add unset on navigators (#318)

packages/immutadot/src/core/unset.js

@@ -1,6 +1,6 @@
-import { apply } from 'path/apply'
-
-const unsetOperation = (obj, prop) => { delete obj[prop] }
+import { curry } from './curry'
+import { nav } from 'nav/nav'
+import { toPath } from '@immutadot/parser'
 
 /**
  * Removes the property at <code>path</code> of <code>object</code>.
@@ -12,6 +12,6 @@ const unsetOperation = (obj, prop) => { delete obj[prop] }
  * @example unset({ nested: { prop: 'value' } }, 'nested.prop') // => { nested: {} }
  * @since 1.0.0
  */
-const unset = apply(unsetOperation)
+const unset = curry((obj, path) => nav(toPath(path))(obj).unset(), { fixedArity: true })
 
 export { unset }

packages/immutadot/src/core/unset.spec.js

@@ -1,7 +1,9 @@
 /* eslint-env jest */
 import { immutaTest } from 'test.utils'
 import { unset } from 'core'
+
 describe('core.unset', () => {
+
   it('should unset a prop', () => {
     immutaTest({
       nested: { prop: 'initial' },
@@ -15,4 +17,49 @@ describe('core.unset', () => {
       return output
     })
   })
+
+  it('should unset an index', () => {
+    immutaTest({
+      nested: { arr: [1, 2, 3] },
+      other: {},
+    }, ['nested.arr.1'], input => {
+      const output = unset(input, 'nested.arr[1]')
+      expect(output).toEqual({
+        nested: { arr: [1, , 3] }, // eslint-disable-line no-sparse-arrays
+        other: {},
+      })
+      return output
+    })
+  })
+
+  it('should unset several props', () => {
+    immutaTest({
+      nested: {
+        prop1: 'initial1',
+        prop2: 'initial2',
+      },
+      other: {},
+    }, ['nested.prop1', 'nested.prop2'], input => {
+      const output = unset(input, 'nested.{prop1,prop2}')
+      expect(output).toEqual({
+        nested: {},
+        other: {},
+      })
+      return output
+    })
+  })
+
+  it('should unset a slice', () => {
+    immutaTest({
+      nested: { arr: [1, 2, 3, 4] },
+      other: {},
+    }, ['nested.arr.1', 'nested.arr.2'], input => {
+      const output = unset(input, 'nested.arr[1:3]')
+      expect(output).toEqual({
+        nested: { arr: [1, , , 4] }, // eslint-disable-line no-sparse-arrays
+        other: {},
+      })
+      return output
+    })
+  })
 })

packages/immutadot/src/nav/baseNav.js

@@ -3,4 +3,8 @@ export class BaseNav {
     this.value = value
     this._next = next
   }
+
+  get final() {
+    return false
+  }
 }

packages/immutadot/src/nav/finalNav.js

@@ -0,0 +1,21 @@
+class FinalNav {
+  constructor(value) {
+    this.value = value
+  }
+
+  get() {
+    return this.value
+  }
+
+  update(updater) {
+    return updater(this.value)
+  }
+
+  get final() {
+    return true
+  }
+}
+
+export function finalNav(value) {
+  return new FinalNav(value)
+}

packages/immutadot/src/nav/indexNav.js

@@ -28,6 +28,15 @@ class IndexNav extends ArrayNav {
     copy[this.index] = this.next.update(updater)
     return copy
   }
+
+  unset() {
+    const copy = this.copy()
+    if (this.next.final)
+      delete copy[this.index]
+    else
+      copy[this.index] = this.next.unset()
+    return copy
+  }
 }
 
 export function indexNav(index, next) {

packages/immutadot/src/nav/nav.js

@@ -1,4 +1,5 @@
 import { allProps, index, list, prop, slice } from '@immutadot/parser/consts'
+import { finalNav } from './finalNav'
 import { indexNav } from './indexNav'
 import { propNav } from './propNav'
 import { propsNav } from './propsNav'
@@ -21,21 +22,3 @@ function toNav(type) {
   default: throw TypeError(type)
   }
 }
-
-class FinalNav {
-  constructor(value) {
-    this.value = value
-  }
-
-  get() {
-    return this.value
-  }
-
-  update(updater) {
-    return updater(this.value)
-  }
-}
-
-function finalNav(value) {
-  return new FinalNav(value)
-}

packages/immutadot/src/nav/propNav.js

@@ -21,6 +21,15 @@ class PropNav extends ObjectNav {
     copy[this.key] = this.next.update(updater)
     return copy
   }
+
+  unset() {
+    const copy = this.copy()
+    if (this.next.final)
+      delete copy[this.key]
+    else
+      copy[this.key] = this.next.unset()
+    return copy
+  }
 }
 
 export function propNav(key, next) {

packages/immutadot/src/nav/propsNav.js

@@ -28,6 +28,20 @@ class PropsNav extends ObjectNav {
     for (const key of keys) copy[key] = _next(value[key]).update(updater)
     return copy
   }
+
+  unset() {
+    const { _next, keys, value } = this
+
+    const copy = this.copy()
+    for (const key of keys) {
+      const next = _next(value[key])
+      if (next.final)
+        delete copy[key]
+      else
+        copy[key] = next.unset()
+    }
+    return copy
+  }
 }
 
 export function propsNav(keys, next) {

packages/immutadot/src/nav/sliceNav.js

@@ -43,6 +43,20 @@ class SliceNav extends ArrayNav {
     for (const index of range) copy[index] = _next(value[index]).update(updater)
     return copy
   }
+
+  unset() {
+    const { _next, value, range } = this
+
+    const copy = this.copy()
+    for (const index of range) {
+      const next = _next(value[index])
+      if (next.final)
+        delete copy[index]
+      else
+        copy[index] = next.unset()
+    }
+    return copy
+  }
 }
 
 export function sliceNav(bounds, next) {