Convert all keyword functions to arrow functions

Author: rekmarks

src/asMiddleware.test.ts

@@ -7,7 +7,7 @@ import {
 
 const jsonrpc = '2.0' as const;
 
-describe('asMiddleware', function () {
+describe('asMiddleware', () => {
   it('basic', async () => {
     const engine = new JsonRpcEngine();
     const subengine = new JsonRpcEngine();

src/createAsyncMiddleware.test.ts

@@ -6,7 +6,7 @@ import {
 
 const jsonrpc = '2.0' as const;
 
-describe('createAsyncMiddleware', function () {
+describe('createAsyncMiddleware', () => {
   it('basic middleware test', async () => {
     const engine = new JsonRpcEngine();
 

src/createScaffoldMiddleware.test.ts

@@ -6,7 +6,7 @@ import {
   assertIsJsonRpcFailure,
 } from '.';
 
-describe('createScaffoldMiddleware', function () {
+describe('createScaffoldMiddleware', () => {
   it('basic middleware test', async () => {
     const engine = new JsonRpcEngine();
 

src/engine.test.ts

@@ -7,15 +7,15 @@ import {
 
 const jsonrpc = '2.0' as const;
 
-describe('JsonRpcEngine', function () {
-  it('handle: throws on truthy, non-function callback', function () {
+describe('JsonRpcEngine', () => {
+  it('handle: throws on truthy, non-function callback', () => {
     const engine: any = new JsonRpcEngine();
     expect(() => engine.handle({}, true)).toThrow(
       '"callback" must be a function if provided.',
     );
   });
 
-  it('handle: returns error for invalid request parameter', async function () {
+  it('handle: returns error for invalid request parameter', async () => {
     const engine = new JsonRpcEngine();
     let response: any = await engine.handle(null as any);
     expect(response.error.code).toStrictEqual(-32600);
@@ -26,7 +26,7 @@ describe('JsonRpcEngine', function () {
     expect(response.result).toBeUndefined();
   });
 
-  it('handle: returns error for invalid request method', async function () {
+  it('handle: returns error for invalid request method', async () => {
     const engine = new JsonRpcEngine();
     let response: any = await engine.handle({ method: null } as any);
     expect(response.error.code).toStrictEqual(-32600);
@@ -79,7 +79,7 @@ describe('JsonRpcEngine', function () {
     });
   });
 
-  it('handle (async): basic middleware test', async function () {
+  it('handle (async): basic middleware test', async () => {
     const engine = new JsonRpcEngine();
 
     engine.push(function (_req, res, _next, end) {
@@ -299,7 +299,7 @@ describe('JsonRpcEngine', function () {
     });
   });
 
-  it('handle: batch payloads (async signature)', async function () {
+  it('handle: batch payloads (async signature)', async () => {
     const engine = new JsonRpcEngine();
 
     engine.push(function (req, res, _next, end) {
@@ -329,7 +329,7 @@ describe('JsonRpcEngine', function () {
     expect(res[4].result).toStrictEqual(5);
   });
 
-  it('handle: batch payload with bad request object', async function () {
+  it('handle: batch payload with bad request object', async () => {
     const engine = new JsonRpcEngine();
 
     engine.push(function (req, res, _next, end) {
@@ -518,7 +518,7 @@ describe('JsonRpcEngine', function () {
     });
   });
 
-  it('handles batch request processing error (async)', async function () {
+  it('handles batch request processing error (async)', async () => {
     const engine = new JsonRpcEngine();
     jest
       .spyOn(engine as any, '_promiseHandle')

src/idRemapMiddleware.test.ts

@@ -1,6 +1,6 @@
 import { JsonRpcEngine, createIdRemapMiddleware } from '.';
 
-describe('idRemapMiddleware', function () {
+describe('idRemapMiddleware', () => {
   it('basic middleware test', async () => {
     const engine = new JsonRpcEngine();
 

src/mergeMiddleware.test.ts

@@ -7,7 +7,7 @@ import {
 
 const jsonrpc = '2.0' as const;
 
-describe('mergeMiddleware', function () {
+describe('mergeMiddleware', () => {
   it('basic', async () => {
     const engine = new JsonRpcEngine();
     let originalReq: JsonRpcRequest<unknown>;

src/utils.test.ts

@@ -6,8 +6,8 @@ import {
   assertIsJsonRpcFailure,
 } from '.';
 
-describe('isJsonRpcSuccess', function () {
-  it('correctly identifies JSON-RPC response objects', function () {
+describe('isJsonRpcSuccess', () => {
+  it('correctly identifies JSON-RPC response objects', () => {
     (
       [
         [{ result: 'success' }, true],
@@ -21,8 +21,8 @@ describe('isJsonRpcSuccess', function () {
   });
 });
 
-describe('isJsonRpcFailure', function () {
-  it('correctly identifies JSON-RPC response objects', function () {
+describe('isJsonRpcFailure', () => {
+  it('correctly identifies JSON-RPC response objects', () => {
     (
       [
         [{ error: 'failure' }, true],
@@ -36,8 +36,8 @@ describe('isJsonRpcFailure', function () {
   });
 });
 
-describe('assertIsJsonRpcSuccess', function () {
-  it('correctly identifies JSON-RPC response objects', function () {
+describe('assertIsJsonRpcSuccess', () => {
+  it('correctly identifies JSON-RPC response objects', () => {
     ([{ result: 'success' }, { result: null }] as any[]).forEach((input) => {
       expect(() => assertIsJsonRpcSuccess(input)).not.toThrow();
     });
@@ -50,8 +50,8 @@ describe('assertIsJsonRpcSuccess', function () {
   });
 });
 
-describe('assertIsJsonRpcFailure', function () {
-  it('correctly identifies JSON-RPC response objects', function () {
+describe('assertIsJsonRpcFailure', () => {
+  it('correctly identifies JSON-RPC response objects', () => {
     ([{ error: 'failure' }, { error: null }] as any[]).forEach((input) => {
       expect(() => assertIsJsonRpcFailure(input)).not.toThrow();
     });
@@ -64,7 +64,7 @@ describe('assertIsJsonRpcFailure', function () {
   });
 });
 
-describe('getJsonRpcIdValidator', function () {
+describe('getJsonRpcIdValidator', () => {
   const getInputs = () => {
     return {
       // invariant with respect to options
@@ -91,7 +91,7 @@ describe('getJsonRpcIdValidator', function () {
     }
   };
 
-  it('performs as expected with default options', function () {
+  it('performs as expected with default options', () => {
     const inputs = getInputs();
 
     // The default options are:
@@ -101,7 +101,7 @@ describe('getJsonRpcIdValidator', function () {
     expect(() => validateAll(getJsonRpcIdValidator(), inputs)).not.toThrow();
   });
 
-  it('performs as expected with "permitEmptyString: false"', function () {
+  it('performs as expected with "permitEmptyString: false"', () => {
     const inputs = getInputs();
     inputs.emptyString.expected = false;
 
@@ -115,7 +115,7 @@ describe('getJsonRpcIdValidator', function () {
     ).not.toThrow();
   });
 
-  it('performs as expected with "permitFractions: true"', function () {
+  it('performs as expected with "permitFractions: true"', () => {
     const inputs = getInputs();
     inputs.fraction.expected = true;
 
@@ -129,7 +129,7 @@ describe('getJsonRpcIdValidator', function () {
     ).not.toThrow();
   });
 
-  it('performs as expected with "permitNull: false"', function () {
+  it('performs as expected with "permitNull: false"', () => {
     const inputs = getInputs();
     inputs.null.expected = false;