Make warnings fail
Original: 4543b563a9
This commit is contained in:
Vincent LE GOFF 2019-04-24 13:41:23 +02:00 committed by Ryan Dahl
parent e1f7a60bb3
commit dcd01dd025
88 changed files with 948 additions and 806 deletions

View file

@ -13,16 +13,18 @@ import * as path from "./path/mod.ts";
const testdataDir = path.resolve("fs", "testdata");
test(async function moveDirectoryIfSrcNotExists() {
test(async function moveDirectoryIfSrcNotExists(): Promise<void> {
const srcDir = path.join(testdataDir, "move_test_src_1");
const destDir = path.join(testdataDir, "move_test_dest_1");
// if src directory not exist
await assertThrowsAsync(async () => {
await move(srcDir, destDir);
});
await assertThrowsAsync(
async (): Promise<void> => {
await move(srcDir, destDir);
}
);
});
test(async function moveDirectoryIfDestNotExists() {
test(async function moveDirectoryIfDestNotExists(): Promise<void> {
const srcDir = path.join(testdataDir, "move_test_src_2");
const destDir = path.join(testdataDir, "move_test_dest_2");
@ -30,7 +32,7 @@ test(async function moveDirectoryIfDestNotExists() {
// if dest directory not exist
await assertThrowsAsync(
async () => {
async (): Promise<void> => {
await move(srcDir, destDir);
throw new Error("should not throw error");
},
@ -41,17 +43,19 @@ test(async function moveDirectoryIfDestNotExists() {
await Deno.remove(destDir);
});
test(async function moveFileIfSrcNotExists() {
test(async function moveFileIfSrcNotExists(): Promise<void> {
const srcFile = path.join(testdataDir, "move_test_src_3", "test.txt");
const destFile = path.join(testdataDir, "move_test_dest_3", "test.txt");
// if src directory not exist
await assertThrowsAsync(async () => {
await move(srcFile, destFile);
});
await assertThrowsAsync(
async (): Promise<void> => {
await move(srcFile, destFile);
}
);
});
test(async function moveFileIfDestExists() {
test(async function moveFileIfDestExists(): Promise<void> {
const srcDir = path.join(testdataDir, "move_test_src_4");
const destDir = path.join(testdataDir, "move_test_dest_4");
const srcFile = path.join(srcDir, "test.txt");
@ -74,7 +78,7 @@ test(async function moveFileIfDestExists() {
// move it without override
await assertThrowsAsync(
async () => {
async (): Promise<void> => {
await move(srcFile, destFile);
},
Error,
@ -83,7 +87,7 @@ test(async function moveFileIfDestExists() {
// move again with overwrite
await assertThrowsAsync(
async () => {
async (): Promise<void> => {
await move(srcFile, destFile, { overwrite: true });
throw new Error("should not throw error");
},
@ -101,7 +105,7 @@ test(async function moveFileIfDestExists() {
]);
});
test(async function moveDirectory() {
test(async function moveDirectory(): Promise<void> {
const srcDir = path.join(testdataDir, "move_test_src_5");
const destDir = path.join(testdataDir, "move_test_dest_5");
const srcFile = path.join(srcDir, "test.txt");
@ -126,7 +130,9 @@ test(async function moveDirectory() {
await Deno.remove(destDir, { recursive: true });
});
test(async function moveIfSrcAndDestDirectoryExistsAndOverwrite() {
test(async function moveIfSrcAndDestDirectoryExistsAndOverwrite(): Promise<
void
> {
const srcDir = path.join(testdataDir, "move_test_src_6");
const destDir = path.join(testdataDir, "move_test_dest_6");
const srcFile = path.join(srcDir, "test.txt");
@ -156,14 +162,14 @@ test(async function moveIfSrcAndDestDirectoryExistsAndOverwrite() {
await Deno.remove(destDir, { recursive: true });
});
test(async function moveIntoSubDir() {
test(async function moveIntoSubDir(): Promise<void> {
const srcDir = path.join(testdataDir, "move_test_src_7");
const destDir = path.join(srcDir, "nest");
await ensureDir(destDir);
await assertThrowsAsync(
async () => {
async (): Promise<void> => {
await move(srcDir, destDir);
},
Error,
@ -172,16 +178,18 @@ test(async function moveIntoSubDir() {
await Deno.remove(srcDir, { recursive: true });
});
test(function moveSyncDirectoryIfSrcNotExists() {
test(function moveSyncDirectoryIfSrcNotExists(): void {
const srcDir = path.join(testdataDir, "move_sync_test_src_1");
const destDir = path.join(testdataDir, "move_sync_test_dest_1");
// if src directory not exist
assertThrows(() => {
moveSync(srcDir, destDir);
});
assertThrows(
(): void => {
moveSync(srcDir, destDir);
}
);
});
test(function moveSyncDirectoryIfDestNotExists() {
test(function moveSyncDirectoryIfDestNotExists(): void {
const srcDir = path.join(testdataDir, "move_sync_test_src_2");
const destDir = path.join(testdataDir, "move_sync_test_dest_2");
@ -189,7 +197,7 @@ test(function moveSyncDirectoryIfDestNotExists() {
// if dest directory not exist
assertThrows(
() => {
(): void => {
moveSync(srcDir, destDir);
throw new Error("should not throw error");
},
@ -200,17 +208,19 @@ test(function moveSyncDirectoryIfDestNotExists() {
Deno.removeSync(destDir);
});
test(function moveSyncFileIfSrcNotExists() {
test(function moveSyncFileIfSrcNotExists(): void {
const srcFile = path.join(testdataDir, "move_sync_test_src_3", "test.txt");
const destFile = path.join(testdataDir, "move_sync_test_dest_3", "test.txt");
// if src directory not exist
assertThrows(() => {
moveSync(srcFile, destFile);
});
assertThrows(
(): void => {
moveSync(srcFile, destFile);
}
);
});
test(function moveSyncFileIfDestExists() {
test(function moveSyncFileIfDestExists(): void {
const srcDir = path.join(testdataDir, "move_sync_test_src_4");
const destDir = path.join(testdataDir, "move_sync_test_dest_4");
const srcFile = path.join(srcDir, "test.txt");
@ -232,7 +242,7 @@ test(function moveSyncFileIfDestExists() {
// move it without override
assertThrows(
() => {
(): void => {
moveSync(srcFile, destFile);
},
Error,
@ -241,7 +251,7 @@ test(function moveSyncFileIfDestExists() {
// move again with overwrite
assertThrows(
() => {
(): void => {
moveSync(srcFile, destFile, { overwrite: true });
throw new Error("should not throw error");
},
@ -257,7 +267,7 @@ test(function moveSyncFileIfDestExists() {
Deno.removeSync(destDir, { recursive: true });
});
test(function moveSyncDirectory() {
test(function moveSyncDirectory(): void {
const srcDir = path.join(testdataDir, "move_sync_test_src_5");
const destDir = path.join(testdataDir, "move_sync_test_dest_5");
const srcFile = path.join(srcDir, "test.txt");
@ -280,7 +290,7 @@ test(function moveSyncDirectory() {
Deno.removeSync(destDir, { recursive: true });
});
test(function moveSyncIfSrcAndDestDirectoryExistsAndOverwrite() {
test(function moveSyncIfSrcAndDestDirectoryExistsAndOverwrite(): void {
const srcDir = path.join(testdataDir, "move_sync_test_src_6");
const destDir = path.join(testdataDir, "move_sync_test_dest_6");
const srcFile = path.join(srcDir, "test.txt");
@ -307,14 +317,14 @@ test(function moveSyncIfSrcAndDestDirectoryExistsAndOverwrite() {
Deno.removeSync(destDir, { recursive: true });
});
test(function moveSyncIntoSubDir() {
test(function moveSyncIntoSubDir(): void {
const srcDir = path.join(testdataDir, "move_sync_test_src_7");
const destDir = path.join(srcDir, "nest");
ensureDirSync(destDir);
assertThrows(
() => {
(): void => {
moveSync(srcDir, destDir);
},
Error,