mirate to typescript

Author: NShahri

src/__tests__/abuse.test.js renamed to src/__tests__/abuse.test.ts

@@ -3,25 +3,24 @@
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
- *
- * @flow
  */
 
-const DataLoader = require('..');
+import DataLoader from '../index.ts';
+import { describe, it, expect } from '@jest/globals';
 
 describe('Provides descriptive error messages for API abuse', () => {
   it('Loader creation requires a function', () => {
     expect(() => {
-      // $FlowExpectError
-      new DataLoader(); // eslint-disable-line no-new
+      // @ts-expect-error testing invalid usage
+      new DataLoader();
     }).toThrow(
       'DataLoader must be constructed with a function which accepts ' +
         'Array<key> and returns Promise<Array<value>>, but got: undefined.',
     );
 
     expect(() => {
-      // $FlowExpectError
-      new DataLoader({}); // eslint-disable-line no-new
+      // @ts-expect-error testing invalid usage
+      new DataLoader({});
     }).toThrow(
       'DataLoader must be constructed with a function which accepts ' +
         'Array<key> and returns Promise<Array<value>>, but got: [object Object].',
@@ -32,15 +31,15 @@ describe('Provides descriptive error messages for API abuse', () => {
     const idLoader = new DataLoader<number, number>(async keys => keys);
 
     expect(() => {
-      // $FlowExpectError
+      // @ts-expect-error testing invalid usage
       idLoader.load();
     }).toThrow(
       'The loader.load() function must be called with a value, ' +
         'but got: undefined.',
     );
 
     expect(() => {
-      // $FlowExpectError
+      // @ts-expect-error testing invalid usage
       idLoader.load(null);
     }).toThrow(
       'The loader.load() function must be called with a value, ' +
@@ -57,15 +56,15 @@ describe('Provides descriptive error messages for API abuse', () => {
     const idLoader = new DataLoader<number, number>(async keys => keys);
 
     expect(() => {
-      // $FlowExpectError
+      // @ts-expect-error testing invalid usage
       idLoader.loadMany();
     }).toThrow(
       'The loader.loadMany() function must be called with Array<key> ' +
         'but got: undefined.',
     );
 
     expect(() => {
-      // $FlowExpectError
+      // @ts-expect-error testing invalid usage
       idLoader.loadMany(1, 2, 3);
     }).toThrow(
       'The loader.loadMany() function must be called with Array<key> ' +
@@ -79,7 +78,7 @@ describe('Provides descriptive error messages for API abuse', () => {
   });
 
   it('Batch function must return a Promise, not null', async () => {
-    // $FlowExpectError
+    // @ts-expect-error testing invalid usage
     const badLoader = new DataLoader<number, number>(() => null);
 
     let caughtError;
@@ -89,7 +88,7 @@ describe('Provides descriptive error messages for API abuse', () => {
       caughtError = error;
     }
     expect(caughtError).toBeInstanceOf(Error);
-    expect((caughtError: any).message).toBe(
+    expect((caughtError as Error).message).toBe(
       'DataLoader must be constructed with a function which accepts ' +
         'Array<key> and returns Promise<Array<value>>, but the function did ' +
         'not return a Promise: null.',
@@ -108,7 +107,7 @@ describe('Provides descriptive error messages for API abuse', () => {
       caughtError = error;
     }
     expect(caughtError).toBeInstanceOf(Error);
-    expect((caughtError: any).message).toBe(
+    expect((caughtError as Error).message).toBe(
       'DataLoader must be constructed with a function which accepts ' +
         'Array<key> and returns Promise<Array<value>>, but the function ' +
         'errored synchronously: Error: Mock Synchronous Error.',
@@ -117,7 +116,7 @@ describe('Provides descriptive error messages for API abuse', () => {
 
   it('Batch function must return a Promise, not a value', async () => {
     // Note: this is returning the keys directly, rather than a promise to keys.
-    // $FlowExpectError
+    // @ts-expect-error testing invalid usage
     const badLoader = new DataLoader<number, number>(keys => keys);
 
     let caughtError;
@@ -127,7 +126,7 @@ describe('Provides descriptive error messages for API abuse', () => {
       caughtError = error;
     }
     expect(caughtError).toBeInstanceOf(Error);
-    expect((caughtError: any).message).toBe(
+    expect((caughtError as Error).message).toBe(
       'DataLoader must be constructed with a function which accepts ' +
         'Array<key> and returns Promise<Array<value>>, but the function did ' +
         'not return a Promise: 1.',
@@ -136,7 +135,7 @@ describe('Provides descriptive error messages for API abuse', () => {
 
   it('Batch function must return a Promise of an Array, not null', async () => {
     // Note: this resolves to undefined
-    // $FlowExpectError
+    // @ts-expect-error testing invalid usage
     const badLoader = new DataLoader<number, number>(async () => null);
 
     let caughtError;
@@ -146,7 +145,7 @@ describe('Provides descriptive error messages for API abuse', () => {
       caughtError = error;
     }
     expect(caughtError).toBeInstanceOf(Error);
-    expect((caughtError: any).message).toBe(
+    expect((caughtError as Error).message).toBe(
       'DataLoader must be constructed with a function which accepts ' +
         'Array<key> and returns Promise<Array<value>>, but the function did ' +
         'not return a Promise of an Array: null.',
@@ -164,7 +163,7 @@ describe('Provides descriptive error messages for API abuse', () => {
       caughtError = error;
     }
     expect(caughtError).toBeInstanceOf(Error);
-    expect((caughtError: any).message).toBe(
+    expect((caughtError as Error).message).toBe(
       'DataLoader must be constructed with a function which accepts ' +
         'Array<key> and returns Promise<Array<value>>, but the function did ' +
         'not return a Promise of an Array of the same length as the Array ' +
@@ -180,17 +179,17 @@ describe('Provides descriptive error messages for API abuse', () => {
     }
 
     expect(() => {
-      // $FlowExpectError
       const incompleteMap = new IncompleteMap();
       const options = { cacheMap: incompleteMap };
-      new DataLoader(async keys => keys, options); // eslint-disable-line no-new
+      // @ts-expect-error testing invalid usage
+      new DataLoader(async keys => keys, options);
     }).toThrow('Custom cacheMap missing methods: set, delete, clear');
   });
 
   it('Requires a number for maxBatchSize', () => {
     expect(
       () =>
-        // $FlowExpectError
+        // @ts-expect-error testing invalid usage
         new DataLoader(async keys => keys, { maxBatchSize: null }),
     ).toThrow('maxBatchSize must be a positive number: null');
   });
@@ -204,15 +203,15 @@ describe('Provides descriptive error messages for API abuse', () => {
   it('Requires a function for cacheKeyFn', () => {
     expect(
       () =>
-        // $FlowExpectError
+        // @ts-expect-error testing invalid usage
         new DataLoader(async keys => keys, { cacheKeyFn: null }),
     ).toThrow('cacheKeyFn must be a function: null');
   });
 
   it('Requires a function for batchScheduleFn', () => {
     expect(
       () =>
-        // $FlowExpectError
+        // @ts-expect-error testing invalid usage
         new DataLoader(async keys => keys, { batchScheduleFn: null }),
     ).toThrow('batchScheduleFn must be a function: null');
   });

src/__tests__/browser.test.js renamed to src/__tests__/browser.test.ts

@@ -3,17 +3,19 @@
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
- *
- * @flow
  */
 
-// Mock out process.nextTick as not existing for this test before requiring.
-process.nextTick = (null: any);
-const DataLoader = require('..');
+// Set up mocks to simulate a recent browser environment.
+// Remove process.nextTick.
+// This mock must be imported before importing DataLoader.
+import './nextTick.mock.ts';
+
+import DataLoader from '../index.ts';
+import { describe, it, expect } from '@jest/globals';
 
 describe('Browser support', () => {
   it('batches multiple requests without process.nextTick', async () => {
-    const loadCalls = [];
+    const loadCalls: ReadonlyArray<number>[] = [];
     const identityLoader = new DataLoader<number, number>(async keys => {
       loadCalls.push(keys);
       return keys;