Minor tweaks

Author: sindresorhus

lib/linux.js

@@ -54,11 +54,11 @@ export default async function linux(paths) {
 
 	const trashPathsCache = new Map();
 
-	const getDeviceTrashPaths = async developmentId => {
-		let trashPathsPromise = trashPathsCache.get(developmentId);
+	const getDeviceTrashPaths = async deviceId => {
+		let trashPathsPromise = trashPathsCache.get(deviceId);
 		if (!trashPathsPromise) {
 			trashPathsPromise = (async () => {
-				const trashPath = await xdgTrashdir(mountPointMap.get(developmentId));
+				const trashPath = await xdgTrashdir(mountPointMap.get(deviceId));
 				const paths = {
 					filesPath: path.join(trashPath, 'files'),
 					infoPath: path.join(trashPath, 'info'),
@@ -67,7 +67,7 @@ export default async function linux(paths) {
 				await fs.promises.mkdir(paths.infoPath, {mode: 0o700, recursive: true});
 				return paths;
 			})();
-			trashPathsCache.set(developmentId, trashPathsPromise);
+			trashPathsCache.set(deviceId, trashPathsPromise);
 		}
 
 		return trashPathsPromise;

test.js

@@ -195,3 +195,39 @@ test('glob with nested directories', async () => {
 	assert.ok(!fs.existsSync(directory2));
 	assert.ok(!fs.existsSync(directory3));
 });
+
+test('empty array', async () => {
+	await assert.doesNotReject(trash([]));
+});
+
+test('mixed existing and non-existing files', async () => {
+	fs.writeFileSync('exists1', '');
+	fs.writeFileSync('exists2', '');
+	assert.ok(fs.existsSync('exists1'));
+	assert.ok(fs.existsSync('exists2'));
+	assert.ok(!fs.existsSync('does-not-exist'));
+
+	await trash(['exists1', 'does-not-exist', 'exists2']);
+
+	assert.ok(!fs.existsSync('exists1'));
+	assert.ok(!fs.existsSync('exists2'));
+	assert.ok(!fs.existsSync('does-not-exist'));
+});
+
+test('single file path', async () => {
+	fs.writeFileSync('single-file', '');
+	assert.ok(fs.existsSync('single-file'));
+
+	await trash('single-file');
+
+	assert.ok(!fs.existsSync('single-file'));
+});
+
+test('empty directory', async () => {
+	fs.mkdirSync('empty-dir');
+	assert.ok(fs.existsSync('empty-dir'));
+
+	await trash('empty-dir');
+
+	assert.ok(!fs.existsSync('empty-dir'));
+});