fix: Catch database exceptions at the end of transactions. (serverpod…

…#2851)

packages/serverpod_test/lib/src/transaction_manager.dart

@@ -44,7 +44,13 @@ class TransactionManager {
           await _endTransactionScopeCompleter.future;
         },
         isUserCall: false,
-      ),
+      ).catchError((error, stackTrace) {
+        // no-op:
+        // If a database exception occurred during the transaction,
+        // but the exception was caught and the transactions was allowed to complete,
+        // then the transaction will rethrow it when it completes.
+        // This has to be caught, otherwise the dart test runner will fail the test suite.
+      }),
     );
 
     await transactionStartedCompleter.future;

tests/serverpod_test_server/config/passwords.yaml

@@ -14,3 +14,7 @@ production:
   database: 'password'
   redis: 'password'
   serviceSecret: 'super_SECRET_password'
+test:
+  database: 'password'
+  redis: 'password'
+  serviceSecret: 'super_SECRET_password'

tests/serverpod_test_server/config/test.yaml

@@ -0,0 +1,30 @@
+# Duplicate of production.yaml
+# The test tools use test by default, but the other integration tests use production as run mode
+apiServer:
+  port: 8080
+  publicHost: serverpod_test_server
+  publicPort: 8080
+  publicScheme: http
+
+insightsServer:
+  port: 8081
+  publicHost: serverpod_test_server
+  publicPort: 8081
+  publicScheme: http
+
+webServer:
+  port: 8082
+  publicHost: serverpod_test_server
+  publicPort: 8082
+  publicScheme: http
+
+database:
+  host: postgres
+  port: 5432
+  name: serverpod_test
+  user: postgres
+
+redis:
+  enabled: true
+  host: redis
+  port: 6379

tests/serverpod_test_server/test_integration/method_streaming_cancel_test.dart

@@ -1,4 +1,3 @@
-import 'package:serverpod/serverpod.dart';
 import 'package:test/test.dart';
 
 import 'test_tools/serverpod_test_tools.dart';
@@ -59,6 +58,5 @@ void main() {
         );
       });
     },
-    runMode: ServerpodRunMode.production,
   );
 }

tests/serverpod_test_server/test_integration/test_tools/authentication_validation_test.dart

@@ -305,6 +305,5 @@ void main() {
         });
       });
     },
-    runMode: ServerpodRunMode.production,
   );
 }

tests/serverpod_test_server/test_integration/test_tools/database_operations_in_endpoint_test.dart

@@ -88,7 +88,6 @@ void main() {
         });
       });
     },
-    runMode: ServerpodRunMode.production,
   );
 
   withServerpod(
@@ -168,7 +167,6 @@ void main() {
         });
       });
     },
-    runMode: ServerpodRunMode.production,
     rollbackDatabase: RollbackDatabase.afterEach,
   );
 
@@ -200,7 +198,6 @@ void main() {
             expect(simpleDatas[1].num, 123);
           });
         },
-        runMode: ServerpodRunMode.production,
         rollbackDatabase: RollbackDatabase.afterAll,
       );
 
@@ -214,7 +211,6 @@ void main() {
             expect(simpleDatas, hasLength(0));
           });
         },
-        runMode: ServerpodRunMode.production,
       );
     });
 
@@ -240,7 +236,6 @@ void main() {
           });
         },
         rollbackDatabase: RollbackDatabase.afterAll,
-        runMode: ServerpodRunMode.production,
       );
 
       withServerpod(
@@ -253,7 +248,6 @@ void main() {
             expect(simpleDatas, hasLength(0));
           });
         },
-        runMode: ServerpodRunMode.production,
       );
     });
 
@@ -280,7 +274,6 @@ void main() {
           });
         },
         rollbackDatabase: RollbackDatabase.afterAll,
-        runMode: ServerpodRunMode.production,
       );
 
       withServerpod(
@@ -294,7 +287,6 @@ void main() {
             expect(simpleDatas, hasLength(0));
           });
         },
-        runMode: ServerpodRunMode.production,
       );
     });
   });
@@ -326,7 +318,6 @@ void main() {
             expect(simpleDatas[1].num, 123);
           });
         },
-        runMode: ServerpodRunMode.production,
         rollbackDatabase: RollbackDatabase.disabled,
       );
 
@@ -351,7 +342,6 @@ void main() {
             expect(simpleDatas[1].num, 123);
           });
         },
-        runMode: ServerpodRunMode.production,
         rollbackDatabase: RollbackDatabase.disabled,
       );
     });
@@ -377,7 +367,6 @@ void main() {
           });
         },
         rollbackDatabase: RollbackDatabase.disabled,
-        runMode: ServerpodRunMode.production,
       );
 
       withServerpod(
@@ -399,7 +388,6 @@ void main() {
           });
         },
         rollbackDatabase: RollbackDatabase.disabled,
-        runMode: ServerpodRunMode.production,
       );
     });
 
@@ -426,7 +414,6 @@ void main() {
         });
       },
       rollbackDatabase: RollbackDatabase.disabled,
-      runMode: ServerpodRunMode.production,
     );
   });
 }