next(self._msg_iter)
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _create_graph(src_comp_cls, sink_comp_cls, flt_comp_cls=None):
def __init__(self, config, self_port_output):
nonlocal flt_iter_initialized
flt_iter_initialized = True
- self._up_iter = self._create_input_port_message_iterator(
+ self._up_iter = self._create_message_iterator(
self._component._input_ports['in']
)
def test_create_user_error(self):
# This tests both error handling by
- # _UserSinkComponent._create_input_port_message_iterator
- # and _UserMessageIterator._create_input_port_message_iterator, as they
+ # _UserSinkComponent._create_message_iterator
+ # and _UserMessageIterator._create_message_iterator, as they
# are both used in the graph.
class MySourceIter(bt2._UserMessageIterator):
def __init__(self, config, self_port_output):
def __init__(self, config, self_port_output):
# This is expected to raise because of the error in
# MySourceIter.__init__.
- self._create_input_port_message_iterator(
- self._component._input_ports['in']
- )
+ self._create_message_iterator(self._component._input_ports['in'])
class MyFilter(bt2._UserFilterComponent, message_iterator_class=MyFilterIter):
def __init__(self, config, params, obj):
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
nonlocal can_seek_forward
class MyFilterIter(bt2._UserMessageIterator):
def __init__(self, port):
input_port = port.user_data
- self._upstream_iter = self._create_input_port_message_iterator(
- input_port
- )
+ self._upstream_iter = self._create_message_iterator(input_port)
def __next__(self):
return next(self._upstream_iter)
with self.assertRaises(bt2.TryAgain):
next(it)
+ def test_error_in_iterator_with_cycle_after_having_created_upstream_iterator(self):
+ # Test a failure that triggered an abort in libbabeltrace2, in this situation:
+ #
+ # - The filter iterator creates an upstream iterator.
+ # - The filter iterator creates a reference cycle, including itself.
+ # - An exception is raised, causing the filter iterator's
+ # initialization method to fail.
+ class MySourceIter(bt2._UserMessageIterator):
+ pass
+
+ class MySource(bt2._UserSourceComponent, message_iterator_class=MySourceIter):
+ def __init__(self, config, params, obj):
+ self._add_output_port('out')
+
+ class MyFilterIter(bt2._UserMessageIterator):
+ def __init__(self, config, port):
+ # First, create an upstream iterator.
+ self._upstream_iter = self._create_message_iterator(
+ self._component._input_ports['in']
+ )
+
+ # Then, voluntarily make a reference cycle that will keep this
+ # Python object alive, which will keep the upstream iterator
+ # Babeltrace object alive.
+ self._self = self
+
+ # Finally, raise an exception to make __init__ fail.
+ raise ValueError('woops')
+
+ class MyFilter(bt2._UserFilterComponent, message_iterator_class=MyFilterIter):
+ def __init__(self, config, params, obj):
+ self._in = self._add_input_port('in')
+ self._out = self._add_output_port('out')
+
+ class MySink(bt2._UserSinkComponent):
+ def __init__(self, config, params, obj):
+ self._input_port = self._add_input_port('in')
+
+ def _user_graph_is_configured(self):
+ self._upstream_iter = self._create_message_iterator(self._input_port)
+
+ def _user_consume(self):
+ # We should not reach this.
+ assert False
+
+ g = bt2.Graph()
+ src = g.add_component(MySource, 'src')
+ flt = g.add_component(MyFilter, 'flt')
+ snk = g.add_component(MySink, 'snk')
+ g.connect_ports(src.output_ports['out'], flt.input_ports['in'])
+ g.connect_ports(flt.output_ports['out'], snk.input_ports['in'])
+
+ with self.assertRaisesRegex(bt2._Error, 'ValueError: woops'):
+ g.run()
+
def _setup_seek_test(
sink_cls,
class MyFilterIter(bt2._UserMessageIterator):
def __init__(self, config, port):
- self._upstream_iter = self._create_input_port_message_iterator(
+ self._upstream_iter = self._create_message_iterator(
self._component._input_ports['in']
)
config.can_seek_forward = self._upstream_iter.can_seek_forward
class UserMessageIteratorSeekBeginningTestCase(unittest.TestCase):
+ def test_can_seek_beginning_without_seek_beginning(self):
+ with self.assertRaisesRegex(
+ bt2._IncompleteUserClass,
+ "cannot create component class 'MySource': message iterator class implements _user_can_seek_beginning but not _user_seek_beginning",
+ ):
+ _setup_seek_test(SimpleSink, user_can_seek_beginning=lambda: None)
+
def test_can_seek_beginning(self):
class MySink(bt2._UserSinkComponent):
def __init__(self, config, params, obj):
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
nonlocal can_seek_beginning
return input_port_iter_can_seek_beginning
graph = _setup_seek_test(
- MySink, user_can_seek_beginning=_user_can_seek_beginning
+ MySink,
+ user_can_seek_beginning=_user_can_seek_beginning,
+ user_seek_beginning=lambda: None,
)
input_port_iter_can_seek_beginning = True
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
nonlocal can_seek_beginning
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
nonlocal can_seek_beginning
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
# This is expected to raise.
raise ValueError('moustiquaire')
graph = _setup_seek_test(
- MySink, user_can_seek_beginning=_user_can_seek_beginning
+ MySink,
+ user_can_seek_beginning=_user_can_seek_beginning,
+ user_seek_beginning=lambda: None,
)
with self.assertRaises(bt2._Error) as ctx:
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
# This is expected to raise.
return 'Amqui'
graph = _setup_seek_test(
- MySink, user_can_seek_beginning=_user_can_seek_beginning
+ MySink,
+ user_can_seek_beginning=_user_can_seek_beginning,
+ user_seek_beginning=lambda: None,
)
with self.assertRaises(bt2._Error) as ctx:
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
nonlocal do_seek_beginning
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
self._msg_iter.seek_beginning()
class UserMessageIteratorSeekNsFromOriginTestCase(unittest.TestCase):
+ def test_can_seek_ns_from_origin_without_seek_ns_from_origin(self):
+ # Test the case where:
+ #
+ # - can_seek_ns_from_origin: Returns True (don't really care, as long
+ # as it's provided)
+ # - seek_ns_from_origin provided: No
+ # - can the iterator seek beginning: Don't care
+ # - can the iterator seek forward: Don't care
+ for can_seek_ns_from_origin in (False, True):
+ for iter_can_seek_beginning in (False, True):
+ for iter_can_seek_forward in (False, True):
+ with self.assertRaisesRegex(
+ bt2._IncompleteUserClass,
+ "cannot create component class 'MySource': message iterator class implements _user_can_seek_ns_from_origin but not _user_seek_ns_from_origin",
+ ):
+ self._can_seek_ns_from_origin_test(
+ None,
+ user_can_seek_ns_from_origin_ret_val=True,
+ user_seek_ns_from_origin_provided=False,
+ iter_can_seek_beginning=iter_can_seek_beginning,
+ iter_can_seek_forward=iter_can_seek_forward,
+ )
+
def test_can_seek_ns_from_origin_returns_true(self):
# Test the case where:
#
# - can_seek_ns_from_origin: returns True
- # - seek_ns_from_origin provided: Don't care
+ # - seek_ns_from_origin provided: Yes
# - can the iterator seek beginning: Don't care
# - can the iterator seek forward: Don't care
#
# We expect iter.can_seek_ns_from_origin to return True.
- for user_seek_ns_from_origin_provided in (False, True):
- for iter_can_seek_beginning in (False, True):
- for iter_can_seek_forward in (False, True):
- self._can_seek_ns_from_origin_test(
- expected_outcome=True,
- user_can_seek_ns_from_origin_ret_val=True,
- user_seek_ns_from_origin_provided=user_seek_ns_from_origin_provided,
- iter_can_seek_beginning=iter_can_seek_beginning,
- iter_can_seek_forward=iter_can_seek_forward,
- )
+ for iter_can_seek_beginning in (False, True):
+ for iter_can_seek_forward in (False, True):
+ self._can_seek_ns_from_origin_test(
+ expected_outcome=True,
+ user_can_seek_ns_from_origin_ret_val=True,
+ user_seek_ns_from_origin_provided=True,
+ iter_can_seek_beginning=iter_can_seek_beginning,
+ iter_can_seek_forward=iter_can_seek_forward,
+ )
def test_can_seek_ns_from_origin_returns_false_can_seek_beginning_forward_seekable(
- self
+ self,
):
# Test the case where:
#
# - can_seek_ns_from_origin: returns False
- # - seek_ns_from_origin provided: Don't care
+ # - seek_ns_from_origin provided: Yes
# - can the iterator seek beginning: Yes
# - can the iterator seek forward: Yes
#
# We expect iter.can_seek_ns_from_origin to return True.
- for user_seek_ns_from_origin_provided in (False, True):
- self._can_seek_ns_from_origin_test(
- expected_outcome=True,
- user_can_seek_ns_from_origin_ret_val=False,
- user_seek_ns_from_origin_provided=user_seek_ns_from_origin_provided,
- iter_can_seek_beginning=True,
- iter_can_seek_forward=True,
- )
+ self._can_seek_ns_from_origin_test(
+ expected_outcome=True,
+ user_can_seek_ns_from_origin_ret_val=False,
+ user_seek_ns_from_origin_provided=True,
+ iter_can_seek_beginning=True,
+ iter_can_seek_forward=True,
+ )
def test_can_seek_ns_from_origin_returns_false_can_seek_beginning_not_forward_seekable(
- self
+ self,
):
# Test the case where:
#
# - can_seek_ns_from_origin: returns False
- # - seek_ns_from_origin provided: Don't care
+ # - seek_ns_from_origin provided: Yes
# - can the iterator seek beginning: Yes
# - can the iterator seek forward: No
#
# We expect iter.can_seek_ns_from_origin to return False.
- for user_seek_ns_from_origin_provided in (False, True):
- self._can_seek_ns_from_origin_test(
- expected_outcome=False,
- user_can_seek_ns_from_origin_ret_val=False,
- user_seek_ns_from_origin_provided=user_seek_ns_from_origin_provided,
- iter_can_seek_beginning=True,
- iter_can_seek_forward=False,
- )
+ self._can_seek_ns_from_origin_test(
+ expected_outcome=False,
+ user_can_seek_ns_from_origin_ret_val=False,
+ user_seek_ns_from_origin_provided=True,
+ iter_can_seek_beginning=True,
+ iter_can_seek_forward=False,
+ )
def test_can_seek_ns_from_origin_returns_false_cant_seek_beginning_forward_seekable(
- self
+ self,
):
# Test the case where:
#
# - can_seek_ns_from_origin: returns False
- # - seek_ns_from_origin provided: Don't care
+ # - seek_ns_from_origin provided: Yes
# - can the iterator seek beginning: No
# - can the iterator seek forward: Yes
#
# We expect iter.can_seek_ns_from_origin to return False.
- # for user_seek_ns_from_origin_provided in (False, True):
self._can_seek_ns_from_origin_test(
expected_outcome=False,
user_can_seek_ns_from_origin_ret_val=False,
- user_seek_ns_from_origin_provided=False,
+ user_seek_ns_from_origin_provided=True,
iter_can_seek_beginning=False,
iter_can_seek_forward=True,
)
def test_can_seek_ns_from_origin_returns_false_cant_seek_beginning_not_forward_seekable(
- self
+ self,
):
# Test the case where:
#
# - can_seek_ns_from_origin: returns False
- # - seek_ns_from_origin provided: Don't care
+ # - seek_ns_from_origin provided: Yes
# - can the iterator seek beginning: No
# - can the iterator seek forward: No
#
# We expect iter.can_seek_ns_from_origin to return False.
- for user_seek_ns_from_origin_provided in (False, True):
- self._can_seek_ns_from_origin_test(
- expected_outcome=False,
- user_can_seek_ns_from_origin_ret_val=False,
- user_seek_ns_from_origin_provided=user_seek_ns_from_origin_provided,
- iter_can_seek_beginning=False,
- iter_can_seek_forward=False,
- )
+ self._can_seek_ns_from_origin_test(
+ expected_outcome=False,
+ user_can_seek_ns_from_origin_ret_val=False,
+ user_seek_ns_from_origin_provided=True,
+ iter_can_seek_beginning=False,
+ iter_can_seek_forward=False,
+ )
def test_no_can_seek_ns_from_origin_seek_ns_from_origin(self):
# Test the case where:
)
def test_no_can_seek_ns_from_origin_no_seek_ns_from_origin_can_seek_beginning_forward_seekable(
- self
+ self,
):
# Test the case where:
#
)
def test_no_can_seek_ns_from_origin_no_seek_ns_from_origin_can_seek_beginning_not_forward_seekable(
- self
+ self,
):
# Test the case where:
#
)
def test_no_can_seek_ns_from_origin_no_seek_ns_from_origin_cant_seek_beginning_forward_seekable(
- self
+ self,
):
# Test the case where:
#
)
def test_no_can_seek_ns_from_origin_no_seek_ns_from_origin_cant_seek_beginning_not_forward_seekable(
- self
+ self,
):
# Test the case where:
#
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
nonlocal can_seek_ns_from_origin
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
# This is expected to raise.
raise ValueError('Joutel')
graph = _setup_seek_test(
- MySink, user_can_seek_ns_from_origin=_user_can_seek_ns_from_origin
+ MySink,
+ user_can_seek_ns_from_origin=_user_can_seek_ns_from_origin,
+ user_seek_ns_from_origin=lambda: None,
)
with self.assertRaises(bt2._Error) as ctx:
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
# This is expected to raise.
return 'Nitchequon'
graph = _setup_seek_test(
- MySink, user_can_seek_ns_from_origin=_user_can_seek_ns_from_origin
+ MySink,
+ user_can_seek_ns_from_origin=_user_can_seek_ns_from_origin,
+ user_seek_ns_from_origin=lambda: None,
)
with self.assertRaises(bt2._Error) as ctx:
self._add_input_port('in')
def _user_graph_is_configured(self):
- self._msg_iter = self._create_input_port_message_iterator(
- self._input_ports['in']
- )
+ self._msg_iter = self._create_message_iterator(self._input_ports['in'])
def _user_consume(self):
self._msg_iter.seek_ns_from_origin(17)