(shardingsphere) branch master updated: Add more test cases on JDBCQueryResultMetaData (#33217)

[zhangliang]

This is an automated email from the ASF dual-hosted git repository.

zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 867b3da43f5 Add more test cases on JDBCQueryResultMetaData (#33217)
867b3da43f5 is described below

commit 867b3da43f5b420c2c71a84332e6b043c7fa513d
Author: Liang Zhang <zhangli...@apache.org>
AuthorDate: Fri Oct 11 19:30:35 2024 +0800

    Add more test cases on JDBCQueryResultMetaData (#33217)
---
 .../jdbc/metadata/JDBCQueryResultMetaDataTest.java | 23 ++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git 
a/infra/executor/src/test/java/org/apache/shardingsphere/infra/executor/sql/execute/result/query/impl/driver/jdbc/metadata/JDBCQueryResultMetaDataTest.java
 
b/infra/executor/src/test/java/org/apache/shardingsphere/infra/executor/sql/execute/result/query/impl/driver/jdbc/metadata/JDBCQueryResultMetaDataTest.java
index 7caa2f20cd8..0e2a25fc453 100644
--- 
a/infra/executor/src/test/java/org/apache/shardingsphere/infra/executor/sql/execute/result/query/impl/driver/jdbc/metadata/JDBCQueryResultMetaDataTest.java
+++ 
b/infra/executor/src/test/java/org/apache/shardingsphere/infra/executor/sql/execute/result/query/impl/driver/jdbc/metadata/JDBCQueryResultMetaDataTest.java
@@ -22,10 +22,12 @@ import org.junit.jupiter.api.Test;
 
 import java.sql.ResultSetMetaData;
 import java.sql.SQLException;
+import java.sql.SQLFeatureNotSupportedException;
 import java.sql.Types;
 
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.MatcherAssert.assertThat;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
@@ -80,6 +82,13 @@ class JDBCQueryResultMetaDataTest {
         assertThat(queryResultMetaData.getTableName(1), is("order"));
     }
     
+    @Test
+    void assertGetTableNameWithException() throws SQLException {
+        ResultSetMetaData resultSetMetaData = mock(ResultSetMetaData.class);
+        when(resultSetMetaData.getTableName(1)).thenThrow(new 
SQLFeatureNotSupportedException());
+        assertThat(new 
JDBCQueryResultMetaData(resultSetMetaData).getTableName(1), is(""));
+    }
+    
     @Test
     void assertGetColumnType() throws SQLException {
         assertThat(queryResultMetaData.getColumnType(1), is(Types.INTEGER));
@@ -100,11 +109,25 @@ class JDBCQueryResultMetaDataTest {
         assertTrue(queryResultMetaData.isSigned(1));
     }
     
+    @Test
+    void assertIsSignedWithException() throws SQLException {
+        ResultSetMetaData resultSetMetaData = mock(ResultSetMetaData.class);
+        when(resultSetMetaData.isSigned(1)).thenThrow(new 
SQLFeatureNotSupportedException());
+        assertFalse(new 
JDBCQueryResultMetaData(resultSetMetaData).isSigned(1));
+    }
+    
     @Test
     void assertIsNotNull() throws SQLException {
         assertTrue(queryResultMetaData.isNotNull(1));
     }
     
+    @Test
+    void assertIsNull() throws SQLException {
+        ResultSetMetaData resultSetMetaData = mock(ResultSetMetaData.class);
+        
when(resultSetMetaData.isNullable(1)).thenReturn(ResultSetMetaData.columnNullableUnknown);
+        assertFalse(new 
JDBCQueryResultMetaData(resultSetMetaData).isNotNull(1));
+    }
+    
     @Test
     void assertIsAutoIncrement() throws SQLException {
         assertTrue(queryResultMetaData.isAutoIncrement(1));