Skip to content

expands tests to run on osx-arm64 and linux-x64#638

Open
prathikr wants to merge 59 commits intomainfrom
prathikrao/add-macos-tests
Open

expands tests to run on osx-arm64 and linux-x64#638
prathikr wants to merge 59 commits intomainfrom
prathikrao/add-macos-tests

Conversation

@prathikr
Copy link
Copy Markdown
Contributor

@prathikr prathikr commented Apr 15, 2026

Adds mac/linux test coverage for FL Core & SDK and retires github actions macos tests. Also removes our reliance on PATs.

NOTE: The Linux JS test job is currently disabled due to intermittent SSL errors when running get_model_list. This issue is under investigation.

@prathikr prathikr changed the title expands tests to run on osx-arm64 machines expands tests to run on linux/macos machines Apr 15, 2026
@prathikr prathikr changed the title expands tests to run on linux/macos machines expands tests to run on win-arm64, osx-arm64, and linux-x64 Apr 15, 2026
Prathik Rao added 3 commits April 15, 2026 15:12
@prathikr prathikr changed the title expands tests to run on win-arm64, osx-arm64, and linux-x64 expands tests to run on osx-arm64 and linux-x64 Apr 15, 2026
@prathikr prathikr changed the base branch from prathikrao/add-linux-tests to main April 15, 2026 22:16
Copilot AI review requested due to automatic review settings April 15, 2026 23:20
Copy link
Copy Markdown
Contributor

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

Note

Copilot was unable to run its full agentic suite in this review.

Expands CI test execution across additional OS/architecture combos (notably macOS arm64 and Linux x64) by splitting SDK build and test steps into dedicated Azure Pipelines templates and wiring new test stages/jobs into the main packaging pipeline.

Changes:

  • Add new Azure Pipelines test-step templates for C#, JS, Python, and Rust SDKs and move test execution out of build templates.
  • Update packaging pipeline to run SDK tests on win-x64, linux-x64, and osx-arm64 (with JS linux-x64 currently commented out).
  • Remove legacy GitHub Actions SDK build workflows in favor of Azure DevOps orchestration.

Reviewed changes

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

Show a summary per file
File Description
sdk/js/test/detail/modelLoadManager.test.ts Increases Mocha timeout to reduce macOS CI flakiness from slow network fetches.
.pipelines/templates/test-rust-steps.yml New reusable Rust SDK test steps (extract native bits, install toolchain, run unit/integration tests).
.pipelines/templates/test-python-steps.yml New reusable Python SDK test steps (select Python arch, install wheels/deps, run pytest).
.pipelines/templates/test-js-steps.yml New reusable JS SDK test steps (install deps, place native bits, build, run tests).
.pipelines/templates/test-cs-steps.yml New reusable C# SDK test steps (NuGet config, restore/build/test).
.pipelines/templates/build-rust-steps.yml Removes Rust test execution from build template (now in test template).
.pipelines/templates/build-python-steps.yml Adjusts ORT packages on Linux; removes Python test execution from build template.
.pipelines/templates/build-js-steps.yml Removes JS test execution from build template.
.pipelines/templates/build-cs-steps.yml Removes C# test execution from build template.
.pipelines/templates/build-core-steps.yml Expands condition for restoring FLC tests to include linux-x64 and osx-x64 flavors.
.pipelines/foundry-local-packaging.yml Adds SDK test stages/jobs for standard + WinML; adds osx-arm64 pool usage; updates job display names; comments out JS linux test job.
.github/workflows/foundry-local-sdk-build.yml Removes legacy SDK build orchestration workflow.
.github/workflows/build-rust-steps.yml Removes legacy GitHub Actions Rust SDK workflow.
.github/workflows/build-python-steps.yml Removes legacy GitHub Actions Python SDK workflow.
.github/workflows/build-js-steps.yml Removes legacy GitHub Actions JS SDK workflow.
.github/workflows/build-cs-steps.yml Removes legacy GitHub Actions C# SDK workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .pipelines/templates/test-js-steps.yml
Comment thread .pipelines/templates/test-js-steps.yml
Comment thread .pipelines/templates/test-python-steps.yml
Comment thread .pipelines/foundry-local-packaging.yml
Comment thread .pipelines/foundry-local-packaging.yml
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