hana 1.3.3 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 54a300a1f4f135540a4c590cf3e7e1b8ad617b4a
-  data.tar.gz: 499cba6697fe8fca1bd5a4788042cb5492ed01d4
+SHA256:
+  metadata.gz: 556063c84c12bf3c25887a6021dbe2231ab3f4a63ef142ad9497031c7fdb6f14
+  data.tar.gz: 41a7f89c78c4cec3c97c5e8a1f9e409e3a7f6b7edc84ac8f21c7b794fce665f5
 SHA512:
-  metadata.gz: 50c03edacec8defe7b7e82289a87bfe3c52e77255d019dc96abdc1407be82c9ceb027ebae98bfb2dfb7c2a987d42a73e4ea08e639f5217282de93cabbc856384
-  data.tar.gz: fb38bcb65aedbffa1e1cdf3d109776abcf1a7bf6cba5ca91dd32f41df67983f10438027dffc0f3df632c819ddb3b20e4d6da1af8f1aadf8d486250075df5cb4f
+  metadata.gz: 6922f6fccc85b67181b4d262df950bbb76df8ace31be13acd1ad943aae40b3ab2f936a41af2e18cf546345409d51236c8a74a8a7be6bc05ec6a14c5456e9442b
+  data.tar.gz: 15c8575ff781dfe4dbb01bc77f80383d4a216a69d8e0282c64175837da52e29d5b92d88bb8ad0a891159ce33c2b9185d895337410400a8038650e623f0931c49

data/Manifest.txt

@@ -1,11 +1,12 @@
-.autotest
-CHANGELOG.rdoc
 Manifest.txt
 README.md
 Rakefile
 lib/hana.rb
 test/helper.rb
+test/json-patch-tests/.editorconfig
+test/json-patch-tests/.npmignore
 test/json-patch-tests/README.md
+test/json-patch-tests/package.json
 test/json-patch-tests/spec_tests.json
 test/json-patch-tests/tests.json
 test/mine.json

data/Rakefile

@@ -12,7 +12,6 @@ Hoe.spec 'hana' do
   developer('Aaron Patterson', 'aaron@tenderlovemaking.com')
   license 'MIT'
   self.readme_file   = 'README.md'
-  self.history_file  = 'CHANGELOG.rdoc'
   self.extra_rdoc_files  = FileList['*.rdoc']
   extra_dev_deps << ["minitest", "~> 5.0"]
 end

data/lib/hana.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Hana
-  VERSION = '1.3.3'
+  VERSION = '1.3.4'
 
   class Pointer
     include Enumerable
@@ -100,7 +100,11 @@ module Hana
       if key
         add_op dest, key, obj
       else
-        dest.replace obj
+        if doc.equal? dest
+          doc = obj
+        else
+          dest.replace obj
+        end
       end
       doc
     end
@@ -157,6 +161,7 @@ module Hana
         raise Patch::IndexError unless key =~ /\A\d+\Z/
         obj[key.to_i] = ins.fetch VALUE
       else
+        raise Patch::MissingTargetException unless obj
         obj[key] = ins.fetch VALUE
       end
       doc

data/test/json-patch-tests/.editorconfig

@@ -0,0 +1,10 @@
+# EditorConfig is awesome: http://EditorConfig.org
+
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+charset = utf-8
+trim_trailing_whitespace = true
+indent_style = space

data/test/json-patch-tests/.npmignore

@@ -0,0 +1,2 @@
+.editorconfig
+.gitignore

data/test/json-patch-tests/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "json-patch-test-suite",
+  "version": "1.1.0",
+  "description": "JSON Patch RFC 6902 test suite",
+  "repository": "github:json-patch/json-patch-tests",
+  "homepage": "https://github.com/json-patch/json-patch-tests",
+  "bugs": "https://github.com/json-patch/json-patch-tests/issues",
+  "keywords": [
+    "JSON",
+    "Patch",
+    "test",
+    "suite"
+  ],
+  "license": "Apache-2.0"
+}

data/test/json-patch-tests/tests.json

@@ -55,6 +55,21 @@
       "expected": "bar",
       "disabled": true },
 
+    { "comment": "replace object document with array document?",
+      "doc": {},
+      "patch": [{"op": "add", "path": "", "value": []}],
+      "expected": [] },
+
+    { "comment": "replace array document with object document?",
+      "doc": [],
+      "patch": [{"op": "add", "path": "", "value": {}}],
+      "expected": {} },
+
+    { "comment": "append to root array document?",
+      "doc": [],
+      "patch": [{"op": "add", "path": "/-", "value": "hi"}],
+      "expected": ["hi"] },
+
     { "comment": "Add, / target",
       "doc": {},
       "patch": [ {"op": "add", "path": "/", "value":1 } ],
@@ -187,6 +202,11 @@
       "patch": [{"op": "replace", "path": "", "value": {"baz": "qux"}}],
       "expected": {"baz": "qux"} },
 
