fix: ensuring gitignored filenames are exact, using $ (#13)

patrickleet · Jul 18, 2018 · b106216 · b106216
1 parent 4054d35
commit b106216
Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/__tests__/gitignored.js b/__tests__/gitignored.js
@@ -12,7 +12,7 @@ describe('lib/gitignored', () => {
   })
 
   it('.gitignoredRegExpString() returns a string that is a regexp that matches all entries from array', () => {
-    expect(regExpStr).toBe(`(node_modules|\\.DS_Store|(.+)\\.log|\\.git)`)
+    expect(regExpStr).toBe(`(node_modules|\\.DS_Store|(.+)\\.log|\\.git)$`)
   })
 
   it('.gitignoreRegExp() returns a regexp', () => {

diff --git a/lib/gitignored.js b/lib/gitignored.js
@@ -17,14 +17,14 @@ module.exports.getGitignored = function(gitignorePath) {
 }
 
 const gitignoredRegExpString = function(gitignored) {
-  return gitignored.reduce((regex, rule, index) => {
+  return `${gitignored.reduce((regex, rule, index) => {
     return index === gitignored.length - 1 ? `${regex}${rule})` : `${regex}${rule}|`
   }, '(')
     .replace(/\./g, '\\.')
-    .replace(/\*/g, '(.+)')
+    .replace(/\*/g, '(.+)')}$`
 }
 module.exports.gitignoredRegExpString = gitignoredRegExpString
 
 module.exports.gitignoreRegExp = function(gitignored) {
   return new RegExp(gitignoredRegExpString(gitignored), 'g')
-}
+}