fix(tests): string concatenation and formatting

- Fixed incorrect string concatenation in test_commands.py.
- Updated old-style string formatting to f-strings in workflow.py.
- Adjusted literal string formatting in test_dataflow.py for consistency.

Signed-off-by: Yves Bastide <yves@botify.com>

Author: ybastide

tests/integration/test_commands.py

@@ -110,7 +110,7 @@ def test_simpleflow_idempotent(self):
             if (
                 e["eventType"] == "ActivityTaskScheduled"
                 and e["activityTaskScheduledEventAttributes"]["activityType"]["name"]
-                == "tests.integration.workflow" ".get_uuid"
+                == "tests.integration.workflow.get_uuid"
             )
         ]
         expect(activities).should.have.length_of(2)

tests/integration/workflow.py

@@ -34,13 +34,13 @@ def get_uuid(unused=None):
 
 @activity.with_attributes(task_list="quickstart", version="example")
 def increment(x):
-    print("increment: %d" % x)
+    print(f"increment: {x:d}")
     return x + 1
 
 
 @activity.with_attributes(task_list="quickstart", version="example")
 def double(y):
-    print("double: %d" % y)
+    print(f"double: {y:d}")
     return y * 2
 
 

tests/test_simpleflow/test_dataflow.py

@@ -1176,7 +1176,7 @@ def test_workflow_activity_raises_on_failure(mock_decref_workflow):
 
     workflow_failed = simpleflow.swf.mapper.models.decision.WorkflowExecutionDecision()
     workflow_failed.fail(
-        reason="Workflow execution error in " "activity-tests.data.activities.raise_on_failure: " '"error"',
+        reason='Workflow execution error in activity-tests.data.activities.raise_on_failure: "error"',
         details=builder.DEFAULT_DETAILS,
     )