Skip to content

chore(CI): push mocha test results as semaphore artifacts for easier triage#3355

Merged
Dave Shoup (shouples) merged 3 commits intomainfrom
djs/mocha-test-triage
Apr 1, 2026
Merged

chore(CI): push mocha test results as semaphore artifacts for easier triage#3355
Dave Shoup (shouples) merged 3 commits intomainfrom
djs/mocha-test-triage

Conversation

@shouples
Copy link
Copy Markdown
Contributor

@shouples Dave Shoup (shouples) commented Mar 29, 2026

Up until now, we've had to copy output from Semaphore jobs and hope we have enough information; this PR pushes the XML as well as extension+sidecar logs as artifacts, similar to the E2E test behavior. (However, unlike E2E tests, we only run a single VS Code window/workspace for the entire collection of tests.)

Sample run artifacts:
image
image

Copilot AI review requested due to automatic review settings March 29, 2026 18:34
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves CI triage in Semaphore by capturing Mocha JUnit XML output together with extension + sidecar logs as build artifacts, similar to existing E2E artifact behavior.

Changes:

  • Copy extension and sidecar log files to the workspace root during Mocha test runs (TEST-extension.log, TEST-sidecar.log).
  • Bundle Mocha results + logs into a single zip artifact and upload it from Semaphore (Linux + Windows).
  • Ignore the generated logs/zips via .gitignore.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/testing.ts Copies extension/sidecar logs next to TEST-result.xml to support CI artifact bundling.
.semaphore/semaphore.yml Creates/uploads a “mocha triage” zip artifact and standardizes artifact destination naming.
.gitignore Prevents generated test logs and triage zips from being committed.

Comment thread src/testing.ts
Comment thread .semaphore/semaphore.yml Outdated
Comment thread .semaphore/semaphore.yml Outdated
@sonarqube-confluent
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@shouples Dave Shoup (shouples) marked this pull request as ready for review March 30, 2026 14:55
@shouples Dave Shoup (shouples) requested a review from a team as a code owner March 30, 2026 14:55
@shouples Dave Shoup (shouples) merged commit 2082eab into main Apr 1, 2026
14 checks passed
@shouples Dave Shoup (shouples) deleted the djs/mocha-test-triage branch April 1, 2026 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants