handle more path types

* Buffer
* Uint8Array
* URL

index.js

@@ -1,10 +1,11 @@
 'use strict';
 
-const {promisify} = require('util');
+const {types: {isUint8Array}, promisify} = require('util');
 const {unlink} = require('fs');
 
-const ARGLEN_ERROR = 'Expected 1 argument (<string>)';
-const ERROR = 'Expected a path of a file or symbolic link (<string>)';
+const PATH_TYPES = '(<string|Buffer|Uint8Array|URL>)';
+const ARGLEN_ERROR = `Expected 1 argument ${PATH_TYPES}`;
+const ERROR = `Expected a path of a file or symbolic link ${PATH_TYPES}`;
 const promisifiedUnlink = promisify(unlink);
 
 module.exports = async function rmf(...args) {
@@ -30,6 +31,16 @@ module.exports = async function rmf(...args) {
 		throw new Error(`${ERROR}, but got '' (empty string).`);
 	}
 
+	if (path.length === 0) {
+		if (Buffer.isBuffer(path)) {
+			throw new Error(`${ERROR}, but got an empty Buffer.`);
+		}
+
+		if (isUint8Array(path)) {
+			throw new Error(`${ERROR}, but got an empty Uint8Array.`);
+		}
+	}
+
 	try {
 		await promisifiedUnlink(path);
 	} catch (err) {

package.json

@@ -25,6 +25,7 @@
 	"devDependencies": {
 		"@shinnn/eslint-config": "^6.7.1",
 		"eslint": "^5.6.0",
+		"file-url": "^2.0.2",
 		"nyc": "^13.0.1",
 		"nyc-config-common": "^1.0.0",
 		"tape": "^4.9.1"

test.js

@@ -3,6 +3,7 @@
 const {join} = require('path');
 const {access, writeFile} = require('fs').promises;
 
+const fileUrl = require('file-url');
 const rmf = require('.');
 const test = require('tape');
 
@@ -37,32 +38,50 @@ test('rmf()', async t => {
 	}
 
 	t.equal(
-		await rmf(join(__dirname, '__this_file_does_not_exist__')),
+		await rmf(Buffer.from(join(__dirname, '__this_file_does_not_exist__'))),
 		false,
 		'should return false when the target does not exist.'
 	);
 
 	t.equal(
-		(await getError(join(__filename, 'file'))).code,
+		(await getError(new URL(fileUrl(join(__filename, 'file'))))).code,
 		'ENOTDIR',
-		'should return false when it tries to remove a directory.'
+		'should throw an error when it tries to remove a directory.'
 	);
 
 	t.equal(
 		(await getError('')).toString(),
-		'Error: Expected a path of a file or symbolic link (<string>), but got \'\' (empty string).',
-		'should return false when it takes an empty path.'
+		'Error: Expected a path of a file or symbolic link (<string|Buffer|Uint8Array|URL>), but got \'\' (empty string).',
+		'should return false when it takes an empty string.'
+	);
+
+	t.equal(
+		(await getError(Buffer.alloc(0))).toString(),
+		'Error: Expected a path of a file or symbolic link (<string|Buffer|Uint8Array|URL>), but got an empty Buffer.',
+		'should return false when it takes an empty Buffer.'
+	);
+
+	t.equal(
+		(await getError(new Uint8Array())).toString(),
+		'Error: Expected a path of a file or symbolic link (<string|Buffer|Uint8Array|URL>), but got an empty Uint8Array.',
+		'should return false when it takes an empty Uint8Array.'
+	);
+
+	t.equal(
+		(await getError(new Int8Array())).code,
+		'ERR_INVALID_ARG_TYPE',
+		'should return false when it takes an invalid path-type value.'
 	);
 
 	t.equal(
 		(await getError()).toString(),
-		'RangeError: Expected 1 argument (<string>), but got no arguments.',
+		'RangeError: Expected 1 argument (<string|Buffer|Uint8Array|URL>), but got no arguments.',
 		'should return false when it takes no arguments.'
 	);
 
 	t.equal(
 		(await getError('a', 'b')).toString(),
-		'RangeError: Expected 1 argument (<string>), but got 2 arguments.',
+		'RangeError: Expected 1 argument (<string|Buffer|Uint8Array|URL>), but got 2 arguments.',
 		'should return false when it takes too many arguments.'
 	);