Example: For refactor / JSON parse log (#2690)

* feat(dispatcher): add production mode support for error handling and extend URI validation tests

- Introduced `productionRouter` in `DummyTestRouter` to enable production mode configuration.
- Updated `DispatchingInterceptor` to adjust error messages in production mode.
- Expanded `DispatchingInterceptorTest` with production-specific URI validation.

* refactor(orchestration): simplify for-loop logic, improve JSON response formatting, and enhance test validation

Author: predic8

core/src/main/java/com/predic8/membrane/core/lang/ScriptingUtils.java

@@ -83,7 +83,7 @@ public static HashMap<String, Object> createParameterBindings(Router router, Exc
             params.put("headers", headerMap);
             if (includeJsonObject) {
                 try {
-                    log.info("Parsing body as JSON for scripting plugins");
+                    log.debug("Parsing body as JSON for scripting plugins");
                     params.put("json", om.readValue(readInputStream(msg.getBodyAsStreamDecoded()), Map.class));
                 } catch (Exception e) {
                     log.warn("Can't parse body as JSON", e);

distribution/examples/orchestration/for-loop/apis.yaml

@@ -1,40 +1,41 @@
 # yaml-language-server: $schema=https://www.membrane-api.io/v7.0.6.json
+
 api:
   port: 2000
   flow:
     - request:
-        # Fetch the complete list of products (limit set to avoid pagination)
         - call:
-            url: https://api.predic8.de/shop/v2/products?limit=1000
-        - setProperty:
-            name: products
-            value: ${$.products}
-            language: jsonpath
+            url: https://api.predic8.de/shop/v2/products?limit=7
         # Iterate over each product to get additional details (price)
         - for:
-            in: property.products
+            in: $.products
+            language: jsonpath
             flow:
               - call:
                   url: https://api.predic8.de/shop/v2/products/${property.it['id']}
-              - setProperty:
-                  name: price
-                  value: ${$.price}
-                  language: jsonpath
+                  language: spel
               - groovy:
-                  src: property.it.price = property.price
+                  src: |
+                    property.result = property.result ?: []
+                    property.result.add(
+                      [ "name": property.it.name,
+                        "price": fn.jsonPath('$.price')
+                      ])
         # Render a simplified JSON response with only product name and price
         - template:
             contentType: application/json
             pretty: true
             src: |
               {
-                    "products": [
-                      <% property.products.eachWithIndex { p, idx -> %>
-                        {
-                            "name": "<%= p.name %>",
-                            "price": "<%= p.price %>"
-                        }<%= idx < property.products.size() - 1 ? ',' : '' %>
-                        <% } %>
+                  "products": [
+                    <% property.result.eachWithIndex { p, idx -> %>
+                      {
+                          "name": "${p.name}",
+                          "price": ${p.price}
+                      }${idx < property.result.size() - 1 ? ',' : ''}
+                    <% } %>
                   ]
               }
-    - return: {}
+    - return:
+        status: 200
+

distribution/src/test/java/com/predic8/membrane/examples/withinternet/test/orchestration/ForLoopExampleTest.java

@@ -14,11 +14,11 @@
 
 package com.predic8.membrane.examples.withinternet.test.orchestration;
 
-import com.predic8.membrane.examples.util.AbstractSampleMembraneStartStopTestcase;
-import org.junit.jupiter.api.Test;
+import com.predic8.membrane.examples.util.*;
+import org.junit.jupiter.api.*;
 
-import static io.restassured.RestAssured.given;
-import static org.hamcrest.CoreMatchers.containsString;
+import static io.restassured.RestAssured.*;
+import static org.hamcrest.Matchers.*;
 
 public class ForLoopExampleTest extends AbstractSampleMembraneStartStopTestcase {
 
@@ -27,18 +27,18 @@ protected String getExampleDirName() {
         return "orchestration/for-loop";
     }
 
-    // @formatter:off
     @Test
-    void testCall() {
+    void call() {
+        // @formatter:off
         given()
         .when()
             .get("http://localhost:2000")
         .then()
-            .body(
-                containsString("\"products\""),
-                containsString("\"name\""),
-                containsString("\"price\"")
-            ).statusCode(200);
+            .body("products[0].name", notNullValue())
+            .body("products[0].price", instanceOf(Number.class))
+            .body("products.size()", greaterThan(2))
+            .statusCode(200);
+        // @formatter:on
     }
-    // @formatter:on
+
 }