From: Simon Marchi Date: Tue, 24 Sep 2019 19:20:06 +0000 (-0400) Subject: Fix: bt2: duplicate test name in test_field.py X-Git-Url: http://git.efficios.com/?p=babeltrace.git;a=commitdiff_plain;h=d4db9beb2e8e00eaa60a5f89f745b1add05a7d64 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 --- 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)