Require Node.js 6 and add 100% code coverage (sindresorhus#3)

.gitignore

@@ -1,2 +1,4 @@
 node_modules
 yarn.lock
+.nyc_output
+coverage

.travis.yml

@@ -1,5 +1,9 @@
+os:
+  - osx
+  - linux
+  - windows
 language: node_js
 node_js:
+  - '10'
   - '8'
   - '6'
-  - '4'

index.js

@@ -9,12 +9,12 @@ function type(fn, fn2, fp) {
 
 	return pify(fs[fn])(fp)
 		.then(stats => stats[fn2]())
-		.catch(err => {
-			if (err.code === 'ENOENT') {
+		.catch(error => {
+			if (error.code === 'ENOENT') {
 				return false;
 			}
 
-			throw err;
+			throw error;
 		});
 }
 
@@ -25,12 +25,12 @@ function typeSync(fn, fn2, fp) {
 
 	try {
 		return fs[fn](fp)[fn2]();
-	} catch (err) {
-		if (err.code === 'ENOENT') {
+	} catch (error) {
+		if (error.code === 'ENOENT') {
 			return false;
 		}
 
-		throw err;
+		throw error;
 	}
 }
 

package.json

@@ -10,10 +10,10 @@
 		"url": "sindresorhus.com"
 	},
 	"engines": {
-		"node": ">=4"
+		"node": ">=6"
 	},
 	"scripts": {
-		"test": "xo && ava"
+		"test": "xo && nyc ava"
 	},
 	"files": [
 		"index.js"
@@ -36,10 +36,11 @@
 		"filesystem"
 	],
 	"dependencies": {
-		"pify": "^3.0.0"
+		"pify": "^4.0.1"
 	},
 	"devDependencies": {
 		"ava": "*",
+		"nyc": "^13.3.0",
 		"xo": "*"
 	}
 }

test/eacces.js

@@ -0,0 +1,30 @@
+import fs from 'fs';
+import test from 'ava';
+import m from '..';
+
+function fakeError(fp) {
+	const error = new Error(`EACCES: permission denied, stat '${fp}'`);
+	error.code = 'EACCES';
+	return error;
+}
+
+Object.defineProperties(fs, {
+	stat: {
+		value(fp, cb) {
+			cb(fakeError(fp));
+		}
+	},
+	statSync: {
+		value(fp) {
+			throw fakeError(fp);
+		}
+	}
+});
+
+test('throws on EACCES error - async', async t => {
+	await t.throwsAsync(m.file('/root/private'));
+});
+
+test('throws on EACCES error - sync', t => {
+	t.throws(() => m.fileSync('/root/private'));
+});

test/nominal.js

@@ -0,0 +1,49 @@
+import test from 'ava';
+import m from '..';
+
+test('.file()', async t => {
+	t.true(await m.file('package.json'));
+	await t.throwsAsync(m.file(false));
+});
+
+test('.dir()', async t => {
+	t.true(await m.dir('.'));
+	await t.throwsAsync(m.dir(false));
+});
+
+if (process.platform !== 'win32') {
+	test('.symlink()', async t => {
+		t.true(await m.symlink('symlink'));
+		await t.throwsAsync(m.symlink(false));
+	});
+}
+
+test('.fileSync()', t => {
+	t.true(m.fileSync('package.json'));
+});
+
+test('.dirSync()', t => {
+	t.true(m.dirSync('.'));
+});
+
+if (process.platform !== 'win32') {
+	test('.symlinkSync()', t => {
+		t.true(m.symlinkSync('symlink'));
+	});
+}
+
+test('return false if path doesn\'t exist - async', async t => {
+	t.false(await m.file('unicorn'));
+});
+
+test('return false if path doesn\'t exist - sync', t => {
+	t.false(m.fileSync('unicorn'));
+});
+
+test('throws invalid argument - async', async t => {
+	await t.throwsAsync(m.file(false));
+});
+
+test('throws on invalid argument - sync', t => {
+	t.throws(() => m.fileSync(false));
+});