Fix/npm test (#139)

* Remove "npm run useChrome" directive from the test script.

* Add typescript typechecking to the test script

* Add missing run keyword

.github/workflows/node.js.yml

@@ -6,23 +6,26 @@ name: Test and Build
 on:
   push:
     branches:
-     - main
-     - develop
+      - main
+      - develop
   pull_request:
     branches:
-     - main
-     - develop
+      - main
+      - develop
 jobs:
   build:
-
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js 16
-      uses: actions/setup-node@v2
-      with:
-        node-version: '16.x'
-        cache: 'npm'
-    - run: npm ci
-    - run: npm run test
+      - uses: actions/checkout@v2
+      - name: Use Node.js 16
+        uses: actions/setup-node@v2
+        with:
+          node-version: "16.x"
+          cache: "npm"
+      # Install dependencies
+      - run: npm ci
+      # Use chrome manifest
+      - run: npm run useChrome
+      # Run tests
+      - run: npm run test

package.json

@@ -5,7 +5,8 @@
   "main": "index.js",
   "scripts": {
     "lint": "npx eslint --ext .js,.ts,.vue --fix .",
-    "test": "npm run lint && npm run useChrome && npm run build",
+    "ts-check": "npx tsc --noEmit",
+    "test": "npm run ts-check && npm run lint && npm run build",
     "dev": "snowpack build --watch",
     "build": "snowpack build",
     "useChrome": "cp src/manifest.chrome.json src/manifest.json",

src/contentScripts/other/hisqis/newTable.ts

@@ -109,6 +109,10 @@ import { DataTable } from 'simple-datatables'
 
   oldTable.parentNode?.insertBefore(newTable, oldTable)
 
+  // If "TS2578: Unused '@ts-expect-error' directive" appears then they probably fixxed their typescript
+  // definition and the following @ts-expect-error directive should be removed.
+  // See: https://fiduswriter.github.io/simple-datatables/documentation/Getting-Started#browser
+  // @ts-expect-error
   // eslint-disable-next-line no-unused-vars
   const _dataTable = new DataTable(newTable, {
     sortable: true,