[GitHub] jena pull request #114: JENA-632: Generate JSON from SPARQL directly

[afs]

Github user afs commented on a diff in the pull request:

    https://github.com/apache/jena/pull/114#discussion_r175915508
  
    --- Diff: 
jena-arq/src/main/java/org/apache/jena/sparql/serializer/QuerySerializer.java 
---
    @@ -142,7 +144,19 @@ public void visitAskResultForm(Query query)
             out.print("ASK") ;
             out.newline() ;
         }
    -    
    +
    +    @Override
    +    public void visitJsonResultForm(Query query) {
    +        out.print("JSON {");
    --- End diff --
    
    More test suggestions: `testParse("JSON { "s": "FOO" , "p" : ?p , "o" : ?o 
} WHERE { }")` to parse to Q, serialize, reparse to Q2 and check Q.equals(Q2).
    
    e.g.a new `org.apache.jena.sparql.syntax.TestJSONParsing`
    
    `JSON { "s": "FOO" }` becomes `JSON { "s": FOO }`


---