Fix Windows compatibility (#78)

.travis.yml

@@ -1,6 +1,7 @@
 os:
   - linux
   - osx
+  - windows
 language: node_js
 node_js:
   - '12'

index.js

@@ -39,7 +39,6 @@ const preprocessSourcePath = (source, options) => path.resolve(options.cwd ? opt
 
 const preprocessDestinationPath = (source, destination, options) => {
 	let basename = path.basename(source);
-	const dirname = path.dirname(source);
 
 	if (typeof options.rename === 'string') {
 		basename = options.rename;
@@ -52,7 +51,9 @@ const preprocessDestinationPath = (source, destination, options) => {
 	}
 
 	if (options.parents) {
-		return path.join(destination, dirname, basename);
+		const dirname = path.dirname(source);
+		const parsedDirectory = path.parse(dirname);
+		return path.join(destination, dirname.replace(parsedDirectory.root, path.sep), basename);
 	}
 
 	return path.join(destination, basename);

package.json

@@ -54,9 +54,10 @@
 		"p-map": "^3.0.0"
 	},
 	"devDependencies": {
-		"ava": "^2.1.0",
+		"ava": "^3.12.1",
+		"proxyquire": "^2.1.3",
 		"rimraf": "^3.0.0",
-		"tempfile": "^3.0.0",
+		"tempy": "^0.6.0",
 		"tsd": "^0.11.0",
 		"xo": "^0.25.3"
 	}

test.js

@@ -1,39 +1,42 @@
-import path from 'path';
-import fs from 'fs';
-import crypto from 'crypto';
-import rimraf from 'rimraf';
-import test from 'ava';
-import tempfile from 'tempfile';
-import CpyError from './cpy-error';
-import cpy from '.';
+const path = require('path');
+const fs = require('fs');
+const crypto = require('crypto');
+const rimraf = require('rimraf');
+const test = require('ava');
+const tempy = require('tempy');
+const proxyquire = require('proxyquire');
+const CpyError = require('./cpy-error');
+const cpy = require('.');
 
 const read = (...args) => fs.readFileSync(path.join(...args), 'utf8');
+const cpyMockedError = module => {
+	return proxyquire('.', {
+		[module]() {
+			throw new Error(`${module}:\tERROR`);
+		}
+	});
+};
 
 test.beforeEach(t => {
-	t.context.tmp = tempfile();
-	t.context.EPERM = tempfile('EPERM');
-	fs.mkdirSync(t.context.EPERM, 0);
+	t.context.tmp = tempy.file();
+	t.context.dir = tempy.directory();
 });
 
 test.afterEach(t => {
 	rimraf.sync(t.context.tmp);
-	rimraf.sync(t.context.EPERM);
+	rimraf.sync(t.context.dir);
 });
 
 test('reject Errors on missing `source`', async t => {
-	const error1 = await t.throwsAsync(cpy, /`source`/);
-	t.true(error1 instanceof CpyError);
+	await t.throwsAsync(cpy, {message: /`source`/, instanceOf: CpyError});
 
-	const error2 = await t.throwsAsync(cpy(null, 'destination'), /`source`/);
-	t.true(error2 instanceof CpyError);
+	await t.throwsAsync(cpy(null, 'destination'), {message: /`source`/, instanceOf: CpyError});
 
-	const error3 = await t.throwsAsync(cpy([], 'destination'), /`source`/);
-	t.true(error3 instanceof CpyError);
+	await t.throwsAsync(cpy([], 'destination'), {message: /`source`/, instanceOf: CpyError});
 });
 
 test('reject Errors on missing `destination`', async t => {
-	const error = await t.throwsAsync(cpy('TARGET'), /`destination`/);
-	t.true(error instanceof CpyError);
+	await t.throwsAsync(cpy('TARGET'), {message: /`destination`/, instanceOf: CpyError});
 });
 
 test('copy single file', async t => {
@@ -57,21 +60,21 @@ test('throws on invalid concurrency value', async t => {
 test('copy array of files with filter', async t => {
 	await cpy(['license', 'package.json'], t.context.tmp, {
 		filter: file => {
-			if (file.path.endsWith('/license')) {
+			if (file.path.endsWith(`${path.sep}license`)) {
 				t.is(file.path, path.join(process.cwd(), 'license'));
 				t.is(file.relativePath, 'license');
 				t.is(file.name, 'license');
 				t.is(file.nameWithoutExtension, 'license');
 				t.is(file.extension, '');
-			} else if (file.path.endsWith('/package.json')) {
+			} else if (file.path.endsWith(`${path.sep}package.json`)) {
 				t.is(file.path, path.join(process.cwd(), 'package.json'));
 				t.is(file.relativePath, 'package.json');
 				t.is(file.name, 'package.json');
 				t.is(file.nameWithoutExtension, 'package');
 				t.is(file.extension, 'json');
 			}
 
-			return !file.path.endsWith('/license');
+			return !file.path.endsWith(`${path.sep}license`);
 		}
 	});
 
@@ -82,21 +85,21 @@ test('copy array of files with filter', async t => {
 test('copy array of files with async filter', async t => {
 	await cpy(['license', 'package.json'], t.context.tmp, {
 		filter: async file => {
-			if (file.path.endsWith('/license')) {
+			if (file.path.endsWith(`${path.sep}license`)) {
 				t.is(file.path, path.join(process.cwd(), 'license'));
 				t.is(file.relativePath, 'license');
 				t.is(file.name, 'license');
 				t.is(file.nameWithoutExtension, 'license');
 				t.is(file.extension, '');
-			} else if (file.path.endsWith('/package.json')) {
+			} else if (file.path.endsWith(`${path.sep}package.json`)) {
 				t.is(file.path, path.join(process.cwd(), 'package.json'));
 				t.is(file.relativePath, 'package.json');
 				t.is(file.name, 'package.json');
 				t.is(file.nameWithoutExtension, 'package');
 				t.is(file.extension, 'json');
 			}
 
-			return !file.path.endsWith('/license');
+			return !file.path.endsWith(`${path.sep}license`);
 		}
 	});
 
@@ -140,7 +143,8 @@ test('path structure', async t => {
 
 	await cpy([path.join(t.context.tmp, 'cwd/hello.js')], t.context.tmp, {parents: true});
 
-	t.is(read(t.context.tmp, 'cwd/hello.js'), read(t.context.tmp, t.context.tmp, 'cwd/hello.js'));
+	const {root} = path.parse(t.context.tmp);
+	t.is(read(t.context.tmp, 'cwd/hello.js'), read(t.context.tmp, t.context.tmp.replace(root, path.sep), 'cwd/hello.js'));
 });
 
 test('rename filenames but not filepaths', async t => {
@@ -175,19 +179,14 @@ test('rename filenames using a function', async t => {
 	t.is(read(t.context.tmp, 'source/bar.js'), read(t.context.tmp, 'destination/subdir/source/prefix-bar.js'));
 });
 
-test('cp-file errors are not glob errors', async t => {
-	const error = await t.throwsAsync(cpy('license', t.context.EPERM), /EPERM/);
-	t.notRegex(error.message, /glob/);
-});
-
 test('cp-file errors are CpyErrors', async t => {
-	const error = await t.throwsAsync(cpy('license', t.context.EPERM), /EPERM/);
-	t.true(error instanceof CpyError);
+	const cpy = cpyMockedError('cp-file');
+	await t.throwsAsync(cpy('license', t.context.dir), {message: /cp-file/, instanceOf: CpyError});
 });
 
 test('glob errors are CpyErrors', async t => {
-	const error = await t.throwsAsync(cpy(t.context.EPERM + '/**', t.context.tmp), /EPERM/);
-	t.true(error instanceof CpyError);
+	const cpy = cpyMockedError('globby');
+	await t.throwsAsync(cpy(path.join(t.context.dir, '/**'), t.context.tmp), {message: /globby/, instanceOf: CpyError});
 });
 
 test('throws on non-existing file', async t => {