hana 1.3.1 → 1.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ccb71fa0fbe43f9887bd8a7f3a1b5ad9273acbcb
-  data.tar.gz: 807c25b1c8b8dcc67ce9f109799224f91bff37ad
+  metadata.gz: 4fb4c95fd3541788a19d1c4b9d7bd1931242191a
+  data.tar.gz: 83e628bf62efb6e476121837a710c0d64bfc7590
 SHA512:
-  metadata.gz: 44ebd19bc18f9aac58a3e22d97ef0af22249bd2d559d5b25e876d0054fd12c5d4ed4bc8def868da96c71d1655809006578d6ca3bfe0931c3abf42b00549154b0
-  data.tar.gz: f8882b84a664e16bb252138a101af07640c3dbcd33c037ed777d33838815a1b7c458c710b3877ca1abd39c484cea61981b4e63ace77c6d495a1ca02a808c4f61
+  metadata.gz: 5b60dc61f8c64a99d6595b7b8e8192b2d1f658a47eab4a74bfca692edc4883dc25979b41bc4a68b3f1a5b02f69f832fdfa9ca744aea279ef7480d836b3ba0463
+  data.tar.gz: 52c58c0d9d5690a9f37c8e24621ae21dba3fd651566730c723c4603ab41c6b843da2f3b13eca22cf31a7bd0eb12e476b1f0c3f5d50a45d4eac57474b0e896480

data/README.md

