Generate async files

Author: github-actions[bot]

src/NHibernate.Test/Async/SystemTransactions/SystemTransactionFixture.cs

@@ -531,7 +531,10 @@ public async Task EnforceConnectionUsageRulesOnTransactionCompletionAsync()
 		[Test]
 		public async Task SupportsTransactionTimeoutAsync()
 		{
-			Assume.That(TestDialect.SupportsTransactionScopeTimeouts, Is.True);
+			Assume.That(TestDialect.SupportsTransactionScopeTimeouts, Is.True, "The tested dialect is not supported for transaction scope timeouts.");
+			// ODBC always freezes the session during transaction scopes timeouts.
+			Assume.That(Sfi.ConnectionProvider.Driver, Is.Not.InstanceOf(typeof(OdbcDriver)), "ODBC is not supported for transaction scope timeouts.");
+
 			// Test case adapted from https://github.com/kaksmet/NHibBugRepro
 
 			// Create some test data.
@@ -552,7 +555,7 @@ public async Task SupportsTransactionTimeoutAsync()
 				await (t.CommitAsync());
 			}
 
-			// Setup unhandler exception catcher
+			// Setup unhandled exception catcher.
 			_unhandledExceptions = new ConcurrentBag<object>();
 			AppDomain.CurrentDomain.UnhandledException += CurrentDomain_UnhandledException;
 			try
@@ -579,18 +582,63 @@ public async Task SupportsTransactionTimeoutAsync()
 						// Assume that is a transaction timeout. It may cause various failures, of which some are hard to identify.
 						timeoutsCount++;
 					}
+					// If in need of checking some specific failures, the following code may be used instead:
+					/*
+					catch (Exception ex)
+					{
+						var currentEx = ex;
+						// Depending on where the transaction aborption has broken NHibernate processing, we may
+						// get various exceptions, like directly a TransactionAbortedException with an inner
+						// TimeoutException, or a HibernateException encapsulating a TransactionException with a
+						// timeout, ...
+						bool isTransactionException, isTimeout;
+						do
+						{
+							isTransactionException = currentEx is System.Transactions.TransactionException;
+							isTimeout = isTransactionException && currentEx is TransactionAbortedException;
+							currentEx = currentEx.InnerException;
+						}
+						while (!isTransactionException && currentEx != null);
+						while (!isTimeout && currentEx != null)
+						{
+							isTimeout = currentEx is TimeoutException;
+							currentEx = currentEx?.InnerException;
+						}
+
+						if (!isTimeout)
+						{
+							// We may also get a GenericADOException with an InvalidOperationException stating the
+							// transaction associated to the connection is no more active but not yet suppressed,
+							// and that for executing some SQL, we need to suppress it. That is a weak way of
+							// identifying the case, especially with the many localizations of the message.
+							currentEx = ex;
+							do
+							{
+								isTimeout = currentEx is InvalidOperationException && currentEx.Message.Contains("SQL");
+								currentEx = currentEx?.InnerException;
+							}
+							while (!isTimeout && currentEx != null);
+						}
+
+						if (isTimeout)
+							timeoutsCount++;
+						else
+							throw;
+					}
+					*/
 				}
 
-				// Despite the Thread sleep and the count of entities to load, this test may get the timeout only for slightly
-				// more than 10% of the attempts.
-				Assert.That(timeoutsCount, Is.GreaterThan(5), "The test should have generated more timeouts.");
 				Assert.That(
 					_unhandledExceptions.Count,
 					Is.EqualTo(0),
 					"Unhandled exceptions have occurred: {0}",
 					string.Join(@"
 
 ", _unhandledExceptions));
+
+				// Despite the Thread sleep and the count of entities to load, this test may get the timeout only for slightly
+				// more than 10% of the attempts.
+				Warn.Unless(timeoutsCount, Is.GreaterThan(5), "The test should have generated more timeouts.");
 			}
 			finally
 			{