Add comprehensive tests for record support to improve code coverage

Co-authored-by: ozlerhakan <[email protected]>

Author: Copilot

src/test/java/com/poiji/deserialize/RecordDeserializationTest.java

@@ -110,4 +110,184 @@ public void shouldMapExcelToRecordWithExcelCell() {
             assertThat(calculation.toDate().toString(), is("2018-06-30"));
         }
     }
+
+    @Test
+    public void shouldMapExcelToRecordFromInputStream() {
+        // Test with InputStream
+        try (java.io.FileInputStream stream = new java.io.FileInputStream(
+                new File("src/test/resources/employees.xlsx"))) {
+            List<EmployeeRecord> employees = Poiji.fromExcel(
+                    stream,
+                    com.poiji.exception.PoijiExcelType.XLSX,
+                    EmployeeRecord.class
+            );
+
+            assertThat(employees, notNullValue());
+            assertThat(employees.size(), is(3));
+
+            EmployeeRecord firstEmployee = employees.get(0);
+            assertThat(firstEmployee.employeeId(), is(123923L));
+            assertThat(firstEmployee.name(), is("Joe"));
+        } catch (Exception e) {
+            throw new RuntimeException(e);
+        }
+    }
+
+    @Test
+    public void shouldMapExcelToRecordFromInputStreamWithOptions() {
+        // Test with InputStream and options
+        PoijiOptions options = PoijiOptions.PoijiOptionsBuilder.settings()
+                .headerStart(0)
+                .build();
+
+        try (java.io.FileInputStream stream = new java.io.FileInputStream(
+                new File("src/test/resources/employees.xlsx"))) {
+            List<EmployeeRecord> employees = Poiji.fromExcel(
+                    stream,
+                    com.poiji.exception.PoijiExcelType.XLSX,
+                    EmployeeRecord.class,
+                    options
+            );
+
+            assertThat(employees, notNullValue());
+            assertThat(employees.size(), is(3));
+        } catch (Exception e) {
+            throw new RuntimeException(e);
+        }
+    }
+
+    @Test
+    public void shouldMapXLSFromInputStream() {
+        // Test XLS with InputStream
+        try (java.io.FileInputStream stream = new java.io.FileInputStream(
+                new File("src/test/resources/employees.xls"))) {
+            List<EmployeeRecord> employees = Poiji.fromExcel(
+                    stream,
+                    com.poiji.exception.PoijiExcelType.XLS,
+                    EmployeeRecord.class
+            );
+
+            assertThat(employees, notNullValue());
+            assertThat(employees.size(), is(3));
+
+            EmployeeRecord firstEmployee = employees.get(0);
+            assertThat(firstEmployee.employeeId(), is(123923L));
+        } catch (Exception e) {
+            throw new RuntimeException(e);
+        }
+    }
+
+    @Test
+    public void shouldHandleRecordWithConsumer() {
+        // Test with consumer interface
+        final java.util.List<EmployeeRecord> collectedEmployees = new java.util.ArrayList<>();
+        
+        Poiji.fromExcel(
+                new File("src/test/resources/employees.xlsx"),
+                EmployeeRecord.class,
+                collectedEmployees::add
+        );
+
+        assertThat(collectedEmployees.size(), is(3));
+        assertThat(collectedEmployees.get(0).name(), is("Joe"));
+    }
+
+    @Test
+    public void shouldHandleRecordWithConsumerAndOptions() {
+        // Test with consumer interface and options
+        PoijiOptions options = PoijiOptions.PoijiOptionsBuilder.settings()
+                .headerStart(0)
+                .build();
+
+        final java.util.List<EmployeeRecord> collectedEmployees = new java.util.ArrayList<>();
+        
+        Poiji.fromExcel(
+                new File("src/test/resources/employees.xlsx"),
+                EmployeeRecord.class,
+                options,
+                collectedEmployees::add
+        );
+
+        assertThat(collectedEmployees.size(), is(3));
+    }
+
+    @Test
+    public void shouldMapExcelToRecordFromSheet() {
+        // Test with Sheet API
+        try {
+            org.apache.poi.ss.usermodel.Workbook workbook = org.apache.poi.ss.usermodel.WorkbookFactory.create(
+                    new File("src/test/resources/employees.xlsx"));
+            org.apache.poi.ss.usermodel.Sheet sheet = workbook.getSheetAt(0);
+
+            List<EmployeeRecord> employees = Poiji.fromExcel(
+                    sheet,
+                    EmployeeRecord.class
+            );
+
+            assertThat(employees, notNullValue());
+            assertThat(employees.size(), is(3));
+            assertThat(employees.get(0).name(), is("Joe"));
+
+            workbook.close();
+        } catch (Exception e) {
+            throw new RuntimeException(e);
+        }
+    }
+
+    @Test
+    public void shouldMapExcelToRecordFromSheetWithOptions() {
+        // Test with Sheet API and options
+        PoijiOptions options = PoijiOptions.PoijiOptionsBuilder.settings()
+                .headerStart(0)
+                .build();
+
+        try {
+            org.apache.poi.ss.usermodel.Workbook workbook = org.apache.poi.ss.usermodel.WorkbookFactory.create(
+                    new File("src/test/resources/employees.xlsx"));
+            org.apache.poi.ss.usermodel.Sheet sheet = workbook.getSheetAt(0);
+
+            List<EmployeeRecord> employees = Poiji.fromExcel(
+                    sheet,
+                    EmployeeRecord.class,
+                    options
+            );
+
+            assertThat(employees, notNullValue());
+            assertThat(employees.size(), is(3));
+
+            workbook.close();
+        } catch (Exception e) {
+            throw new RuntimeException(e);
+        }
+    }
+
+    @Test
+    public void shouldMapExcelToRecordFromSheetWithConsumer() {
+        // Test with Sheet API and consumer
+        PoijiOptions options = PoijiOptions.PoijiOptionsBuilder.settings()
+                .headerStart(0)
+                .build();
+
+        final java.util.List<EmployeeRecord> collectedEmployees = new java.util.ArrayList<>();
+
+        try {
+            org.apache.poi.ss.usermodel.Workbook workbook = org.apache.poi.ss.usermodel.WorkbookFactory.create(
+                    new File("src/test/resources/employees.xlsx"));
+            org.apache.poi.ss.usermodel.Sheet sheet = workbook.getSheetAt(0);
+
+            Poiji.fromExcel(
+                    sheet,
+                    EmployeeRecord.class,
+                    options,
+                    collectedEmployees::add
+            );
+
+            assertThat(collectedEmployees.size(), is(3));
+            assertThat(collectedEmployees.get(0).name(), is("Joe"));
+
+            workbook.close();
+        } catch (Exception e) {
+            throw new RuntimeException(e);
+        }
+    }
 }

