Skip to content

Commit 8a7e206

Browse files
committed
ci: update concurrency group
1 parent 1a35b80 commit 8a7e206

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

.github/workflows/ui-tests.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,9 @@ jobs:
8888
build-and-test-unity6-macos: # Unity 6 requires a full build cycle to compile AltTester packages properly. This doesn't work well in Game CI, so we have to build it manually.
8989
name: Build & Test Unity 6 macOS 🛠️🧪
9090
runs-on: [self-hosted, macOS]
91+
concurrency:
92+
group: ui-tests-email-inbox-macos
93+
cancel-in-progress: false # Let tests complete rather than canceling
9194
if: github.event_name != 'workflow_dispatch' || github.event.inputs.targetPlatform == 'All' || github.event.inputs.targetPlatform == 'StandaloneOSX-Unity6'
9295
steps:
9396
- name: Cleanup old builds

sample/Assets/Editor/MacBuilderUnity6.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ private static void BuildPlayer(string defaultBuildPath, bool setupForAltTester
6969

7070
Debug.Log($"Build options: {options.options}");
7171
Debug.Log($"Starting build to: {buildPath}");
72-
72+
7373
var result = BuildPipeline.BuildPlayer(options);
7474

7575
if (result.summary.result == UnityEditor.Build.Reporting.BuildResult.Succeeded)
@@ -90,15 +90,15 @@ private static void BuildPlayer(string defaultBuildPath, bool setupForAltTester
9090
Debug.Log("Build Profile restored");
9191

9292
if (setupForAltTester)
93-
{
93+
{
9494
// Clean up AltTester settings after build
9595
AltBuilder.RemoveAltTesterFromScriptingDefineSymbols(BuildTargetGroup.Standalone);
9696

9797
// Clean up custom e2e testing define
9898
var defineSymbols = PlayerSettings.GetScriptingDefineSymbolsForGroup(BuildTargetGroup.Standalone);
9999
defineSymbols = defineSymbols.Replace("IMMUTABLE_E2E_TESTING;", "").Replace(";IMMUTABLE_E2E_TESTING", "").Replace("IMMUTABLE_E2E_TESTING", "");
100100
PlayerSettings.SetScriptingDefineSymbolsForGroup(BuildTargetGroup.Standalone, defineSymbols);
101-
101+
102102
var cleanedDefineSymbols = PlayerSettings.GetScriptingDefineSymbolsForGroup(BuildTargetGroup.Standalone);
103103

104104
RemoveAltFromScene(scenes[0]);
@@ -129,7 +129,7 @@ private static string[] GetScenesToBuild(bool setupForAltTester = false)
129129
return new[]
130130
{
131131
"Assets/Scenes/Passport/Initialisation.unity",
132-
"Assets/Scenes/Passport/UnauthenticatedScene.unity",
132+
"Assets/Scenes/Passport/UnauthenticatedScene.unity",
133133
"Assets/Scenes/Passport/AuthenticatedScene.unity",
134134
"Assets/Scenes/Passport/ZkEvm/ZkEvmGetBalance.unity",
135135
"Assets/Scenes/Passport/ZkEvm/ZkEvmGetTransactionReceipt.unity",
@@ -143,7 +143,7 @@ private static string[] GetScenesToBuild(bool setupForAltTester = false)
143143
private static void SetupAltTester(string[] scenes)
144144
{
145145
AltBuilder.AddAltTesterInScriptingDefineSymbolsGroup(BuildTargetGroup.Standalone);
146-
146+
147147
var defineSymbols = PlayerSettings.GetScriptingDefineSymbolsForGroup(BuildTargetGroup.Standalone);
148148
if (!defineSymbols.Contains("IMMUTABLE_E2E_TESTING"))
149149
{
@@ -165,7 +165,7 @@ private static void SetupAltTester(string[] scenes)
165165
{
166166
instrumentationSettings.AltServerPort = int.Parse(port);
167167
}
168-
168+
169169
instrumentationSettings.ResetConnectionData = true;
170170
AltBuilder.InsertAltInScene(scenes[0], instrumentationSettings);
171171
}

0 commit comments

Comments
 (0)