Skip to content

[build] add new test job for running emu smoke tests#502

Merged
jaygmsft merged 18 commits into
mainfrom
user/v-davidz/add_emu_smoke
Jul 2, 2026
Merged

[build] add new test job for running emu smoke tests#502
jaygmsft merged 18 commits into
mainfrom
user/v-davidz/add_emu_smoke

Conversation

@zimmy87

@zimmy87 zimmy87 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

This change adds 2 new test jobs: test_ubuntu_smoke & test_windows_smoke

Copilot AI review requested due to automatic review settings June 30, 2026 22:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 extends the xtask nextest runner to support narrowing runs to a specific test target and name filter(s), and adds two GitHub Actions jobs intended to run a small “emu smoke” slice on Ubuntu and Windows.

Changes:

  • Add --test and --filter passthrough flags to cargo xtask nextest and cargo xtask precheck.
  • Add test_ubuntu_smoke and test_windows_smoke workflow jobs to run emu smoke tests for azihsm_ddi_mbor_types.
  • Add a dedicated nextest profile (ci-mbor-smoke) for JUnit output.

Reviewed changes

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

Show a summary per file
File Description
xtask/src/precheck.rs Plumbs new --test/--filter options into precheck’s nextest invocations; adds an unconditional emu smoke run to the default nextest path.
xtask/src/nextest.rs Adds --test and positional test-name filter forwarding to cargo nextest run.
xtask/src/integration_tests.rs Updates Nextest { .. } construction to include new fields.
.github/workflows/rust.yml Adds Ubuntu/Windows smoke-test jobs that invoke precheck/nextest with --test and --filter.
.config/nextest.toml Adds a ci-mbor-smoke profile section for JUnit output.

Comment thread .github/workflows/rust.yml Outdated
Comment thread .github/workflows/rust.yml Outdated
Comment thread xtask/src/precheck.rs Outdated
Copilot AI review requested due to automatic review settings July 1, 2026 00:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 1, 2026 18:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Comment thread xtask/src/precheck.rs
Comment thread .github/workflows/rust.yml
Comment thread .github/workflows/rust.yml Outdated
Comment thread .github/workflows/rust.yml Outdated
Copilot AI review requested due to automatic review settings July 1, 2026 22:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread xtask/src/precheck.rs Outdated
Comment thread xtask/src/nextest_report.rs
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 1, 2026 22:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

@zimmy87 zimmy87 marked this pull request as ready for review July 1, 2026 22:57
Copilot AI review requested due to automatic review settings July 2, 2026 18:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread xtask/src/nextest.rs
Comment thread xtask/src/coverage.rs
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 2, 2026 19:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Comment thread xtask/src/nextest_report.rs
zimmy87 and others added 2 commits July 2, 2026 13:21
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 2, 2026 20:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

@jaygmsft jaygmsft enabled auto-merge July 2, 2026 22:03
@jaygmsft jaygmsft added this pull request to the merge queue Jul 2, 2026
Merged via the queue into main with commit 49e35d6 Jul 2, 2026
25 checks passed
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