#59: fixes trailing backslashes before ]

.github/workflows/nodejs.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [8.x, 10.x, 12.x]
+        node-version: [10.x, 12.x, 14.x]
 
     steps:
     - uses: actions/checkout@v1

index.js

@@ -40,6 +40,12 @@ const sanitizeRange = range => range.replace(
     : EMPTY
 )
 
+// See fixtures #59
+const cleanRangeBackSlash = slashes => slashes.slice(
+  0,
+  parseInt(slashes.length / 2, 10) * 2
+)
+
 // > If the pattern ends with a slash,
 // > it is removed for the purpose of the following description,
 // > but it would only find a match with a directory.
@@ -217,7 +223,7 @@ const REPLACERS = [
     /(\\)?\[([^\]/]*?)(\\*)($|\])/g,
     (match, leadEscape, range, endEscape, close) => leadEscape === ESCAPE
       // '\\[bar]' -> '\\\\[bar\\]'
-      ? `\\[${range}${close}`
+      ? `\\[${range}${cleanRangeBackSlash(endEscape)}${close}`
       : close === ']'
         ? endEscape.length % 2 === 0
           // A normal case, and it is a range notation

test/fixtures/cases.js

@@ -23,12 +23,12 @@ const cases = [
   [
     '#59 and more cases about range notation',
     [
-      'src/\\[foo\\]',
-      'src/\\[foo2\\\\]',
-      'src/\\[foo3\\\\\\]',
-      'src/\\[foo4\\\\\\\\]',
-      'src/\\[foo5\\\\\\\\\\]',
-      'src/\\[foo6\\\\\\\\\\\\]',
+      'src/\\[foo\\]',              // 1 -> 0
+      'src/\\[foo2\\\\]',           // 2 -> 1
+      'src/\\[foo3\\\\\\]',         // 3 -> 1
+      'src/\\[foo4\\\\\\\\]',       // 4 -> 2
+      'src/\\[foo5\\\\\\\\\\]',     // 5 -> 2
+      'src/\\[foo6\\\\\\\\\\\\]',   // 6 -> 3
 
       'src/\\[bar]',