Add comprehensive UI component unit tests (#4110)

* Add comprehensive UI component unit tests

* Fix search and gradient unit test compilation issues

* Fix failing UI component unit tests

* Stabilize UI unit tests under form harness

* Stabilize SearchBar and InfiniteContainer unit tests

* Harden UI unit tests for deterministic expectations

* Align SearchBar and TextSelection tests with runtime behavior

* Adjust SearchBar test expectations

Author: shai-almog

maven/core-unittests/src/test/java/com/codename1/testing/TestCodenameOneImplementation.java

@@ -8,11 +8,13 @@
 import com.codename1.location.LocationManager;
 import com.codename1.media.Media;
 import com.codename1.media.MediaRecorderBuilder;
+import com.codename1.ui.Button;
 import com.codename1.ui.Display;
 import com.codename1.ui.PeerComponent;
 import com.codename1.ui.Stroke;
 import com.codename1.ui.TextArea;
 import com.codename1.ui.TextField;
+import com.codename1.ui.TextSelection;
 import com.codename1.ui.events.ActionListener;
 import com.codename1.ui.geom.Dimension;
 import com.codename1.ui.util.ImageIO;
@@ -99,13 +101,63 @@ public class TestCodenameOneImplementation extends CodenameOneImplementation {
     private boolean nativeBrowserWindowShowInvoked;
     private boolean nativeBrowserWindowCleanupInvoked;
     private boolean nativeBrowserWindowHideInvoked;
+    private boolean nativeImageCacheSupported;
+    private int initializeTextSelectionCount;
+    private int deinitializeTextSelectionCount;
+    private TextSelection lastInitializedTextSelection;
+    private TextSelection lastDeinitializedTextSelection;
+    private final Map<Object, HeavyButtonPeerState> heavyButtonPeers = new HashMap<Object, HeavyButtonPeerState>();
+    private boolean requiresHeavyButton;
 
 
     public TestCodenameOneImplementation() {
         this(true);
         instance = this;
     }
 
+    public static class HeavyButtonPeerState {
+        private final List<ActionListener> listeners = new ArrayList<ActionListener>();
+        private int x;
+        private int y;
+        private int width;
+        private int height;
+        private boolean initCalled;
+        private boolean deinitCalled;
+        private int updateCount;
+
+        public List<ActionListener> getListeners() {
+            return listeners;
+        }
+
+        public int getX() {
+            return x;
+        }
+
+        public int getY() {
+            return y;
+        }
+
+        public int getWidth() {
+            return width;
+        }
+
+        public int getHeight() {
+            return height;
+        }
+
+        public boolean isInitCalled() {
+            return initCalled;
+        }
+
+        public boolean isDeinitCalled() {
+            return deinitCalled;
+        }
+
+        public int getUpdateCount() {
+            return updateCount;
+        }
+    }
+
     @Override
     public InputStream getResourceAsStream(Class cls, String resource) {
         return resourceAsStreams.get(resource);
@@ -115,6 +167,45 @@ public void putResource(String s, InputStream i) {
         resourceAsStreams.put(s, i);
     }
 
+    public void setSupportsNativeImageCache(boolean supported) {
+        nativeImageCacheSupported = supported;
+    }
+
+    public void resetTextSelectionTracking() {
+        initializeTextSelectionCount = 0;
+        deinitializeTextSelectionCount = 0;
+        lastInitializedTextSelection = null;
+        lastDeinitializedTextSelection = null;
+    }
+
+    public int getInitializeTextSelectionCount() {
+        return initializeTextSelectionCount;
+    }
+
+    public int getDeinitializeTextSelectionCount() {
+        return deinitializeTextSelectionCount;
+    }
+
+    public TextSelection getLastInitializedTextSelection() {
+        return lastInitializedTextSelection;
+    }
+
+    public TextSelection getLastDeinitializedTextSelection() {
+        return lastDeinitializedTextSelection;
+    }
+
+    public void setRequiresHeavyButton(boolean requiresHeavyButton) {
+        this.requiresHeavyButton = requiresHeavyButton;
+    }
+
+    public void resetHeavyButtonTracking() {
+        heavyButtonPeers.clear();
+    }
+
+    public HeavyButtonPeerState getHeavyButtonPeerState(Object peer) {
+        return heavyButtonPeers.get(peer);
+    }
+
     @Override
     public boolean isNativeTitle() {
         return nativeTitle;
@@ -133,6 +224,11 @@ public void setMutableImagesFast(boolean mutableImagesFast) {
         this.mutableImagesFast = mutableImagesFast;
     }
 
+    @Override
+    public boolean supportsNativeImageCache() {
+        return nativeImageCacheSupported;
+    }
+
     @Override
     public AsyncResource<Media> createBackgroundMediaAsync(String uri) {
         return backgroundMediaAsync;
@@ -174,6 +270,18 @@ public void setMediaAsync(AsyncResource<Media> mediaAsync) {
         this.mediaAsync = mediaAsync;
     }
 
+    @Override
+    public void initializeTextSelection(TextSelection aThis) {
+        initializeTextSelectionCount++;
+        lastInitializedTextSelection = aThis;
+    }
+
+    @Override
+    public void deinitializeTextSelection(TextSelection aThis) {
+        deinitializeTextSelectionCount++;
+        lastDeinitializedTextSelection = aThis;
+    }
+
     @Override
     public void startRemoteControl() {
         startRemoteControlInvocations++;
@@ -184,6 +292,62 @@ public void stopRemoteControl() {
         stopRemoteControlInvocations++;
     }
 
+    @Override
+    public Object createHeavyButton(Button aThis) {
+        HeavyButtonPeerState state = new HeavyButtonPeerState();
+        heavyButtonPeers.put(state, state);
+        return state;
+    }
+
+    @Override
+    public void addHeavyActionListener(Object peer, ActionListener l) {
+        HeavyButtonPeerState state = heavyButtonPeers.get(peer);
+        if (state != null) {
+            state.getListeners().add(l);
+        }
+    }
+
+    @Override
+    public void removeHeavyActionListener(Object peer, ActionListener l) {
+        HeavyButtonPeerState state = heavyButtonPeers.get(peer);
+        if (state != null) {
+            state.getListeners().remove(l);
+        }
+    }
+
+    @Override
+    public void updateHeavyButtonBounds(Object peer, int x, int y, int width, int height) {
+        HeavyButtonPeerState state = heavyButtonPeers.get(peer);
+        if (state != null) {
+            state.x = x;
+            state.y = y;
+            state.width = width;
+            state.height = height;
+            state.updateCount++;
+        }
+    }
+
+    @Override
+    public void initHeavyButton(Object peer) {
+        HeavyButtonPeerState state = heavyButtonPeers.get(peer);
+        if (state != null) {
+            state.initCalled = true;
+        }
+    }
+
+    @Override
+    public void deinitializeHeavyButton(Object peer) {
+        HeavyButtonPeerState state = heavyButtonPeers.get(peer);
+        if (state != null) {
+            state.deinitCalled = true;
+        }
+    }
+
+    @Override
+    public boolean requiresHeavyButtonForCopyToClipboard() {
+        return requiresHeavyButton;
+    }
+
     public int getStartRemoteControlInvocations() {
         return startRemoteControlInvocations;
     }
@@ -523,6 +687,18 @@ public void setTransformScale(Object transform, float scaleX, float scaleY, floa
         ((TestTransform) transform).setScale(scaleX, scaleY, scaleZ);
     }
 
+    @Override
+    public Object makeTransformRotation(float angle, float x, float y, float z) {
+        TestTransform transform = new TestTransform();
+        transform.setRotation(angle, x, y);
+        return transform;
+    }
+
+    @Override
+    public void setTransformRotation(Object transform, float angle, float x, float y, float z) {
+        ((TestTransform) transform).setRotation(angle, x, y);
+    }
+
     @Override
     public Object makeTransformAffine(double m00, double m10, double m01, double m11, double m02, double m12) {
         TestTransform transform = new TestTransform();
@@ -1169,6 +1345,10 @@ public String getPlatformName() {
 
     @Override
     public L10NManager getLocalizationManager() {
+        if (localizationManager == null) {
+            localizationManager = new L10NManager("en", "US") {
+            };
+        }
         return localizationManager;
     }
 
@@ -1380,6 +1560,18 @@ void setScale(float sx, float sy, float sz) {
             m22 = sz;
         }
 
+        void setRotation(float angle, float px, float py) {
+            setIdentity();
+            float cos = (float) Math.cos(angle);
+            float sin = (float) Math.sin(angle);
+            m00 = cos;
+            m01 = -sin;
+            m10 = sin;
+            m11 = cos;
+            m02 = px - px * cos + py * sin;
+            m12 = py - px * sin - py * cos;
+        }
+
         void setAffine(float nm00, float nm01, float nm02, float nm10, float nm11, float nm12) {
             m00 = nm00;
             m01 = nm01;

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

@@ -0,0 +1,65 @@
+package com.codename1.ui;
+
+import com.codename1.junit.FormTest;
+import com.codename1.junit.UITestBase;
+import com.codename1.ui.Button;
+import com.codename1.ui.Label;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+class ComponentGroupTest extends UITestBase {
+
+    @FormTest
+    void testForceGroupAssignsUiids() {
+        ComponentGroup group = new ComponentGroup();
+        group.setForceGroup(true);
+        Button first = new Button("First");
+        Button middle = new Button("Middle");
+        Button last = new Button("Last");
+        group.add(first);
+        group.add(middle);
+        group.add(last);
+
+        group.refreshTheme(true);
+        assertEquals("ButtonGroupFirst", first.getUIID());
+        assertEquals("ButtonGroup", middle.getUIID());
+        assertEquals("ButtonGroupLast", last.getUIID());
+
+        group.removeComponent(middle);
+        group.refreshTheme(true);
+        assertEquals("ButtonGroupFirst", first.getUIID());
+        assertEquals("ButtonGroupLast", last.getUIID());
+        assertEquals("Button", middle.getUIID());
+    }
+
+    @FormTest
+    void testHorizontalModeUpdatesUiids() {
+        ComponentGroup group = new ComponentGroup();
+        group.setForceGroup(true);
+        group.setHorizontal(true);
+        Button first = new Button("One");
+        Button second = new Button("Two");
+        group.add(first);
+        group.add(second);
+
+        group.refreshTheme(true);
+        assertTrue(group.isHorizontal());
+        assertEquals("ToggleButtonFirst", first.getUIID());
+        assertEquals("ToggleButtonLast", second.getUIID());
+
+        group.setHorizontal(false);
+        group.refreshTheme(true);
+        assertFalse(group.isHorizontal());
+    }
+
+    @FormTest
+    void testStaticEncloseHelpers() {
+        Label l1 = new Label("A");
+        Label l2 = new Label("B");
+        ComponentGroup vertical = ComponentGroup.enclose(l1, l2);
+        assertFalse(vertical.isHorizontal());
+
+        ComponentGroup horizontal = ComponentGroup.encloseHorizontal(new Label("X"), new Label("Y"));
+        assertTrue(horizontal.isHorizontal());
+    }
+}