Release 5.9.0: Group support

docs/releasenotes/5.9.0.rst

@@ -0,0 +1,34 @@
+:orphan:
+
+=============
+Robocop 5.8.0
+=============
+
+Update Robocop rules with GROUP support from Robot Framework 7.2.
+
+Following rules are updated & tested with GROUPs:
+
+- ``invalid-if``
+- ``missing-keyword-name``
+- ``parsing-error``
+- ``return-in-test-case``
+- ``too-many-calls-in-keyword``
+- ``too-many-calls-in-test-case``
+- ``possible-variable-overwriting``
+- ``bad-indent``
+- ``bad-block-indent``
+
+Acknowledgements
+================
+
+Thanks to the whole community for submitting bug reports and feature requests.
+Without you, Robocop wouldn't be in the place where it is now. All the feedback
+is essential to drive the tool towards higher quality and better user
+experience.
+
+If you want to help us more, consider contributing to the project directly.
+We can offer our constant support to make the work fun and effective. We do
+our best to create a supportive and welcoming environment for everyone.
+Feel free to ping us on our official `#robocop-linter Slack channel`_ anytime.
+
+.. _#robocop-linter Slack channel: https://robotframework.slack.com/archives/C01AWSNKC2H

robocop/version.py

@@ -1 +1 @@
-__version__ = "5.8.0"
+__version__ = "5.9.0"

tests/atest/rules/errors/parsing_error/groups.robot

@@ -0,0 +1,38 @@
+*** Test Cases ***
+END missing
+    [Documentation]    FAIL    GROUP must have closing END.
+    GROUP    This is not closed
+        Fail    Not run
+
+Empty
+    [Documentation]    FAIL    GROUP cannot be empty.
+    GROUP    This is empty
+    END
+    Log    Outside
+
+Multiple parameters
+    [Documentation]    FAIL    GROUP accepts only one argument as name, got 3 arguments 'Too', 'many' and 'values'.
+    GROUP    Too    many    values
+        Fail    Not run
+    END
+    Log   Last Keyword
+
+Non-existing variable in name
+    [Documentation]    FAIL    Variable '\${non_existing_var}' not found.
+    GROUP    ${non_existing_var} in name
+        Fail    Not run
+    END
+    Log   Last Keyword
+
+Invalid data is not reported after failures
+    [Documentation]    FAIL    Something bad happened!
+    # We probably should validate syntax before even executing the test and report
+    # such failures early. That should then be done also with other control structures.
+    Fail    Something bad happened!
+    GROUP    ${non_existing_non_executed_variable_is_ok}
+        Fail    Not run
+    END
+    GROUP    Empty non-executed GROUP is ok
+    END
+    GROUP    Even missing END is ok
+        Fail    Not run