fix: trim pattern

Overlooked that went missing at some point. Fix: #156
isaacs · Feb 15, 2022 · 6ade2da · 6ade2da
1 parent a6f52b0
commit 6ade2da
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/minimatch.js b/minimatch.js
@@ -134,6 +134,8 @@ function Minimatch (pattern, options) {
 
   if (!options) options = {}
 
+  pattern = pattern.trim()
+
   // windows support: need to use /, not \
   if (path.sep !== '/') {
     pattern = pattern.split(path.sep).join('/')

diff --git a/test/basic.js b/test/basic.js
@@ -117,13 +117,13 @@ t.test('whitespace handling', t => {
   t.equal(mm('x/y', 'y'), false)
   t.equal(mm('x/y', 'y', { matchBase: true }), true)
   t.equal(mm('x/ ', ' '), false)
-  t.equal(mm('x/ ', ' ', { matchBase: true }), true)
+  t.equal(mm('x/ ', ' ', { matchBase: true }), false)
   t.equal(mm('x/', ''), false)
   t.equal(mm('x/', '', { matchBase: true }), false)
   t.equal(mm('', ''), true)
   t.equal(mm(' ', ''), false)
-  t.equal(mm('', ' '), false)
-  t.equal(mm(' ', ' '), true)
+  t.equal(mm('', ' '), true)
+  t.equal(mm(' ', ' '), false)
   t.end()
 })
 
@@ -165,6 +165,11 @@ t.test('flipNegate', t => {
   t.end()
 })
 
+t.test('pattern should be trimmed', t => {
+  t.equal(mm('x', ' x '), true)
+  t.end()
+})
+
 function alpha (a, b) {
   return a > b ? 1 : -1
 }