From b24c604a47b13a1367a5a0e8571e120ad1290052 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Tue, 24 Sep 2019 15:20:06 -0400 Subject: [PATCH] Fix: bt2: duplicate test name in test_field.py Flake8 reports: ./tests/bindings/python/bt2/test_field.py:2170:5: F811 redefinition of unused 'test_has_field_prop_true' from line 2166 ./tests/bindings/python/bt2/test_field.py:2178:5: F811 redefinition of unused 'test_bool_op_true' from line 2174 I think that in both cases, the second instance was meant to be renamed to `_false`. Reported-by: flake8 Change-Id: I8b3143bb3f86982e220a6b516f028fca9e6e24ab Signed-off-by: Simon Marchi Reviewed-on: https://review.lttng.org/c/babeltrace/+/2093 --- tests/bindings/python/bt2/test_field.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/bindings/python/bt2/test_field.py b/tests/bindings/python/bt2/test_field.py index 47755f2f..53906b97 100644 --- a/tests/bindings/python/bt2/test_field.py +++ b/tests/bindings/python/bt2/test_field.py @@ -2167,7 +2167,7 @@ class OptionFieldTestCase(unittest.TestCase): self._def.has_field = True self.assertTrue(self._def.has_field) - def test_has_field_prop_true(self): + def test_has_field_prop_false(self): self._def.has_field = False self.assertFalse(self._def.has_field) @@ -2175,7 +2175,7 @@ class OptionFieldTestCase(unittest.TestCase): self._def.value = 'allo' self.assertTrue(self._def) - def test_bool_op_true(self): + def test_bool_op_false(self): self._def.has_field = False self.assertFalse(self._def) -- 2.34.1