Eslint fixes (denoland/deno_std#356)

Make warnings fail
Original: https://github.com/denoland/deno_std/commit/4543b563a9a01c8c168aafcbfd9d4634effba7fc

1 files changed, 44 insertions, 34 deletions

diff --git a/fs/move_test.ts b/fs/move_test.ts
index aa49b4cca..fae951e1f 100644
--- a/fs/move_test.ts
+++ b/fs/move_test.ts
@@ -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,