Merge pull request #1346 from github/dependabot/npm_and_yarn/eslint-plugin-jest-28.9.0

Bump eslint-plugin-jest from 27.9.0 to 28.9.0

Author: landongrindheim

Diff for: .eslintrc.json

@@ -1,6 +1,14 @@
 {
-  "plugins": ["github", "jest", "@typescript-eslint"],
-  "extends": ["plugin:github/recommended", "plugin:github/typescript", "prettier"],
+  "plugins": [
+    "github",
+    "jest",
+    "@typescript-eslint"
+  ],
+  "extends": [
+    "plugin:github/recommended",
+    "plugin:github/typescript",
+    "prettier"
+  ],
   "parser": "@typescript-eslint/parser",
   "parserOptions": {
     "ecmaVersion": 9,
@@ -18,14 +26,23 @@
     "no-unused-vars": "off",
     "camelcase": "off",
     "@typescript-eslint/no-unused-vars": "error",
-    "@typescript-eslint/explicit-member-accessibility": ["error", {"accessibility": "no-public"}],
+    "@typescript-eslint/explicit-member-accessibility": [
+      "error",
+      {
+        "accessibility": "no-public"
+      }
+    ],
     "@typescript-eslint/no-require-imports": "error",
     "@typescript-eslint/array-type": "error",
     "@typescript-eslint/await-thenable": "error",
     "@typescript-eslint/ban-ts-comment": "error",
     "@typescript-eslint/consistent-type-assertions": "error",
-    "@typescript-eslint/explicit-function-return-type": ["error", {"allowExpressions": true}],
-    "@typescript-eslint/func-call-spacing": ["error", "never"],
+    "@typescript-eslint/explicit-function-return-type": [
+      "error",
+      {
+        "allowExpressions": true
+      }
+    ],
     "@typescript-eslint/no-array-constructor": "error",
     "@typescript-eslint/no-empty-interface": "error",
     "@typescript-eslint/no-explicit-any": "off",
@@ -46,7 +63,6 @@
     "@typescript-eslint/promise-function-async": "error",
     "@typescript-eslint/require-array-sort-compare": "error",
     "@typescript-eslint/restrict-plus-operands": "error",
-    "@typescript-eslint/type-annotation-spacing": "error",
     "@typescript-eslint/unbound-method": "error",
     "@typescript-eslint/no-shadow": "error",
     "no-shadow": "off",
@@ -55,7 +71,9 @@
   },
   "overrides": [
     {
-      "files": ["*.ts"],
+      "files": [
+        "*.ts"
+      ],
       "rules": {
         "no-undef": "off"
       }
@@ -66,4 +84,4 @@
     "es6": true,
     "jest/globals": true
   }
-}
+}

Diff for: __tests__/helpers.ts

@@ -16,7 +16,7 @@ export const removeDanglingUpdaterContainers = async (): Promise<void> => {
       try {
         await docker.getContainer(container.Id).remove({v: true, force: true})
       } catch (e) {
-        // ignore
+        console.log(e) // eslint-disable-line no-console
       }
     }
   }

Diff for: __tests__/updater-integration.test.ts

@@ -62,7 +62,7 @@ integration('Updater', () => {
   })
 
   afterEach(async () => {
-    server && server() // teardown server process
+    server && server() // eslint-disable-line @typescript-eslint/no-unused-expressions
     await removeDanglingUpdaterContainers()
     fs.rmSync(workingDirectory, {recursive: true})
   })