diff options
| -rw-r--r-- | checkers/stdlib.py | 2 | ||||
| -rw-r--r-- | test/functional/redundant_unittest_assert.py | 8 |
2 files changed, 9 insertions, 1 deletions
diff --git a/checkers/stdlib.py b/checkers/stdlib.py index 216d640d5..b6b802623 100644 --- a/checkers/stdlib.py +++ b/checkers/stdlib.py @@ -134,7 +134,7 @@ class StdlibChecker(BaseChecker): def _check_redundant_assert(self, node, infer): if (isinstance(infer, astroid.BoundMethod) and - isinstance(node.args[0], astroid.Const) and + node.args and isinstance(node.args[0], astroid.Const) and infer.name in ['assertTrue', 'assertFalse']): self.add_message('redundant-unittest-assert', args=(infer.name, node.args[0].value, ), diff --git a/test/functional/redundant_unittest_assert.py b/test/functional/redundant_unittest_assert.py index 4628824b2..2eb73e3d4 100644 --- a/test/functional/redundant_unittest_assert.py +++ b/test/functional/redundant_unittest_assert.py @@ -28,3 +28,11 @@ class Tests(unittest.TestCase): self.assertTrue('should be' in some_var, some_var) self.assertTrue(some_var, some_var) + + +@unittest.skip("don't run this") +class RegressionWithArgs(unittest.TestCase): + '''Don't fail if the bound method doesn't have arguments.''' + + def test(self): + self.run() |