diff options
| author | Rodolfo Carvalho <rhcarvalho@gmail.com> | 2017-03-06 10:35:15 +0100 |
|---|---|---|
| committer | Rodolfo Carvalho <rhcarvalho@gmail.com> | 2017-03-06 14:03:57 +0100 |
| commit | 3577754a29953c00f65f9b0b7ad257597c9ceb2d (patch) | |
| tree | c818a1ed6e8f6994668ed76a1f9806756a5911fc | |
| parent | 73b1a388623cf3336e97502280c1a97852b31411 (diff) | |
| download | openshift-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-x | roles/lib_utils/src/test/unit/test_yedit.py | 2 |
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): |