X-Git-Url: http://git.efficios.com/?p=babeltrace.git;a=blobdiff_plain;f=tests%2Fbindings%2Fpython%2Fbt2%2Ftest_query_executor.py;h=e7186e8843b3e0ec929d250f7ac35aa80bf6ac93;hp=cdaccdcf087a7a5305aa1f865e5429a5a07e2436;hb=0235b0db7de5bcacdb3650c92461f2ce5eb2143d;hpb=976c241df475c9d99161a1100517c3c308074d7f diff --git a/tests/bindings/python/bt2/test_query_executor.py b/tests/bindings/python/bt2/test_query_executor.py index cdaccdcf..e7186e88 100644 --- a/tests/bindings/python/bt2/test_query_executor.py +++ b/tests/bindings/python/bt2/test_query_executor.py @@ -1,132 +1,290 @@ -from bt2 import values +# SPDX-License-Identifier: GPL-2.0-only +# +# Copyright (C) 2019 EfficiOS Inc. +# + import unittest -import copy import bt2 +import re -@unittest.skip("this is broken") class QueryExecutorTestCase(unittest.TestCase): + def test_default_interrupter(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + query_exec = bt2.QueryExecutor(MySink, 'obj') + interrupter = query_exec.default_interrupter + self.assertIs(type(interrupter), bt2.Interrupter) + def test_query(self): class MySink(bt2._UserSinkComponent): - def _consume(self): + def _user_consume(self): pass @classmethod - def _query(cls, query_exec, obj, params): + def _user_query(cls, priv_query_exec, obj, params, method_obj): nonlocal query_params query_params = params - return { - 'null': None, - 'bt2': 'BT2', - } + return {'null': None, 'bt2': 'BT2'} query_params = None params = { 'array': ['coucou', 23, None], - 'other_map': { - 'yes': 'yeah', - '19': 19, - 'minus 1.5': -1.5, - }, + 'other_map': {'yes': 'yeah', '19': 19, 'minus 1.5': -1.5}, 'null': None, } - res = bt2.QueryExecutor().query(MySink, 'obj', params) + res = bt2.QueryExecutor(MySink, 'obj', params).query() + self.assertIs(type(res), bt2._MapValueConst) + self.assertIs(type(res['bt2']), bt2._StringValueConst) self.assertEqual(query_params, params) - self.assertEqual(res, { - 'null': None, - 'bt2': 'BT2', - }) + self.assertEqual(res, {'null': None, 'bt2': 'BT2'}) del query_params def test_query_params_none(self): class MySink(bt2._UserSinkComponent): - def _consume(self): + def _user_consume(self): pass @classmethod - def _query(cls, query_exec, obj, params): + def _user_query(cls, priv_query_exec, obj, params, method_obj): nonlocal query_params query_params = params query_params = 23 - res = bt2.QueryExecutor().query(MySink, 'obj', None) - self.assertEqual(query_params, None) + bt2.QueryExecutor(MySink, 'obj', None).query() + self.assertIs(query_params, None) del query_params + def test_query_no_params(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + nonlocal query_params + query_params = params + + query_params = 23 + bt2.QueryExecutor(MySink, 'obj').query() + self.assertIs(query_params, None) + del query_params + + def test_query_with_method_obj(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + nonlocal query_method_obj + query_method_obj = method_obj + + query_method_obj = None + method_obj = object() + bt2.QueryExecutor(MySink, 'obj', method_obj=method_obj).query() + self.assertIs(query_method_obj, method_obj) + del query_method_obj + + def test_query_with_method_obj_del_ref(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + nonlocal query_method_obj + query_method_obj = method_obj + + class Custom: + pass + + query_method_obj = None + method_obj = Custom() + method_obj.hola = 'hello' + query_exec = bt2.QueryExecutor(MySink, 'obj', method_obj=method_obj) + del method_obj + query_exec.query() + self.assertIsInstance(query_method_obj, Custom) + self.assertEqual(query_method_obj.hola, 'hello') + del query_method_obj + + def test_query_with_none_method_obj(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + nonlocal query_method_obj + query_method_obj = method_obj + + query_method_obj = object() + bt2.QueryExecutor(MySink, 'obj').query() + self.assertIsNone(query_method_obj) + del query_method_obj + + def test_query_with_method_obj_non_python_comp_cls(self): + plugin = bt2.find_plugin('text', find_in_user_dir=False, find_in_sys_dir=False) + assert plugin is not None + cc = plugin.source_component_classes['dmesg'] + assert cc is not None + + with self.assertRaisesRegex( + ValueError, + re.escape(r'cannot pass a Python object to a non-Python component class'), + ): + bt2.QueryExecutor(cc, 'obj', method_obj=object()).query() + + def test_query_logging_level(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + nonlocal query_log_level + query_log_level = priv_query_exec.logging_level + + query_log_level = None + query_exec = bt2.QueryExecutor(MySink, 'obj', None) + query_exec.logging_level = bt2.LoggingLevel.INFO + query_exec.query() + self.assertEqual(query_log_level, bt2.LoggingLevel.INFO) + del query_log_level + def test_query_gen_error(self): class MySink(bt2._UserSinkComponent): - def _consume(self): + def _user_consume(self): pass @classmethod - def _query(cls, query_exec, obj, params): + def _user_query(cls, priv_query_exec, obj, params, method_obj): raise ValueError - with self.assertRaises(bt2.Error): - res = bt2.QueryExecutor().query(MySink, 'obj', [17, 23]) + with self.assertRaises(bt2._Error) as ctx: + bt2.QueryExecutor(MySink, 'obj', [17, 23]).query() - def test_query_invalid_object(self): + exc = ctx.exception + self.assertEqual(len(exc), 3) + cause = exc[0] + self.assertIsInstance(cause, bt2._ComponentClassErrorCause) + self.assertIn('raise ValueError', cause.message) + self.assertEqual(cause.component_class_type, bt2.ComponentClassType.SINK) + self.assertEqual(cause.component_class_name, 'MySink') + + def test_query_unknown_object(self): class MySink(bt2._UserSinkComponent): - def _consume(self): + def _user_consume(self): pass @classmethod - def _query(cls, query_exec, obj, params): - raise bt2.InvalidQueryObject + def _user_query(cls, priv_query_exec, obj, params, method_obj): + raise bt2.UnknownObject - with self.assertRaises(bt2.InvalidQueryObject): - res = bt2.QueryExecutor().query(MySink, 'obj', [17, 23]) + with self.assertRaises(bt2.UnknownObject): + bt2.QueryExecutor(MySink, 'obj', [17, 23]).query() - def test_query_invalid_params(self): + def test_query_logging_level_invalid_type(self): class MySink(bt2._UserSinkComponent): - def _consume(self): + def _user_consume(self): pass @classmethod - def _query(cls, query_exec, obj, params): - raise bt2.InvalidQueryParams + def _user_query(cls, priv_query_exec, obj, params, method_obj): + pass + + query_exec = bt2.QueryExecutor(MySink, 'obj', [17, 23]) + + with self.assertRaises(TypeError): + query_exec.logging_level = 'yeah' - with self.assertRaises(bt2.InvalidQueryParams): - res = bt2.QueryExecutor().query(MySink, 'obj', [17, 23]) + def test_query_logging_level_invalid_value(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + pass + + query_exec = bt2.QueryExecutor(MySink, 'obj', [17, 23]) + + with self.assertRaises(ValueError): + query_exec.logging_level = 12345 def test_query_try_again(self): class MySink(bt2._UserSinkComponent): - def _consume(self): + def _user_consume(self): pass @classmethod - def _query(cls, query_exec, obj, params): + def _user_query(cls, priv_query_exec, obj, params, method_obj): raise bt2.TryAgain with self.assertRaises(bt2.TryAgain): - res = bt2.QueryExecutor().query(MySink, 'obj', [17, 23]) + bt2.QueryExecutor(MySink, 'obj', [17, 23]).query() + + def test_query_add_interrupter(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + nonlocal interrupter2 + test_self.assertFalse(query_exec.is_interrupted) + interrupter2.set() + test_self.assertTrue(query_exec.is_interrupted) + interrupter2.reset() + test_self.assertFalse(query_exec.is_interrupted) - def test_cancel_no_query(self): - query_exec = bt2.QueryExecutor() - self.assertFalse(query_exec.is_canceled) - query_exec.cancel() - self.assertTrue(query_exec.is_canceled) + interrupter1 = bt2.Interrupter() + interrupter2 = bt2.Interrupter() + test_self = self + query_exec = bt2.QueryExecutor(MySink, 'obj', [17, 23]) + query_exec.add_interrupter(interrupter1) + query_exec.add_interrupter(interrupter2) + query_exec.query() - def test_query_canceled(self): + def test_query_interrupt(self): class MySink(bt2._UserSinkComponent): - def _consume(self): + def _user_consume(self): pass @classmethod - def _query(cls, query_exec, obj, params): - raise bt2.TryAgain + def _user_query(cls, priv_query_exec, obj, params, method_obj): + test_self.assertFalse(query_exec.is_interrupted) + query_exec.default_interrupter.set() + test_self.assertTrue(query_exec.is_interrupted) + + test_self = self + query_exec = bt2.QueryExecutor(MySink, 'obj', [17, 23]) + query_exec.query() + + def test_query_priv_executor_invalid_after(self): + class MySink(bt2._UserSinkComponent): + def _user_consume(self): + pass + + @classmethod + def _user_query(cls, priv_query_exec, obj, params, method_obj): + nonlocal test_priv_query_exec + test_priv_query_exec = priv_query_exec + + test_priv_query_exec = None + query_exec = bt2.QueryExecutor(MySink, 'obj', [17, 23]) + query_exec.query() + assert test_priv_query_exec is not None - query_exec = bt2.QueryExecutor() - query_exec.cancel() + with self.assertRaises(RuntimeError): + test_priv_query_exec.logging_level - with self.assertRaises(bt2.QueryExecutorCanceled): - res = query_exec.query(MySink, 'obj', [17, 23]) + del test_priv_query_exec - def test_eq(self): - query_exec = bt2.QueryExecutor() - self.assertEqual(query_exec, query_exec) - def test_eq_invalid(self): - query_exec = bt2.QueryExecutor() - self.assertNotEqual(query_exec, 23) +if __name__ == '__main__': + unittest.main()