Merge branch 'tests' into 'master'
Enhance tracetools_test See merge request micro-ROS/ros2_tracing!14
This commit is contained in:
		
						commit
						9d16302f83
					
				
					 7 changed files with 71 additions and 41 deletions
				
			
		| 
						 | 
				
			
			@ -45,12 +45,12 @@ class TestIntra(TraceTestCase):
 | 
			
		|||
            'topic_name',
 | 
			
		||||
            '/the_topic/_intra',
 | 
			
		||||
            sub_init_events)
 | 
			
		||||
        self.assertEqual(
 | 
			
		||||
            len(sub_init_normal_events),
 | 
			
		||||
        self.assertNumEvents(
 | 
			
		||||
            sub_init_normal_events,
 | 
			
		||||
            1,
 | 
			
		||||
            'none or more than 1 sub init event for normal sub')
 | 
			
		||||
        self.assertEqual(
 | 
			
		||||
            len(sub_init_intra_events),
 | 
			
		||||
        self.assertNumEvents(
 | 
			
		||||
            sub_init_intra_events,
 | 
			
		||||
            1,
 | 
			
		||||
            'none or more than 1 sub init event for intra sub')
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -67,8 +67,8 @@ class TestIntra(TraceTestCase):
 | 
			
		|||
            sub_handle_intra,
 | 
			
		||||
            self.get_events_with_name(
 | 
			
		||||
                'ros2:rclcpp_subscription_callback_added'))
 | 
			
		||||
        self.assertEqual(
 | 
			
		||||
            len(callback_added_events),
 | 
			
		||||
        self.assertNumEvents(
 | 
			
		||||
            callback_added_events,
 | 
			
		||||
            1,
 | 
			
		||||
            'none or more than 1 callback added event')
 | 
			
		||||
        callback_added_event = callback_added_events[0]
 | 
			
		||||
| 
						 | 
				
			
			@ -78,8 +78,8 @@ class TestIntra(TraceTestCase):
 | 
			
		|||
        start_events = self.get_events_with_name('ros2:callback_start')
 | 
			
		||||
        end_events = self.get_events_with_name('ros2:callback_end')
 | 
			
		||||
        # Should still have two start:end pairs (1 normal + 1 intra)
 | 
			
		||||
        self.assertEqual(len(start_events), 2, 'does not have 2 callback start events')
 | 
			
		||||
        self.assertEqual(len(end_events), 2, 'does not have 2 callback end events')
 | 
			
		||||
        self.assertNumEvents(start_events, 2, 'does not have 2 callback start events')
 | 
			
		||||
        self.assertNumEvents(end_events, 2, 'does not have 2 callback end events')
 | 
			
		||||
        start_events_intra = self.get_events_with_field_value(
 | 
			
		||||
            'callback',
 | 
			
		||||
            callback_handle_intra,
 | 
			
		||||
| 
						 | 
				
			
			@ -88,12 +88,12 @@ class TestIntra(TraceTestCase):
 | 
			
		|||
            'callback',
 | 
			
		||||
            callback_handle_intra,
 | 
			
		||||
            end_events)
 | 
			
		||||
        self.assertEqual(
 | 
			
		||||
            len(start_events_intra),
 | 
			
		||||
        self.assertNumEvents(
 | 
			
		||||
            start_events_intra,
 | 
			
		||||
            1,
 | 
			
		||||
            'none or more than one intra start event')
 | 
			
		||||
        self.assertEqual(
 | 
			
		||||
            len(end_events_intra),
 | 
			
		||||
        self.assertNumEvents(
 | 
			
		||||
            end_events_intra,
 | 
			
		||||
            1,
 | 
			
		||||
            'none or more than one intra end event')
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -110,8 +110,8 @@ class TestIntra(TraceTestCase):
 | 
			
		|||
            'callback',
 | 
			
		||||
            callback_handle_intra,
 | 
			
		||||
            start_events)
 | 
			
		||||
        self.assertEqual(
 | 
			
		||||
            len(start_events_not_intra),
 | 
			
		||||
        self.assertNumEvents(
 | 
			
		||||
            start_events_not_intra,
 | 
			
		||||
            1,
 | 
			
		||||
            'none or more than one normal start event')
 | 
			
		||||
        start_event_not_intra = start_events_not_intra[0]
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue