ci: update smoke tests

Author: mato533

.github/workflows/ci-lint.yml

@@ -1,4 +1,4 @@
-name: Lint
+name: Static code analysis
 
 on:
   workflow_call:
@@ -28,13 +28,6 @@ jobs:
         with:
           node-version: ${{ matrix.node-version }}
 
-      - name: ⬇️ Download Build Archive
-        uses: ./.github/workflows/actions/download-archive
-        with:
-          name: vscode-webdriverio
-          path: .
-          filename: vscode-webdriverio-build.zip
-
       - name: 📃 Run the lint
         run: pnpm run style:fix
         shell: bash

.github/workflows/ci.yml

@@ -21,33 +21,32 @@ jobs:
       os: 'ubuntu-latest'
 
   lint:
-    name: Lint
-    needs: [build]
+    name: Static code analysis
     uses: ./.github/workflows/ci-lint.yml
 
   typecheck:
     name: Typecheck
-    needs: [build]
+    needs: [lint, build]
     uses: ./.github/workflows/ci-typecheck.yml
 
   unit:
     name: Unit
-    needs: [build]
+    needs: [lint, build]
     uses: ./.github/workflows/ci-unit.yml
 
   e2e:
     name: E2E
-    needs: [build]
+    needs: [lint, build]
     uses: ./.github/workflows/ci-e2e.yml
 
   compatibility:
     name: Compatibility
-    needs: [build]
+    needs: [lint, build]
     uses: ./.github/workflows/ci-e2e.yml
     with:
       compatibility-mode: 'yes'
 
   smoke:
     name: Smoke
-    needs: [build, e2e]
+    needs: [lint, build, e2e, compatibility]
     uses: ./.github/workflows/ci-smoke.yml

e2e/tests/updateConfig.spec.ts

@@ -85,20 +85,23 @@ describe('VS Code Extension Testing (Update config)', function () {
 
         // Emulate the changing configuration
         shell.cp('-f', afterConfig, beforeConfig)
-        await new Promise((resolve) => setTimeout(resolve, 1000))
         await browser.waitUntil(
             async () => {
-                if (!(await items[0].isExpanded())) {
-                    await items[0].expand()
-                }
-
-                const children = await items[0].getChildren()
-                const target = children[0]
-                if (!target) {
+                try {
+                    if (!(await items[0].isExpanded())) {
+                        await items[0].expand()
+                    }
+
+                    const children = await items[0].getChildren()
+                    const target = children[0]
+                    if (!target) {
+                        return false
+                    }
+                    const regex = new RegExp('after.test.ts')
+                    return regex.test(await target.getLabel())
+                } catch {
                     return false
                 }
-                const regex = new RegExp('after.test.ts')
-                return regex.test(await target.getLabel())
             },
             { timeoutMsg: 'The label "after.test.ts" is not found.' }
         )

e2e/tests/updateErrorSpec.spec.ts

@@ -89,20 +89,23 @@ describe('VS Code Extension Testing (Update config)', function () {
 
         // Emulate the changing configuration
         shell.cp('-f', spec.after, spec.before)
-        await new Promise((resolve) => setTimeout(resolve, 1000))
         await browser.waitUntil(
             async () => {
-                if (!(await items[0].isExpanded())) {
-                    await items[0].expand()
-                }
-
-                const children = await items[0].getChildren()
-                const target = children[0]
-                if (!target) {
+                try {
+                    if (!(await items[0].isExpanded())) {
+                        await items[0].expand()
+                    }
+
+                    const children = await items[0].getChildren()
+                    const target = children[0]
+                    if (!target) {
+                        return false
+                    }
+                    const regex = new RegExp('before.spec.ts')
+                    return regex.test(await target.getLabel())
+                } catch {
                     return false
                 }
-                const regex = new RegExp('before.spec.ts')
-                return regex.test(await target.getLabel())
             },
             {
                 timeoutMsg: 'The label "before.spec.ts" is not found.',

e2e/tests/updateSpec.spec.ts

@@ -89,20 +89,23 @@ describe('VS Code Extension Testing (Update config)', function () {
 
         // Emulate the changing configuration
         shell.cp('-f', spec.after, spec.before)
-        await new Promise((resolve) => setTimeout(resolve, 1000))
         await browser.waitUntil(
             async () => {
-                if (!(await items[0].isExpanded())) {
-                    await items[0].expand()
-                }
-
-                const children = await items[0].getChildren()
-                const target = children[0]
-                if (!target) {
+                try {
+                    if (!(await items[0].isExpanded())) {
+                        await items[0].expand()
+                    }
+
+                    const children = await items[0].getChildren()
+                    const target = children[0]
+                    if (!target) {
+                        return false
+                    }
+                    const regex = new RegExp('before.spec.ts')
+                    return regex.test(await target.getLabel())
+                } catch {
                     return false
                 }
-                const regex = new RegExp('before.spec.ts')
-                return regex.test(await target.getLabel())
             },
             {
                 timeoutMsg: 'The label "before.spec.ts" is not found.',