(ofbiz-framework) branch trunk updated: Fixed: Update end bracket detection en groovy scriplet (OFBIZ-13178)

[nmalin]

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

nmalin pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new b97957d3ab Fixed: Update end bracket detection en groovy scriplet 
(OFBIZ-13178)
b97957d3ab is described below

commit b97957d3ab6bd801b92e4c501f0aa8448c09b26f
Author: Nicolas Malin <nicolas.ma...@nereide.fr>
AuthorDate: Tue Nov 19 10:53:51 2024 +0100

    Fixed: Update end bracket detection en groovy scriplet (OFBIZ-13178)
    
    Fixed the long size line issue on introduced tests.
---
 .../apache/ofbiz/base/util/string/FlexibleStringExpanderTests.java  | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/framework/base/src/test/java/org/apache/ofbiz/base/util/string/FlexibleStringExpanderTests.java
 
b/framework/base/src/test/java/org/apache/ofbiz/base/util/string/FlexibleStringExpanderTests.java
index 0bc528f419..81c39731ce 100644
--- 
a/framework/base/src/test/java/org/apache/ofbiz/base/util/string/FlexibleStringExpanderTests.java
+++ 
b/framework/base/src/test/java/org/apache/ofbiz/base/util/string/FlexibleStringExpanderTests.java
@@ -316,8 +316,10 @@ public class FlexibleStringExpanderTests {
         fseTest("UEL integration: missing", "${noList[0]}", testMap, null, 
null, "", null, false);
         fseTest("Escaped expression", "This is an \\${escaped} expression", 
testMap, "This is an ${escaped} expression", false);
         fseTest("Escaped(groovy) expression", "This is an \\${groovy:escaped} 
expression", testMap, "This is an ${groovy:escaped} expression", false);
-        fseTest("Bracket en groovy", "This is a groovy ${groovy: if (true) 
{return 'bracket'}} expression", testMap, "This is a groovy bracket 
expression", false);
-        fseTest("Bracket en groovy again", "This is a groovy ${groovy: if 
(true) {if (true) {return 'with 2 brackets'}}} expression", testMap, "This is a 
groovy with 2 brackets expression", false);
+        fseTest("Bracket en groovy", "This is a groovy ${groovy: if (true) 
{return 'bracket'}} expression", testMap,
+                "This is a groovy bracket expression", false);
+        fseTest("Bracket en groovy again", "This is a groovy ${groovy: if 
(true) {if (true) {return 'with 2 brackets'}}} expression", testMap,
+                "This is a groovy with 2 brackets expression", false);
 
         // TODO: Find a better way to setup or handle the big decimal value. 
If new ones are not instantiated in the test
         // it fails because of the comparison between object pointers..