Skip to content

Close test coverage gaps - #226

Merged
stefankoegl merged 1 commit into
masterfrom
claude/test-coverage-gaps
Oct 10, 2026
Merged

stefankoegl merged 1 commit into
masterfrom
claude/test-coverage-gaps

Conversation

@stefankoegl

Copy link
Copy Markdown
Owner
  • test_js_file now checks the error cases of tests.js, which it skipped
    along with all other cases that have no 'expected' result
  • Test the jsondiff and jsonpatch scripts in bin/, which had no tests
  • Test that _ArrayItems spreads out its labels again once they run out
  • Test that string patches with duplicate 'op' or 'path' members are
    rejected, and that add and replace in a tuple raise JsonPatchConflict

Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_015o6rtuujv61fERXZhQaL9i

- test_js_file now checks the error cases of tests.js, which it skipped
  along with all other cases that have no 'expected' result
- Test the jsondiff and jsonpatch scripts in bin/, which had no tests
- Test that _ArrayItems spreads out its labels again once they run out
- Test that string patches with duplicate 'op' or 'path' members are
  rejected, and that add and replace in a tuple raise JsonPatchConflict

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015o6rtuujv61fERXZhQaL9i
@stefankoegl
stefankoegl merged commit e01227d into master Oct 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants