(shardingsphere) branch master updated: Add more test cases on FirebirdQueryHeaderBuilderTest (#38061)

[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 68da19b9ecd Add more test cases on FirebirdQueryHeaderBuilderTest 
(#38061)
68da19b9ecd is described below

commit 68da19b9ecda1e491c1bfd39e0695e007d723335
Author: Liang Zhang <zhangli...@apache.org>
AuthorDate: Mon Feb 16 18:12:58 2026 +0800

    Add more test cases on FirebirdQueryHeaderBuilderTest (#38061)
---
 .../query/FirebirdQueryHeaderBuilderTest.java      | 26 +++++++++++++++-------
 1 file changed, 18 insertions(+), 8 deletions(-)

diff --git 
a/proxy/backend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/backend/firebird/response/header/query/FirebirdQueryHeaderBuilderTest.java
 
b/proxy/backend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/backend/firebird/response/header/query/FirebirdQueryHeaderBuilderTest.java
index 79427a4241a..785671033b0 100644
--- 
a/proxy/backend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/backend/firebird/response/header/query/FirebirdQueryHeaderBuilderTest.java
+++ 
b/proxy/backend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/backend/firebird/response/header/query/FirebirdQueryHeaderBuilderTest.java
@@ -17,8 +17,12 @@
 
 package org.apache.shardingsphere.proxy.backend.firebird.response.header.query;
 
+import 
org.apache.shardingsphere.database.connector.core.spi.DatabaseTypedSPILoader;
+import org.apache.shardingsphere.database.connector.core.type.DatabaseType;
 import 
org.apache.shardingsphere.infra.executor.sql.execute.result.query.QueryResultMetaData;
+import org.apache.shardingsphere.infra.spi.type.typed.TypedSPILoader;
 import 
org.apache.shardingsphere.proxy.backend.response.header.query.QueryHeader;
+import 
org.apache.shardingsphere.proxy.backend.response.header.query.QueryHeaderBuilder;
 import org.junit.jupiter.api.Test;
 
 import java.sql.SQLException;
@@ -31,18 +35,24 @@ import static org.mockito.Mockito.when;
 
 class FirebirdQueryHeaderBuilderTest {
     
+    private final DatabaseType databaseType = 
TypedSPILoader.getService(DatabaseType.class, "Firebird");
+    
+    private final QueryHeaderBuilder queryHeaderBuilder = 
DatabaseTypedSPILoader.getService(QueryHeaderBuilder.class, databaseType);
+    
     @Test
-    void assertBuildPostgreSQLQueryHeader() throws SQLException {
-        final int columnIndex = 1;
+    void assertBuild() throws SQLException {
         QueryResultMetaData queryResultMetaData = 
mock(QueryResultMetaData.class);
-        
when(queryResultMetaData.getColumnLabel(columnIndex)).thenReturn("label");
-        
when(queryResultMetaData.getColumnType(columnIndex)).thenReturn(Types.INTEGER);
-        
when(queryResultMetaData.getColumnTypeName(columnIndex)).thenReturn("int");
-        when(queryResultMetaData.getColumnLength(columnIndex)).thenReturn(11);
-        QueryHeader actual = new 
FirebirdQueryHeaderBuilder().build(queryResultMetaData, null, null, 
queryResultMetaData.getColumnLabel(columnIndex), columnIndex);
-        assertThat(actual.getColumnLabel(), is("label"));
+        when(queryResultMetaData.getColumnType(1)).thenReturn(Types.INTEGER);
+        when(queryResultMetaData.getColumnTypeName(1)).thenReturn("int");
+        when(queryResultMetaData.getColumnLength(1)).thenReturn(11);
+        QueryHeader actual = queryHeaderBuilder.build(queryResultMetaData, 
null, null, "foo_label", 1);
+        assertThat(actual.getSchema(), is(""));
+        assertThat(actual.getTable(), is(""));
+        assertThat(actual.getColumnLabel(), is("foo_label"));
+        assertThat(actual.getColumnName(), is(""));
         assertThat(actual.getColumnType(), is(Types.INTEGER));
         assertThat(actual.getColumnTypeName(), is("int"));
         assertThat(actual.getColumnLength(), is(11));
+        assertThat(actual.getDecimals(), is(0));
     }
 }