json-pointer - npm Package Compare versions

Comparing version 0.5.0 to 0.6.0

index.js

@@ -107,3 +107,14 @@ 'use strict';
     }
-    delete api.get(obj, refTokens.slice(0, -1))[finalToken];
+
+    var parent = api.get(obj, refTokens.slice(0, -1));
+    if (Array.isArray(parent)) {
+      var index = +finalToken;
+      if (finalToken === '' && isNaN(index)) {
+        throw new Error('Invalid array index: "' + finalToken + '"');
+      }
+
+      Array.prototype.splice.call(parent, index, 1);
+    } else {
+      delete parent[finalToken];
+    }
 };
@@ -110,0 +121,0 @@

package.json

 {
   "name": "json-pointer",
   "description": "Some utilities for JSON pointers described by RFC 6901",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "author": "Manuel Stofer <[e-mail]>",
@@ -6,0 +6,0 @@ "license": "MIT",

Readme.md

@@ -18,9 +18,3 @@ # json-pointer
 
-[component](https://github.com/component/component)
 
-```bash
-$ component install manuelstofer/json-pointer
-```
-
-
 ## API
@@ -27,0 +21,0 @@

test/test.js

@@ -180,20 +180,45 @@ /*global describe, it, beforeEach*/
         each(Object.keys(rfcValues), function (p) {
-            if (p !== '') {
-                it('should work for "' + p + '"', function () {
-                    pointer.remove(rfcExample, p);
-                    expect(pointer.get.bind(pointer, rfcExample, p)).to.throw(Error);
-                });
-            }
+            if (p === '' || p === '/foo/0') return;
+
+            it('should work for "' + p + '"', function () {
+                pointer.remove(rfcExample, p);
+                expect(pointer.get.bind(pointer, rfcExample, p)).to.throw(Error);
+            });
         });
 
+        it('should work for "/foo/0"', function () {
+            var p = '/foo/0';
+            pointer.remove(rfcExample, p);
+            expect(pointer.get(rfcExample, p)).to.equal('baz');
+        });
+
+        it('should work for "/foo/1"', function () {
+            var p = '/foo/1';
+            pointer.remove(rfcExample, p);
+            expect(pointer.get.bind(pointer, rfcExample, p)).to.throw(Error);
+        });
+
         each(Object.keys(rfcParsed), function (p) {
-            if (p !== '') {
-                it('should work for ' + JSON.stringify(rfcParsed[p].tokens), function () {
-                    pointer.remove(rfcExample, immutable(rfcParsed[p].tokens));
-                    expect(function() {
-                        pointer.get(pointer, rfcExample, immutable(rfcParsed[p].tokens));
-                    }).to.throw(Error);
-                });
-            }
+            if (p === '' || p === '/foo/0') return;
+
+            it('should work for ' + JSON.stringify(rfcParsed[p].tokens), function () {
+                pointer.remove(rfcExample, immutable(rfcParsed[p].tokens));
+                expect(function() {
+                    pointer.get(rfcExample, immutable(rfcParsed[p].tokens));
+                }).to.throw(Error);
+            });
         });
+
+        it('should work for ["foo","0"]', function () {
+            var p = immutable(['foo', '0']);
+            pointer.remove(rfcExample, p);
+            expect(pointer.get(rfcExample, p)).to.equal('baz');
+        });
+
+        it('should work for ["foo","1"]', function () {
+            var p = immutable(['foo', '1']);
+            pointer.remove(rfcExample, p);
+            expect(pointer.get.bind(pointer, rfcExample, p)).to.throw(Error);
+        });
+
     });
@@ -200,0 +225,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

25584

increased by2.89%

Lines of code

565

increased by2.36%

Worsened metrics

Number of package files

9

decreased by-10%

Number of lines in readme file

160

decreased by-3.61%

No dependency changes