* isolated the shutdown node test

EventStore · Nov 2, 2023 · 35bcee4 · 35bcee4
1 parent e8e3182
commit 35bcee4
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 11 deletions.
diff --git a/test/EventStore.Client.Operations.Tests/MergeIndexesTests.cs b/test/EventStore.Client.Operations.Tests/MergeIndexesTests.cs
@@ -1,7 +1,7 @@
 namespace EventStore.Client.Operations.Tests;
 
-public class @admin : IClassFixture<InsecureClientTestFixture> {
-	public admin(ITestOutputHelper output, InsecureClientTestFixture fixture) =>
+public class MergeIndexesTests : IClassFixture<InsecureClientTestFixture> {
+	public MergeIndexesTests(ITestOutputHelper output, InsecureClientTestFixture fixture) =>
 		Fixture = fixture.With(x => x.CaptureTestRun(output));
 
 	InsecureClientTestFixture Fixture { get; }

diff --git a/test/EventStore.Client.Operations.Tests/ShutdownNodeAuthenticationTests.cs b/test/EventStore.Client.Operations.Tests/ShutdownNodeAuthenticationTests.cs
@@ -0,0 +1,12 @@
+namespace EventStore.Client.Operations.Tests;
+
+public class ShutdownNodeAuthenticationTests : IClassFixture<InsecureClientTestFixture> {
+	public ShutdownNodeAuthenticationTests(ITestOutputHelper output, InsecureClientTestFixture fixture) =>
+		Fixture = fixture.With(x => x.CaptureTestRun(output));
+
+	InsecureClientTestFixture Fixture { get; }
+
+	[Fact]
+	public async Task shutdown_without_credentials_throws() =>
+		await Fixture.Operations.ShutdownAsync().ShouldThrowAsync<AccessDeniedException>();
+}
diff --git a/test/EventStore.Client.Operations.Tests/ShutdownNodeTests.cs b/test/EventStore.Client.Operations.Tests/ShutdownNodeTests.cs
@@ -1,5 +1,3 @@
-using FlakyTest.XUnit.Attributes;
-
 namespace EventStore.Client.Operations.Tests;
 
 public class ShutdownNodeTests : IClassFixture<InsecureClientTestFixture> {
@@ -9,10 +7,6 @@ public ShutdownNodeTests(ITestOutputHelper output, InsecureClientTestFixture fix
 	InsecureClientTestFixture Fixture { get; }
 
 	[Fact]
-	public async Task shutdown_does_not_throw() => 
+	public async Task shutdown_does_not_throw() =>
 		await Fixture.Operations.ShutdownAsync(userCredentials: TestCredentials.Root).ShouldNotThrowAsync();
-
-	[MaybeFixedFact(1)]
-	public async Task shutdown_without_credentials_throws() =>
-		await Fixture.Operations.ShutdownAsync().ShouldThrowAsync<AccessDeniedException>();
 }
diff --git a/test/EventStore.Client.Tests.Common/Fixtures/EventStoreFixture.cs b/test/EventStore.Client.Tests.Common/Fixtures/EventStoreFixture.cs
@@ -112,11 +112,11 @@ async Task WarmUp() {
 	}
 
 	public async Task InitializeAsync() {
-		await Service.Start().ShouldNotThrowAsync();
+		await Service.Start();
 
 		await WarmUp();
 
-		await OnSetup().ShouldNotThrowAsync("Failed to run OnSetup!");
+		await OnSetup();
 	}
 
 	public async Task DisposeAsync() {