Modernize our TypeScript usage

.gitignore

@@ -1,5 +1,6 @@
 /.git
 dist
+tsconfig.tsbuildinfo
 npm-debug.log*
 yarn-debug.log*
 yarn-error.log*

__tests__/__snapshots__/crontab.test.ts.snap

@@ -1,11 +1,11 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`parses crontab file correctly 1`] = `
-Array [
-  Object {
+[
+  {
     "identifier": "simple",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": undefined,
       "jobKeyMode": undefined,
@@ -17,10 +17,10 @@ Array [
     "task": "simple",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "every_day_at_4_am",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": undefined,
       "jobKeyMode": undefined,
@@ -32,10 +32,10 @@ Array [
     "task": "every_day_at_4_am",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "every_sunday_at_4_am",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": undefined,
       "jobKeyMode": undefined,
@@ -47,10 +47,10 @@ Array [
     "task": "every_sunday_at_4_am",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "sunday_7",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": undefined,
       "jobKeyMode": undefined,
@@ -62,46 +62,46 @@ Array [
     "task": "every_sunday_at_4_am",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "every_tuesday_at_4_am",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": undefined,
       "jobKeyMode": undefined,
       "maxAttempts": undefined,
       "priority": undefined,
       "queueName": undefined,
     },
-    "payload": Object {
+    "payload": {
       "isTuesday": true,
     },
     "task": "every_tuesday_at_4_am",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "stuff",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 2685660000,
       "jobKey": undefined,
       "jobKeyMode": undefined,
       "maxAttempts": 3,
       "priority": 3,
       "queueName": "my_queue",
     },
-    "payload": Object {
-      "myExtraPayload": Object {
+    "payload": {
+      "myExtraPayload": {
         "stuff": "here with # hash char",
       },
     },
     "task": "one",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "lots_of_spaces",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": undefined,
       "jobKeyMode": undefined,
@@ -113,10 +113,10 @@ Array [
     "task": "lots_of_spaces",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "with_key",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": "my_key",
       "jobKeyMode": "replace",
@@ -128,10 +128,10 @@ Array [
     "task": "with_key",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "with_key_and_mode",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": "my_key",
       "jobKeyMode": "preserve_run_at",
@@ -143,10 +143,10 @@ Array [
     "task": "with_key_and_mode",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "nested/task",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": "my_key",
       "jobKeyMode": "preserve_run_at",
@@ -158,10 +158,10 @@ Array [
     "task": "nested/task",
     Symbol(isParsed): true,
   },
-  Object {
+  {
     "identifier": "nested/folder/task",
     "match": [Function],
-    "options": Object {
+    "options": {
       "backfillPeriod": 0,
       "jobKey": "my_key",
       "jobKeyMode": "preserve_run_at",

__tests__/batchJobs.test.ts

@@ -1,3 +1,4 @@
+import { jest } from "@jest/globals";
 import { Task, TaskList, WorkerSharedOptions } from "../src/interfaces";
 import { runTaskListOnce } from "../src/main";
 import {
@@ -45,20 +46,20 @@ test("on success, deletes job", () =>
 
     const job3: Task = jest.fn((o) => {
       expect(o).toMatchInlineSnapshot(`
-        Array [
-          Object {
+        [
+          {
             "a": 1,
           },
-          Object {
+          {
             "a": 2,
           },
-          Object {
+          {
             "a": 3,
           },
         ]
       `);
     });
-    const job4: Task = jest.fn();
+    const job4: Task = jest.fn(() => {});
     const tasks: TaskList = {
       job3,
       job4,
@@ -83,7 +84,7 @@ test("on failure, re-enqueues job", () =>
     const job3: Task = jest.fn(() => {
       throw new Error("RETRY");
     });
-    const job4: Task = jest.fn();
+    const job4: Task = jest.fn(() => {});
     const tasks: TaskList = {
       job3,
       job4,
@@ -115,7 +116,7 @@ test("on partial fail, re-enqueues job with just failed elements", () =>
         a % 2 === 1 ? Promise.reject(new Error(String(a))) : null,
       );
     });
-    const job4: Task = jest.fn();
+    const job4: Task = jest.fn(() => {});
     const tasks: TaskList = {
       job3,
       job4,
@@ -147,7 +148,7 @@ test("on partial fail (promise), re-enqueues job with just failed elements", ()
         a % 2 === 1 ? Promise.reject(new Error(String(a))) : null,
       );
     });
-    const job4: Task = jest.fn();
+    const job4: Task = jest.fn(() => {});
     const tasks: TaskList = {
       job3,
       job4,

__tests__/events.test.ts

@@ -1,5 +1,6 @@
+import { jest } from "@jest/globals";
 import { EventEmitter } from "events";
-import { Pool } from "pg";
+import type { Pool } from "pg";
 
 import { run } from "../src";
 import deferred, { Deferred } from "../src/deferred";

__tests__/fixtures-esm/tasks/wouldyoulike.js → __tests__/fixtures-esm/tasks/wouldyoulike.mjs

@@ -1,4 +1,4 @@
-import { rand } from "../blah.js";
+import { rand } from "../blah.mjs";
 export default (_payload, helpers) => {
   helpers.logger.debug(rand());
   return "some sausages";

__tests__/forbiddenFlags.test.ts

@@ -1,3 +1,4 @@
+import { jest } from "@jest/globals";
 import {
   makeWorkerUtils,
   runTaskListOnce,
@@ -28,7 +29,7 @@ test("supports the flags API", () =>
     expect(jobs[0]).toHaveProperty("flags");
     expect(jobs[0].flags).toEqual({ a: true, b: true });
 
-    const task: Task = jest.fn();
+    const task: Task = jest.fn(() => {});
     const taskList: TaskList = { task };
     await runTaskListOnce(options, taskList, pgClient);
   }));
@@ -49,8 +50,8 @@ test.each([
   withPgPool(async (pgPool) => {
     await reset(pgPool, options);
 
-    const shouldRun = jest.fn();
-    const shouldSkip = jest.fn();
+    const shouldRun = jest.fn(() => {});
+    const shouldSkip = jest.fn(() => {});
 
     const job: Task = async (_payload, helpers) => {
       const flags = helpers.job.flags || {};
@@ -111,8 +112,8 @@ test.each([
   withPgPool(async (pgPool) => {
     await reset(pgPool, options);
 
-    const ranWithoutDFlag = jest.fn();
-    const ranWithDFlag = jest.fn();
+    const ranWithoutDFlag = jest.fn(() => {});
+    const ranWithDFlag = jest.fn(() => {});
 
     const job: Task = async (_payload, helpers) => {
       const flags = helpers.job.flags || {};