Fix TS errors and add type check to CI

Author: IvanGoncharov

.github/workflows/ci.yml

@@ -22,6 +22,9 @@ jobs:
       - name: Install Dependencies
         run: npm ci
 
+      - name: Check Types
+        run: npm run check
+
       - name: Lint Prettier
         run: npm run prettier:check
 

package.json

@@ -8,7 +8,8 @@
     "node": ">= 8.x"
   },
   "scripts": {
-    "test": "npm run prettier:check",
+    "test": "npm run check && npm run prettier:check",
+    "check": "tsc --noEmit",
     "start": "nodemon src/index.ts",
     "debug": "ts-node --inspect --compilerOptions '{\"inlineSources\":true}' src/index.ts",
     "start:editor": "webpack-dev-server --config webpack.config.js",

src/fake.ts

@@ -184,12 +184,14 @@ const fakeFunctions = {
 
   // System section
   // Skipped: faker.system.fileName
-  // TODO: Add ext and type
+  // TODO: Add ext
+  // @ts-expect-error Remove once fix is merged https://github.com/DefinitelyTyped/DefinitelyTyped/pull/52229
   filename: () => faker.system.commonFileName(),
   mimeType: () => faker.system.mimeType(),
   // Skipped: faker.system.fileType
   // Skipped: faker.system.commonFileType
   // Skipped: faker.system.commonFileExt
+  // @ts-expect-error Remove once fix is merged https://github.com/DefinitelyTyped/DefinitelyTyped/pull/52229
   fileExtension: () => faker.system.fileExt(),
   semver: () => faker.system.semver(),
 };

src/fake_schema.ts

@@ -143,7 +143,7 @@ export const fakeFieldResolver: GraphQLFieldResolver<unknown, unknown> = async (
     return args && (() => getRandomItem(args.values));
   }
 
-  function getListLength(object): ListLengthArgs {
+  function getListLength(object) {
     const listLength = schema.getDirective('listLength');
     const args = getDirectiveArgs(listLength, object) as ListLengthArgs;
     return args ? getRandomInt(args.min, args.max) : getRandomInt(2, 4);