From 3f0cf9619fce71a8898c495501df4bdb0e07e735 Mon Sep 17 00:00:00 2001 From: Leo K Date: Thu, 5 Aug 2021 13:08:58 +0200 Subject: refactor(cli/tests): remove unnecessary void return types (#11577) --- cli/tests/unit/read_text_file_test.ts | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) (limited to 'cli/tests/unit/read_text_file_test.ts') diff --git a/cli/tests/unit/read_text_file_test.ts b/cli/tests/unit/read_text_file_test.ts index 2223d9c18..a7eb1a730 100644 --- a/cli/tests/unit/read_text_file_test.ts +++ b/cli/tests/unit/read_text_file_test.ts @@ -7,14 +7,14 @@ import { unitTest, } from "./test_util.ts"; -unitTest({ perms: { read: true } }, function readTextFileSyncSuccess(): void { +unitTest({ perms: { read: true } }, function readTextFileSyncSuccess() { const data = Deno.readTextFileSync("cli/tests/fixture.json"); assert(data.length > 0); const pkg = JSON.parse(data); assertEquals(pkg.name, "deno"); }); -unitTest({ perms: { read: true } }, function readTextFileSyncByUrl(): void { +unitTest({ perms: { read: true } }, function readTextFileSyncByUrl() { const data = Deno.readTextFileSync( pathToAbsoluteFileUrl("cli/tests/fixture.json"), ); @@ -23,13 +23,13 @@ unitTest({ perms: { read: true } }, function readTextFileSyncByUrl(): void { assertEquals(pkg.name, "deno"); }); -unitTest({ perms: { read: false } }, function readTextFileSyncPerm(): void { +unitTest({ perms: { read: false } }, function readTextFileSyncPerm() { assertThrows(() => { Deno.readTextFileSync("cli/tests/fixture.json"); }, Deno.errors.PermissionDenied); }); -unitTest({ perms: { read: true } }, function readTextFileSyncNotFound(): void { +unitTest({ perms: { read: true } }, function readTextFileSyncNotFound() { assertThrows(() => { Deno.readTextFileSync("bad_filename"); }, Deno.errors.NotFound); @@ -37,7 +37,7 @@ unitTest({ perms: { read: true } }, function readTextFileSyncNotFound(): void { unitTest( { perms: { read: true } }, - async function readTextFileSuccess(): Promise { + async function readTextFileSuccess() { const data = await Deno.readTextFile("cli/tests/fixture.json"); assert(data.length > 0); const pkg = JSON.parse(data); @@ -45,9 +45,7 @@ unitTest( }, ); -unitTest({ perms: { read: true } }, async function readTextFileByUrl(): Promise< - void -> { +unitTest({ perms: { read: true } }, async function readTextFileByUrl() { const data = await Deno.readTextFile( pathToAbsoluteFileUrl("cli/tests/fixture.json"), ); @@ -56,15 +54,13 @@ unitTest({ perms: { read: true } }, async function readTextFileByUrl(): Promise< assertEquals(pkg.name, "deno"); }); -unitTest({ perms: { read: false } }, async function readTextFilePerm(): Promise< - void -> { +unitTest({ perms: { read: false } }, async function readTextFilePerm() { await assertThrowsAsync(async () => { await Deno.readTextFile("cli/tests/fixture.json"); }, Deno.errors.PermissionDenied); }); -unitTest({ perms: { read: true } }, function readTextFileSyncLoop(): void { +unitTest({ perms: { read: true } }, function readTextFileSyncLoop() { for (let i = 0; i < 256; i++) { Deno.readTextFileSync("cli/tests/fixture.json"); } @@ -72,7 +68,7 @@ unitTest({ perms: { read: true } }, function readTextFileSyncLoop(): void { unitTest( { perms: { read: true } }, - async function readTextFileDoesNotLeakResources(): Promise { + async function readTextFileDoesNotLeakResources() { const resourcesBefore = Deno.resources(); await assertThrowsAsync(async () => await Deno.readTextFile("cli")); assertEquals(resourcesBefore, Deno.resources()); @@ -81,7 +77,7 @@ unitTest( unitTest( { perms: { read: true } }, - function readTextFileSyncDoesNotLeakResources(): void { + function readTextFileSyncDoesNotLeakResources() { const resourcesBefore = Deno.resources(); assertThrows(() => Deno.readTextFileSync("cli")); assertEquals(resourcesBefore, Deno.resources()); -- cgit v1.2.3