Merge pull request #4 from json-patch/master

get the latest.

Author: mikemccabe

spec_tests.json

@@ -1,4 +1,12 @@
 [
+  {
+    "comment": "4.1. add with missing object",
+    "doc": { "q": { "bar": 2 } },
+    "patch": [ {"op": "add", "path": "/a/b", "value": 1} ],
+    "error":
+       "path /a does not exist -- missing objects are not created recursively"
+  },
+
   {
     "comment": "A.1.  Adding an Object Member",
     "doc": {
@@ -14,7 +22,7 @@
   },
 
   {
-    "comment": "A.2.  Adding an Array Element", 
+    "comment": "A.2.  Adding an Array Element",
     "doc": {
   "foo": [ "bar", "baz" ]
 },
@@ -58,7 +66,7 @@
     "doc": {
   "baz": "qux",
   "foo": "bar"
-}, 
+},
     "patch": [
   { "op": "replace", "path": "/baz", "value": "boo" }
 ],
@@ -78,10 +86,10 @@
   "qux": {
     "corge": "grault"
   }
-}, 
+},
     "patch": [
   { "op": "move", "from": "/foo/waldo", "path": "/qux/thud" }
-], 
+],
     "expected": {
   "foo": {
     "bar": "baz"
@@ -97,10 +105,10 @@
     "comment": "A.7.  Moving an Array Element",
     "doc": {
   "foo": [ "all", "grass", "cows", "eat" ]
-}, 
+},
     "patch": [
   { "op": "move", "from": "/foo/1", "path": "/foo/3" }
-], 
+],
     "expected": {
   "foo": [ "all", "cows", "eat", "grass" ]
 }
@@ -131,7 +139,7 @@
     "patch": [
   { "op": "test", "path": "/baz", "value": "bar" }
 ],
-    "error": "string not equivalent" 
+    "error": "string not equivalent"
   },
 
   {
@@ -150,21 +158,21 @@
   }
 }
   },
-  
+
   {
     "comment": "A.11.  Ignoring Unrecognized Elements",
     "doc": {
   "foo":"bar"
-}, 
+},
     "patch": [
   { "op": "add", "path": "/baz", "value": "qux", "xyz": 123 }
 ],
     "expected": {
-  "foo":"bar", 
+  "foo":"bar",
   "baz":"qux"
 }
   },
-  
+
  {
     "comment": "A.12.  Adding to a Non-existant Target",
     "doc": {
@@ -175,9 +183,9 @@
 ],
     "error": "add to a non-existant target"
   },
-  
+
  {
-    "comment": "Invalid JSON Patch Document",
+    "comment": "A.13 Invalid JSON Patch Document",
     "doc": {
      "foo": "bar"
     },
@@ -186,18 +194,39 @@
 ],
     "error": "operation has two 'op' members"
   },
-  
-  { 
-    "comment": "~ Escape Ordering",
+
+  {
+    "comment": "A.14. ~ Escape Ordering",
     "doc": {
        "/": 9,
-       "~1": 10 
+       "~1": 10
     },
-    "patch": [{"op": "test", "path": "/~01", "value":"10"}],
+    "patch": [{"op": "test", "path": "/~01", "value": 10}],
     "expected": {
        "/": 9,
-       "~1": 10 
+       "~1": 10
+    }
+  },
+
+  {
+    "comment": "A.15. Comparing Strings and Numbers",
+    "doc": {
+       "/": 9,
+       "~1": 10
+    },
+    "patch": [{"op": "test", "path": "/~01", "value": "10"}],
+    "error": "number is not equal to string"
+  },
+
+  {
+    "comment": "A.16. Adding an Array Value",
+    "doc": {
+       "foo": ["bar"]
+    },
+    "patch": [{ "op": "add", "path": "/foo/-", "value": ["abc", "def"] }],
+    "expected": {
+      "foo": ["bar", ["abc", "def"]]
     }
   }
-    
-]
+
+]

tests.json

@@ -108,9 +108,19 @@
       "expected": ["bar", "foo", "sil"] },
 
     { "doc": ["foo", "sil"],
-      "patch": [{"op":" add", "path": "/2", "value": "bar"}],
+      "patch": [{"op":"add", "path": "/2", "value": "bar"}],
       "expected": ["foo", "sil", "bar"] },
 
+    { "comment": "test against implementation-specific numeric parsing",
+      "doc": {"1e0": "foo"},
+      "patch": [{"op": "test", "path": "/1e0", "value": "foo"}],
+      "expected": {"1e0": "foo"} },
+
+    { "comment": "test with bad number should fail",
+      "doc": ["foo", "bar"],
+      "patch": [{"op": "test", "path": "/1e0", "value": "bar"}],
+      "error": "test op shouldn't get array element 1" },
+
     { "doc": ["foo", "sil"],
       "patch": [{"op": "add", "path": "/bar", "value": 42}],
       "error": "Object operation on array target" },
@@ -223,20 +233,35 @@
 
     { "comment": "Move to same location has no effect",
       "doc": {"foo": 1},
-      "patch": [{"op": "move", "path": "/foo", "to": "/foo"}],
+      "patch": [{"op": "move", "from": "/foo", "path": "/foo"}],
       "expected": {"foo": 1} },
 
     { "doc": {"foo": 1, "baz": [{"qux": "hello"}]},
-      "patch": [{"op": "move", "path": "/foo", "to": "/bar"}],
+      "patch": [{"op": "move", "from": "/foo", "path": "/bar"}],
       "expected": {"baz": [{"qux": "hello"}], "bar": 1} },
 
     { "doc": {"baz": [{"qux": "hello"}], "bar": 1},
-      "patch": [{"op": "move", "path": "/baz/0/qux", "to": "/baz/1"}],
+      "patch": [{"op": "move", "from": "/baz/0/qux", "path": "/baz/1"}],
       "expected": {"baz": [{}, "hello"], "bar": 1} },
 
     { "doc": {"baz": [{"qux": "hello"}], "bar": 1},
-      "patch": [{"op": "copy", "path": "/baz/0", "to": "/boo"}],
+      "patch": [{"op": "copy", "from": "/baz/0", "path": "/boo"}],
       "expected": {"baz":[{"qux":"hello"}],"bar":1,"boo":{"qux":"hello"}} },
 
+    { "comment": "replacing the root of the document is possible with add",
+      "doc": {"foo": "bar"},
+      "patch": [{"op": "add", "path": "", "value": {"baz": "qux"}}],
+      "expected": {"baz":"qux"}},
+
+    { "comment": "Adding to \"/-\" adds to the end of the array",
+      "doc": [ 1, 2 ],
+      "patch": [ { "op": "add", "path": "/-", "value": { "foo": [ "bar", "baz" ] } } ],
+      "expected": [ 1, 2, { "foo": [ "bar", "baz" ] } ]},
+
+    { "comment": "Adding to \"/-\" adds to the end of the array, even n levels down",
+      "doc": [ 1, 2, [ 3, [ 4, 5 ] ] ],
+      "patch": [ { "op": "add", "path": "/2/1/-", "value": { "foo": [ "bar", "baz" ] } } ],
+      "expected": [ 1, 2, [ 3, [ 4, 5, { "foo": [ "bar", "baz" ] } ] ] ]},
+
     { "comment": "tests complete" }
 ]