Add graphics pipeline screenshot test (#4142)

* Add graphics pipeline screenshot test

* Add comprehensive graphics method screenshot tests

* Fix graphics screenshot test dependencies

* Fix LinearGradientPaint construction in screenshot test

* Add per-method graphics screenshots and trigger iOS runs

* Fix graphics screenshot compatibility and PR workflow triggers

* Remove reflection in graphics methods screenshot test

Author: shai-almog

.github/workflows/pr.yml

@@ -5,6 +5,8 @@ on:
     branches:
       - master
     paths-ignore:
+      - '.github/workflows/scripts-android.yml'
+      - '.github/workflows/scripts-ios.yml'
       - 'scripts/**'
       - 'docs/**'
       - '**/*.md'
@@ -14,6 +16,8 @@ on:
     branches:
       - master
     paths-ignore:
+      - '.github/workflows/scripts-android.yml'
+      - '.github/workflows/scripts-ios.yml'
       - 'scripts/**'
       - 'docs/**'
       - '**/*.md'

.github/workflows/scripts-ios.yml

@@ -8,6 +8,7 @@ on:
       - 'scripts/build-ios-port.sh'
       - 'scripts/build-ios-app.sh'
       - 'scripts/run-ios-ui-tests.sh'
+      - 'scripts/device-runner-app/**/*.java'
       - 'scripts/ios/tests/**'
       - 'scripts/ios/screenshots/**'
       - 'scripts/templates/**'
@@ -29,6 +30,7 @@ on:
       - 'scripts/build-ios-port.sh'
       - 'scripts/build-ios-app.sh'
       - 'scripts/run-ios-ui-tests.sh'
+      - 'scripts/device-runner-app/**/*.java'
       - 'scripts/ios/tests/**'
       - 'scripts/ios/screenshots/**'
       - 'scripts/templates/**'

scripts/device-runner-app/tests/AbstractGraphicsScreenshotTest.java

@@ -0,0 +1,35 @@
+package com.codenameone.examples.hellocodenameone.tests;
+
+import com.codename1.testing.AbstractTest;
+import com.codename1.ui.Component;
+import com.codename1.ui.Form;
+import com.codename1.ui.layouts.BorderLayout;
+
+abstract class AbstractGraphicsScreenshotTest extends AbstractTest {
+    protected abstract Component createContent();
+
+    protected abstract String screenshotName();
+
+    @Override
+    public boolean runTest() throws Exception {
+        final Form[] holder = new Form[1];
+        Cn1ssDeviceRunnerHelper.runOnEdtSync(() -> {
+            Form form = new Form("Graphics", new BorderLayout());
+            form.add(BorderLayout.CENTER, createContent());
+            holder[0] = form;
+            form.show();
+        });
+
+        Cn1ssDeviceRunnerHelper.waitForMillis(1200);
+
+        final boolean[] result = new boolean[1];
+        Cn1ssDeviceRunnerHelper.runOnEdtSync(() -> {
+            if (holder[0] != null) {
+                holder[0].revalidate();
+                holder[0].repaint();
+            }
+            result[0] = Cn1ssDeviceRunnerHelper.emitCurrentFormScreenshot(screenshotName());
+        });
+        return result[0];
+    }
+}

scripts/device-runner-app/tests/Cn1ssDeviceRunner.java

@@ -9,7 +9,12 @@ public final class Cn1ssDeviceRunner extends DeviceRunner {
     private static final String[] TEST_CLASSES = new String[] {
             MainScreenScreenshotTest.class.getName(),
             BrowserComponentScreenshotTest.class.getName(),
-            MediaPlaybackScreenshotTest.class.getName()
+            MediaPlaybackScreenshotTest.class.getName(),
+            GraphicsPipelineScreenshotTest.class.getName(),
+            GraphicsShapesAndGradientsScreenshotTest.class.getName(),
+            GraphicsStateAndTextScreenshotTest.class.getName(),
+            GraphicsTransformationsScreenshotTest.class.getName(),
+            GraphicsMethodsScreenshotTest.class.getName()
     };
 
     public void runSuite() {