@@ -17,7 +17,7 @@ process it, then emit as JSON again.
 
 ```ruby
 patch = Hana::Patch.new [
-  { 'add' => '/baz', 'value' => 'qux' }
+  { 'op' => 'add', 'path' => '/baz', 'value' => 'qux' }
 ]
 
 patch.apply('foo' => 'bar') # => {'baz' => 'qux', 'foo' => 'bar'}
@@ -35,7 +35,7 @@ patch.apply('foo' => 'bar') # => {'baz' => 'qux', 'foo' => 'bar'}
 
 (The MIT License)
 
-Copyright (c) 2012-2013 Aaron Patterson
+Copyright (c) 2012-2016 Aaron Patterson
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/Rakefile

@@ -10,6 +10,7 @@ Hoe.plugin :git     # `gem install hoe-git`
 
 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']

data/lib/hana.rb

@@ -1,5 +1,5 @@
 module Hana
-  VERSION = '1.3.1'
+  VERSION = '1.3.2'
 
   class Pointer
     include Enumerable
@@ -31,9 +31,12 @@ module Hana
     def self.parse path
       return [''] if path == '/'
 
-      path.sub(/^\//, '').split(/(?<!\^)\//).each { |part|
+      parts = path.sub(/^\//, '').split(/(?<!\^)\//).each { |part|
         part.gsub!(/\^[\/^]|~[01]/) { |m| ESC[m] }
       }
+
+      parts.push("") if path[-1] == '/'
+      parts
     end
   end
 

data/test/helper.rb

@@ -21,7 +21,7 @@ module Hana
             patch = Hana::Patch.new patch
 
             if test['error']
-              assert_raises(*ex(test['error'])) do
+              assert_raises(*ex(test['error']), test['error']) do
                 patch.apply doc
               end
             else
@@ -43,6 +43,7 @@ module Hana
     def ex msg
       case msg
       when /Out of bounds/i then [Hana::Patch::OutOfBoundsException]
+      when /index is greater than/i then [Hana::Patch::OutOfBoundsException]
       when /Object operation on array/ then
         [Hana::Patch::ObjectOperationOnArrayException]
       when /test op shouldn't get array element/ then

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

@@ -60,6 +60,11 @@
       "patch": [ {"op": "add", "path": "/", "value":1 } ],
       "expected": {"":1} },
 
+    { "comment": "Add, /foo/ deep target (trailing slash)",
+      "doc": {"foo": {}},
+      "patch": [ {"op": "add", "path": "/foo/", "value":1 } ],
+      "expected": {"foo":{"": 1}} },
+
     { "comment": "Add composite value at top level",
       "doc": {"foo": 1},
       "patch": [{"op": "add", "path": "/bar", "value": [1, 2]}],
@@ -107,10 +112,16 @@
       "patch": [{"op": "add", "path": "/0", "value": "bar"}],
       "expected": ["bar", "foo", "sil"] },
 
-    { "doc": ["foo", "sil"],
+    { "comment": "push item to array via last index + 1",
+      "doc": ["foo", "sil"],
       "patch": [{"op":"add", "path": "/2", "value": "bar"}],
       "expected": ["foo", "sil", "bar"] },
 
+    { "comment": "add item to array at index > length should fail",
+      "doc": ["foo", "sil"],
+      "patch": [{"op":"add", "path": "/3", "value": "bar"}],
+      "error": "index is greater than number of items in array" },
+      
     { "comment": "test against implementation-specific numeric parsing",
       "doc": {"1e0": "foo"},
       "patch": [{"op": "test", "path": "/1e0", "value": "foo"}],
@@ -183,7 +194,32 @@
 
     { "doc": {"foo": null},
       "patch": [{"op": "test", "path": "/foo", "value": null}],
-      "comment": "null value should still be valid obj property" },
+      "comment": "null value should be valid obj property" },
+
+    { "doc": {"foo": null},
+      "patch": [{"op": "replace", "path": "/foo", "value": "truthy"}],
+      "expected": {"foo": "truthy"},
+      "comment": "null value should be valid obj property to be replaced with something truthy" },
+
+    { "doc": {"foo": null},
+      "patch": [{"op": "move", "from": "/foo", "path": "/bar"}],
+      "expected": {"bar": null},
+      "comment": "null value should be valid obj property to be moved" },
+
+    { "doc": {"foo": null},
+      "patch": [{"op": "copy", "from": "/foo", "path": "/bar"}],
+      "expected": {"foo": null, "bar": null},
+      "comment": "null value should be valid obj property to be copied" },
+
+    { "doc": {"foo": null},
+      "patch": [{"op": "remove", "path": "/foo"}],
+      "expected": {},
+      "comment": "null value should be valid obj property to be removed" },
+
+    { "doc": {"foo": "bar"},
+      "patch": [{"op": "replace", "path": "/foo", "value": null}],
+      "expected": {"foo": null},
+      "comment": "null value should still be valid obj property replace other value" },
 
     { "doc": {"foo": {"foo": 1, "bar": 2}},
       "patch": [{"op": "test", "path": "/foo", "value": {"bar": 2, "foo": 1}}],

data/test/test_hana.rb

@@ -23,6 +23,11 @@ class TestHana < Hana::TestCase
     assert_equal %w{ foo bar baz }, pointer.to_a
   end
 
+  def test_split_with_trailing
+    pointer = Hana::Pointer.new '/foo/'
+    assert_equal ["foo", ""], pointer.to_a
+  end
+
   def test_root
     pointer = Hana::Pointer.new '/'
     assert_equal [''], pointer.to_a

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hana
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.2
 platform: ruby
 authors:
 - Aaron Patterson
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-05 00:00:00.000000000 Z
+date: 2016-05-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.5'
+        version: '5.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.5'
+        version: '5.9'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.13'
+        version: '3.15'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.13'
+        version: '3.15'
 description: Implementation of [JSON Patch][1] and [JSON Pointer][2] RFC.
 email:
 - aaron@tenderlovemaking.com
@@ -63,7 +63,6 @@ extra_rdoc_files:
 - README.md
 files:
 - ".autotest"
-- ".gemtest"
 - CHANGELOG.rdoc
 - Manifest.txt
 - README.md
@@ -98,10 +97,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.6.4
 signing_key: 
 specification_version: 4
 summary: Implementation of [JSON Patch][1] and [JSON Pointer][2] RFC.
-test_files:
-- test/test_hana.rb
-- test/test_ietf.rb
+test_files: []