test review

KSemenenko · KSemenenko · commit c9e8401c7183 · 2026-04-01T15:26:38.000+02:00
diff --git a/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserIntegrationCollection.cs b/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserIntegrationCollection.cs
@@ -1,9 +1,7 @@
-using Xunit;
-
 namespace ManagedCode.Storage.Tests.Storages.Browser;
 
-[CollectionDefinition(Name, DisableParallelization = true)]
-public sealed class BrowserIntegrationCollection : ICollectionFixture<BrowserServerHostFixture>, ICollectionFixture<BrowserWasmHostFixture>
+internal static class BrowserIntegrationCollection
 {
-    public const string Name = nameof(BrowserIntegrationCollection);
+    public const string Server = BrowserServerHostCollection.Name;
+    public const string Wasm = BrowserWasmHostCollection.Name;
 }
diff --git a/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserServerHostCollection.cs b/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserServerHostCollection.cs
@@ -5,4 +5,5 @@ namespace ManagedCode.Storage.Tests.Storages.Browser;
 [CollectionDefinition(nameof(BrowserServerHostCollection), DisableParallelization = true)]
 public sealed class BrowserServerHostCollection : ICollectionFixture<BrowserServerHostFixture>
 {
+    public const string Name = nameof(BrowserServerHostCollection);
 }
diff --git a/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserServerStorageIntegrationTests.cs b/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserServerStorageIntegrationTests.cs
@@ -6,7 +6,7 @@
 
 namespace ManagedCode.Storage.Tests.Storages.Browser;
 
-[Collection(BrowserIntegrationCollection.Name)]
+[Collection(BrowserIntegrationCollection.Server)]
 public sealed class BrowserServerStorageIntegrationTests(BrowserServerHostFixture fixture)
 {
     [Fact]
diff --git a/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserServerVfsIntegrationTests.cs b/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserServerVfsIntegrationTests.cs
@@ -6,7 +6,7 @@
 
 namespace ManagedCode.Storage.Tests.Storages.Browser;
 
-[Collection(BrowserIntegrationCollection.Name)]
+[Collection(BrowserIntegrationCollection.Server)]
 public sealed class BrowserServerVfsIntegrationTests(BrowserServerHostFixture fixture)
 {
     [Fact]
diff --git a/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserWasmHostCollection.cs b/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserWasmHostCollection.cs
@@ -5,4 +5,5 @@ namespace ManagedCode.Storage.Tests.Storages.Browser;
 [CollectionDefinition(nameof(BrowserWasmHostCollection), DisableParallelization = true)]
 public sealed class BrowserWasmHostCollection : ICollectionFixture<BrowserWasmHostFixture>
 {
+    public const string Name = nameof(BrowserWasmHostCollection);
 }
diff --git a/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserWasmStorageIntegrationTests.cs b/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserWasmStorageIntegrationTests.cs
@@ -6,7 +6,7 @@
 
 namespace ManagedCode.Storage.Tests.Storages.Browser;
 
-[Collection(BrowserIntegrationCollection.Name)]
+[Collection(BrowserIntegrationCollection.Wasm)]
 public sealed class BrowserWasmStorageIntegrationTests(BrowserWasmHostFixture fixture)
 {
     [Fact]
diff --git a/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserWasmVfsIntegrationTests.cs b/Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserWasmVfsIntegrationTests.cs
@@ -6,7 +6,7 @@
 
 namespace ManagedCode.Storage.Tests.Storages.Browser;
 
-[Collection(BrowserIntegrationCollection.Name)]
+[Collection(BrowserIntegrationCollection.Wasm)]
 public sealed class BrowserWasmVfsIntegrationTests(BrowserWasmHostFixture fixture)
 {
     [Fact]

Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,7 @@`
`1`		`-using Xunit;`
`2`		`-`
`3`	`1`	`namespace ManagedCode.Storage.Tests.Storages.Browser;`
`4`	`2`
`5`		`-[CollectionDefinition(Name, DisableParallelization = true)]`
`6`		`-public sealed class BrowserIntegrationCollection : ICollectionFixture<BrowserServerHostFixture>, ICollectionFixture<BrowserWasmHostFixture>`
	`3`	`+internal static class BrowserIntegrationCollection`
`7`	`4`	`{`
`8`		`- public const string Name = nameof(BrowserIntegrationCollection);`
	`5`	`+ public const string Server = BrowserServerHostCollection.Name;`
	`6`	`+ public const string Wasm = BrowserWasmHostCollection.Name;`
`9`	`7`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,4 +5,5 @@ namespace ManagedCode.Storage.Tests.Storages.Browser;`
`5`	`5`	`[CollectionDefinition(nameof(BrowserServerHostCollection), DisableParallelization = true)]`
`6`	`6`	`public sealed class BrowserServerHostCollection : ICollectionFixture<BrowserServerHostFixture>`
`7`	`7`	`{`
	`8`	`+ public const string Name = nameof(BrowserServerHostCollection);`
`8`	`9`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@`
`6`	`6`
`7`	`7`	`namespace ManagedCode.Storage.Tests.Storages.Browser;`
`8`	`8`
`9`		`-[Collection(BrowserIntegrationCollection.Name)]`
	`9`	`+[Collection(BrowserIntegrationCollection.Server)]`
`10`	`10`	`public sealed class BrowserServerStorageIntegrationTests(BrowserServerHostFixture fixture)`
`11`	`11`	`{`
`12`	`12`	`[Fact]`
Original file line number	Diff line number	Diff line change
`@@ -5,4 +5,5 @@ namespace ManagedCode.Storage.Tests.Storages.Browser;`
`5`	`5`	`[CollectionDefinition(nameof(BrowserWasmHostCollection), DisableParallelization = true)]`
`6`	`6`	`public sealed class BrowserWasmHostCollection : ICollectionFixture<BrowserWasmHostFixture>`
`7`	`7`	`{`
	`8`	`+ public const string Name = nameof(BrowserWasmHostCollection);`
`8`	`9`	`}`