robotframework-robocop 5.7.0__tar.gz → 5.8.1__tar.gz

{robotframework_robocop-5.7.0/robotframework_robocop.egg-info → robotframework_robocop-5.8.1}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.2
 Name: robotframework-robocop
-Version: 5.7.0
+Version: 5.8.1
 Summary: Static code analysis tool (linter) for Robot Framework
 Home-page: https://github.com/MarketSquare/robotframework-robocop
 Download-URL: https://pypi.org/project/robotframework-robocop
@@ -19,6 +19,8 @@ Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
 Classifier: Framework :: Robot Framework
 Classifier: Framework :: Robot Framework :: Tool
 Classifier: Topic :: Software Development :: Testing
@@ -29,7 +31,7 @@ Requires-Python: >=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: jinja2<4.0,>=3.0
-Requires-Dist: robotframework<7.2,>=3.2.2
+Requires-Dist: robotframework<7.3,>=3.2.2
 Requires-Dist: pathspec<0.13,>=0.9
 Requires-Dist: tomli>=2.0.0
 Requires-Dist: pytz>=2022.7
@@ -49,6 +51,21 @@ Requires-Dist: sphinx_design; extra == "doc"
 Requires-Dist: sphinx-copybutton; extra == "doc"
 Requires-Dist: sphinxemoji; extra == "doc"
 Requires-Dist: pygments; extra == "doc"
