Merge pull request #349 from Timmoth/fix/flaky-git-static

Fix the Git service test racing every other test that registers services

.gitignore

@@ -427,3 +427,4 @@ FodyWeavers.xsd
 
 # macOS
 .DS_Store
+ISSUE_TRIAGE.md

Tests/Api/InventoryEndpointStartupTests.cs

@@ -9,6 +9,7 @@ namespace Tests.Api;
 ///     <c>rpk discover --push</c> on a timer. It therefore has to work on a server that
 ///     nobody has opened a page on yet.
 /// </summary>
+[Collection("Process-wide static state")]
 public class InventoryEndpointStartupTests(ITestOutputHelper output) : ApiTestBase(output) {
     private const string _existingConfig = """
                                            version: 3

Tests/Api/InventoryEndpointTests.cs

@@ -5,6 +5,7 @@ using Xunit.Abstractions;
 
 namespace Tests.Api;
 
+[Collection("Process-wide static state")]
 public class InventoryEndpointTests(ITestOutputHelper output) : ApiTestBase(output) {
     [Fact]
     public async Task DryRun_Add_New_Resource_Does_Not_Persist() {

Tests/Api/InventoryEndpointUnconfiguredTests.cs

@@ -4,6 +4,7 @@ using Xunit.Abstractions;
 
 namespace Tests.Api;
 
+[Collection("Process-wide static state")]
 public class InventoryEndpointUnconfiguredTests(ITestOutputHelper output) : ApiTestBase(output) {
     protected override void ConfigureTestConfiguration(IDictionary<string, string?> config) =>
         config.Remove("RPK_API_KEY");

Tests/EndToEnd/AccessPointTests/AccessPointCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.AccessPointTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class AccessPointCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/AccessPointTests/AccessPointErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.AccessPointTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class AccessPointErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/AccessPointTests/AccessPointWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.AccessPointTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class AccessPointWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/CliCommandsWorkflowTests.cs

@@ -7,7 +7,7 @@ namespace Tests.EndToEnd;
 /// Comprehensive E2E test covering all CLI commands with varied input data.
 /// Tests happy paths for CRUD operations, components, labels, and exporters.
 /// </summary>
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class CliCommandsWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ConnectionTests/ConnectionRemoveWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ConnectionTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ConnectionRemoveWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ConnectionTests/ConnectionWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ConnectionTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ConnectionWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ConnectionTests/PortConnectionWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ConnectionTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class PortConnectionWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ConnectionTests/RenameResourceTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ConnectionTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class RenameResourceTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
 

Tests/EndToEnd/CorruptConfigTests.cs

@@ -15,7 +15,7 @@ namespace Tests.EndToEnd;
 // leaves valid YAML that is indistinguishable from a smaller inventory. Nothing at
 // load time can detect it, which is precisely why the writer-side fix (atomic,
 // durable saves — see PhysicalTextFileStoreTests) is the primary remedy.
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class CorruptConfigTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     // Exactly what the serializer writes for three bare servers.

Tests/EndToEnd/DeleteResourceTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class DeleteResourceTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/DesktopTests/DesktopCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.DesktopTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class DesktopCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/DesktopTests/DesktopErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.DesktopTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class DesktopErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/DesktopTests/DesktopWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.DesktopTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class DesktopWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/DiscoveryTests/DiscoverNetworkValidationTests.cs

@@ -7,7 +7,7 @@ namespace Tests.EndToEnd.DiscoveryTests;
 ///     `rpk discover network` argument validation. Every case here fails before any
 ///     probing starts, so these tests never send a packet anywhere.
 /// </summary>
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class DiscoverNetworkValidationTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<string> ExecuteAsync(params string[] args) =>

Tests/EndToEnd/ExporterTests/AnsibleInventoryWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ExporterTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class AnsibleInventoryWorkflowTests(
     TempYamlCliFixture fs,
     ITestOutputHelper outputHelper)

Tests/EndToEnd/ExporterTests/HostsExportWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ExporterTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class HostsExportWorkflowTests(
     TempYamlCliFixture fs,
     ITestOutputHelper outputHelper)

Tests/EndToEnd/ExporterTests/SshExportWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ExporterTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class SshExportWorkflowTests(
     TempYamlCliFixture fs,
     ITestOutputHelper outputHelper)

Tests/EndToEnd/FirewallTests/FirewallCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.FirewallTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class FirewallCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/FirewallTests/FirewallErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.FirewallTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class FirewallErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/FirewallTests/FirewallWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.FirewallTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class FirewallWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/GenerateAnsibleInventoryTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class GenerateAnsibleInventoryTests(
     TempYamlCliFixture fs,
     ITestOutputHelper outputHelper)

Tests/EndToEnd/Graph/GraphTopologyCliTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.Graph;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class GraphTopologyCliTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<string> ExecuteAsync(params string[] args) {

Tests/EndToEnd/Infra/YamlCliTestCollection.cs

@@ -1,6 +1,23 @@
 namespace Tests.EndToEnd.Infra;
 
-[CollectionDefinition("Yaml CLI tests", DisableParallelization = true)]
+/// <summary>
+///     Every test that registers RackPeek's services, directly or through the CLI host.
+///     <para>
+///         Registration writes process-wide statics as a side effect —
+///         <c>RpkConstants.HasGitServices</c> among them — which is harmless in
+///         production, where a process registers once, and a race in a test run, where
+///         dozens of classes register with different configuration. Two classes in
+///         different collections run in parallel, so a test asserting on one of those
+///         statics can read a value another class set microseconds earlier.
+///     </para>
+///     <para>
+///         One collection with parallelisation off is what makes those assertions mean
+///         anything. The name is deliberately about the shared state rather than about
+///         the YAML CLI, because membership is decided by "does this touch the statics",
+///         not by what the test is nominally exercising.
+///     </para>
+/// </summary>
+[CollectionDefinition("Process-wide static state", DisableParallelization = true)]
 public class YamlCliTestCollection
     : ICollectionFixture<TempYamlCliFixture> {
 }

Tests/EndToEnd/Labels/LabelsWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.Labels;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class LabelsWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/LaptopTests/LaptopCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.LaptopTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class LaptopCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/LaptopTests/LaptopErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.LaptopTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class LaptopErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/LaptopTests/LaptopNicWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.LaptopTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class LaptopNicWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/LaptopTests/LaptopWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.LaptopTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class LaptopWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/OtherTests/OtherCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.OtherTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class OtherCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/OtherTests/OtherErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.OtherTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class OtherErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/OtherTests/OtherPortWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.OtherTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class OtherPortWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/OtherTests/OtherWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.OtherTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class OtherWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/RouterTests/RouterCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.RouterTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class RouterCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/RouterTests/RouterErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.RouterTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class RouterErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/RouterTests/RouterWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.RouterTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class RouterWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ServerTests/ServerCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ServerTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ServerCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ServerTests/ServerErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ServerTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ServerErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ServerTests/ServerWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ServerTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ServerWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ServiceTests/ServiceCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ServiceTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ServiceCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ServiceTests/ServiceErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ServiceTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ServiceErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/ServiceTests/ServiceWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.ServiceTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class ServiceWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/SwitchTests/SwitchCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.SwitchTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class SwitchCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/SwitchTests/SwitchErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.SwitchTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class SwitchErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/SwitchTests/SwitchWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.SwitchTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class SwitchWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/SystemTests/SystemCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.SystemTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class SystemCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/SystemTests/SystemErrorTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.SystemTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class SystemErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/SystemTests/SystemWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.SystemTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class SystemWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/Tags/TagsWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.Tags;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class TagsWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string output, string yaml)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/UpsTests/UpsCommandTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.UpsTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class UpsCommandTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/UpsTests/UpsErrorTest.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.UpsTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class UpsErrorTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/UpsTests/UpsPortWorkflowTests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.UpsTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class UpsPortWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/EndToEnd/UpsTests/UpsWorkflowtests.cs

@@ -3,7 +3,7 @@ using Xunit.Abstractions;
 
 namespace Tests.EndToEnd.UpsTests;
 
-[Collection("Yaml CLI tests")]
+[Collection("Process-wide static state")]
 public class UpsWorkflowTests(TempYamlCliFixture fs, ITestOutputHelper outputHelper)
     : IClassFixture<TempYamlCliFixture> {
     private async Task<(string, string)> ExecuteAsync(params string[] args) {

Tests/Git/GitConfigurationTests.cs

@@ -6,7 +6,7 @@ using RackPeek.Domain.Git;
 
 namespace Tests.Git;
 
-[Collection("Git static state")]
+[Collection("Process-wide static state")]
 public sealed class GitConfigurationTests : IDisposable {
     private readonly string _tempDir;