Browse Source

Merge pull request #196 from jochenberger/fix-not-exists-serialization

PR for #178
pull/325/merge
kallestenflo 8 years ago
parent
commit
c6c092a091
  1. 2
      json-path/src/main/java/com/jayway/jsonpath/internal/filter/ValueNode.java
  2. 9
      json-path/src/test/java/com/jayway/jsonpath/FilterCompilerTest.java

2
json-path/src/main/java/com/jayway/jsonpath/internal/filter/ValueNode.java

@ -801,7 +801,7 @@ public abstract class ValueNode {
@Override
public String toString() {
return path.toString();
return existsCheck && ! shouldExist ? Utils.concat("!" , path.toString()) : path.toString();
}
public ValueNode evaluate(Predicate.PredicateContext ctx) {

9
json-path/src/test/java/com/jayway/jsonpath/FilterCompilerTest.java

@ -84,6 +84,15 @@ public class FilterCompilerTest {
assertInvalidPathException("[?(!'foo')]");
}
@Test
// issue #178
public void compile_and_serialize_not_exists_filter(){
Filter compiled = compile("[?(!@.foo)]");
String serialized = compiled.toString();
assertThat(serialized).isEqualTo("[?(!@['foo'])]");
}
private void assertInvalidPathException(String filter){
try {

Loading…
Cancel
Save