+Dynamic: author
+Dynamic: author-email
+Dynamic: classifier
+Dynamic: description
+Dynamic: description-content-type
+Dynamic: download-url
+Dynamic: home-page
+Dynamic: keywords
+Dynamic: license
+Dynamic: platform
+Dynamic: project-url
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: requires-python
+Dynamic: summary
 
 [![Unit tests](https://github.com/MarketSquare/robotframework-robocop/actions/workflows/unit-tests.yml/badge.svg)](https://github.com/MarketSquare/robotframework-robocop/actions/workflows/unit-tests.yml "GitHub Workflow Unit Tests Status")
 ![Codecov](https://img.shields.io/codecov/c/github/MarketSquare/robotframework-robocop/master "Code coverage on master branch")

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1}/robocop/__init__.py

@@ -3,4 +3,4 @@ from robocop.config import Config
 from robocop.run import Robocop, run_robocop
 from robocop.version import __version__
 
-__all__ = ["__version__", "run_robocop", "checkers", "Robocop", "Config"]
+__all__ = ["Config", "Robocop", "__version__", "checkers", "run_robocop"]

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1}/robocop/checkers/misc.py

@@ -1,6 +1,7 @@
 """Miscellaneous checkers"""
 
 import ast
+import re
 from dataclasses import dataclass
 from pathlib import Path
 from typing import Optional
@@ -882,12 +883,20 @@ rules = {
         misreading your keyword arguments, explicitly state that the value is empty using the
         built-in ``${EMPTY}`` variable.
 
-        If your argument is falsly flagged by this rule, escape the ``=`` character in your argument
-        value by like so: ``\\=``.
-
         Example of a rule violation::
 
             My Amazing Keyword    argument_name=
+
+        Positional arguments that end with a ``=`` character can be falsly flagged by this rule. You
+        can fix this by making the situation more explicit:
+
+        1. Escape the ``=`` character::
+
+            Log    value\\=
+
+        2. Make it a named argument instead::
+
+            Log    message=value=
         """,
     ),
 }
@@ -1383,7 +1392,7 @@ class LoopStatementsChecker(VisitorChecker):
                 )
 
     def check_statement_in_loop(self, node, token_type):
-        if self.loops or node.errors and f"{token_type} can only be used inside a loop." not in node.errors:
+        if self.loops or (node.errors and f"{token_type} can only be used inside a loop." not in node.errors):
             return
         self.report(
             "statement-outside-loop",
@@ -1642,6 +1651,11 @@ class UnusedVariablesChecker(VisitorChecker):
         if node.next:
             self.visit_Try(node.next)
 
+    def visit_Group(self, node):  # noqa: N802
+        for token in node.header.get_tokens(Token.ARGUMENT):
+            self.find_not_nested_variable(token.value, is_var=False)
+        self.generic_visit(node)
+
     def visit_KeywordCall(self, node):  # noqa: N802
         for token in node.get_tokens(Token.ARGUMENT, Token.KEYWORD):  # argument can be used in keyword name
             self.find_not_nested_variable(token.value, is_var=False)
@@ -1992,6 +2006,7 @@ class UndefinedArgumentDefaultChecker(VisitorChecker):
         "undefined-argument-default",
         "undefined-argument-value",
     )
+    valid_argument_name = re.compile(r"[a-zA-Z0-9-_ ]+")
 
     def visit_Arguments(self, node: Arguments):  # noqa: N802
         for token in node.get_tokens(Token.ARGUMENT):
@@ -2029,10 +2044,19 @@ class UndefinedArgumentDefaultChecker(VisitorChecker):
                 # `=` is escaped
                 continue
 
+            if arg_name.endswith(" "):
+                # Space before `=` is not a named arg
+                continue
+
             if default_val != "":
                 # Has a value
                 continue
 
+            is_plain_var_name = self.valid_argument_name.fullmatch(arg_name)
+            if is_plain_var_name is None:
+                # Argument name includes invalid chars
+                continue
+
             # Falsly triggers if a positional argument ends with `=`
             # The language server has the same behavior
             self.report(

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1}/robocop/checkers/spacing.py

@@ -469,7 +469,7 @@ class EmptyLinesChecker(VisitorChecker):
         self.verify_consecutive_empty_lines(node.body, check_trailing=True)
         self.generic_visit(node)
 
-    visit_ForLoop = visit_While = visit_Try = visit_If = visit_For  # noqa: N815
+    visit_ForLoop = visit_While = visit_Try = visit_If = visit_Group = visit_For  # noqa: N815
 
     def visit_File(self, node):  # noqa: N802
         for section in node.sections:
@@ -707,7 +707,7 @@ class UnevenIndentChecker(VisitorChecker):
                 self.visit(child)
         self.visit_Statement(node.end)
 
-    visit_While = visit_ForLoop = visit_For  # noqa: N815
+    visit_While = visit_ForLoop = visit_Group = visit_For  # noqa: N815
 
     def get_common_if_indent(self, node):
         indents = count_indents(node)

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1}/robocop/utils/disablers.py

@@ -73,7 +73,7 @@ class DisablersVisitor(ModelVisitor):
         self.parse_disablers_in_node(node)
         self.is_first_comment_section = False
 
-    visit_TestCase = visit_Keyword = visit_Try = visit_For = visit_ForLoop = visit_While = visit_Section  # noqa: N815
+    visit_TestCase = visit_Keyword = visit_Try = visit_For = visit_ForLoop = visit_While = visit_Group = visit_Section  # noqa: N815
 
     def visit_If(self, node):  # noqa: N802
         last_line = node.body[-1].end_lineno if node.body else None

robotframework_robocop-5.8.1/robocop/version.py

@@ -0,0 +1 @@
+__version__ = "5.8.1"

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1/robotframework_robocop.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.2
 Name: robotframework-robocop
-Version: 5.7.0
+Version: 5.8.1
 Summary: Static code analysis tool (linter) for Robot Framework
 Home-page: https://github.com/MarketSquare/robotframework-robocop
 Download-URL: https://pypi.org/project/robotframework-robocop
@@ -19,6 +19,8 @@ Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
 Classifier: Framework :: Robot Framework
 Classifier: Framework :: Robot Framework :: Tool
 Classifier: Topic :: Software Development :: Testing
@@ -29,7 +31,7 @@ Requires-Python: >=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: jinja2<4.0,>=3.0
-Requires-Dist: robotframework<7.2,>=3.2.2
+Requires-Dist: robotframework<7.3,>=3.2.2
 Requires-Dist: pathspec<0.13,>=0.9
 Requires-Dist: tomli>=2.0.0
 Requires-Dist: pytz>=2022.7
@@ -49,6 +51,21 @@ Requires-Dist: sphinx_design; extra == "doc"
 Requires-Dist: sphinx-copybutton; extra == "doc"
 Requires-Dist: sphinxemoji; extra == "doc"
 Requires-Dist: pygments; extra == "doc"
+Dynamic: author
+Dynamic: author-email
+Dynamic: classifier
+Dynamic: description
+Dynamic: description-content-type
+Dynamic: download-url
+Dynamic: home-page
+Dynamic: keywords
+Dynamic: license
+Dynamic: platform
+Dynamic: project-url
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: requires-python
+Dynamic: summary
 
 [![Unit tests](https://github.com/MarketSquare/robotframework-robocop/actions/workflows/unit-tests.yml/badge.svg)](https://github.com/MarketSquare/robotframework-robocop/actions/workflows/unit-tests.yml "GitHub Workflow Unit Tests Status")
 ![Codecov](https://img.shields.io/codecov/c/github/MarketSquare/robotframework-robocop/master "Code coverage on master branch")

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1}/robotframework_robocop.egg-info/requires.txt

@@ -1,5 +1,5 @@
 jinja2<4.0,>=3.0
-robotframework<7.2,>=3.2.2
+robotframework<7.3,>=3.2.2
 pathspec<0.13,>=0.9
 tomli>=2.0.0
 pytz>=2022.7

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1}/setup.py

@@ -14,6 +14,8 @@ Programming Language :: Python
 Programming Language :: Python :: 3.9
 Programming Language :: Python :: 3.10
 Programming Language :: Python :: 3.11
+Programming Language :: Python :: 3.12
+Programming Language :: Python :: 3.13
 Framework :: Robot Framework
 Framework :: Robot Framework :: Tool
 Topic :: Software Development :: Testing
@@ -50,7 +52,7 @@ setup(
     include_package_data=True,
     install_requires=[
         "jinja2>=3.0,<4.0",
-        "robotframework>=3.2.2,<7.2",
+        "robotframework>=3.2.2,<7.3",
         "pathspec>=0.9,<0.13",
         "tomli>=2.0.0",
         "pytz>=2022.7",

robotframework_robocop-5.7.0/robocop/version.py

@@ -1 +0,0 @@
-__version__ = "5.7.0"

{robotframework_robocop-5.7.0 → robotframework_robocop-5.8.1}/robocop/utils/__init__.py

@@ -22,11 +22,11 @@ from robocop.utils.misc import (
 )
 
 __all__ = [  # TODO: replace with direct imports
+    "ROBOT_VERSION",
+    "AssignmentTypeDetector",
     "DisablersFinder",
     "FileType",
     "FileTypeChecker",
-    "ROBOT_VERSION",
-    "AssignmentTypeDetector",
     "RecommendationFinder",
     "find_robot_vars",
     "get_errors",