Stop supporting multiple targets in XHarness Helix SDK

src/Microsoft.DotNet.Helix/Sdk.Tests/Microsoft.DotNet.Helix.Sdk.Tests/CreateXHarnessAppleWorkItemsTests.cs

@@ -191,7 +191,7 @@ private ITaskItem CreateAppBundle(
         {
             var mockBundle = new Mock<ITaskItem>();
             mockBundle.SetupGet(x => x.ItemSpec).Returns(path);
-            mockBundle.Setup(x => x.GetMetadata(CreateXHarnessAppleWorkItems.MetadataNames.Targets)).Returns(targets);
+            mockBundle.Setup(x => x.GetMetadata(CreateXHarnessAppleWorkItems.MetadataNames.Target)).Returns(targets);
             mockBundle.Setup(x => x.GetMetadata(CreateXHarnessAppleWorkItems.MetadataNames.IncludesTestRunner)).Returns(includesTestRunner.ToString());
 
             if (workItemTimeout != null)

src/Microsoft.DotNet.Helix/Sdk.Tests/Microsoft.DotNet.Helix.Sdk.Tests/ProvisioningProfileProviderTests.cs

@@ -133,7 +133,7 @@ private static ITaskItem CreateAppBundle(string path, string targets)
         {
             var mockBundle = new Mock<ITaskItem>();
             mockBundle.SetupGet(x => x.ItemSpec).Returns(path);
-            mockBundle.Setup(x => x.GetMetadata(CreateXHarnessAppleWorkItems.MetadataNames.Targets)).Returns(targets);
+            mockBundle.Setup(x => x.GetMetadata(CreateXHarnessAppleWorkItems.MetadataNames.Target)).Returns(targets);
             return mockBundle.Object;
         }
     }

src/Microsoft.DotNet.Helix/Sdk/CreateXHarnessAppleWorkItems.cs

@@ -20,7 +20,7 @@ public class CreateXHarnessAppleWorkItems : XHarnessTaskBase
 
         public static class MetadataNames
         {
-            public const string Targets = "Targets";
+            public const string Target = "Target";
             public const string LaunchTimeout = "LaunchTimeout";
             public const string IncludesTestRunner = "IncludesTestRunner";
             public const string ResetSimulator = "ResetSimulator";
@@ -102,14 +102,14 @@ private async Task<ITaskItem> PrepareWorkItem(
             var (testTimeout, workItemTimeout, expectedExitCode, customCommands) = ParseMetadata(appBundleItem);
 
             // Validation of any metadata specific to iOS stuff goes here
-            if (!appBundleItem.TryGetMetadata(MetadataNames.Targets, out string targets))
+            if (!appBundleItem.TryGetMetadata(MetadataNames.Target, out string target))
             {
-                Log.LogError($"'{MetadataNames.Targets}' metadata must be specified - " +
+                Log.LogError($"'{MetadataNames.Target}' metadata must be specified - " +
                     "expecting list of target device/simulator platforms to execute tests on (e.g. ios-simulator-64)");
                 return null;
             }
 
-            targets = targets.ToLowerInvariant();
+            target = target.ToLowerInvariant();
 
             // Optional timeout for the how long it takes for the app to be installed, booted and tests start executing
             TimeSpan launchTimeout = s_defaultLaunchTimeout;
@@ -151,7 +151,7 @@ private async Task<ITaskItem> PrepareWorkItem(
                 customCommands = $"xharness apple {(includesTestRunner ? "test" : "run")} " +
                     "--app \"$app\" " +
                     "--output-directory \"$output_directory\" " +
-                    "--targets \"$targets\" " +
+                    "--target \"$target\" " +
                     "--timeout \"$timeout\" " +
                     (includesTestRunner
                         ? $"--launch-timeout \"$launch_timeout\" "
@@ -163,7 +163,7 @@ private async Task<ITaskItem> PrepareWorkItem(
             }
 
             string appName = fileSystem.GetFileName(appBundleItem.ItemSpec);
-            string helixCommand = GetHelixCommand(appName, targets, testTimeout, launchTimeout, includesTestRunner, expectedExitCode, resetSimulator);
+            string helixCommand = GetHelixCommand(appName, target, testTimeout, launchTimeout, includesTestRunner, expectedExitCode, resetSimulator);
             string payloadArchivePath = await CreateZipArchiveOfFolder(zipArchiveManager, fileSystem, appFolderPath, customCommands);
 
             Log.LogMessage($"Creating work item with properties Identity: {workItemName}, Payload: {appFolderPath}, Command: {helixCommand}");
@@ -179,7 +179,7 @@ private async Task<ITaskItem> PrepareWorkItem(
 
         private string GetHelixCommand(
             string appName,
-            string targets,
+            string target,
             TimeSpan testTimeout,
             TimeSpan launchTimeout,
             bool includesTestRunner,
@@ -188,7 +188,7 @@ private string GetHelixCommand(
             =>
             $"chmod +x {EntryPointScript} && ./{EntryPointScript} " +
             $"--app \"{appName}\" " +
-            $"--targets \"{targets}\" " +
+            $"--target \"{target}\" " +
             $"--timeout \"{testTimeout}\" " +
             $"--launch-timeout \"{launchTimeout}\" " +
             (includesTestRunner ? "--includes-test-runner " : string.Empty) +

src/Microsoft.DotNet.Helix/Sdk/ProvisioningProfileProvider.cs

@@ -62,7 +62,7 @@ public void AddProfilesToBundles(ITaskItem[] appBundles)
 
             foreach (var appBundle in appBundles)
             {
-                if (!appBundle.TryGetMetadata(CreateXHarnessAppleWorkItems.MetadataNames.Targets, out string bundleTargets))
+                if (!appBundle.TryGetMetadata(CreateXHarnessAppleWorkItems.MetadataNames.Target, out string bundleTargets))
                 {
                     _log.LogError("'Targets' metadata must be specified - " +
                         "expecting list of target device/simulator platforms to execute tests on (e.g. ios-simulator-64)");

src/Microsoft.DotNet.Helix/Sdk/tools/xharness-runner/Readme.md

@@ -59,13 +59,13 @@ To execute .app bundles, declare one or more `XHarnessAppBundleToTest` items:
 <ItemGroup>
   <!-- Find all directories named *.app -->
   <XHarnessAppBundleToTest Include="$([System.IO.Directory]::GetDirectories('$(TestArchiveTestsRoot)', '*.app', System.IO.SearchOption.AllDirectories))">
-    <Targets>ios-simulator-64_13.5</Targets>
+    <Target>ios-simulator-64_13.5</Targets>
   </XHarnessAppBundleToTest>
 </ItemGroup>
 ```
 
-The `<Targets>` metadata is a required configuration that tells XHarness which kind of device/Simulator to target.
-Use the XHarness CLI help command to find more (see the `--targets` option).
+The `<Target>` metadata is a required configuration that tells XHarness which kind of device/Simulator to target.
+Use the XHarness CLI help command to find more (see the `--target` option).
 
 You can also specify some additional metadata that will help you configure the run better:
 
@@ -163,17 +163,17 @@ Example:
 ```xml
 <ItemGroup>
   <XHarnessAppBundleToTest Include="path\to\Some.iOS.app">
-    <Targets>ios-simulator-64</Targets>
+    <Target>ios-simulator-64</Target>
     <WorkItemTimeout>00:12:00</WorkItemTimeout>
     <CustomCommands>
       set -e
-      deviceId=`xharness apple device $targets`
-      xharness apple install -t $targets --device "$deviceId" -o "$output_directory" --app=$app
+      deviceId=`xharness apple device $target`
+      xharness apple install -t $target --device "$deviceId" -o "$output_directory" --app=$app
       set +e
       result=0
-      xharness apple just-test -t $targets --device "$deviceId" -o "$output_directory" --app net.dot.Some.iOS --timeout 00:08:00
+      xharness apple just-test -t $target --device "$deviceId" -o "$output_directory" --app net.dot.Some.iOS --timeout 00:08:00
       ((result|=$?))
-      xharness apple uninstall -t $targets --device "$deviceId" -o "$output_directory" --app net.dot.Some.iOS
+      xharness apple uninstall -t $target --device "$deviceId" -o "$output_directory" --app net.dot.Some.iOS
       ((result|=$?))
       exit $result
     </CustomCommands>
@@ -187,7 +187,7 @@ When using `CustomCommands`, several variables will be defined for you for easie
 - `$app` - path to the application
 - `$output_directory` - path under which all files will be uploaded to Helix at the end of the job
   - If a file named `testResults.xml` is found containing xUnit results, it will be uploaded back to Azure DevOps
-- `$targets`, `$timeout`, `$launch_timeout`, `$expected_exit_code`, `$includes_test_runner` - parsed metadata defined on the original `XHarnessAppBundleToTest` MSBuild item
+- `$target`, `$timeout`, `$launch_timeout`, `$expected_exit_code`, `$includes_test_runner` - parsed metadata defined on the original `XHarnessAppBundleToTest` MSBuild item
 
 #### Variables defined for Android scenarios
 Android is currently not supported - coming soon!

tests/XHarness/XHarness.Device.AppleRun.proj

@@ -20,7 +20,7 @@
 
     <ItemGroup>
       <XHarnessAppBundleToTest Include="$(ArtifactsTmpDir)XHarness.RunDeviceAppBundle/$(XHarnessRunAppBundleName)">
-        <Targets>ios-device</Targets>
+        <Target>ios-device</Target>
         <WorkItemTimeout>00:05:00</WorkItemTimeout>
         <TestTimeout>00:04:00</TestTimeout>
         <LaunchTimeout>00:03:00</LaunchTimeout>

tests/XHarness/XHarness.Simulator.AppleRun.proj

@@ -20,7 +20,7 @@
 
     <ItemGroup>
       <XHarnessAppBundleToTest Include="$(ArtifactsTmpDir)XHarness.RunAppBundle/$(XHarnessRunAppBundleName)">
-        <Targets>ios-simulator-64</Targets>
+        <Target>ios-simulator-64</Target>
         <WorkItemTimeout>00:12:00</WorkItemTimeout>
         <TestTimeout>00:08:00</TestTimeout>
         <LaunchTimeout>00:05:00</LaunchTimeout>

tests/XHarness/XHarness.Simulator.AppleTest.proj

@@ -20,7 +20,7 @@
 
     <ItemGroup>
       <XHarnessAppBundleToTest Include="$(ArtifactsTmpDir)XHarness.TestAppBundle/$(XHarnessTestAppBundleName)">
-        <Targets>ios-simulator-64</Targets>
+        <Target>ios-simulator-64</Target>
         <TestTimeout>00:05:00</TestTimeout>
         <WorkItemTimeout>00:15:00</WorkItemTimeout>
         <LaunchTimeout>00:06:00</LaunchTimeout>

tests/XHarness/XHarness.Simulator.CustomCommands.proj

@@ -20,7 +20,7 @@
 
     <ItemGroup>
       <XHarnessAppBundleToTest Include="$(ArtifactsTmpDir)XHarness.CustomCommandsAppBundle/$(XHarnessAppBundleName).app">
-        <Targets>ios-simulator-64</Targets>
+        <Target>ios-simulator-64</Target>
         <WorkItemTimeout>00:15:00</WorkItemTimeout>
         <TestTimeout>00:05:00</TestTimeout>
         <LaunchTimeout>00:05:00</LaunchTimeout>