17 May
2017
17 May
'17
5:23 p.m.
On 17.5.2017 17:08, Stephen Warren wrote:
On 05/17/2017 03:17 AM, Michal Simek wrote:
All tests in test_hush_if_test depends on hush parser to be present. This patch simplify test dependencies by using global pytestmark.
Nit: That seems to have been accidentally line-wrapped.
diff --git a/test/py/tests/test_hush_if_test.py b/test/py/tests/test_hush_if_test.py
+pytestmark = pytest.mark.buildconfigspec('hush_parser')
This is fine to unify the marking, but I was actually talking about adding a new dependency on cmd_echo, which I assume can be turned off even with hush_parser enabled?
Definitely dependencies should be for both.
Thanks, Michal