From bb509e672cee1e11525f1bf29a2208d157969f51 Mon Sep 17 00:00:00 2001 From: Christophe Bedard Date: Sat, 16 Nov 2019 15:27:08 -0800 Subject: [PATCH] Update style for test_subscription_callback --- tracetools_test/test/test_subscription_callback.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tracetools_test/test/test_subscription_callback.py b/tracetools_test/test/test_subscription_callback.py index cfe463f..e1e2483 100644 --- a/tracetools_test/test/test_subscription_callback.py +++ b/tracetools_test/test/test_subscription_callback.py @@ -42,14 +42,14 @@ class TestSubscriptionCallback(TraceTestCase): self.assertIsInstance(is_intra_process_value, int, 'is_intra_process not int') self.assertTrue( is_intra_process_value in [0, 1], - f'invalid value for is_intra_process: {is_intra_process_value}') + f'invalid value for is_intra_process: {is_intra_process_value}', + ) end_events = self.get_events_with_name('ros2:callback_end') for event in end_events: self.assertValidHandle(event, 'callback') # Check that a start:end pair has a common callback handle - # Note: might be unstable if tracing is disabled too early ping_events = self.get_events_with_procname('test_ping') pong_events = self.get_events_with_procname('test_pong') ping_events_start = self.get_events_with_name('ros2:callback_start', ping_events) @@ -59,13 +59,15 @@ class TestSubscriptionCallback(TraceTestCase): ping_start, 'callback', 'ros2:callback_end', - ping_events) + ping_events, + ) for pong_start in pong_events_start: self.assertMatchingField( pong_start, 'callback', 'ros2:callback_end', - pong_events) + pong_events, + ) if __name__ == '__main__':