Refactor test tracer for consistency

Quinn-With-Two-Ns · Dec 6, 2024 · b6e8ba6 · b6e8ba6
1 parent 7ed7c9d
commit b6e8ba6
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/temporal-sdk/src/test/java/io/temporal/workflow/nexus/CancelAsyncOperationTest.java b/temporal-sdk/src/test/java/io/temporal/workflow/nexus/CancelAsyncOperationTest.java
@@ -65,9 +65,9 @@ public void asyncOperationImmediatelyCancelled() {
         .setExpected(
             "interceptExecuteWorkflow " + SDKTestWorkflowRule.UUID_REGEXP,
             "newThread workflow-method",
-            "executeNexusOperation TestNexusService1.operation",
-            "startNexusOperation TestNexusService1.operation",
-            "cancelNexusOperation TestNexusService1.operation");
+            "executeNexusOperation TestNexusService1 operation",
+            "startNexusOperation TestNexusService1 operation",
+            "cancelNexusOperation TestNexusService1 operation");
   }
 
   @Test
@@ -85,13 +85,13 @@ public void asyncOperationCancelled() {
         .setExpected(
             "interceptExecuteWorkflow " + SDKTestWorkflowRule.UUID_REGEXP,
             "newThread workflow-method",
-            "executeNexusOperation TestNexusService1.operation",
-            "startNexusOperation TestNexusService1.operation",
+            "executeNexusOperation TestNexusService1 operation",
+            "startNexusOperation TestNexusService1 operation",
             "interceptExecuteWorkflow " + SDKTestWorkflowRule.UUID_REGEXP,
             "registerSignalHandlers unblock",
             "newThread workflow-method",
             "await await",
-            "cancelNexusOperation TestNexusService1.operation");
+            "cancelNexusOperation TestNexusService1 operation");
   }
 
   public static class TestNexus implements TestWorkflows.TestWorkflow1 {

diff --git a/temporal-sdk/src/test/java/io/temporal/workflow/nexus/SyncClientOperationTest.java b/temporal-sdk/src/test/java/io/temporal/workflow/nexus/SyncClientOperationTest.java
@@ -70,8 +70,8 @@ public void syncClientOperationSuccess() {
             "interceptExecuteWorkflow " + SDKTestWorkflowRule.UUID_REGEXP,
             "registerUpdateHandlers update",
             "newThread workflow-method",
-            "executeNexusOperation TestNexusService1.operation",
-            "startNexusOperation TestNexusService1.operation");
+            "executeNexusOperation TestNexusService1 operation",
+            "startNexusOperation TestNexusService1 operation");
     // Test metrics all tasks should have
     Map<String, String> nexusWorkerTags =
         ImmutableMap.<String, String>builder()

diff --git a/temporal-testing/src/main/java/io/temporal/testing/internal/TracingWorkerInterceptor.java b/temporal-testing/src/main/java/io/temporal/testing/internal/TracingWorkerInterceptor.java
@@ -197,7 +197,7 @@ public <R> ChildWorkflowOutput<R> executeChildWorkflow(ChildWorkflowInput<R> inp
     public <R> ExecuteNexusOperationOutput<R> executeNexusOperation(
         ExecuteNexusOperationInput<R> input) {
       if (!WorkflowUnsafe.isReplaying()) {
-        trace.add("executeNexusOperation " + input.getService() + "." + input.getOperation());
+        trace.add("executeNexusOperation " + input.getService() + " " + input.getOperation());
       }
       return next.executeNexusOperation(input);
     }
@@ -499,7 +499,7 @@ public StartOperationOutput startOperation(StartOperationInput input)
       trace.add(
           "startNexusOperation "
               + input.getOperationContext().getService()
-              + "."
+              + " "
               + input.getOperationContext().getOperation());
       return next.startOperation(input);
     }
@@ -509,7 +509,7 @@ public CancelOperationOutput cancelOperation(CancelOperationInput input) {
       trace.add(
           "cancelNexusOperation "
               + input.getOperationContext().getService()
-              + "."
+              + " "
               + input.getOperationContext().getOperation());
       return next.cancelOperation(input);
     }