src/test/java/com/poiji/util/ReflectUtilRecordTest.java

@@ -0,0 +1,63 @@
+package com.poiji.util;
+
+import com.poiji.exception.PoijiInstantiationException;
+import org.junit.Test;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.fail;
+
+/**
+ * Tests for ReflectUtil record support
+ */
+public class ReflectUtilRecordTest {
+
+    // Simple POJO for testing
+    public static class SimplePojo {
+        private String name;
+        
+        public SimplePojo() {}
+        
+        public String getName() {
+            return name;
+        }
+        
+        public void setName(String name) {
+            this.name = name;
+        }
+    }
+
+    @Test
+    public void shouldReturnFalseForNonRecordClass() {
+        // Test that isRecord returns false for regular classes
+        assertFalse(ReflectUtil.isRecord(SimplePojo.class));
+        assertFalse(ReflectUtil.isRecord(String.class));
+        assertFalse(ReflectUtil.isRecord(Integer.class));
+    }
+
+    @Test
+    public void shouldThrowExceptionWhenCreatingRecordInstanceFromNonRecord() {
+        // Test that newRecordInstance throws exception for non-record types
+        Map<String, Object> values = new HashMap<>();
+        values.put("name", "test");
+        
+        try {
+            ReflectUtil.newRecordInstance(SimplePojo.class, values);
+            fail("Expected PoijiInstantiationException");
+        } catch (PoijiInstantiationException e) {
+            // Expected
+            assertThat(e.getMessage().contains("is not a record"), is(true));
+        }
+    }
+
+    @Test
+    public void shouldCreatePojoInstanceWithNewInstanceOf() {
+        // Test that regular class instantiation still works
+        SimplePojo pojo = ReflectUtil.newInstanceOf(SimplePojo.class);
+        assertThat(pojo != null, is(true));
+    }
+}