fix: pass eslint (#530)

* fix: pass eslint

There are two types of errors that should not be treated as errors, configure
eslint in the comments to ignore them.

1. @typescript-eslint/require-await, the code does not use await in async, but
we do want a Promise.
2. @typescript-eslint/explicit-module-boundary-types, there is no explicit
return statement in the function, but we really don't care about its return
value.

In order for eslint to make sense, I didn't disable these rules in the file
but to line.

* fix: update code to pass eslint instead of using comment to disable lint rules

Author: Arondight

src/Low.test.ts

@@ -17,14 +17,14 @@ function createJSONFile(obj: unknown): string {
   return file
 }
 
-export async function testNoAdapter() {
+export function testNoAdapter(): void {
   // Ignoring TypeScript error and pass incorrect argument
   // eslint-disable-next-line @typescript-eslint/ban-ts-comment
   // @ts-ignore
   throws(() => new Low(), MissingAdapterError)
 }
 
-export async function testLow() {
+export async function testLow(): Promise<void> {
   type Data = {
     a?: number
     b?: number
@@ -52,7 +52,7 @@ export async function testLow() {
   deepEqual(JSON.parse(data), newObj)
 }
 
-export async function testLodash() {
+export async function testLodash(): Promise<void> {
   // Extend with lodash
   class LowWithLodash<T> extends Low<T> {
     chain: lodash.ExpChain<this['data']> = lodash.chain(this).get('data')

src/LowSync.test.ts

@@ -12,14 +12,14 @@ function createJSONFile(obj: unknown): string {
   return file
 }
 
-export async function testNoAdapter() {
+export function testNoAdapter(): void {
   // Ignoring TypeScript error and pass incorrect argument
   // eslint-disable-next-line @typescript-eslint/ban-ts-comment
   // @ts-ignore
   throws(() => new LowSync(), MissingAdapterError)
 }
 
-export async function testLowSync() {
+export function testLowSync(): void {
   type Data = {
     a?: number
     b?: number

src/adapters/JSONFile.test.ts

@@ -3,7 +3,7 @@ import tempy from 'tempy'
 
 import { JSONFile } from './JSONFile.js'
 
-export async function testJSONFile() {
+export async function testJSONFile(): Promise<void> {
   const obj = { a: 1 }
 
   const filename = tempy.file()

src/adapters/JSONFileSync.test.ts

@@ -3,7 +3,7 @@ import tempy from 'tempy'
 
 import { JSONFileSync } from './JSONFileSync.js'
 
-export async function testJSONFileSync() {
+export function testJSONFileSync(): void {
   const obj = { a: 1 }
 
   const filename = tempy.file()

src/adapters/LocalStorage.test.ts

@@ -21,7 +21,7 @@ global.localStorage = {
   },
 }
 
-export async function testLocalStorage() {
+export function testLocalStorage(): void {
   const obj = { a: 1 }
   const storage = new LocalStorage('key')
 

src/adapters/Memory.test.ts

@@ -2,7 +2,7 @@ import { deepStrictEqual as deepEqual, strictEqual as equal } from 'assert'
 
 import { Memory } from './Memory.js'
 
-export async function testMemory() {
+export async function testMemory(): Promise<void> {
   const obj = { a: 1 }
 
   const memory = new Memory()

src/adapters/MemorySync.test.ts

@@ -2,7 +2,7 @@ import { deepStrictEqual as deepEqual, strictEqual as equal } from 'assert'
 
 import { MemorySync } from './MemorySync.js'
 
-export function testMemorySync() {
+export function testMemorySync(): void {
   const obj = { a: 1 }
 
   const memory = new MemorySync()

src/adapters/TextFile.test.ts

@@ -3,7 +3,7 @@ import tempy from 'tempy'
 
 import { TextFile } from './TextFile.js'
 
-export async function testTextFile() {
+export async function testTextFile(): Promise<void> {
   const str = 'foo'
 
   const filename = tempy.file()
@@ -19,7 +19,7 @@ export async function testTextFile() {
   deepEqual(await file.read(), str)
 }
 
-export async function testRaceCondition() {
+export async function testRaceCondition(): Promise<void> {
   const filename = tempy.file()
   const file = new TextFile(filename)
   const promises: Promise<void>[] = []

src/adapters/TextFileSync.test.ts

@@ -3,7 +3,7 @@ import tempy from 'tempy'
 
 import { TextFileSync } from './TextFileSync.js'
 
-export async function testTextFileSync() {
+export function testTextFileSync(): void {
   const str = 'foo'
 
   const filename = tempy.file()