Update parser to treat file extensions as case-insensitive (#743)

parcel-bundler · Feb 5, 2018 · ab713a3 · ab713a3
1 parent 55b9640
commit ab713a3
Show file tree

Hide file tree

Showing 8 changed files with 105 additions and 3 deletions.
diff --git a/src/Parser.js b/src/Parser.js
@@ -49,15 +49,15 @@ class Parser {
       ext = '.' + ext;
     }
 
-    this.extensions[ext] = parser;
+    this.extensions[ext.toLowerCase()] = parser;
   }
 
   findParser(filename) {
     if (glob.hasMagic(filename)) {
       return GlobAsset;
     }
 
-    let extension = path.extname(filename);
+    let extension = path.extname(filename).toLowerCase();
     let parser = this.extensions[extension] || RawAsset;
     if (typeof parser === 'string') {
       parser = this.extensions[extension] = require(parser);

diff --git a/test/integration/parser-case-insensitive-ext/icons.SVG b/test/integration/parser-case-insensitive-ext/icons.SVG
diff --git a/test/integration/parser-case-insensitive-ext/index.cSs b/test/integration/parser-case-insensitive-ext/index.cSs
@@ -0,0 +1,3 @@
+body {
+  background: red;
+}
diff --git a/test/integration/parser-case-insensitive-ext/index.html b/test/integration/parser-case-insensitive-ext/index.html
@@ -0,0 +1,17 @@
+<!doctype html>
+<html>
+<head>
+  <link rel="stylesheet" href="index.cSs" />
+</head>
+<body>
+  <h1>Hello world</h1>
+  <p>Linking to <a href="other.HTM">another page</a></p>
+  <a href="#hash_link">Hash link</a>
+  <a href="mailto:someone@acme.com">Mailto link</a>
+  <a href="tel:+33636757575">Tel link</a>
+  <script src="index.js"></script>
+  <script src="https://unpkg.com/parcel-bundler"></script>
+  <i>hello</i> <i>world</i>
+  <svg><use xlink:href="icons.SVG#icon-repo-pull"></use></svg>
+</body>
+</html>
diff --git a/test/integration/parser-case-insensitive-ext/index.js b/test/integration/parser-case-insensitive-ext/index.js
@@ -0,0 +1 @@
+alert('Hi');
diff --git a/test/integration/parser-case-insensitive-ext/other.HTM b/test/integration/parser-case-insensitive-ext/other.HTM
@@ -0,0 +1,10 @@
+<!doctype html>
+<html>
+<head>
+  <link rel="stylesheet" href="index.cSs" />
+</head>
+<body>
+  <h1>Other page</h1>
+  <script src="index.js"></script>
+</body>
+</html>
diff --git a/test/parser.js b/test/parser.js
@@ -0,0 +1,66 @@
+const assert = require('assert');
+const fs = require('fs');
+const {bundle, assertBundleTree} = require('./utils');
+
+describe('parser', function() {
+  it('should support case-insensitive file extension', async function() {
+    let b = await bundle(
+      __dirname + '/integration/parser-case-insensitive-ext/index.html'
+    );
+
+    assertBundleTree(b, {
+      name: 'index.html',
+      assets: ['index.html'],
+      childBundles: [
+        {
+          type: 'svg',
+          assets: ['icons.SVG'],
+          childBundles: []
+        },
+        {
+          type: 'css',
+          assets: ['index.cSs'],
+          childBundles: []
+        },
+        {
+          type: 'js',
+          assets: ['index.js'],
+          childBundles: [
+            {
+              type: 'map'
+            }
+          ]
+        },
+        {
+          type: 'html',
+          assets: ['other.HTM'],
+          childBundles: [
+            {
+              type: 'css',
+              assets: ['index.cSs'],
+              childBundles: []
+            },
+            {
+              type: 'js',
+              assets: ['index.js'],
+              childBundles: [
+                {
+                  type: 'map'
+                }
+              ]
+            }
+          ]
+        }
+      ]
+    });
+
+    let files = fs.readdirSync(__dirname + '/dist');
+    let html = fs.readFileSync(__dirname + '/dist/index.html');
+    for (let file of files) {
+      let ext = file.match(/\.([0-9a-z]+)(?:[?#]|$)/i)[0];
+      if (file !== 'index.html' && ext !== '.map') {
+        assert(html.includes(file));
+      }
+    }
+  });
+});
diff --git a/test/utils.js b/test/utils.js
@@ -147,7 +147,7 @@ function assertBundleTree(bundle, tree) {
   }
 
   if (tree.type) {
-    assert.equal(bundle.type, tree.type);
+    assert.equal(bundle.type.toLowerCase(), tree.type.toLowerCase());
   }
 
   if (tree.assets) {