JSON.parse .jsonld and extend PATCH to .n3 and .rdf

Author: bourgeoa

lib/handlers/patch.js

@@ -18,8 +18,17 @@ const PATCH_PARSERS = {
   'text/n3': require('./patch/n3-patch-parser.js')
 }
 
+// use mime-type as contentType for new rdf resource
 const DEFAULT_FOR_NEW_CONTENT_TYPE = 'text/turtle'
 
+function contentTypeForNew (req) {
+  let contentTypeForNew = DEFAULT_FOR_NEW_CONTENT_TYPE
+  if (req.path.endsWith('.jsonld')) contentTypeForNew = 'application/ld+json'
+  else if (req.path.endsWith('.n3')) contentTypeForNew = 'text/n3'
+  else if (req.path.endsWith('.rdf')) contentTypeForNew = 'application/rdf+xml'
+  return contentTypeForNew
+}
+
 // Handles a PATCH request
 async function patchHandler (req, res, next) {
   debug(`PATCH -- ${req.originalUrl}`)
@@ -33,9 +42,9 @@ async function patchHandler (req, res, next) {
       // First check if the file already exists
       ({ path, contentType } = await ldp.resourceMapper.mapUrlToFile({ url: req }))
     } catch (err) {
-      // If the file doesn't exist, request one to be created with the default content type
+      // If the file doesn't exist, request one to be created with the file mime type as contentType
       ({ path, contentType } = await ldp.resourceMapper.mapUrlToFile(
-        { url: req, createIfNotExists: true, contentType: DEFAULT_FOR_NEW_CONTENT_TYPE }))
+        { url: req, createIfNotExists: true, contentType: contentTypeForNew(req) }))
       // check if a folder with same name exists
       await ldp.checkItemName(req)
       resourceExists = false
@@ -93,13 +102,14 @@ function readGraph (resource) {
         // If the file does not exist, assume empty contents
         // (it will be created after a successful patch)
         if (err.code === 'ENOENT') {
-          fileContents = ''
+          fileContents = resource.contentType.includes('json') ? JSON.stringify('{}') : ''
           // Fail on all other errors
         } else {
           return reject(error(500, `Original file read error: ${err}`))
         }
       }
       debug('PATCH -- Read target file (%d bytes)', fileContents.length)
+      fileContents = resource.contentType.includes('json') ? JSON.parse(fileContents) : fileContents
       resolve(fileContents)
     })
   )

test/integration/patch-test.js

@@ -84,6 +84,17 @@ describe('PATCH', () => {
       result: '@prefix : </new.ttl#>.\n@prefix tim: </>.\n\ntim:x tim:y tim:z.\n\n'
     }))
 
+    describe('on a non-existing jsonld file', describePatch({
+      path: '/new.jsonld',
+      exists: false,
+      patch: `<> a solid:InsertDeletePatch;
+                 solid:inserts { <x> <y> <z>. }.`
+    }, { // expected:
+      status: 200,
+      text: 'Patch applied successfully',
+      result: '[{"@id":"https://tim.localhost:7777/x","https://tim.localhost:7777/y":[{"@id":"https://tim.localhost:7777/z"}]}]'
+    }))
+
     describe('on a resource with read-only access', describePatch({
       path: '/read-only.ttl',
       patch: `<> a solid:InsertDeletePatch;