Always throw if unable to overwrite

index.js

@@ -30,9 +30,7 @@ const cpFileAsync = async (source, destination, options, progressEmitter) => {
 		progressEmitter.written = progressEmitter.size;
 		updateStats = true;
 	} catch (error) {
-		if (options.overwrite || error.code !== 'EEXIST') {
-			throw new CpFileError(`Cannot write to \`${destination}\`: ${error.message}`, error);
-		}
+		throw new CpFileError(`Cannot write to \`${destination}\`: ${error.message}`, error);
 	}
 
 	if (readError) {

test/async.js

@@ -7,9 +7,9 @@ import del from 'del';
 import test from 'ava';
 import uuid from 'uuid';
 import sinon from 'sinon';
-import cpFile from '..';
 import assertDateEqual from './helpers/_assert';
 import {buildEACCES, buildEIO, buildENOSPC, buildENOENT, buildEPERM} from './helpers/_fs-errors';
+import cpFile from '..';
 
 const THREE_HUNDRED_KILO = (100 * 3 * 1024) + 1;
 
@@ -75,8 +75,9 @@ test('overwrite when options are undefined', async t => {
 
 test('do not overwrite when disabled', async t => {
 	fs.writeFileSync(t.context.destination, '');
-	await cpFile('license', t.context.destination, {overwrite: false});
-	t.is(fs.readFileSync(t.context.destination, 'utf8'), '');
+	const error = await t.throwsAsync(cpFile('license', t.context.destination, {overwrite: false}));
+	t.is(error.name, 'CpFileError', error.message);
+	t.is(error.code, 'EEXIST', error.message);
 });
 
 test('do not create `destination` on unreadable `source`', async t => {

test/sync.js

@@ -5,9 +5,9 @@ import del from 'del';
 import test from 'ava';
 import uuid from 'uuid';
 import sinon from 'sinon';
-import cpFile from '..';
 import assertDateEqual from './helpers/_assert';
 import {buildEACCES, buildENOSPC, buildEBADF, buildEPERM} from './helpers/_fs-errors';
+import cpFile from '..';
 
 const THREE_HUNDRED_KILO = (100 * 3 * 1024) + 1;