Merge pull request #111 from microting/copilot/fix-test-class-constructors

Fix multiple public constructors for xUnit v3 compatibility

Author: renemadsen

test/EFCore.MySql.FunctionalTests/Query/EntitySplittingQueryMySqlTest.cs

@@ -10,7 +10,7 @@ namespace Pomelo.EntityFrameworkCore.MySql.FunctionalTests.Query;
 
 public class EntitySplittingQueryMySqlTest(NonSharedFixture fixture) : EntitySplittingQueryTestBase(fixture)
 {
-    public EntitySplittingQueryMySqlTest(ITestOutputHelper testOutputHelper) : this(new NonSharedFixture())
+    internal EntitySplittingQueryMySqlTest(ITestOutputHelper testOutputHelper) : this(new NonSharedFixture())
     {
         // Fixture.TestSqlLoggerFactory.SetTestOutputHelper(testOutputHelper);
     }

test/EFCore.MySql.FunctionalTests/TestUtilities/TestMySqlRetryingExecutionStrategy.cs

@@ -20,7 +20,7 @@ public class TestMySqlRetryingExecutionStrategy : MySqlRetryingExecutionStrategy
             42019 // CREATE DATABASE operation failed
         };
 
-        public TestMySqlRetryingExecutionStrategy()
+        internal TestMySqlRetryingExecutionStrategy()
             : base(
                 new DbContext(
                     new DbContextOptionsBuilder()
@@ -30,12 +30,12 @@ public TestMySqlRetryingExecutionStrategy()
         {
         }
 
-        public TestMySqlRetryingExecutionStrategy(DbContext context)
+        internal TestMySqlRetryingExecutionStrategy(DbContext context)
             : base(context, DefaultMaxRetryCount, DefaultMaxDelay, _additionalErrorNumbers)
         {
         }
 
-        public TestMySqlRetryingExecutionStrategy(DbContext context, TimeSpan maxDelay)
+        internal TestMySqlRetryingExecutionStrategy(DbContext context, TimeSpan maxDelay)
             : base(context, DefaultMaxRetryCount, maxDelay, _additionalErrorNumbers)
         {
         }

test/EFCore.MySql.FunctionalTests/TestUtilities/Xunit/MySqlConditionalFactTestCase.cs

@@ -1,4 +1,5 @@
 using System;
+using System.ComponentModel;
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.EntityFrameworkCore.TestUtilities.Xunit;
@@ -13,6 +14,7 @@ namespace Pomelo.EntityFrameworkCore.MySql.FunctionalTests.TestUtilities.Xunit
     public sealed class MySqlConditionalFactTestCase : XunitTestCase
     {
         [Obsolete("Called by the de-serializer; should only be called by deriving classes for de-serialization purposes")]
+        [EditorBrowsable(EditorBrowsableState.Never)]
         public MySqlConditionalFactTestCase()
         {
         }

test/EFCore.MySql.FunctionalTests/TestUtilities/Xunit/MySqlConditionalTheoryTestCase.cs

@@ -1,4 +1,5 @@
 using System;
+using System.ComponentModel;
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.EntityFrameworkCore.TestUtilities.Xunit;
@@ -13,6 +14,7 @@ namespace Pomelo.EntityFrameworkCore.MySql.FunctionalTests.TestUtilities.Xunit
     public sealed class MySqlConditionalTheoryTestCase : XunitTheoryTestCase
     {
         [Obsolete("Called by the de-serializer; should only be called by deriving classes for de-serialization purposes")]
+        [EditorBrowsable(EditorBrowsableState.Never)]
         public MySqlConditionalTheoryTestCase()
         {
         }