add some tests

Signed-off-by: Michael Sprauer <[email protected]>

Author: MichaelSp

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/jdbc/DatabaseDriver.java

@@ -16,14 +16,14 @@
 
 package org.springframework.boot.jdbc;
 
+import org.springframework.util.Assert;
+import org.springframework.util.StringUtils;
+
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.Locale;
 
-import org.springframework.util.Assert;
-import org.springframework.util.StringUtils;
-
 /**
  * Enumeration of common database drivers.
  *
@@ -51,11 +51,6 @@ public enum DatabaseDriver {
 	 * H2.
 	 */
 	H2("H2", "org.h2.Driver", "org.h2.jdbcx.JdbcDataSource", "SELECT 1"),
-	
-	/**
-	 * HDB - SAP Hana Database.
-	 */
-	HDB("HDB", "com.sap.db.jdbc.Driver", "com.sap.db.jdbcext.XADataSourceSAP", "SELECT 1 FROM DUMMY"),
 
 	/**
 	 * HyperSQL DataBase.
@@ -104,6 +99,12 @@ public String getId() {
 	POSTGRESQL("PostgreSQL", "org.postgresql.Driver", "org.postgresql.xa.PGXADataSource",
 			"SELECT 1"),
 
+	/**
+	 * SAP - SAP Hana Database - HDB.
+	 */
+	SAP("HDB", "com.sap.db.jdbc.Driver", "com.sap.db.jdbcext.XADataSourceSAP",
+			"SELECT 1 FROM DUMMY"),
+
 	/**
 	 * jTDS. As it can be used for several databases, there isn't a single product name we
 	 * could rely on.

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/jdbc/DatabaseDriverTests.java

@@ -74,6 +74,7 @@ public void databaseProductNameLookups() {
 		assertThat(DatabaseDriver.fromProductName("Apache Derby"))
 				.isEqualTo(DatabaseDriver.DERBY);
 		assertThat(DatabaseDriver.fromProductName("H2")).isEqualTo(DatabaseDriver.H2);
+		assertThat(DatabaseDriver.fromProductName("HDB")).isEqualTo(DatabaseDriver.SAP);
 		assertThat(DatabaseDriver.fromProductName("HSQL Database Engine"))
 				.isEqualTo(DatabaseDriver.HSQLDB);
 		assertThat(DatabaseDriver.fromProductName("SQLite"))
@@ -113,6 +114,8 @@ public void databaseJdbcUrlLookups() {
 				.isEqualTo(DatabaseDriver.DERBY);
 		assertThat(DatabaseDriver.fromJdbcUrl("jdbc:h2:~/sample"))
 				.isEqualTo(DatabaseDriver.H2);
+		assertThat(DatabaseDriver.fromJdbcUrl("jdbc:sap:localhost"))
+				.isEqualTo(DatabaseDriver.SAP);
 		assertThat(DatabaseDriver.fromJdbcUrl("jdbc:hsqldb:hsql://localhost"))
 				.isEqualTo(DatabaseDriver.HSQLDB);
 		assertThat(DatabaseDriver.fromJdbcUrl("jdbc:sqlite:sample.db"))