MPT-16566 Fix ruff rules after dependabot bump

Author: d3rky

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.12.11
+    rev: v0.14.10
     hooks:
       - id: ruff-check
       - id: ruff-format

swo_aws_extension/flows/fulfillment/pipelines.py

@@ -25,7 +25,7 @@ def strip_trace_id(traceback_msg: str) -> str:
 # Should be removed with the new SDK error handling mechanism.
 def pipeline_error_handler(error: Exception, context: Context, next_step):
     """Custom error handler for AWS pipelines."""
-    logger.error("%s - Unexpected error in AWS pipeline: %s", context.order_id, str(error))
+    logger.error("%s - Unexpected error in AWS pipeline: %s", context.order_id, error)
     traceback_id = strip_trace_id(traceback.format_exc())
     TeamsNotificationManager().notify_one_time_error(
         "Order fulfillment unhandled exception!",

swo_aws_extension/flows/steps/base.py

@@ -33,14 +33,16 @@ def __call__(
             self.pre_step(context)
         except ConfigurationStepError as error:
             logger.info(
-                "%s - Stop - Stop step due to configuration error: %s", context.order_id, str(error)
+                "%s - Stop - Stop step due to configuration error: %s",
+                context.order_id,
+                error,
             )
             return
 
         try:
             self.process(context)
         except UnexpectedStopError as error:
-            logger.info("%s - Unexpected Stop: %s", context.order_id, str(error))
+            logger.info("%s - Unexpected Stop: %s", context.order_id, error)
             TeamsNotificationManager().notify_one_time_error(error.title, error.message)
             return
 

tests/swo/mpt/sync/test_syncer.py

@@ -418,9 +418,11 @@ def test_dry_run(self, mpt_client, caplog, agreement_factory, mock_update_agreem
 
         assert caplog.messages == [
             "AGR-2119-4550-8674-5962 - synchronizing responsibility transfer ID: PMA-123456",
-            "AGR-2119-4550-8674-5962 - dry run mode - skipping update with parameters: "
-            "{'fulfillment': [{'externalId': 'responsibilityTransferId', 'value': "
-            "'PMA-123456'}]}",
+            (
+                "AGR-2119-4550-8674-5962 - dry run mode - skipping update with parameters: "
+                "{'fulfillment': [{'externalId': 'responsibilityTransferId', 'value': "
+                "'PMA-123456'}]}"
+            ),
         ]
         mock_update_agreement.assert_not_called()