+    { "comment": "test replace with missing parent key should fail",
+      "doc": {"bar": "baz"},
+      "patch": [{"op": "replace", "path": "/foo/bar", "value": false}],
+      "error": "replace op should fail with missing parent key" },
+
     { "comment": "spurious patch properties",
       "doc": {"foo": 1},
       "patch": [{"op": "test", "path": "/foo", "value": 1, "spurious": 1}],
@@ -394,7 +414,7 @@
       "doc": ["foo", "bar"],
       "patch": [{"op": "test", "path": "/01", "value": "bar"}],
       "error": "test op should reject the array value, it has leading zeros" },
-    
+
     { "comment": "Removing nonexistent field",
       "doc": {"foo" : "bar"},
       "patch": [{"op": "remove", "path": "/baz"}],
@@ -403,6 +423,12 @@
     { "comment": "Removing nonexistent index",
       "doc": ["foo", "bar"],
       "patch": [{"op": "remove", "path": "/2"}],
-      "error": "removing a nonexistent index should fail" }
+      "error": "removing a nonexistent index should fail" },
+
+    { "comment": "Patch with different capitalisation than doc",
+       "doc": {"foo":"bar"},
+       "patch": [{"op": "add", "path": "/FOO", "value": "BAR"}],
+       "expected": {"foo": "bar", "FOO": "BAR"}
+    }
 
 ]

data/test/test_hana.rb

@@ -69,4 +69,43 @@ class TestHana < Hana::TestCase
     pointer = Hana::Pointer.new '/foo/bar/baz'
     assert_nil pointer.eval('foo' => nil)
   end
+
+  def test_remove_missing_object_key
+    patch = Hana::Patch.new [
+      { 'op' => 'remove', 'path' => '/missing_key' }
+    ]
+    assert_raises(Hana::Patch::IndexError) do
+      patch.apply('foo' => 'bar')
+    end
+  end
+
+  def test_remove_missing_array_index
+    patch = Hana::Patch.new [
+      { 'op' => 'remove', 'path' => '/1' }
+    ]
+    assert_raises(Hana::Patch::OutOfBoundsException) do
+      patch.apply([0])
+    end
+  end
+
+  def test_remove_missing_object_key_in_array
+    patch = Hana::Patch.new [
+      { 'op' => 'remove', 'path' => '/1/baz' }
+    ]
+    assert_raises(Hana::Patch::IndexError) do
+      patch.apply([
+        { 'foo' => 'bar' },
+        { 'foo' => 'bar' }
+      ])
+    end
+  end
+
+  def test_replace_missing_key
+    patch = Hana::Patch.new [
+      { 'op' => 'replace', 'path' => '/missing_key/field', 'value' => 'asdf' }
+    ]
+    assert_raises(Hana::Patch::MissingTargetException) do
+      patch.apply('foo' => 'bar')
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hana
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Aaron Patterson
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-14 00:00:00.000000000 Z
+date: 2018-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.10'
+        version: '5.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.10'
+        version: '5.11'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -58,18 +58,18 @@ email:
 executables: []
 extensions: []
 extra_rdoc_files:
-- CHANGELOG.rdoc
 - Manifest.txt
 - README.md
 files:
-- ".autotest"
-- CHANGELOG.rdoc
 - Manifest.txt
 - README.md
 - Rakefile
 - lib/hana.rb
 - test/helper.rb
+- test/json-patch-tests/.editorconfig
+- test/json-patch-tests/.npmignore
 - test/json-patch-tests/README.md
+- test/json-patch-tests/package.json
 - test/json-patch-tests/spec_tests.json
 - test/json-patch-tests/tests.json
 - test/mine.json
@@ -97,7 +97,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Implementation of [JSON Patch][1] and [JSON Pointer][2] RFC.

data/.autotest

@@ -1,8 +0,0 @@
-# -*- ruby -*-
-
-require 'autotest/restart'
-
-Autotest.add_hook :initialize do |at|
-  at.testlib          = 'minitest/autorun'
-  at.find_directories = ARGV unless ARGV.empty?
-end

data/CHANGELOG.rdoc

@@ -1,12 +0,0 @@
-Wed May 22 19:28:10 2013  Aaron Patterson <aaron@tenderlovemaking.com>
-
-	* All IETF compliant
-
-Fri Sep  7 18:54:17 2012  Aaron Patterson <aaron@tenderlovemaking.com>
-
-	* Bumping the version.
-
-Fri Sep  7 18:53:16 2012  Aaron Patterson <aaron@tenderlovemaking.com>
-
-	* validate commands an raise an exception Just In Case
-