dohertj2/ScadaBridge
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

test(e2e): guard ApiSurfaceFixture partial-init delete + seal TransportImportTests (final review nits)

Browse Source
This commit is contained in:
Joseph Doherty
2026-06-06 12:37:21 -04:00
parent 09f14f18ea
commit 0700777e2f
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/ZB.MOM.WW.ScadaBridge.CentralUI.PlaywrightTests/Admin/ApiSurfaceFixture.cs
+6 -1
View File
@@ -28,7 +28,12 @@ public sealed class ApiSurfaceFixture : IAsyncLifetime
} }
catch catch
{ {
await CliRunner.DeleteApiMethodAsync(MethodId); // Partial-init guard: MethodId is non-zero only if CreateApiMethodAsync succeeded,
// so skip the delete when nothing was created (avoids a phantom delete --id 0).
if (MethodId != 0)
{
await CliRunner.DeleteApiMethodAsync(MethodId);
}
Available = false; Available = false;
throw; throw;
} }
tests/ZB.MOM.WW.ScadaBridge.CentralUI.PlaywrightTests/Transport/TransportImportTests.cs
+1 -1
View File
@@ -35,7 +35,7 @@ namespace ZB.MOM.WW.ScadaBridge.CentralUI.PlaywrightTests.Transport;
/// </para> /// </para>
/// </summary> /// </summary>
[Collection("Playwright")] [Collection("Playwright")]
public class TransportImportTests public sealed class TransportImportTests
{ {
private readonly PlaywrightFixture _fixture; private readonly PlaywrightFixture _fixture;