Handle reserved Windows filenames (#2)

Author: daveseco7

.travis.yml

@@ -3,5 +3,3 @@ language: node_js
 node_js:
   - '5'
   - '4'
-  - '0.12'
-  - '0.10'

index.js

@@ -7,7 +7,7 @@ var stripOuter = require('strip-outer');
 // doesn't make sense to have longer filenames
 var MAX_FILENAME_LENGTH = 100;
 
-var reControlChars = /[\x00-\x1f\x80-\x9f]/g;
+var reControlChars = /[\x00-\x1f\x80-\x9f]/g; // eslint-disable-line no-control-regex
 var reRelativePath = /^\.+/;
 
 var fn = module.exports = function (str, opts) {
@@ -32,6 +32,7 @@ var fn = module.exports = function (str, opts) {
 		str = str.length > 1 ? stripOuter(str, replacement) : str;
 	}
 
+	str = filenameReservedRegex.windowsNames().test(str) ? str + replacement : str;
 	str = str.slice(0, MAX_FILENAME_LENGTH);
 
 	return str;

package.json

@@ -10,7 +10,7 @@
     "url": "sindresorhus.com"
   },
   "engines": {
-    "node": ">=0.10.0"
+    "node": ">=4"
   },
   "scripts": {
     "test": "xo && ava"
@@ -32,7 +32,7 @@
     "dirname"
   ],
   "dependencies": {
-    "filename-reserved-regex": "^1.0.0",
+    "filename-reserved-regex": "^2.0.0",
     "strip-outer": "^1.0.0",
     "trim-repeated": "^1.0.0"
   },

test.js

@@ -14,6 +14,11 @@ test('filnamify()', t => {
 	t.is(m('..'), '!');
 	t.is(m('./'), '!');
 	t.is(m('../'), '!');
+	t.is(m('con'), 'con!');
+	t.is(m('foo/bar/nul'), 'foo!bar!nul');
+	t.is(m('con', {replacement: '🐴🐴'}), 'con🐴🐴');
+	t.is(m('c/n', {replacement: 'o'}), 'cono');
+	t.is(m('c/n', {replacement: 'con'}), 'cconn');
 });
 
 test('filenamify.path()', t => {