search

json-patch / json-patch-tests

Tests for implementations of json-patch
68 stars 21 forks source link

Add "Append item to array" test #20

Closed sonnyp closed 9 years ago

sonnyp commented 9 years ago

https://github.com/sonnyp/json-patch-tests/blob/master/tests.json#L102 ?

nope; my test specifically test pushing a value at the last index + 1

I made the test because all tests would pass except this because my implementation believed it was out of bound

mikemccabe commented 9 years ago

https://github.com/sonnyp/json-patch-tests/blob/master/tests.json#L110

sonnyp commented 9 years ago

oopsie.

sonnyp commented 9 years ago

https://github.com/json-patch/json-patch-tests/pull/21