[.NET] Move all tests that touch LibCantera into the same collection

Due to an idiosyncrasy in the native lib, separate loggers are created per thread. We need all tests that call the native lib to be on the same thread to ensure the LogWriter_MessageLogged test passes.
Cantera · Aug 20, 2023 · 689266a · 689266a
1 parent 6d3e101
commit 689266a
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 6 deletions.
diff --git a/interfaces/dotnet/Cantera.Tests/src/ApplicationTest.cs b/interfaces/dotnet/Cantera.Tests/src/ApplicationTest.cs
@@ -8,7 +8,7 @@
 
 namespace Cantera.Tests;
 
-[Collection("Application")]
+[Collection(LibCanteraFixture.Collection)]
 public class ApplicationTest
 {
     class FooException : Exception { }

diff --git a/interfaces/dotnet/Cantera.Tests/src/ExceptionTest.cs b/interfaces/dotnet/Cantera.Tests/src/ExceptionTest.cs
@@ -9,6 +9,7 @@
 
 namespace Cantera.Tests;
 
+[Collection(LibCanteraFixture.Collection)]
 public class ExceptionTest
 {
     class FooException : Exception { }

diff --git a/interfaces/dotnet/Cantera.Tests/src/TestFixtures.cs b/interfaces/dotnet/Cantera.Tests/src/TestFixtures.cs
@@ -5,13 +5,15 @@
 
 namespace Cantera.Tests;
 
-public class ApplicationFixture
+public class LibCanteraFixture
 {
-    public ApplicationFixture()
+    internal const string Collection = "LibCantera";
+
+    public LibCanteraFixture()
     {
         Application.DataDirectories.AddAssemblyDirectory();
     }
 }
 
-[CollectionDefinition("Application")]
-public class DatabaseCollection : ICollectionFixture<ApplicationFixture> { }
+[CollectionDefinition(LibCanteraFixture.Collection)]
+public class LibCanteraCollection : ICollectionFixture<LibCanteraFixture> { }
diff --git a/interfaces/dotnet/Cantera.Tests/src/ThermoPhaseTest.cs b/interfaces/dotnet/Cantera.Tests/src/ThermoPhaseTest.cs
@@ -5,7 +5,7 @@
 
 namespace Cantera.Tests;
 
-[Collection("Application")]
+[Collection(LibCanteraFixture.Collection)]
 public class ThermoPhaseTest
 {
     [Fact]