diff options
| author | Dylan Baker <baker.dylan.c@gmail.com> | 2014-04-23 17:10:55 -0700 |
|---|---|---|
| committer | Dylan Baker <baker.dylan.c@gmail.com> | 2014-05-13 16:06:49 -0700 |
| commit | 0dddc26971c66dc663d439ad96ffd3768eedb6f5 (patch) | |
| tree | 1c336c45799d30d8e41550fb4ae12f4fe02ed8ea /framework | |
| parent | b594165abba86d14038ff655c1f0dda7cc1e4cac (diff) | |
profile.py: Cleanup a few pylint warnings
This adds self._dmesg definition to the TestProfile constructor (this
isn't necessary but it's good form), and renames self.__dmesg to
self._dmesg. A double underscore is not about making an name or method
private, it's about making it invisible to that class's children. We
don't want to hide self._dmesg from any children of TestProfile.
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
Diffstat (limited to 'framework')
| -rw-r--r-- | framework/profile.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/framework/profile.py b/framework/profile.py index 25f82b0c8..7decef08f 100644 --- a/framework/profile.py +++ b/framework/profile.py @@ -62,13 +62,14 @@ class TestProfile(object): self.test_list = {} self.filters = [] # Sets a default of a Dummy + self._dmesg = None self.dmesg = False self.results_dir = None @property def dmesg(self): """ Return dmesg """ - return self.__dmesg + return self._dmesg @dmesg.setter def dmesg(self, not_dummy): @@ -79,7 +80,7 @@ class TestProfile(object): will get a DummyDmesg """ - self.__dmesg = get_dmesg(not_dummy) + self._dmesg = get_dmesg(not_dummy) def flatten_group_hierarchy(self): """ Flatten nested dictionary structure |