Skip to content

[Swift shipping] Setup MacCatalyst tests on the CI#2998

Merged
kotlarmilos merged 51 commits intodotnet:feature/swift-bindingsfrom
kotlarmilos:swift-bindings/standup-device-tests
Feb 20, 2025
Merged

[Swift shipping] Setup MacCatalyst tests on the CI#2998
kotlarmilos merged 51 commits intodotnet:feature/swift-bindingsfrom
kotlarmilos:swift-bindings/standup-device-tests

Conversation

@kotlarmilos
Copy link
Member

@kotlarmilos kotlarmilos commented Feb 12, 2025

Description

This PR introduces MacCatalyst tests using MAUI test runners and the xUnit framework. The goal is to update our infrastructure to support writing xUnit tests within MAUI projects. This will allow us to reference a NuGet package with bindings and run framework tests, such as in-app purchase.

Changes

  • Added a MAUI project for running tests using XHarness
  • Added a new test-job.yml for device tests
  • Moved the HikingApp to the samples directory

Contributes to #2850

@kotlarmilos kotlarmilos added * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) area-SwiftBindings Swift bindings for .NET labels Feb 12, 2025
@kotlarmilos kotlarmilos self-assigned this Feb 12, 2025
@kotlarmilos kotlarmilos changed the title [Swift shipping] Run device tests on the CI [Swift shipping] Setup maccatalyst tests on the CI Feb 17, 2025
@kotlarmilos kotlarmilos changed the title [Swift shipping] Setup maccatalyst tests on the CI [Swift shipping] Setup MacCatalyst tests on the CI Feb 17, 2025
@kotlarmilos kotlarmilos removed the * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) label Feb 17, 2025
@kotlarmilos kotlarmilos marked this pull request as ready for review February 17, 2025 13:41
parameters:
osGroup: OSX
platform: maccatalyst
archType: x64
Copy link

Choose a reason for hiding this comment

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

Why x64? Is this a HW availability limitation?

Copy link
Member Author

@kotlarmilos kotlarmilos Feb 18, 2025

Choose a reason for hiding this comment

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

Yes, this is the default image. There is a macOS-arm64 image and we need to figure out how to use it.

https://learn.microsoft.com/en-us/azure/devops/pipelines/agents/hosted?view=azure-devops&tabs=yaml

Copy link
Member Author

@kotlarmilos kotlarmilos Feb 18, 2025

Choose a reason for hiding this comment

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

The hostArchitecture: Arm64 was missing.

Copy link
Member Author

Choose a reason for hiding this comment

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

@akoeplinger Do you know if we can run macOS-arm64 containers?

Copy link
Member

Choose a reason for hiding this comment

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

not if you run tests directly on the AzDO build machine, the arm64 macOS pool is only available in internal builds, not in public.

Copy link
Member Author

@kotlarmilos kotlarmilos Feb 19, 2025

Choose a reason for hiding this comment

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

Is the arm64 job redundant? Should we use Helix instead for arm64 jobs?

Copy link
Member

Choose a reason for hiding this comment

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

which arm64 job? the one in this file only builds for arm64 which does work on x64.

if you can send to Helix then yeah you could send to an arm64 queue

Copy link
Member Author

@kotlarmilos kotlarmilos Feb 19, 2025

Choose a reason for hiding this comment

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

which arm64 job? the one in this file only builds for arm64 which does work on x64.

Yes, that was my question. So even it builds for arm64, it runs on x64.

Copy link
Member

Choose a reason for hiding this comment

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

Yes

@kotlarmilos kotlarmilos merged commit ac33925 into dotnet:feature/swift-bindings Feb 20, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-SwiftBindings Swift bindings for .NET

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants