(shardingsphere) branch master updated: Add more test cases on MemoryMergedResultTest (#33182)

[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 bdd80e0c0a5 Add more test cases on MemoryMergedResultTest (#33182)
bdd80e0c0a5 is described below

commit bdd80e0c0a5dc9e718609edbb529975cbf08fb09
Author: Liang Zhang <zhangli...@apache.org>
AuthorDate: Wed Oct 9 12:59:43 2024 +0800

    Add more test cases on MemoryMergedResultTest (#33182)
    
    * Refactor LocalDataMergedResultTest
    
    * Add more test cases on LocalDataQueryResultRow
    
    * Add more test cases on MemoryMergedResultTest
---
 .../merge/result/impl/memory/MemoryMergedResultTest.java     | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git 
a/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/result/impl/memory/MemoryMergedResultTest.java
 
b/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/result/impl/memory/MemoryMergedResultTest.java
index 0cef6266c75..c0f36a43716 100644
--- 
a/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/result/impl/memory/MemoryMergedResultTest.java
+++ 
b/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/result/impl/memory/MemoryMergedResultTest.java
@@ -34,6 +34,7 @@ import java.util.Date;
 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.assertNull;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.Mockito.when;
@@ -59,7 +60,10 @@ class MemoryMergedResultTest {
     @Test
     void assertGetValue() throws SQLException {
         when(memoryResultSetRow.getCell(1)).thenReturn("1");
+        when(memoryResultSetRow.getCell(2)).thenReturn(null);
         assertThat(memoryMergedResult.getValue(1, Object.class).toString(), 
is("1"));
+        assertNull(memoryMergedResult.getValue(2, Object.class));
+        assertTrue(memoryMergedResult.wasNull());
     }
     
     @Test
@@ -90,7 +94,10 @@ class MemoryMergedResultTest {
     @Test
     void assertGetCalendarValue() {
         when(memoryResultSetRow.getCell(1)).thenReturn(new Date(0L));
+        when(memoryResultSetRow.getCell(2)).thenReturn(null);
         assertThat(memoryMergedResult.getCalendarValue(1, Object.class, 
Calendar.getInstance()), is(new Date(0L)));
+        assertNull(memoryMergedResult.getCalendarValue(2, Object.class, 
Calendar.getInstance()));
+        assertTrue(memoryMergedResult.wasNull());
     }
     
     @Test
@@ -98,6 +105,11 @@ class MemoryMergedResultTest {
         assertThrows(SQLFeatureNotSupportedException.class, () -> 
memoryMergedResult.getInputStream(1, "ascii"));
     }
     
+    @Test
+    void assertGetCharacterStream() {
+        assertThrows(SQLFeatureNotSupportedException.class, () -> 
memoryMergedResult.getCharacterStream(1));
+    }
+    
     @Test
     void assertWasNull() {
         assertFalse(memoryMergedResult.wasNull());