feat: add frequent typo-catching rules

CHANGELOG.md

@@ -5,6 +5,9 @@ All notable changes in **salt-lint** are documented below.
 ### Deprecated
 - Drop Python 2.7 support ((#239)[https://github.com/warpnet/salt-lint/pull/239]).
 
+### Added
+- Rule 216, 217 and 218 for catching common typographical errors ((#249)[https://github.com/warpnet/salt-lint/pull/249]).
+
 ## [0.5.2] (2021-01-29)
 ### Fixed
 - Append the contents of the `CHANGELOG.md` file to the long description of the package instead of the duplicate `README.md` contents ([#234](https://github.com/warpnet/salt-lint/pull/234)).

saltlint/linter/rule.py

@@ -154,3 +154,13 @@ def regex(self):
 
     def match(self, file, line):
         return self.regex.search(line)
+
+
+class TypographicalErrorRule(Rule):
+    """Base rule for catching common typographical errors."""
+    severity = 'LOW'
+    languages = [LANGUAGE_SLS]
+    tags = ['formatting']
+
+    def match(self, file, line):
+        return self.regex.search(line)

saltlint/rules/TypoContentsRule.py

@@ -0,0 +1,16 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2016 Will Thames and contributors
+# Copyright (c) 2018 Ansible Project
+# Modified work Copyright (c) 2021 Warpnet B.V.
+# Modified work Copyright (c) 2021 Yury Bushmelev
+
+import re
+from saltlint.linter.rule import TypographicalErrorRule
+
+
+class TypoContentsRule(TypographicalErrorRule):
+    id = '218'
+    shortdesc = '"content" looks like a typo. Did you mean "contents"?'
+    description = '"content" looks like a typo. Did you mean "contents"?'
+    version_added = 'develop'
+    regex = re.compile(r"^\s+- content(|_pillar|_grains|_newline|_delimiter):")

saltlint/rules/TypoOnchangesRule.py

@@ -0,0 +1,16 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2016 Will Thames and contributors
+# Copyright (c) 2018 Ansible Project
+# Modified work Copyright (c) 2021 Warpnet B.V.
+# Modified work Copyright (c) 2021 Yury Bushmelev
+
+import re
+from saltlint.linter.rule import TypographicalErrorRule
+
+
+class TypoOnchangesRule(TypographicalErrorRule):
+    id = '216'
+    shortdesc = '"onchange" looks like a typo. Did you mean "onchanges"?'
+    description = '"onchange" looks like a typo. Did you mean "onchanges"?'
+    version_added = 'develop'
+    regex = re.compile(r"^\s+- onchange(|_in|_any):")

saltlint/rules/TypoRequireRule.py

@@ -0,0 +1,16 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2016 Will Thames and contributors
+# Copyright (c) 2018 Ansible Project
+# Modified work Copyright (c) 2021 Warpnet B.V.
+# Modified work Copyright (c) 2021 Yury Bushmelev
+
+import re
+from saltlint.linter.rule import TypographicalErrorRule
+
+
+class TypoRequireRule(TypographicalErrorRule):
+    id = '217'
+    shortdesc = '"requires" looks like a typo. Did you mean "require"?'
+    description = '"requires" looks like a typo. Did you mean "require"?'
+    version_added = 'develop'
+    regex = re.compile(r"^\s+- requires(|_in|_any):")

setup.py

@@ -45,7 +45,7 @@ def long_description():
 setup(
     name=NAME,
     version=VERSION,
-    description=DESCRIPTION.split('\n')[0],
+    description=DESCRIPTION.split('\n')[0],  # pylint: disable=C0207
     long_description=long_description(),
     long_description_content_type='text/markdown',
     author=__author__,

tests/unit/TestTypoContentsRule.py

@@ -0,0 +1,55 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2013-2018 Will Thames <will@thames.id.au>
+# Copyright (c) 2018 Ansible by Red Hat
+# Modified work Copyright (c) 2020 Warpnet B.V.
+# Copyright (c) 2021 Yury Bushmelev
+
+import unittest
+
+from saltlint.linter.collection import RulesCollection
+from saltlint.rules.TypoContentsRule import TypoContentsRule
+from tests import RunFromText
+
+
+GOOD_CONTENTS_LINE = '''
+testfile:
+  file.managed:
+    - name: /tmp/testfile
+    - user: root
+    - group: root
+    - mode: '0700'
+    - contents: test
+    - contents_pillar: foo
+    - contents_grains: bar
+    - contents_newline: True
+    - contents_delimiter: '!'
+'''
+
+BAD_CONTENTS_LINE = '''
+testfile:
+  file.managed:
+    - name: /tmp/badfile
+    - user: root
+    - group: root
+    - mode: 0700
+    - content: test
+    - content_pillar: foo
+    - content_grains: bar
+    - content_newline: True
+    - content_delimiter: '!'
+'''
+
+class TestTypoContentsRule(unittest.TestCase):
+    collection = RulesCollection()
+
+    def setUp(self):
+        self.collection.register(TypoContentsRule())
+        self.runner = RunFromText(self.collection)
+
+    def test_statement_positive(self):
+        results = self.runner.run_state(GOOD_CONTENTS_LINE)
+        self.assertEqual(0, len(results))
+
+    def test_statement_negative(self):
+        results = self.runner.run_state(BAD_CONTENTS_LINE)
+        self.assertEqual(5, len(results))

tests/unit/TestTypoOnchangesRule.py

@@ -0,0 +1,59 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2013-2018 Will Thames <will@thames.id.au>
+# Copyright (c) 2018 Ansible by Red Hat
+# Modified work Copyright (c) 2020 Warpnet B.V.
+# Copyright (c) 2021 Yury Bushmelev
+
+import unittest
+
+from saltlint.linter.collection import RulesCollection
+from saltlint.rules.TypoOnchangesRule import TypoOnchangesRule
+from tests import RunFromText
+
+
+GOOD_ONCHANGES_LINE = '''
+testfile:
+  file.managed:
+    - name: /tmp/testfile
+    - user: root
+    - group: root
+    - mode: '0700'
+    - onchanges:
+        - otherfile
+    - onchanges_in:
+        - anotherfile
+    - onchanges_any:
+        - yetanotherfile
+        - onemorefile
+'''
+
+BAD_ONCHANGES_LINE = '''
+testfile:
+  file.managed:
+    - name: /tmp/badfile
+    - user: root
+    - group: root
+    - mode: 0700
+    - onchange:
+        - otherfile
+    - onchange_in:
+        - anotherfile
+    - onchange_any:
+        - yetanotherfile
+        - onemorefile
+'''
+
+class TestTypoOnchangesRule(unittest.TestCase):
+    collection = RulesCollection()
+
+    def setUp(self):
+        self.collection.register(TypoOnchangesRule())
+        self.runner = RunFromText(self.collection)
+
+    def test_statement_positive(self):
+        results = self.runner.run_state(GOOD_ONCHANGES_LINE)
+        self.assertEqual(0, len(results))
+
+    def test_statement_negative(self):
+        results = self.runner.run_state(BAD_ONCHANGES_LINE)
+        self.assertEqual(3, len(results))

tests/unit/TestTypoRequireRule.py

@@ -0,0 +1,59 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2013-2018 Will Thames <will@thames.id.au>
+# Copyright (c) 2018 Ansible by Red Hat
+# Modified work Copyright (c) 2020 Warpnet B.V.
+# Copyright (c) 2021 Yury Bushmelev
+
+import unittest
+
+from saltlint.linter.collection import RulesCollection
+from saltlint.rules.TypoRequireRule import TypoRequireRule
+from tests import RunFromText
+
+
+GOOD_REQUIRE_LINE = '''
+testfile:
+  file.managed:
+    - name: /tmp/testfile
+    - user: root
+    - group: root
+    - mode: '0700'
+    - require:
+        - otherfile
+    - require_in:
+        - anotherfile
+    - require_any:
+        - yetanotherfile
+        - onemorefile
+'''
+
+BAD_REQUIRE_LINE = '''
+testfile:
+  file.managed:
+    - name: /tmp/badfile
+    - user: root
+    - group: root
+    - mode: 0700
+    - requires:
+        - otherfile
+    - requires_in:
+        - anotherfile
+    - requires_any:
+        - yetanotherfile
+        - onemorefile
+'''
+
+class TestTypoRequireRule(unittest.TestCase):
+    collection = RulesCollection()
+
+    def setUp(self):
+        self.collection.register(TypoRequireRule())
+        self.runner = RunFromText(self.collection)
+
+    def test_statement_positive(self):
+        results = self.runner.run_state(GOOD_REQUIRE_LINE)
+        self.assertEqual(0, len(results))
+
+    def test_statement_negative(self):
+        results = self.runner.run_state(BAD_REQUIRE_LINE)
+        self.assertEqual(3, len(results))