Skip to content

Conversation

@premun
Copy link
Member

@premun premun commented May 12, 2021

Enables to specify <CustomCommands> metadata on the <XHarnessAppBundleToTest> which will then inject those commands inside of the wrapper scripts we have that set and clean up the iOS simulator/device safely.

Android support will come in a follow-up PR.

premun added 30 commits May 7, 2021 14:12
# Conflicts:
#	src/Common/Microsoft.Arcade.Common/IZipArchiveManager.cs
#	src/Common/Microsoft.Arcade.Common/ZipArchiveManager.cs
#	src/Microsoft.DotNet.Helix/Sdk.Tests/Microsoft.DotNet.Helix.Sdk.Tests/CreateXHarnessAppleWorkItemsTests.cs
#	src/Microsoft.DotNet.Helix/Sdk.Tests/Microsoft.DotNet.Helix.Sdk.Tests/ProvisioningProfileProviderTests.cs
#	src/Microsoft.DotNet.Helix/Sdk/CreateXHarnessAndroidWorkItems.cs
#	src/Microsoft.DotNet.Helix/Sdk/CreateXHarnessAppleWorkItems.cs
#	src/Microsoft.DotNet.Helix/Sdk/ProvisioningProfileProvider.cs
#	src/Microsoft.DotNet.Helix/Sdk/XharnessTaskBase.cs
@premun premun requested review from MattGal and greenEkatherine May 12, 2021 09:23
@premun
Copy link
Member Author

premun commented May 12, 2021

@imhameed this PR will allow you to inject all of the commands you need.

You won't even have to do dotnet exec $XHARNESS_CLI_PATH but just xharness which is an alias created for you.

I have 1 question:
Are you ok with just setting the custom commands and specifying the app bundle which will be the only payload or you need to send some other custom payload with the job as well?

@imhameed
Copy link

I have 1 question:
Are you ok with just setting the custom commands and specifying the app bundle which will be the only payload or you need to send some other custom payload with the job as well?

As far as I know, I only need to send the app bundle to the simulator or device.

@premun premun changed the title Add support for custom XHarness commands in Helix SDK Add support for custom Apple XHarness commands in Helix SDK May 12, 2021
@premun premun merged commit 6b97586 into dotnet:main May 13, 2021
@premun premun deleted the prvysoky/xharness-workitems branch May 13, 2021 08:42
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