This repository was archived by the owner on Nov 7, 2024. It is now read-only.

Commit 4474200

authored

Merge pull request #67 from lukasj/iss59

fixes #59: Add context info for JSON Patch test failure

2 parents afb2f55 + 8a1673f commit 4474200Copy full SHA for 4474200

File tree

-4

lines changed

-4

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -269,8 +269,8 @@ static String PATCH_MOVE_TARGET_NULL(String from) {`
`269`	`269`	`return localize("patch.move.target.null", from);`
`270`	`270`	`}`
`271`	`271`
`272`		`- static String PATCH_TEST_FAILED() {`
`273`		`- return localize("patch.test.failed");`
	`272`	`+ static String PATCH_TEST_FAILED(Stringpath, Stringvalue) {`
	`273`	`+ return localize("patch.test.failed", path, value);`
`274`	`274`	`}`
`275`	`275`
`276`	`276`	`static String PATCH_ILLEGAL_OPERATION(String operation) {`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -193,7 +193,7 @@ private JsonStructure apply(JsonStructure target, JsonObject operation) {`
`193`	`193`	`return pointer.add(from.remove(target), from.getValue(target));`
`194`	`194`	`case TEST:`
`195`	`195`	`if (! getValue(operation).equals(pointer.getValue(target))) {`
`196`		`- throw new JsonException(JsonMessages.PATCH_TEST_FAILED());`
	`196`	`+ throw new JsonException(JsonMessages.PATCH_TEST_FAILED(operation.getString("path"), getValue(operation).toString()));`
`197`	`197`	`}`
`198`	`198`	`return target;`
`199`	`199`	`default:`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -107,6 +107,6 @@ noderef.array.index.err=An array item index is out of range. Index: {0}, Size: {`
`107`	`107`	`patch.must.be.array=A JSON Patch must be an array of JSON Objects`
`108`	`108`	`patch.move.proper.prefix=The ''{0}'' path of the patch operation ''move'' is a proper prefix of the ''{1}'' path`
`109`	`109`	`patch.move.target.null=The ''{0}'' path of the patch operation ''move'' does not exist in target object`
`110`		`-patch.test.failed=The JSON Patch operation ''test'' failed`
	`110`	`+patch.test.failed=The JSON Patch operation ''test'' failed for path ''{0}'' and value ''{1}''`
`111`	`111`	`patch.illegal.operation=Illegal value for the op member of the JSON Patch operation: ''{0}''`
`112`	`112`	`patch.member.missing=The JSON Patch operation ''{0}'' must contain a ''{1}'' member`

Comments

(0)