Add unit tests for various core classes to improve coverage (#4250)

Added unit tests for:
- com.codename1.io.WebServiceProxyCall$WSConnection
- com.codename1.properties.SQLMap
- com.codename1.ui.animations.BubbleTransition (disabled due to headless env issues)
- com.codename1.ui.animations.AnimationObject
- com.codename1.ui.CustomFont
- com.codename1.ui.animations.Timeline
- com.codename1.ui.SideMenuBar$MenuTransition (disabled due to missing resources)
- com.codename1.properties.InstantUI
- com.codename1.ui.BrowserComponent$JSProxy
- com.codename1.charts.views.DialChart

Refactored to avoid reflection and respect constraints.
Tests for BubbleTransition and SideMenuBar are disabled but present for future enablement.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

Author: google-labs-jules[bot]

maven/core-unittests/src/test/java/com/codename1/charts/views/DialChartTest.java

@@ -0,0 +1,31 @@
+package com.codename1.charts.views;
+
+import com.codename1.junit.FormTest;
+import com.codename1.junit.UITestBase;
+import com.codename1.charts.models.CategorySeries;
+import com.codename1.charts.renderers.DialRenderer;
+import com.codename1.charts.compat.Canvas;
+import com.codename1.charts.compat.Paint;
+import com.codename1.ui.Image;
+import com.codename1.ui.Graphics;
+import org.junit.jupiter.api.Assertions;
+
+public class DialChartTest extends UITestBase {
+
+    @FormTest
+    public void testDialChart() {
+        CategorySeries dataset = new CategorySeries("Dial");
+        dataset.add("Speed", 80);
+
+        DialRenderer renderer = new DialRenderer();
+        DialChart chart = new DialChart(dataset, renderer);
+
+        Image img = Image.createImage(200, 200, 0xFFFFFFFF);
+        Graphics g = img.getGraphics();
+        Canvas canvas = new Canvas();
+        canvas.g = g;
+        Paint paint = new Paint();
+
+        chart.draw(canvas, 0, 0, 200, 200, paint);
+    }
+}

maven/core-unittests/src/test/java/com/codename1/io/WebServiceProxyCallTest.java

@@ -0,0 +1,122 @@
+package com.codename1.io;
+
+import com.codename1.junit.FormTest;
+import com.codename1.junit.UITestBase;
+import com.codename1.util.Callback;
+import org.junit.jupiter.api.Assertions;
+import java.io.ByteArrayInputStream;
+import java.io.ByteArrayOutputStream;
+import java.io.DataOutputStream;
+import java.io.IOException;
+
+public class WebServiceProxyCallTest extends UITestBase {
+
+    @FormTest
+    public void testWSConnectionVoid() throws Exception {
+        WebServiceProxyCall.WSDefinition def = new WebServiceProxyCall.WSDefinition();
+        def.url = "http://example.com/void";
+        def.name = "voidMethod";
+        def.returnType = WebServiceProxyCall.TYPE_VOID;
+        def.arguments = new int[]{};
+
+        WebServiceProxyCall.WSConnection conn = new WebServiceProxyCall.WSConnection(def, null);
+
+        ByteArrayOutputStream baos = new ByteArrayOutputStream();
+        // Void returns nothing
+
+        ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
+        conn.readResponse(bais);
+    }
+
+    @FormTest
+    public void testWSConnectionInt() throws Exception {
+        WebServiceProxyCall.WSDefinition def = new WebServiceProxyCall.WSDefinition();
+        def.url = "http://example.com/int";
+        def.name = "intMethod";
+        def.returnType = WebServiceProxyCall.TYPE_INT;
+        def.arguments = new int[]{WebServiceProxyCall.TYPE_INT};
+
+        WebServiceProxyCall.WSConnection conn = new WebServiceProxyCall.WSConnection(def, null, 123);
+
+        ByteArrayOutputStream baos = new ByteArrayOutputStream();
+        DataOutputStream dos = new DataOutputStream(baos);
+        dos.writeInt(456);
+        dos.flush();
+
+        ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
+        conn.readResponse(bais);
+
+        Assertions.assertEquals(456, conn.returnValue);
+
+        ByteArrayOutputStream requestBody = new ByteArrayOutputStream();
+        conn.buildRequestBody(requestBody);
+
+        ByteArrayInputStream requestInput = new ByteArrayInputStream(requestBody.toByteArray());
+        java.io.DataInputStream dis = new java.io.DataInputStream(requestInput);
+        Assertions.assertEquals("intMethod", dis.readUTF());
+        Assertions.assertEquals(123, dis.readInt());
+    }
+
+    @FormTest
+    public void testWSConnectionVariousTypes() throws Exception {
+        WebServiceProxyCall.WSDefinition def = new WebServiceProxyCall.WSDefinition();
+        def.url = "http://example.com/all";
+        def.name = "allMethod";
+        def.returnType = WebServiceProxyCall.TYPE_STRING;
+        def.arguments = new int[]{
+            WebServiceProxyCall.TYPE_BOOLEAN,
+            WebServiceProxyCall.TYPE_STRING,
+            WebServiceProxyCall.TYPE_STRING_ARRAY
+        };
+
+        WebServiceProxyCall.WSConnection conn = new WebServiceProxyCall.WSConnection(def, null, true, "hello", new String[]{"a", "b"});
+
+        ByteArrayOutputStream baos = new ByteArrayOutputStream();
+        DataOutputStream dos = new DataOutputStream(baos);
+        dos.writeBoolean(true); // String presence
+        dos.writeUTF("result");
+        dos.flush();
+
+        ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
+        conn.readResponse(bais);
+
+        Assertions.assertEquals("result", conn.returnValue);
+
+        ByteArrayOutputStream requestBody = new ByteArrayOutputStream();
+        conn.buildRequestBody(requestBody);
+
+        ByteArrayInputStream requestInput = new ByteArrayInputStream(requestBody.toByteArray());
+        java.io.DataInputStream dis = new java.io.DataInputStream(requestInput);
+        Assertions.assertEquals("allMethod", dis.readUTF());
+        Assertions.assertTrue(dis.readBoolean()); // boolean arg
+        Assertions.assertTrue(dis.readBoolean()); // string arg present
+        Assertions.assertEquals("hello", dis.readUTF());
+        Assertions.assertEquals(2, dis.readInt()); // string array length
+        Assertions.assertEquals("a", dis.readUTF());
+        Assertions.assertEquals("b", dis.readUTF());
+    }
+
+    @FormTest
+    public void testWSConnectionArrays() throws Exception {
+         WebServiceProxyCall.WSDefinition def = new WebServiceProxyCall.WSDefinition();
+        def.url = "http://example.com/arrays";
+        def.name = "arraysMethod";
+        def.returnType = WebServiceProxyCall.TYPE_INT_ARRAY;
+        def.arguments = new int[]{};
+
+        WebServiceProxyCall.WSConnection conn = new WebServiceProxyCall.WSConnection(def, null);
+
+        ByteArrayOutputStream baos = new ByteArrayOutputStream();
+        DataOutputStream dos = new DataOutputStream(baos);
+        dos.writeInt(2); // Array length
+        dos.writeInt(10);
+        dos.writeInt(20);
+        dos.flush();
+
+        ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
+        conn.readResponse(bais);
+
+        int[] result = (int[]) conn.returnValue;
+        Assertions.assertArrayEquals(new int[]{10, 20}, result);
+    }
+}

maven/core-unittests/src/test/java/com/codename1/properties/InstantUITest.java

@@ -0,0 +1,36 @@
+package com.codename1.properties;
+
+import com.codename1.junit.FormTest;
+import com.codename1.junit.UITestBase;
+import com.codename1.ui.Container;
+import org.junit.jupiter.api.Assertions;
+
+public class InstantUITest extends UITestBase {
+
+    public static class MyData implements PropertyBusinessObject {
+        public final Property<String, MyData> name = new Property<>("name", String.class);
+        public final Property<Boolean, MyData> active = new Property<>("active", Boolean.class);
+        public final PropertyIndex idx = new PropertyIndex(this, "MyData", name, active);
+
+        @Override
+        public PropertyIndex getPropertyIndex() {
+            return idx;
+        }
+    }
+
+    @FormTest
+    public void testCreateEditUI() {
+        MyData data = new MyData();
+        data.name.set("Test");
+        data.active.set(true);
+
+        InstantUI iui = new InstantUI();
+        Container ui = iui.createEditUI(data, true);
+
+        Assertions.assertNotNull(ui);
+        Assertions.assertTrue(ui.getComponentCount() > 0);
+
+        UiBinding.Binding binding = iui.getBindings(ui);
+        Assertions.assertNotNull(binding);
+    }
+}

maven/core-unittests/src/test/java/com/codename1/properties/SQLMapTest.java

@@ -0,0 +1,113 @@
+package com.codename1.properties;
+
+import com.codename1.junit.FormTest;
+import com.codename1.junit.UITestBase;
+import com.codename1.testing.TestCodenameOneImplementation;
+import com.codename1.db.Database;
+import org.junit.jupiter.api.Assertions;
+import java.util.List;
+
+public class SQLMapTest extends UITestBase {
+
+    public static class MyData implements PropertyBusinessObject {
+        public final Property<String, MyData> name = new Property<>("name", String.class);
+        public final Property<Integer, MyData> age = new Property<>("age", Integer.class);
+        public final PropertyIndex idx = new PropertyIndex(this, "MyData", name, age);
+
+        @Override
+        public PropertyIndex getPropertyIndex() {
+            return idx;
+        }
+    }
+
+    @FormTest
+    public void testCreateTable() throws Exception {
+        Database db = TestCodenameOneImplementation.getInstance().openOrCreateDB("test.db");
+        SQLMap map = SQLMap.create(db);
+        MyData data = new MyData();
+
+        map.setPrimaryKey(data, data.name);
+
+        // Mock query result for table existence check (returns false initially)
+        ((TestCodenameOneImplementation.TestDatabase)db).setQueryResult(new String[]{"type", "name"}, new Object[][]{});
+
+        boolean created = map.createTable(data);
+        Assertions.assertTrue(created);
+
+        List<String> statements = ((TestCodenameOneImplementation.TestDatabase)db).getExecutedStatements();
+        boolean foundCreate = false;
+        for(String s : statements) {
+            if (s.contains("CREATE TABLE")) {
+                foundCreate = true;
+                break;
+            }
+        }
+        Assertions.assertTrue(foundCreate);
+    }
+
+    @FormTest
+    public void testInsert() throws Exception {
+        Database db = TestCodenameOneImplementation.getInstance().openOrCreateDB("test.db");
+        SQLMap map = SQLMap.create(db);
+        MyData data = new MyData();
+        data.name.set("John");
+        data.age.set(30);
+
+        map.insert(data);
+
+        List<String> statements = ((TestCodenameOneImplementation.TestDatabase)db).getExecutedStatements();
+        boolean foundInsert = false;
+        for(String s : statements) {
+            if (s.contains("INSERT INTO")) {
+                foundInsert = true;
+                break;
+            }
+        }
+        Assertions.assertTrue(foundInsert);
+
+        List<String[]> params = ((TestCodenameOneImplementation.TestDatabase)db).getExecutedParameters();
+    }
+
+    @FormTest
+    public void testSelect() throws Exception {
+        Database db = TestCodenameOneImplementation.getInstance().openOrCreateDB("test.db");
+        SQLMap map = SQLMap.create(db);
+        MyData data = new MyData();
+
+        // Mock result
+        ((TestCodenameOneImplementation.TestDatabase)db).setQueryResult(
+            new String[]{"name", "age"},
+            new Object[][]{
+                {"John", 30},
+                {"Jane", 25}
+            }
+        );
+
+        List<PropertyBusinessObject> results = map.select(data, null, true, 0, 0);
+        Assertions.assertEquals(2, results.size());
+        MyData res1 = (MyData) results.get(0);
+        Assertions.assertEquals("John", res1.name.get());
+        Assertions.assertEquals(30, res1.age.get().intValue());
+    }
+
+    @FormTest
+    public void testDelete() throws Exception {
+        Database db = TestCodenameOneImplementation.getInstance().openOrCreateDB("test.db");
+        SQLMap map = SQLMap.create(db);
+        MyData data = new MyData();
+        map.setPrimaryKey(data, data.name);
+        data.name.set("John");
+
+        map.delete(data);
+
+        List<String> statements = ((TestCodenameOneImplementation.TestDatabase)db).getExecutedStatements();
+        boolean foundDelete = false;
+        for(String s : statements) {
+            if (s.contains("DELETE FROM")) {
+                foundDelete = true;
+                break;
+            }
+        }
+        Assertions.assertTrue(foundDelete);
+    }
+}

maven/core-unittests/src/test/java/com/codename1/ui/BrowserComponentJSProxyTest.java

@@ -0,0 +1,32 @@
+package com.codename1.ui;
+
+import com.codename1.junit.FormTest;
+import com.codename1.junit.UITestBase;
+import com.codename1.testing.TestCodenameOneImplementation;
+import org.junit.jupiter.api.Assertions;
+
+public class BrowserComponentJSProxyTest extends UITestBase {
+
+    public static class TestPeer extends PeerComponent {
+        public TestPeer() {
+            super(new Object());
+        }
+    }
+
+    @FormTest
+    public void testJSProxy() {
+        implementation.setBrowserComponent(new TestPeer());
+        BrowserComponent bc = new BrowserComponent();
+        BrowserComponent.JSProxy proxy = bc.createJSProxy("window");
+
+        proxy.call("alert", new Object[]{"Hello"}, null);
+
+        flushSerialCalls();
+
+        java.util.List<String> executed = implementation.getBrowserExecuted();
+        Assertions.assertFalse(executed.isEmpty());
+        String lastScript = executed.get(executed.size() - 1);
+        Assertions.assertTrue(lastScript.contains("window.alert"));
+        Assertions.assertTrue(lastScript.contains("Hello"));
+    }
+}

maven/core-unittests/src/test/java/com/codename1/ui/CustomFontTest.java

@@ -0,0 +1,30 @@
+package com.codename1.ui;
+
+import com.codename1.junit.FormTest;
+import com.codename1.junit.UITestBase;
+import org.junit.jupiter.api.Assertions;
+
+public class CustomFontTest extends UITestBase {
+
+    @FormTest
+    public void testCustomFont() {
+        // Create a dummy bitmap for the font
+        // Assume 2 chars 'A' and 'B', each 10x10 pixels
+        Image bitmap = Image.createImage(20, 10, 0xFF000000);
+        int[] cutOffsets = new int[]{0, 10};
+        int[] charWidth = new int[]{10, 10};
+        String charsets = "AB";
+
+        CustomFont font = new CustomFont(bitmap, cutOffsets, charWidth, charsets);
+
+        Assertions.assertEquals(10, font.charWidth('A'));
+        Assertions.assertEquals(10, font.charWidth('B'));
+        Assertions.assertEquals(0, font.charWidth('C')); // Not in charset
+
+        Assertions.assertEquals(10, font.getHeight());
+        Assertions.assertEquals("AB", font.getCharset());
+
+        Assertions.assertEquals(20, font.stringWidth("AB"));
+        Assertions.assertEquals(10, font.substringWidth("ABC", 0, 1));
+    }
+}