From 61c4ecccc1df222b153236518c9be0ac3091f94a Mon Sep 17 00:00:00 2001 From: Petr Viktorin Date: Tue, 16 Dec 2014 14:45:37 +0100 Subject: Run pylint on tests Drop support for pylint < 1.0 Enable ignoring unknown attributes on modules (both nose and pytest use advanced techniques, support for which only made it to pylint recently) Fix some bugs revealed by pylint Do minor refactoring or add pylint:disable directives where the linter complains. Reviewed-By: Tomas Babej --- ipatests/test_ipapython/test_ipautil.py | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) (limited to 'ipatests/test_ipapython') diff --git a/ipatests/test_ipapython/test_ipautil.py b/ipatests/test_ipapython/test_ipautil.py index c882df2cd..8c0c68909 100644 --- a/ipatests/test_ipapython/test_ipautil.py +++ b/ipatests/test_ipapython/test_ipautil.py @@ -128,6 +128,11 @@ class TestCIDict(object): nose.tools.assert_equal(3, len(self.cidict)) self.cidict.clear() nose.tools.assert_equal(0, len(self.cidict)) + assert self.cidict == {} + assert self.cidict.keys() == [] + assert self.cidict.values() == [] + assert self.cidict.items() == [] + assert self.cidict._keys == {} def test_copy(self): copy = self.cidict.copy() @@ -309,14 +314,6 @@ class TestCIDict(object): assert sorted(dct.keys()) == sorted(['A', 'b', 'C']) assert sorted(dct.values()) == [None] * 3 - def test_clear(self): - self.cidict.clear() - assert self.cidict == {} - assert self.cidict.keys() == [] - assert self.cidict.values() == [] - assert self.cidict.items() == [] - assert self.cidict._keys == {} - class TestTimeParser(object): def test_simple(self): -- cgit