feat: add flipBackslashes option to avoid auto conversion of slashes

mrmlnc · Sep 9, 2019 · a326e3b · a326e3b
1 parent d497548
commit a326e3b
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -36,10  36,19 @@ globParent('path/foo'); // 'path' (see issue #3 for details)
 
 ## API
 
-### `globParent(maybeGlobString)`
 ### `globParent(maybeGlobString, [options])`
 
 Takes a string and returns the part of the path before the glob begins. Be aware of Escaping rules and Limitations below.
 
 #### options
 
 ```js
 {
   // Disables the automatic conversion of slashes for Windows
   flipBackslashes: true
 }
 ```
 
 ## Escaping
 
 The following characters have special significance in glob patterns and must be escaped if you want them to be treated as regular path characters:

diff --git a/index.js b/index.js
@@ -10,9  10,16 @@ var enclosure = /[\{\[].*[\/]*.*[\}\]]$/;
 var globby = /(^|[^\\])([\{\[]|\([^\)] $)/;
 var escaped = /\\([\*\?\|\[\]\(\)\{\}])/g;
 
-module.exports = function globParent(str) {
 /**
  * @param {string} str
  * @param {Object} opts
  * @param {boolean} [opts.flipBackslashes=true]
  */
 module.exports = function globParent(str, opts) {
   var options = Object.assign({ flipBackslashes: true }, opts);
 
   // flip windows path separators
-  if (isWin32 && str.indexOf(slash) < 0) {
   if (options.flipBackslashes && isWin32 && str.indexOf(slash) < 0) {
     str = str.replace(backslash, slash);
   }
 

diff --git a/test/index.test.js b/test/index.test.js
@@ -78,6  78,7 @@ describe('glob-parent', function() {
     assert.equal(gp('[bar]/'), '.');
     assert.equal(gp('./\\[bar]'), './[bar]');
     assert.equal(gp('\\[bar]/'), '[bar]');
     assert.equal(gp('foo-\\(bar\\).md'), 'foo-');
     assert.equal(gp('[bar\\]/'), '.');
     assert.equal(gp('path/foo \\[bar]/'), 'path/foo [bar]');
     assert.equal(gp('path/\\{foo,bar}/'), 'path/{foo,bar}');
@@ -135,6  136,12 @@ describe('glob-parent', function() {
 
     done();
   });
 
   it('should respect disabled auto flip backslashes', function(done) {
     assert.equal(gp('foo-\\(bar\\).md', { flipBackslashes: false }), '.');
 
     done();
   });
 });
 
 describe('glob2base test patterns', function() {