fix: fix issue with tests potentially being double-started, or starte…

…d before being ready (#2)

Also fixes id-token write permission in release workflow, required for npm provenance.

build: add back id-token write permission to release workflow

build: set NODE_AUTH_TOKEN to secrets.GITHUB_TOKEN in Publish step of release

.github/workflows/release.yml

@@ -14,6 +14,7 @@ jobs:
       contents: write
       pull-requests: write
       packages: write
+      id-token: write
     steps:
       - uses: googleapis/release-please-action@v4
         id: release
@@ -41,6 +42,8 @@ jobs:
       - name: Publish
         if: ${{ steps.release.outputs.release_created }}
         run: npm publish --provenance --access public
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       # Suggestion from the release-please-action repo README
       - name: Tag major and minor versions

examples/wdio-cjs/wdio.conf.js

@@ -14,7 +14,7 @@ exports.config = {
   framework: 'mocha',
   reporters: ['spec'],
 
-  services: [['qunit', { autostartDelay: 1000 }]],
+  services: [['qunit', { autostartDelay: 5000 }]],
   mochaOpts: {
     ui: 'bdd',
     timeout: 60000

src/index.ts

@@ -108,7 +108,8 @@ async function QunitFinishedEventInBrowserContext(
     });
     const automaticRunStartPromise = new Promise<void>((resolve) => {
       window.setTimeout(() => {
-        if (!started) {
+        if (!started && !window.QUnit.config.started) {
+          started = true;
           console.debug('QUnit started automatically by service'); // eslint-disable-line no-console
           window.QUnit.start();
           resolve();