summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRodolfo Carvalho <rhcarvalho@gmail.com>2017-03-06 10:35:15 +0100
committerRodolfo Carvalho <rhcarvalho@gmail.com>2017-03-06 14:03:57 +0100
commit3577754a29953c00f65f9b0b7ad257597c9ceb2d (patch)
treec818a1ed6e8f6994668ed76a1f9806756a5911fc
parent73b1a388623cf3336e97502280c1a97852b31411 (diff)
downloadopenshift-3577754a29953c00f65f9b0b7ad257597c9ceb2d.tar.gz
openshift-3577754a29953c00f65f9b0b7ad257597c9ceb2d.tar.bz2
openshift-3577754a29953c00f65f9b0b7ad257597c9ceb2d.tar.xz
openshift-3577754a29953c00f65f9b0b7ad257597c9ceb2d.zip
Remove redundant assertion
That line is testing Python's list.count method, instead of yedit. The assertion right above is a superset of it, as it checks for equality to some expected value.
-rwxr-xr-xroles/lib_utils/src/test/unit/test_yedit.py2
1 files changed, 0 insertions, 2 deletions
diff --git a/roles/lib_utils/src/test/unit/test_yedit.py b/roles/lib_utils/src/test/unit/test_yedit.py
index ce5e027a7..a22cdee56 100755
--- a/roles/lib_utils/src/test/unit/test_yedit.py
+++ b/roles/lib_utils/src/test/unit/test_yedit.py
@@ -200,8 +200,6 @@ class YeditTest(unittest.TestCase):
yed.append('x:y:z', [5, 6])
yed.append('x:y:z', [5, 6])
self.assertTrue(yed.get('x:y:z') == [1, 2, 3, [5, 6], [5, 6]])
- # pylint: disable=maybe-no-member
- self.assertTrue(2 == yed.get('x:y:z').count([5, 6]))
self.assertFalse(yed.exists('x:y:z', 4))
def test_add_item_to_dict(self):