chore: more review comments fixes

Signed-off-by: Javier Aliaga <[email protected]>

Author: javier-aliaga

sdk-tests/src/test/java/io/dapr/it/testcontainers/workflows/crossapp/App2TransformActivity.java

@@ -26,10 +26,10 @@ public class App2TransformActivity implements WorkflowActivity {
   public Object run(WorkflowActivityContext ctx) {
     String input = ctx.getInput(String.class);
     logger.info("=== App2: TransformActivity called ===");
-    logger.info("Input: " + input);
+    logger.info("Input: {}", input);
 
     String output = input.toUpperCase() + " [TRANSFORMED BY APP2]";
-    logger.info("Output: " + output);
+    logger.info("Output: {}", output);
 
     return output;
   }

sdk-tests/src/test/java/io/dapr/it/testcontainers/workflows/crossapp/App3FinalizeActivity.java

@@ -26,10 +26,10 @@ public class App3FinalizeActivity implements WorkflowActivity {
   public Object run(WorkflowActivityContext ctx) {
     String input = ctx.getInput(String.class);
     logger.info("=== App3: FinalizeActivity called ===");
-    logger.info("Input: " + input);
+    logger.info("Input: {}", input);
 
     String output = input + " [FINALIZED BY APP3]";
-    logger.info("Output: " + output);
+    logger.info("Output: {}", output);
 
     return output;
   }

sdk-tests/src/test/java/io/dapr/it/testcontainers/workflows/crossapp/CrossAppWorkflow.java

@@ -19,19 +19,16 @@
 import org.slf4j.Logger;
 
 public class CrossAppWorkflow implements Workflow {
-
-
-
   @Override
   public WorkflowStub create() {
     return ctx -> {
       Logger logger = ctx.getLogger();
       String instanceId = ctx.getInstanceId();
-      logger.info("Starting CrossAppWorkflow: " + ctx.getName());
-      logger.info("Instance ID: " + instanceId);
+      logger.info("Starting CrossAppWorkflow: {}", ctx.getName());
+      logger.info("Instance ID: {}", instanceId);
 
       String input = ctx.getInput(String.class);
-      logger.info("Workflow input: " + input);
+      logger.info("Workflow input: {}", input);
 
       // Call App2TransformActivity in app2
       logger.info("Calling cross-app activity in 'app2'...");
@@ -51,7 +48,7 @@ public WorkflowStub create() {
           String.class
       ).await();
 
-      logger.info("Final cross-app activity result: " + finalizedByApp3);
+      logger.info("Final cross-app activity result: {}", finalizedByApp3);
       ctx.complete(finalizedByApp3);
     };
   }

sdk-tests/src/test/java/io/dapr/it/testcontainers/workflows/crossapp/WorkflowsCrossAppCallActivityIT.java

@@ -224,8 +224,6 @@ public void testCrossAppWorkflow() throws Exception {
     // The workers use host.testcontainers.internal:50001, so we need to use the mapped port
     String grpcEndpoint = "localhost:" + MAIN_WORKFLOW_CONTAINER.getMappedPort(50001);
     String httpEndpoint = "localhost:" + MAIN_WORKFLOW_CONTAINER.getMappedPort(3500);
-    System.setProperty("dapr.grpc.endpoint", grpcEndpoint);
-    System.setProperty("dapr.http.endpoint", httpEndpoint);
     Map<String, String> propertyOverrides = Map.of(
         "dapr.grpc.endpoint", grpcEndpoint,
         "dapr.http.endpoint", httpEndpoint