🚑 Fixed root entrance error

Organized types, optimized their application and removed old types file as if project continues to scale. Types would be better identified in a folder basis.

octokit#584
octokit#583
octokit#106 - Related by setting up tests for proper editing. As before users could not test properly due to project configuration. Will come back to this one once the FATAL error above is confirmed fixed after new release.

jest.config.ts

@@ -71,9 +71,9 @@ export default {
   // maxWorkers: "50%",
 
   // An array of directory names to be searched recursively up from the requiring module's location
-  // moduleDirectories: [
-  //   "node_modules"
-  // ],
+  moduleDirectories: [
+    "node_modules"
+  ],
 
   // An array of file extensions your modules use
   // moduleFileExtensions: [
@@ -121,11 +121,11 @@ export default {
   // restoreMocks: false,
 
   // The root directory that Jest should scan for tests and modules within
-  // rootDir: undefined,
+  // rootDir: "test",
 
   // A list of paths to directories that Jest should use to search for files in
   // roots: [
-  //   "<rootDir>"
+  //   "src/test/**/*.ts"
   // ],
 
   // Allows you to use a custom runner instead of Jest's default test runner
@@ -153,10 +153,10 @@ export default {
   // testLocationInResults: false,
 
   // The glob patterns Jest uses to detect test files
-  // testMatch: [
-  //   "**/__tests__/**/*.[jt]s?(x)",
-  //   "**/?(*.)+(spec|test).[tj]s?(x)"
-  // ],
+  testMatch: [
+    "**/test/**/*.[jt]s?(x)",
+    "**/?(*.)+(spec|test).[tj]s?(x)"
+  ],
 
   // An array of regexp pattern strings that are matched against all test paths, matched tests are skipped
   // testPathIgnorePatterns: [

src/index.ts

@@ -2,21 +2,21 @@
 import BottleneckLight from "bottleneck/light";
 import { Octokit } from "@octokit/core";
 import { OctokitOptions } from "@octokit/core/dist-types/types.d";
-import { Groups } from "./types";
+import { Groups } from "./types/bottle-neck";
 import { VERSION } from "./version";
 
 import { wrapRequest } from "./wrap-request";
 import triggersNotificationPaths from "./generated/triggers-notification-paths";
 import { routeMatcher } from "./route-matcher";
+import Bottleneck from "bottleneck";
 
 // Workaround to allow tests to directly access the triggersNotification function.
 const regex = routeMatcher(triggersNotificationPaths);
 const triggersNotification = regex.test.bind(regex);
 
 const groups: Groups = {};
 
-// @ts-expect-error
-const createGroups = function (Bottleneck, common) {
+const createGroups = function (Bottleneck: Bottleneck | any, common: Object) {
   groups.global = new Bottleneck.Group({
     id: "octokit-global",
     maxConcurrent: 10,
@@ -192,3 +192,5 @@ export function throttling(octokit: Octokit, octokitOptions: OctokitOptions) {
 }
 throttling.VERSION = VERSION;
 throttling.triggersNotification = triggersNotification;
+
+export default throttling;

src/types/bottle-neck.ts

@@ -0,0 +1,8 @@
+import Bottleneck from "bottleneck";
+
+export interface Groups {
+  global?: Bottleneck.Group;
+  write?: Bottleneck.Group;
+  search?: Bottleneck.Group;
+  notifications?: Bottleneck.Group;
+}