(tapestry-5) branch master updated: TAP5-2809: fixing tapestry-jpa runTestAppN tasks

[thiagohp]

This is an automated email from the ASF dual-hosted git repository.

thiagohp pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tapestry-5.git


The following commit(s) were added to refs/heads/master by this push:
     new a917d880e TAP5-2809: fixing tapestry-jpa runTestAppN tasks
a917d880e is described below

commit a917d880efe91920dd92823d54ec59c85bb8c72d
Author: Thiago H. de Paula Figueiredo <thi...@arsmachina.com.br>
AuthorDate: Sun Mar 29 19:02:39 2026 -0300

    TAP5-2809: fixing tapestry-jpa runTestAppN tasks
    
    plus trying to figure out why
    
org.apache.tapestry5.jpa.integration.app6.JpaIntegrationTestWithAnnotationsInServiceImplementation.grid()
    is failing on Jenkins
---
 tapestry-jpa/build.gradle                                        | 9 ++++++---
 ...JpaIntegrationTestWithAnnotationsInServiceImplementation.java | 7 +++++++
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/tapestry-jpa/build.gradle b/tapestry-jpa/build.gradle
index d9ab3fa50..32ffed088 100644
--- a/tapestry-jpa/build.gradle
+++ b/tapestry-jpa/build.gradle
@@ -18,10 +18,13 @@ dependencies {
 }
 
 (1..6).each {
-    tasks.register("runTestApp${it}", JavaExec) {
-        description = "Start app${it} integration test app, useful when 
debugging failing integration tests"
+    
+    var number = it;
+    
+    tasks.register("runTestApp${number}", JavaExec) {
+        description = "Start app${number} integration test app, useful when 
debugging failing integration tests"
         mainClass = 'org.apache.tapestry5.test.JettyRunner'
-        args '-d', "src/test/app${it}", '-p', '8080'
+        args '-d', "src/test/app${number}", '-p', '8080'
         classpath = project.sourceSets.test.runtimeClasspath
     }
 }
diff --git 
a/tapestry-jpa/src/test/java/org/apache/tapestry5/jpa/integration/app6/JpaIntegrationTestWithAnnotationsInServiceImplementation.java
 
b/tapestry-jpa/src/test/java/org/apache/tapestry5/jpa/integration/app6/JpaIntegrationTestWithAnnotationsInServiceImplementation.java
index a806c812e..21a437e71 100644
--- 
a/tapestry-jpa/src/test/java/org/apache/tapestry5/jpa/integration/app6/JpaIntegrationTestWithAnnotationsInServiceImplementation.java
+++ 
b/tapestry-jpa/src/test/java/org/apache/tapestry5/jpa/integration/app6/JpaIntegrationTestWithAnnotationsInServiceImplementation.java
@@ -140,11 +140,18 @@ public class 
JpaIntegrationTestWithAnnotationsInServiceImplementation extends Se
         open("/griddemo");
 
         clickAndWait("link=setup");
+        
+        final String firstNameSelector = "th[data-grid-property=firstName] a";
+        waitForCssSelectorToAppear(firstNameSelector);
 
+        if (!isElementPresent("link=First Name")) {
+            throw new IllegalStateException("link=First Name not found. 
HTML:\n" + getHtmlSource());
+        }
         clickAndWait("link=First Name");
 
         assertText("//td[@data-grid-column-sort='ascending']", "Joe_1");
 
+        waitForCssSelectorToAppear(firstNameSelector);
         clickAndWait("link=First Name");
 
         assertText("//td[@data-grid-column-sort='descending']", "Joe_9");