Merge pull request #1856 from libris/fix/dont-guess-format-without-data-prefix

fix(rest): don't infer format on bare IDs

Author: andersju

rest/src/main/groovy/whelk/rest/api/CrudGetRequest.java

@@ -32,7 +32,7 @@ static CrudGetRequest parse(HttpServletRequest request) {
     private CrudGetRequest(HttpServletRequest request) {
         this.request = request;
         parsePath(getPath());
-        contentType = getBestContentType(getAcceptHeader(request), dataLeaf != null ? dataLeaf : resourceId);
+        contentType = getBestContentType(getAcceptHeader(request), dataLeaf != null ? dataLeaf : "");
         lens = parseLens(request);
         profile = parseProfile(request);
 

rest/src/test/groovy/whelk/rest/api/CrudSpec.groovy

@@ -187,6 +187,25 @@ class CrudSpec extends Specification {
         response.getContentType() == "application/ld+json"
     }
 
+    def "GET /<id> should display requested document if id contains a dot"() {
+        given:
+        def id = "https://id.kb.se/policy/kb/digisaml/1.0"
+        request.getPathInfo() >> {
+            '/' + id
+        }
+        request.getHeader("Accept") >> {
+            "*/*"
+        }
+        storage.load(_, _) >> {
+            new Document(["@graph": [["@id": id, "foo": "bar", 'mainEntity': ["@id": id + '#it']], ["@id": id + '#it']]])
+        }
+        when:
+        crud.doGet(request, response)
+        then:
+        response.getStatus() == HttpServletResponse.SC_OK
+        response.getContentType() == "application/ld+json"
+    }
+
     def "GET /<sameAs ID> should return 302 Found"() {
         given:
         def id = BASE_URI.resolve("/1234").toString()