Build: Restore inclusion of browser run in coverage report

qunitjs · Jun 16, 2024 · be73c94 · be73c94
1 parent fccaa4f
commit be73c94
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.github/workflows/coverage.yaml b/.github/workflows/coverage.yaml
@@ -30,6 +30,7 @@ jobs:
         run: npm run coverage
 
       - name: Upload to coveralls.io
+        if: ${{ github.ref_name == 'main' }} # skip upload on non-main branches
         env:
           COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |

diff --git a/build/coverage-bridge.js b/build/coverage-bridge.js
@@ -12,7 +12,7 @@
     window.__grunt_contrib_qunit__.apply(window, arguments);
   }
 
-  QUnit.done(function () {
+  QUnit.on('runEnd', function () {
     // send coverage data if available
     if (window.__coverage__) {
       sendMessage('qunit.coverage', window.location.pathname, window.__coverage__);