Fix JsonPointer::remove() not handling sequential arrays correctly

[outdooracorn]

The JsonPointer::remove() method doesn't correctly detect sequential vs associative arrays when the TOLERATE_ASSOCIATIVE_ARRAYS is enabled. This causes most sequential arrays to be converted to associative arrays.

The issue is due to the $i counter not being incremented in the foreach loop that detects whether an array is sequential or associative.

While the counter can be incremented in the if statement, like in #65, I've decided to do it in the body of the foreach loop as I believe it to be less "clever" and easier to read. I have also added a test to prove the issue and defend against it in the future.

[codecov]

Codecov Report

Merging #67 (236de58) into master (f4e5117) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 236de58 differs from pull request most recent head 22c33af. Consider uploading reports for the commit 22c33af to get more accurate results

@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   96.95%   96.96%           
=======================================
  Files          15       15           
  Lines         592      593    +1     
=======================================
+ Hits          574      575    +1     
  Misses         18       18           

Files	Coverage Δ
src/JsonPointer.php	94.92% <100.00%> (+0.03%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[outdooracorn]

Hey @vearutop, any chance you're able to find some time to merge this PR? We would love to get the fix pushed out to production as soon as possible. Thanks

[vearutop]

Ah, sorry this took me so long. Thank you for you contribution!