Skip to content

Commit

Permalink
Merge pull request #115 from FOCONIS/use-mariadb-driver-for-mariadb-c…
Browse files Browse the repository at this point in the history
…ontainer

MariaDBContainer should use mariadb driver for health check
  • Loading branch information
rbygrave authored Jan 28, 2025
2 parents af2998b + 4516ca6 commit e2e75f1
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
7 changes: 7 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,13 @@
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.mariadb.jdbc</groupId>
<artifactId>mariadb-java-client</artifactId>
<version>3.5.1</version>
<scope>test</scope>
</dependency>

<dependency>
<groupId>com.sap.cloud.db.jdbc</groupId>
<artifactId>ngdbc</artifactId>
Expand Down
4 changes: 2 additions & 2 deletions src/main/java/io/ebean/test/containers/MariaDBContainer.java
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,12 @@ protected Builder(String platform, int port, int internalPort, String version) {

@Override
protected String buildJdbcUrl() {
return "jdbc:mysql://" + getHost() + ":" + getPort() + "/" + getDbName();
return "jdbc:mariadb://" + getHost() + ":" + getPort() + "/" + getDbName();
}

@Override
protected String buildJdbcAdminUrl() {
return "jdbc:mysql://" + getHost() + ":" + getPort() + "/mysql";
return "jdbc:mariadb://" + getHost() + ":" + getPort() + "/mysql";
}

@Override
Expand Down

0 comments on commit e2e75f1

Please sign in to comment.