add npm-test workflow (#1)

* Update npm-test.yml

* fix: test cases

* fix: build system

* build: update actions

Author: LastLeaf

.github/workflows/npm-test.yml

@@ -9,12 +9,21 @@ on:
 
 jobs:
   test:
-    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        include:
+          - os: windows-latest
+          - os: ubuntu-latest
+          - os: macos-latest
+    runs-on: ${{ matrix.os }}
     steps:
     - uses: actions/checkout@v4
     - uses: actions/setup-node@v4
       with:
         node-version: 20.x
         cache: 'npm'
     - run: npm ci
+    - run: xvfb-run -a npm test
+      if: runner.os == 'Linux'
     - run: npm test
+      if: runner.os != 'Linux'

.vscode/tasks.json

@@ -3,23 +3,6 @@
 {
 	"version": "2.0.0",
 	"tasks": [
-		{
-			"type": "shell",
-			"label": "cp-wasm",
-			"command": "cp node_modules/float-pigment-css/float_pigment_css_bg.wasm dist/",
-			"problemMatcher": "$ts-webpack-watch",
-			"isBackground": true,
-			"presentation": {
-				"reveal": "never",
-				"group": "watchers"
-			},
-			"group": {
-				"kind": "build"
-			},
-			"options": {
-				"cwd": "${workspaceFolder}"
-			}
-		},
 		{
 			"type": "npm",
 			"script": "watch",
@@ -55,16 +38,14 @@
 			"label": "tasks: watch-tests",
 			"dependsOn": [
 				"npm: watch",
-				"cp-wasm",
 				"npm: watch-tests"
 			],
 			"problemMatcher": []
 		},
 		{
 			"label": "tasks: watch",
 			"dependsOn": [
-				"npm: watch",
-				"cp-wasm"
+				"npm: watch"
 			],
 			"problemMatcher": []
 		}

package-lock.json

@@ -87,6 +87,7 @@
     "@vscode/test-cli": "^0.0.10",
     "@vscode/test-electron": "^2.4.1",
     "assert": "^2.1.0",
+    "copy-webpack-plugin": "^12.0.2",
     "eslint": "^7.17.0",
     "eslint-config-airbnb-base": "^14.2.1",
     "eslint-config-prettier": "^8.6.0",

package.json

@@ -4,26 +4,57 @@ import assert from 'assert'
 
 const EXTENSION_DIR = path.resolve(__dirname, '..', '..')
 const TEST_FIXTURE_DIR = path.resolve(EXTENSION_DIR, 'test-fixture')
+const DIAGNOSTIC_SOURCE = 'float-pigment-css-analyzer'
 
 const getUri = (rel: string) => {
   const absPath = path.resolve(TEST_FIXTURE_DIR, rel)
   const uri = vscode.Uri.file(absPath)
   return uri
 }
 
-const sleep = (ms: number): Promise<void> =>
+// const diagChangeCallbacks: ((uris: readonly vscode.Uri[]) => void)[] = []
+// vscode.languages.onDidChangeDiagnostics((e) => {
+//   // eslint-disable-next-line @typescript-eslint/no-floating-promises, promise/catch-or-return
+//   vscode.window.showInformationMessage('!!!')
+//   diagChangeCallbacks.forEach((f) => f(e.uris))
+// })
+const waitDiagnosticsUpdate = (uri: vscode.Uri, source: string): Promise<void> =>
   new Promise((resolve) => {
-    setTimeout(resolve, ms)
+    const diag = vscode.languages.getDiagnostics(uri).find((diag) => diag.source === source)
+    if (diag) {
+      resolve()
+      return
+    }
+    // const cb = (uris: readonly vscode.Uri[]) => {
+    //   if (uris.includes(uri)) {
+    //     const diag = vscode.languages.getDiagnostics(uri).find((diag) => diag.source === source)
+    //     if (diag) {
+    //       const index = diagChangeCallbacks.indexOf(cb)
+    //       diagChangeCallbacks.splice(index, 1)
+    //       resolve()
+    //     }
+    //   }
+    // }
+    // diagChangeCallbacks.push(cb)
+    const cb = () => {
+      const diag = vscode.languages.getDiagnostics(uri).find((diag) => diag.source === source)
+      if (diag) {
+        resolve()
+        return
+      }
+      setTimeout(cb, 100)
+    }
+    setTimeout(cb, 100)
   })
 
 suite('common', () => {
   test('diagnostics for CSS', async () => {
     const uri = getUri('components/index.css')
     await vscode.window.showTextDocument(uri)
-    await sleep(1000)
+    await waitDiagnosticsUpdate(uri, DIAGNOSTIC_SOURCE)
     const diagList = vscode.languages
       .getDiagnostics(uri)
-      .filter((diag) => diag.source === 'float-pigment-css-analyzer')
+      .filter((diag) => diag.source === DIAGNOSTIC_SOURCE)
     assert.equal(diagList.length, 1)
   })
 })