Make text/turtle the default content type for mapping .acl files

Author: rubensworks

lib/resource-mapper.js

@@ -8,13 +8,21 @@ const readdir = promisify(fs.readdir)
 // following the principles of the “sweet spot” discussed in
 // https://www.w3.org/DesignIssues/HTTPFilenameMapping.html
 class ResourceMapper {
-  constructor ({ rootUrl, rootPath, includeHost, defaultContentType, indexName = 'index' }) {
+  constructor ({
+    rootUrl,
+    rootPath,
+    includeHost,
+    defaultContentType,
+    indexName = 'index',
+    overrideTypes = { acl: 'text/turtle' }
+  }) {
     this._rootUrl = this._removeTrailingSlash(rootUrl)
     this._rootPath = this._removeTrailingSlash(rootPath)
     this._includeHost = includeHost
     this._readdir = readdir
     this._defaultContentType = defaultContentType
     this._indexName = indexName
+    this._types = { ...types, ...overrideTypes }
 
     // If the host needs to be replaced on every call, pre-split the root URL
     if (includeHost) {
@@ -128,7 +136,7 @@ class ResourceMapper {
   // Gets the expected content type based on the extension of the path
   _getContentTypeByExtension (path) {
     const extension = /\.([^/.]+)$/.exec(path)
-    return extension && types[extension[1].toLowerCase()] || this._defaultContentType
+    return extension && this._types[extension[1].toLowerCase()] || this._defaultContentType
   }
 
   // Removes a possible trailing slash from a path

test/unit/resource-mapper-test.js

@@ -98,6 +98,28 @@ describe('ResourceMapper', () => {
         contentType: 'image/jpeg'
       })
 
+    itMapsUrl(mapper, 'a URL with an overridden extension that matches the content type',
+      {
+        url: 'http://localhost/space/foo.acl',
+        contentType: 'text/turtle',
+        createIfNotExists: true
+      },
+      {
+        path: `${rootPath}space/foo.acl`,
+        contentType: 'text/turtle'
+      })
+
+    itMapsUrl(mapper, 'a URL with an alternative overridden extension that matches the content type',
+      {
+        url: 'http://localhost/space/foo.acl',
+        contentType: 'text/n3',
+        createIfNotExists: true
+      },
+      {
+        path: `${rootPath}space/foo.acl$.n3`,
+        contentType: 'text/n3'
+      })
+
     // GET/HEAD/POST/DELETE/PATCH base cases
 
     itMapsUrl(mapper, 'a URL of a non-existing file',
@@ -192,6 +214,30 @@ describe('ResourceMapper', () => {
         contentType: 'text/html'
       })
 
+    itMapsUrl(mapper, 'a URL of an existing .acl file',
+      {
+        url: 'http://localhost/space/.acl'
+      },
+      [
+        `${rootPath}space/.acl`
+      ],
+      {
+        path: `${rootPath}space/.acl`,
+        contentType: 'text/turtle'
+      })
+
+    itMapsUrl(mapper, 'a URL of an existing .acl file with a different content type',
+      {
+        url: 'http://localhost/space/.acl'
+      },
+      [
+        `${rootPath}space/.acl$.n3`
+      ],
+      {
+        path: `${rootPath}space/.acl$.n3`,
+        contentType: 'text/n3'
+      })
+
     itMapsUrl(mapper, 'a URL ending with a slash to an index file when index.html is available',
       {
         url: 'http://localhost/space/'
@@ -290,6 +336,13 @@ describe('ResourceMapper', () => {
         contentType: 'text/turtle'
       })
 
+    itMapsFile(mapper, 'an ACL file',
+      { path: `${rootPath}space/.acl` },
+      {
+        url: 'http://localhost/space/.acl',
+        contentType: 'text/turtle'
+      })
+
     itMapsFile(mapper, 'an unknown file type',
       { path: `${rootPath}space/foo.bar` },
       {