Fix linters checks

Since we already have old-style unittest, leaving it as is till mass
migration to pytest. And ingoring raised by this choice warnings.

.flake8

@@ -94,6 +94,7 @@ per-file-ignores =
   test/TestMetaTagValid.py: PT009 D100 D101 D102
   test/TestMetaVideoLinks.py: PT009 D100 D101 D102
   test/TestNoFormattingInWhenRule.py: PT009 D100 D101 D102
+  test/TestNoLogPasswordsRule.py: PT009 D100 D101 D102
   test/TestOctalPermissions.py: PT009 D100 D101 D102
   test/TestPackageIsNotLatest.py: PT009 D100 D101 D102
   test/TestPretaskIncludePlaybook.py: D100 D103

src/ansiblelint/rules/NoLogPasswordsRule.py

@@ -48,4 +48,6 @@ def matchtask(
         # No no_log and no_log: False behave the same way
         # and should return a failure (return True), so we
         # need to invert the boolean
-        return bool(has_password and not convert_to_boolean(task['action'].get('no_log', False)))
+        return bool(
+            has_password and not convert_to_boolean(task['action'].get('no_log', False))
+        )

test/TestNoLogPasswordsRule.py

@@ -1,8 +1,9 @@
+# pylint: disable=preferred-module  # FIXME: remove once migrated per GH-725
 import unittest
 
 from ansiblelint.rules import RulesCollection
-from ansiblelint.runner import Runner
 from ansiblelint.rules.NoLogPasswordsRule import NoLogPasswordsRule
+from ansiblelint.runner import Runner
 
 
 class TestNoLogPasswordsRule(unittest.TestCase):

test/TestRulesCollection.py

@@ -128,4 +128,4 @@ def test_rules_id_format() -> None:
         assert rule_id_re.match(
             rule.id
         ), f"R rule id {rule.id} did not match our required format."
-    assert len(rules) == 39
+    assert len(rules) == 40