Skip to content

FIX Publish Azure partner integration test results - #2901

Merged
Roman Lutz (romanlutz) merged 3 commits into
microsoft:mainfrom
romanlutz:romanlutz-daily-audit-2026-09-25
Oct 3, 2026
Merged

Roman Lutz (romanlutz) merged 3 commits into
microsoft:mainfrom
romanlutz:romanlutz-daily-audit-2026-09-25

Conversation

@romanlutz

@romanlutz Roman Lutz (romanlutz) commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Description

Partner integration tests wrote junit/partner-test-results.xml, which the shared Azure publisher's junit/test-results*.xml pattern does not match. Build 16850 reported 101 passed and 3 skipped, then warned that no result files matched.

Keep filename ownership in the Makefile: give partner tests a target-specific JUNIT_XML=junit/test-results-partner.xml default, and make both partner and E2E recipes use $(JUNIT_XML). Command-line overrides still win, so batches can write distinct reports rather than overwrite one another. The Azure pipeline remains unchanged; no changes to test selection, assertions, or retries.

Related #2777 and #2862 address GitHub Actions reporting and unit/coverage targets, rather than these partner/E2E recipes.

Tests and Documentation

Regression tests run real GNU Make dry-runs for integration, E2E, and partner defaults plus two batch filename overrides, then check publication matching and unrelated-file exclusion. Make-dependent cases explicitly skip where Make is unavailable; validation here used GNU Make 4.4.1 on PATH and ran all 17 cases without skips.

  • uv run --frozen --no-sync pytest -q tests\unit\build_scripts\test_azure_test_results.py: 17 passed.
  • uv run --frozen --no-sync ruff check tests\unit\build_scripts\test_azure_test_results.py: passed.
  • uv run --frozen --no-sync ruff format --check tests\unit\build_scripts\test_azure_test_results.py: passed.
  • uv run --frozen --no-sync ty check tests\unit\build_scripts\test_azure_test_results.py: passed.

Full suite and live Azure pipelines were not rerun for this revision. Hosted ingestion remains to be verified after landing. Documentation: N/A; no additional references to the old partner filename were found.

JupyText: N/A; no notebooks changed.

Include the partner JUnit filename without broadening existing result
patterns. Verify the actual Makefile output and preserved filename scope.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 21bf73f6-4b26-4f5f-bb22-01fa72e49186
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@hannahwestra25 hannahwestra25 self-assigned this Oct 1, 2026
Comment thread .azuredevops/test-job-template.yml Outdated

@hannahwestra25 hannahwestra25 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.

non blocking comment :)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 21bf73f6-4b26-4f5f-bb22-01fa72e49186
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Oct 2, 2026
Merged via the queue into microsoft:main with commit b605e76 Oct 3, 2026
48 checks passed
@romanlutz
Roman Lutz (romanlutz) deleted the romanlutz-daily-audit-2026-09-25 branch October 3, 2026 00:12
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.

2 participants