Changeset: 2cc8de76016d for monetdb-java
URL: https://dev.monetdb.org/hg/monetdb-java/rev/2cc8de76016d
Modified Files:
src/test/java/JDBC_API_Tester.java
Branch: default
Log Message:
Revert "Don't use assert to test whether mserver5 is new enough for the test."
This reverts commit c7a78aaff9cae3974d938564d09caaf3180582dd.
The underlying issue was already fixed by commit
d0b87ad07ebf "MTest runner: do not print stack trace on skipped tests"
Note that assumeTrue (as opposed to assertTrue) is specifically intended
to skip and report as skipped, tests that are somehow not applicable in
the current environment. The problem was that the exception used to do
the skipping was accidentally printed by the Junit-MTest adapter.
diffs (13 lines):
diff --git a/src/test/java/JDBC_API_Tester.java
b/src/test/java/JDBC_API_Tester.java
--- a/src/test/java/JDBC_API_Tester.java
+++ b/src/test/java/JDBC_API_Tester.java
@@ -3822,8 +3822,7 @@ public final class JDBC_API_Tester exten
// Older versions forget to send autocommit status change
// notifications when transaction fails
- if (!isAtLeast56)
- return;
+ assumeTrue(isAtLeast56);
try {
// Connections
_______________________________________________
checkin-list mailing list -- [email protected]
To unsubscribe send an email to [email protected]
