Add tests for case differences.

Author: RubenVerborgh

lib/handlers/resource-mapper.js

@@ -27,7 +27,8 @@ class ResourceMapper {
 
     // Map to the filename on disk, appending the extension of different from the URL
     const extension = contentType in extensions ? `.${extensions[contentType][0]}` : '.unknown'
-    const suffix = extension === urlExtension ? extension : `${urlExtension}$${extension}`
+    const correctExtension = types[urlExtension.substr(1).toLowerCase()] === contentType
+    const suffix = correctExtension ? urlExtension : `${urlExtension}$${extension}`
     const path = `${this._rootPath}${urlPath}${suffix}`
 
     return { path, contentType: contentType || DEFAULT_CONTENTTYPE }

test/unit/resource-mapper-test.js

@@ -59,6 +59,39 @@ describe('ResourceMapper', () => {
         contentType: 'application/octet-stream'
       })
 
+    itMapsUrl(mapper, 'a URL with an alternative extension that matches the content type',
+      {
+        url: 'http://localhost/space/foo.jpeg',
+        contentType: 'image/jpeg',
+        createIfNotExists: true
+      },
+      {
+        path: `${rootPath}space/foo.jpeg`,
+        contentType: 'image/jpeg'
+      })
+
+    itMapsUrl(mapper, 'a URL with an uppercase extension that matches the content type',
+      {
+        url: 'http://localhost/space/foo.JPG',
+        contentType: 'image/jpeg',
+        createIfNotExists: true
+      },
+      {
+        path: `${rootPath}space/foo.JPG`,
+        contentType: 'image/jpeg'
+      })
+
+    itMapsUrl(mapper, 'a URL with a mixed-case extension that matches the content type',
+      {
+        url: 'http://localhost/space/foo.jPeG',
+        contentType: 'image/jpeg',
+        createIfNotExists: true
+      },
+      {
+        path: `${rootPath}space/foo.jPeG`,
+        contentType: 'image/jpeg'
+      })
+
     // GET/HEAD/POST/DELETE/PATCH base cases
 
     itMapsUrl.skip(mapper, 'a URL of a non-existing file',