diff options
| author | Bryn M. Reeves <bmr@redhat.com> | 2013-03-28 17:03:04 +0000 |
|---|---|---|
| committer | Bryn M. Reeves <bmr@redhat.com> | 2013-03-28 17:03:04 +0000 |
| commit | 1f90719d9d5b42d1c553ecded27a85965ce3197c (patch) | |
| tree | db7c9e3dbd01f93a691a9db8c86703ce9461941f /tests/option_tests.py | |
| parent | 2fcb1f4a6fbf5abda746bf1470edb55ccc6b01a6 (diff) | |
| download | sos-1f90719d9d5b42d1c553ecded27a85965ce3197c.tar.gz | |
Rename functions and methods in plugins to comply with pep8
There are lots of historical camelCase function and method names
in the plugin directory even though pep8 very clearly recommends
against these other than for external backwards compatibility.
Rename all the camelCased functions and methods and fix up the
main sosreport code, examples and tests to use the new names.
Fixes Issue #112.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
Diffstat (limited to 'tests/option_tests.py')
| -rw-r--r-- | tests/option_tests.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/option_tests.py b/tests/option_tests.py index 03804dec..6114c9b2 100644 --- a/tests/option_tests.py +++ b/tests/option_tests.py @@ -19,16 +19,16 @@ class GlobalOptionTest(unittest.TestCase): self.plugin.optParms = [{'enabled': False}, {'enabled': None}] def test_simple_lookup(self): - self.assertEquals(self.plugin.getOption('test_option'), 'foobar') + self.assertEquals(self.plugin.get_option('test_option'), 'foobar') def test_multi_lookup(self): - self.assertEquals(self.plugin.getOption(('not_there', 'test_option')), 'foobar') + self.assertEquals(self.plugin.get_option(('not_there', 'test_option')), 'foobar') def test_cascade(self): - self.assertEquals(self.plugin.getOption(('baz')), False) + self.assertEquals(self.plugin.get_option(('baz')), False) def test_none_should_cascade(self): - self.assertEquals(self.plugin.getOption(('empty', 'empty_global')), True) + self.assertEquals(self.plugin.get_option(('empty', 'empty_global')), True) if __name__ == "__main__": unittest.main() |