Fix: Ensure folder names containing dots don't cause a failure (fixes #…

…11)
gulpjs · Jan 3, 2019 · da4fd86 · da4fd86
1 parent af80b79
commit da4fd86
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -5,7 +5,7 @@ const interpret = require('interpret');
 const EXTRE = /^[.]?[^.]+([.].*)$/;
 
 exports.registerFor = function (filepath, cwd) {
-  var match = EXTRE.exec(filepath);
+  var match = EXTRE.exec(path.basename(filepath));
   if (!match) {
     return;
   }

diff --git a/test/fixtures/folder.with.dots/test.yaml b/test/fixtures/folder.with.dots/test.yaml
@@ -0,0 +1,5 @@
+data:
+  trueKey: true
+  falseKey: false
+  subKey:
+    subProp: 1
diff --git a/test/index.js b/test/index.js
@@ -85,6 +85,13 @@ describe('registerFor', function () {
     rechoir.registerFor('./test/fixtures/test.yaml');
     expect(require('./fixtures/test.yaml')).to.deep.equal(expected);
   });
+  it('must not fail on folders with dots', function () {
+    delete require.extensions['.yml'];
+    delete require.extensions['.yaml'];
+    delete require.cache[require.resolve('require-yaml')];
+    rechoir.registerFor('./test/fixtures/folder.with.dots/test.yaml');
+    expect(require('./fixtures/folder.with.dots/test.yaml')).to.deep.equal(expected);
+  });
 });
 
 describe('load', function () {