diff options
Diffstat (limited to 'tools/pylint/plugins/cim_provider_checker.py')
| -rw-r--r-- | tools/pylint/plugins/cim_provider_checker.py | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/tools/pylint/plugins/cim_provider_checker.py b/tools/pylint/plugins/cim_provider_checker.py index 8691b5a..a1b557b 100644 --- a/tools/pylint/plugins/cim_provider_checker.py +++ b/tools/pylint/plugins/cim_provider_checker.py @@ -78,10 +78,6 @@ class CIMProviderChecker(BaseChecker): name = 'cim_provider' msgs = { - 'C9904': ('Invalid provider class name %s', - "Class name representing cim provider should be in inform " - "<prefix>_<name>. Where <prefix> and <name> should be both " - "written in CamelCase."), 'C9905': ('Invalid provider module name %s', "Module containing cim provider(s) should be named as " "<prefix>_<name>. Where both <prefix> and <name> are " @@ -114,21 +110,17 @@ class CIMProviderChecker(BaseChecker): """ if "CIMProvider2" in [a.name for a in node.ancestors()]: supress_cim_provider_messages(self.linter, node) - clsm = _RE_PROVIDER_CLASS_NAME.match(node.name) - if not clsm: - self.add_message('C9904', node=node, args=node.name) parent = node.parent while not isinstance(parent, scoped_nodes.Module): parent = parent.parent + clsm = _RE_PROVIDER_CLASS_NAME.match(node.name) modm = _RE_PROVIDER_MODULE_NAME.match(parent.name) - if not modm: + if clsm and not modm: self.add_message('C9905', node=node, args=parent.name) - if not clsm: - return - if clsm.group('prefix') != modm.group('prefix'): + if clsm and clsm.group('prefix') != modm.group('prefix'): self.add_message('C9906', node=node, args=(modm.group('prefix'), clsm.group('prefix'))) - if clsm.group('prefix') != 'LMI': + if clsm and clsm.group('prefix') != 'LMI': self.add_message('C9910', node=node, args=clsm.group('prefix')) if not 'get_providers' in parent.keys(): self.add_message('E9907', node=parent, args=parent.name) |