Use Maven install target for demo screenshot build (#4077)

* Fix demo screenshot build target

* Trigger demo build when workflow changes

* Update demo build to Codename One 7.0.208

* Enable demos test profile by default

* fixed not running tests during build

* Revert "fixed not running tests during build"

This reverts commit 26c816e.

---------

Co-authored-by: Steve Hannah <[email protected]>

Author: shai-almog

.github/workflows/developer-guide-docs.yml

@@ -31,8 +31,12 @@ jobs:
           filters: |
             demos:
               - 'docs/demos/**'
+              - '.github/workflows/developer-guide-docs.yml'
             docs:
               - 'docs/developer-guide/**'
+              - '.github/workflows/developer-guide-docs.yml'
+            workflow:
+              - '.github/workflows/developer-guide-docs.yml'
 
       - name: Set up Java
         uses: actions/setup-java@v4
@@ -41,24 +45,24 @@ jobs:
           java-version: '11'
 
       - name: Build Codename One demos
-        if: github.event_name != 'pull_request' || steps.changes.outputs.demos == 'true'
+        if: github.event_name != 'pull_request' || steps.changes.outputs.demos == 'true' || steps.changes.outputs.workflow == 'true'
         run: |
           set -euo pipefail
           mkdir -p "$HOME/.codenameone"
           touch "$HOME/.codenameone/guibuilder.jar"
           cp maven/CodeNameOneBuildClient.jar "$HOME/.codenameone/CodeNameOneBuildClient.jar"
-          xvfb-run -a mvn -B -ntp -Dgenerate-gui-sources-done=true -pl common -am -f docs/demos/pom.xml test
+          xvfb-run -a mvn -B -ntp -Dgenerate-gui-sources-done=true -pl common -am -f docs/demos/pom.xml install
 
       - name: Install ImageMagick for screenshot comparison
-        if: github.event_name != 'pull_request' || steps.changes.outputs.demos == 'true'
+        if: github.event_name != 'pull_request' || steps.changes.outputs.demos == 'true' || steps.changes.outputs.workflow == 'true'
         run: |
           set -euo pipefail
           sudo apt-get update
           sudo apt-get install -y --no-install-recommends imagemagick
 
       - name: Compare animation screenshots
         id: compare_animation_screenshots
-        if: github.event_name != 'pull_request' || steps.changes.outputs.demos == 'true'
+        if: github.event_name != 'pull_request' || steps.changes.outputs.demos == 'true' || steps.changes.outputs.workflow == 'true'
         continue-on-error: true
         run: |
           set -euo pipefail
@@ -168,7 +172,7 @@ jobs:
           fi
 
       - name: Build Developer Guide HTML and PDF
-        if: github.event_name != 'pull_request' || steps.changes.outputs.docs == 'true' || steps.changes.outputs.demos == 'true'
+        if: github.event_name != 'pull_request' || steps.changes.outputs.docs == 'true' || steps.changes.outputs.demos == 'true' || steps.changes.outputs.workflow == 'true'
         run: |
           set -euo pipefail
           OUTPUT_ROOT="build/developer-guide"

docs/demos/common/pom.xml

@@ -360,6 +360,14 @@
 
             </plugin>
 
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <configuration>
+                    <skipTests>true</skipTests>
+                </configuration>
+            </plugin>
+
 
         </plugins>
 

docs/demos/javase/pom.xml

@@ -299,9 +299,7 @@
           <!-- Profile for running Unit Tests -->
           <id>test</id>
           <activation>
-              <property>
-                      <name>!skipTests</name>
-              </property>
+              <activeByDefault>true</activeByDefault>
           </activation>
           <properties>
               <codename1.platform>javase</codename1.platform>

docs/demos/pom.xml

@@ -19,8 +19,8 @@
 <module>common</module>
 </modules>
   <properties>
-    <cn1.plugin.version>7.0.175</cn1.plugin.version>
-    <cn1.version>LATEST</cn1.version>
+    <cn1.plugin.version>7.0.208</cn1.plugin.version>
+    <cn1.version>7.0.208</cn1.version>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <java.version>1.8</java.version>
     <java-tests.version>11</java-tests.version>