Support for RF 7.2 GROUP syntax (#733)

Author: bhirsz

robotidy/api.py

@@ -1,6 +1,7 @@
 """
 Methods for transforming Robot Framework ast model programmatically.
 """
+
 from __future__ import annotations
 
 from pathlib import Path
@@ -37,7 +38,8 @@ def transform_model(model, root_dir: str, output: str | None = None, **kwargs) -
     """
     robotidy_class = get_robotidy(root_dir, output, **kwargs)
     disabler_finder = disablers.RegisterDisablers(
-        robotidy_class.config.formatting.start_line, robotidy_class.config.formatting.end_line
+        robotidy_class.config.formatting.start_line,
+        robotidy_class.config.formatting.end_line,
     )
     disabler_finder.visit(model)
     if disabler_finder.is_disabled_in_file(disablers.ALL_TRANSFORMERS):

robotidy/app.py

@@ -61,7 +61,10 @@ def transform_files(self):
                     self.output_diff(model_path, old_model, new_model)
                     changed_files += 1
             except DataError as err:
-                click.echo(f"Failed to decode {source} with an error: {err}\nSkipping file", err=True)
+                click.echo(
+                    f"Failed to decode {source} with an error: {err}\nSkipping file",
+                    err=True,
+                )
                 changed_files = previous_changed_files
                 skipped_files += 1
         return self.formatting_result(all_files, changed_files, skipped_files, stdin)
@@ -141,7 +144,12 @@ def get_line_ending(self, path: str):
                     return f.newlines[0]
         return self.config.formatting.line_sep
 
-    def output_diff(self, path: str, old_model: misc.StatementLinesCollector, new_model: misc.StatementLinesCollector):
+    def output_diff(
+        self,
+        path: str,
+        old_model: misc.StatementLinesCollector,
+        new_model: misc.StatementLinesCollector,
+    ):
         if not self.config.show_diff:
             return
         old = [l + "\n" for l in old_model.text.splitlines()]

robotidy/cli.py

@@ -55,7 +55,10 @@
             "--endline",
         ],
     },
-    {"name": "File exclusion", "options": ["--exclude", "--extend-exclude", "--skip-gitignore"]},
+    {
+        "name": "File exclusion",
+        "options": ["--exclude", "--extend-exclude", "--skip-gitignore"],
+    },
     skip.option_group,
     {
         "name": "Other",
@@ -124,7 +127,11 @@ def print_transformer_docs(transformer):
 @decorators.optional_rich
 def print_description(name: str, target_version: int):
     # TODO: --desc works only for default transformers, it should also print custom transformer desc
-    transformers = load_transformers(TransformConfigMap([], [], []), allow_disabled=True, target_version=target_version)
+    transformers = load_transformers(
+        TransformConfigMap([], [], []),
+        allow_disabled=True,
+        target_version=target_version,
+    )
     transformer_by_names = {transformer.name: transformer for transformer in transformers}
     if name == "all":
         for transformer in transformers:
@@ -159,7 +166,11 @@ def print_transformers_list(global_config: config_module.MainConfig):
     table = Table(title="Transformers", header_style="bold red")
     table.add_column("Name", justify="left", no_wrap=True)
     table.add_column("Enabled")
-    transformers = load_transformers(TransformConfigMap([], [], []), allow_disabled=True, target_version=target_version)
+    transformers = load_transformers(
+        TransformConfigMap([], [], []),
+        allow_disabled=True,
+        target_version=target_version,
+    )
     transformers.extend(_load_external_transformers(transformers, config.transformers_config, target_version))
 
     for transformer in transformers:
@@ -194,7 +205,11 @@ def generate_config(global_config: config_module.MainConfig):
         raise exceptions.MissingOptionalTomliWDependencyError()
     target_version = global_config.default.target_version
     config = global_config.default_loaded
-    transformers = load_transformers(TransformConfigMap([], [], []), allow_disabled=True, target_version=target_version)
+    transformers = load_transformers(
+        TransformConfigMap([], [], []),
+        allow_disabled=True,
+        target_version=target_version,
+    )
     transformers.extend(_load_external_transformers(transformers, config.transformers_config, target_version))
 
     toml_config = {

robotidy/config.py

@@ -98,7 +98,12 @@ def convert_transformers_config(
     is_config: bool = False,
 ) -> list[TransformConfig]:
     return [
-        TransformConfig(tr, force_include=force_included, custom_transformer=custom_transformer, is_config=is_config)
+        TransformConfig(
+            tr,
+            force_include=force_included,
+            custom_transformer=custom_transformer,
+            is_config=is_config,
+        )
         for tr in config.get(param_name, ())
     ]
 
@@ -186,7 +191,10 @@ def from_config_file(self, config: dict, config_path: Path) -> "RawConfig":
         Dictionary key:values needs to be normalized and parsed to correct types.
         """
         options_map = map_class_fields_with_their_types(self)
-        parsed_config = {"defined_in_config": {"defined_in_config", "config_path"}, "config_path": config_path}
+        parsed_config = {
+            "defined_in_config": {"defined_in_config", "config_path"},
+            "config_path": config_path,
+        }
         for key, value in config.items():
             # workaround to be able to use two option names for same action - backward compatibility change
             if key == "load_transformers":
@@ -206,7 +214,10 @@ def from_config_file(self, config: dict, config_path: Path) -> "RawConfig":
                 parsed_config[key] = [convert_transform_config(val, key) for val in value]
             elif key == "src":
                 parsed_config[key] = tuple(value)
-            elif value_type in ("Pattern", Pattern):  # future typing for 3.8 provides type as str
+            elif value_type in (
+                "Pattern",
+                Pattern,
+            ):  # future typing for 3.8 provides type as str
                 parsed_config[key] = misc.validate_regex(value)
             else:
                 parsed_config[key] = value
@@ -273,7 +284,10 @@ def get_sources(self, sources: tuple[str, ...]) -> tuple[str, ...] | None:
 
     def get_sources_with_configs(self):
         sources = files.get_paths(
-            self.sources, self.default.exclude, self.default.extend_exclude, self.default.skip_gitignore
+            self.sources,
+            self.default.exclude,
+            self.default.extend_exclude,
+            self.default.skip_gitignore,
         )
         for source in sources:
             if self.default.config:

robotidy/disablers.py

@@ -66,7 +66,12 @@ def is_line_start(node):
 
 
 class DisablersInFile:
-    def __init__(self, start_line: Optional[int], end_line: Optional[int], file_end: Optional[int] = None):
+    def __init__(
+        self,
+        start_line: Optional[int],
+        end_line: Optional[int],
+        file_end: Optional[int] = None,
+    ):
         self.start_line = start_line
         self.end_line = end_line
         self.file_end = file_end

robotidy/files.py

@@ -128,7 +128,12 @@ def get_path_relative_to_project_root(path: Path, root_parent: Path) -> Path:
         return path
 
 
-def get_paths(src: tuple[str, ...], exclude: Pattern | None, extend_exclude: Pattern | None, skip_gitignore: bool):
+def get_paths(
+    src: tuple[str, ...],
+    exclude: Pattern | None,
+    extend_exclude: Pattern | None,
+    skip_gitignore: bool,
+):
     root = find_project_root(src)
     if skip_gitignore:
         gitignore = None

robotidy/skip.py

@@ -114,7 +114,16 @@ def __init__(self, skip_config: SkipConfig):
 
     @staticmethod
     def parse_skip_settings(skip_config):
-        settings = {"settings", "arguments", "setup", "teardown", "timeout", "template", "return_statement", "tags"}
+        settings = {
+            "settings",
+            "arguments",
+            "setup",
+            "teardown",
+            "timeout",
+            "template",
+            "return_statement",
+            "tags",
+        }
         skip_settings = set()
         for setting in settings:
             if getattr(skip_config, setting):
@@ -156,7 +165,10 @@ def section(self, name):
 documentation_option = click.option("--skip-documentation", is_flag=True, help="Skip formatting of documentation")
 return_values_option = click.option("--skip-return-values", is_flag=True, help="Skip formatting of return values")
 keyword_call_option = click.option(
-    "--skip-keyword-call", type=str, multiple=True, help="Keyword call name that should not be formatted"
+    "--skip-keyword-call",
+    type=str,
+    multiple=True,
+    help="Keyword call name that should not be formatted",
 )
 keyword_call_pattern_option = click.option(
     "--skip-keyword-call-pattern",

robotidy/transformers/AlignSettingsSection.py

@@ -147,7 +147,13 @@ def align_rows(self, statements, look_up):
     def calc_separator(self, index, up_to, indent_arg, token, look_up):
         if index < up_to:
             if self.fixed_width:
-                return max(self.fixed_width - len(token.value), self.formatting_config.space_count) * " "
+                return (
+                    max(
+                        self.fixed_width - len(token.value),
+                        self.formatting_config.space_count,
+                    )
+                    * " "
+                )
             arg_indent = self.argument_indent if indent_arg else 0
             if indent_arg and index != 0:
                 return (

robotidy/transformers/AlignTemplatedTestCases.py

@@ -110,7 +110,13 @@ def align_header(self, statement):
         for index, token in enumerate(statement.data_tokens[:-1]):
             tokens.append(token)
             if self.min_width:
-                separator = max(self.formatting_config.space_count, self.min_width - len(token.value)) * " "
+                separator = (
+                    max(
+                        self.formatting_config.space_count,
+                        self.min_width - len(token.value),
+                    )
+                    * " "
+                )
             else:
                 separator = (self.widths[index] - len(token.value) + self.formatting_config.space_count) * " "
             tokens.append(Token(Token.SEPARATOR, separator))

robotidy/transformers/AlignVariablesSection.py

@@ -46,7 +46,13 @@ class AlignVariablesSection(Transformer):
     To align all columns set ``up_to_column`` to 0.
     """
 
-    def __init__(self, up_to_column: int = 2, skip_types: str = "", min_width: int = None, fixed_width: int = None):
+    def __init__(
+        self,
+        up_to_column: int = 2,
+        skip_types: str = "",
+        min_width: int = None,
+        fixed_width: int = None,
+    ):
         super().__init__()
         self.up_to_column = up_to_column - 1
         self.min_width = min_width
@@ -121,7 +127,13 @@ def align_rows(self, statements, look_up):
     def get_separator(self, index: int, up_to: int, token, look_up: dict[int, int]) -> str:
         if index < up_to:
             if self.fixed_width:
-                return max(self.fixed_width - len(token.value), self.formatting_config.space_count) * " "
+                return (
+                    max(
+                        self.fixed_width - len(token.value),
+                        self.formatting_config.space_count,
+                    )
+                    * " "
+                )
             return (look_up[index] - len(token.value)) * " "
         else:
             return self.formatting_config.separator