fixed warnings and added more tests

src/main/java/com/firebolt/jdbc/resultset/FireboltResultSet.java

@@ -956,7 +956,6 @@ public Statement getStatement() {
 	@Override
 	public Object getObject(int columnIndex, Map<String, Class<?>> map) throws SQLException {
 		FireboltDataType dataType = resultSetMetaData.getColumn(columnIndex).getType().getDataType();
-		BaseType columnType = dataType.getBaseType();
 		Map<String, Class<?>> caseInsensitiveMap = new TreeMap<>(CASE_INSENSITIVE_ORDER);
 		caseInsensitiveMap.putAll(map);
 		Class<?> type = getAllNames(dataType).map(caseInsensitiveMap::get).filter(Objects::nonNull).findFirst()
@@ -969,11 +968,7 @@ private Stream<String> getAllNames(FireboltDataType dataType) {
 	}
 
 	private String getJdbcType(FireboltDataType dataType) {
-		try {
-			return JDBCType.valueOf(dataType.getSqlType()).getName();
-		} catch (IllegalArgumentException e) {
-			return null;
-		}
+		return JDBCType.valueOf(dataType.getSqlType()).getName();
 	}
 
 	@Override

src/test/java/com/firebolt/jdbc/resultset/FireboltResultSetTest.java

@@ -1217,6 +1217,7 @@ void shouldGetDouble() throws SQLException {
 		assertEquals(1.23, resultSet.getDouble(3));
 
 		assertEquals(1.23456789012, resultSet.getObject(4, Double.class));
+		assertEquals(1.23456789012, (double)resultSet.getObject(4, Map.of("double precision", Double.class)), 0.01);
 		assertEquals(new BigDecimal("1.23456789012"), resultSet.getBigDecimal(4));
 
 		assertEquals(1231232.123459999990457054844258706536, resultSet.getObject(5, Double.class), 0.01);

src/test/java/com/firebolt/jdbc/type/FireboltDataTypeTest.java

@@ -0,0 +1,22 @@
+package com.firebolt.jdbc.type;
+
+
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.EnumSource;
+
+import java.sql.JDBCType;
+
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+
+class FireboltDataTypeTest {
+    /**
+     * This test validates that {@link FireboltDataType#getSqlType()} return standard type defined in {@link java.sql.Types} and {@link JDBCType}.
+     * @param type - the type
+     */
+    @ParameterizedTest
+    @EnumSource(FireboltDataType.class)
+    void sqlType(FireboltDataType type) {
+        // assert here is just to satisfy static code analysis. valueOf() either succeeds or throws IllegalArgumentException
+        assertNotNull(JDBCType.valueOf(type.getSqlType()));
+    }
+}