fix: set negatedExtGlob also if it does not span the whole pattern

This way micromatch can detect that this is a negation pattern.
micromatch · Apr 10, 2021 · 243f829 · 243f829
1 parent 4a510d7
commit 243f829
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/lib/parse.js b/lib/parse.js
@@ -247,7 +247,7 @@ const parse = (input, options) => {
         output = token.close = `)$))${extglobStar}`;
       }
 
-      if (token.prev.type === 'bos' && eos()) {
+      if (token.prev.type === 'bos') {
         state.negatedExtglob = true;
       }
     }

diff --git a/test/api.picomatch.js b/test/api.picomatch.js
@@ -347,4 +347,19 @@ describe('picomatch', () => {
       });
     });
   });
+
+  describe('state', () => {
+    describe('negatedExtglob', () => {
+      it('should return true', () => {
+        assert(picomatch('!(abc)', {}, true).state.negatedExtglob);
+        assert(picomatch('!(abc)**', {}, true).state.negatedExtglob);
+        assert(picomatch('!(abc)/**', {}, true).state.negatedExtglob);
+      });
+
+      it('should return false', () => {
+        assert(!picomatch('(!(abc))', {}, true).state.negatedExtglob);
+        assert(!picomatch('**!(abc)', {}, true).state.negatedExtglob);
+      });
+    });
+  });
 });