Skip to content

Commit 888e22f

Browse files
committed
[misc] test correction
1 parent 18fe403 commit 888e22f

File tree

2 files changed

+15
-5
lines changed

2 files changed

+15
-5
lines changed

src/test/java/org/mariadb/jdbc/integration/ConfigurationTest.java

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,8 @@
77
import static org.junit.jupiter.api.Assertions.*;
88

99
import java.sql.*;
10-
import org.junit.jupiter.api.AfterAll;
11-
import org.junit.jupiter.api.Assumptions;
12-
import org.junit.jupiter.api.BeforeAll;
13-
import org.junit.jupiter.api.Test;
10+
11+
import org.junit.jupiter.api.*;
1412

1513
public class ConfigurationTest extends Common {
1614

@@ -59,12 +57,24 @@ public void testSessionVariable() throws SQLException {
5957
Statement stmt = connection.createStatement();
6058
ResultSet rs = stmt.executeQuery("SELECT @@session_track_system_variables");
6159
assertTrue(rs.next());
60+
} catch (SQLException e) {
61+
// in case server check variable validity
62+
Assertions.assertTrue(e.getCause().getMessage().contains("Unknown system variable 'some';f'"));
6263
}
6364
try (Connection connection =
6465
createCon("sessionVariables=session_track_system_variables=\"some\\\";f,'ff'\"")) {
6566
Statement stmt = connection.createStatement();
6667
ResultSet rs = stmt.executeQuery("SELECT @@session_track_system_variables");
6768
assertTrue(rs.next());
69+
} catch (SQLException e) {
70+
// in case server check variable validity
71+
Assertions.assertTrue(e.getCause().getMessage().contains("Unknown system variable 'some\";f'"));
72+
}
73+
try (Connection connection =
74+
createCon("sessionVariables=session_track_system_variables=connect_timeout")) {
75+
Statement stmt = connection.createStatement();
76+
ResultSet rs = stmt.executeQuery("SELECT @@session_track_system_variables");
77+
assertTrue(rs.next());
6878
}
6979
}
7080
}

src/test/java/org/mariadb/jdbc/integration/MultiHostTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -275,7 +275,7 @@ public void masterFailover() throws Exception {
275275
(Connection)
276276
DriverManager.getConnection(
277277
url
278-
+ "&waitReconnectTimeout=300&retriesAllDown=10&connectTimeout=20&deniedListTimeout=20")) {
278+
+ "&waitReconnectTimeout=300&retriesAllDown=10&connectTimeout=20&deniedListTimeout=20&socketTimeout=10")) {
279279
Statement stmt = con.createStatement();
280280
stmt.execute("START TRANSACTION");
281281
stmt.execute("SET @con=1");

0 commit comments

Comments
 (0)