robotcode-robot 0.82.0__py3-none-any.whl → 0.82.2__py3-none-any.whl

robotcode/robot/__version__.py

@@ -1 +1 @@
-__version__ = "0.82.0"
+__version__ = "0.82.2"

robotcode/robot/diagnostics/model_helper.py

@@ -258,6 +258,7 @@ class ModelHelper:
         nodes: Optional[List[ast.AST]],
         position: Optional[Position] = None,
         skip_commandline_variables: bool = False,
+        skip_local_variables: bool = False,
         return_not_found: bool = False,
     ) -> Iterator[Tuple[Token, VariableDefinition]]:
         variable_started = False
@@ -270,6 +271,7 @@ class ModelHelper:
                             nodes,
                             position,
                             skip_commandline_variables=skip_commandline_variables,
+                            skip_local_variables=skip_local_variables,
                             ignore_error=True,
                         )
                         sub_token = Token(
@@ -376,6 +378,7 @@ class ModelHelper:
         nodes: Optional[List[ast.AST]],
         position: Optional[Position] = None,
         skip_commandline_variables: bool = False,
+        skip_local_variables: bool = False,
         return_not_found: bool = False,
     ) -> Iterator[Tuple[Token, VariableDefinition]]:
         def is_number(name: str) -> bool:
@@ -405,6 +408,7 @@ class ModelHelper:
                             nodes,
                             position,
                             skip_commandline_variables=skip_commandline_variables,
+                            skip_local_variables=skip_local_variables,
                             return_not_found=return_not_found,
                         ):
                             yield v
@@ -461,6 +465,7 @@ class ModelHelper:
                     nodes,
                     position,
                     skip_commandline_variables=skip_commandline_variables,
+                    skip_local_variables=skip_local_variables,
                     ignore_error=True,
                 )
                 if var is not None:
@@ -485,6 +490,7 @@ class ModelHelper:
                             nodes,
                             position,
                             skip_commandline_variables=skip_commandline_variables,
+                            skip_local_variables=skip_local_variables,
                             ignore_error=True,
                         )
                         sub_sub_token = Token(

robotcode/robot/diagnostics/namespace.py

@@ -26,13 +26,8 @@ from typing import (
 from robot.errors import VariableError
 from robot.libraries import STDLIBS
 from robot.parsing.lexer.tokens import Token
-from robot.parsing.model.blocks import (
-    Keyword,
-    SettingSection,
-    TestCase,
-    VariableSection,
-)
-from robot.parsing.model.statements import Arguments, Statement
+from robot.parsing.model.blocks import Keyword, SettingSection, TestCase, VariableSection
+from robot.parsing.model.statements import Arguments, Fixture, Statement, Timeout
 from robot.parsing.model.statements import LibraryImport as RobotLibraryImport
 from robot.parsing.model.statements import ResourceImport as RobotResourceImport
 from robot.parsing.model.statements import (
@@ -182,7 +177,7 @@ class VariablesVisitor(Visitor):
             )
 
 
-class BlockVariableVisitor(Visitor):
+class VariableVisitorBase(Visitor):
     def __init__(
         self,
         namespace: "Namespace",
@@ -198,7 +193,79 @@ class BlockVariableVisitor(Visitor):
 
         self._results: Dict[str, VariableDefinition] = {}
         self.current_kw_doc: Optional[KeywordDoc] = None
+        self.current_kw: Optional[Keyword] = None
+
+    def get_variable_token(self, token: Token) -> Optional[Token]:
+        return next(
+            (
+                v
+                for v in itertools.dropwhile(
+                    lambda t: t.type in Token.NON_DATA_TOKENS,
+                    tokenize_variables(token, ignore_errors=True, extra_types={Token.VARIABLE}),
+                )
+                if v.type == Token.VARIABLE
+            ),
+            None,
+        )
+
+
+class ArgumentVisitor(VariableVisitorBase):
+    def __init__(
+        self,
+        namespace: "Namespace",
+        nodes: Optional[List[ast.AST]],
+        position: Optional[Position],
+        in_args: bool,
+        current_kw_doc: Optional[KeywordDoc],
+    ) -> None:
+        super().__init__(namespace, nodes, position, in_args)
+
+        self.current_kw_doc: Optional[KeywordDoc] = current_kw_doc
+
+    def get(self, model: ast.AST) -> Dict[str, VariableDefinition]:
+        self._results = {}
+
+        self.visit(model)
+
+        return self._results
+
+    def visit_Arguments(self, node: Statement) -> None:  # noqa: N802
+        args: List[str] = []
+
+        arguments = node.get_tokens(Token.ARGUMENT)
+
+        for argument_token in arguments:
+            try:
+                argument = self.get_variable_token(argument_token)
+
+                if argument is not None and argument.value != "@{}":
+                    if (
+                        self.in_args
+                        and self.position is not None
+                        and self.position in range_from_token(argument_token)
+                        and self.position > range_from_token(argument).end
+                    ):
+                        break
+
+                    if argument.value not in args:
+                        args.append(argument.value)
+                        arg_def = ArgumentDefinition(
+                            name=argument.value,
+                            name_token=strip_variable_token(argument),
+                            line_no=argument.lineno,
+                            col_offset=argument.col_offset,
+                            end_line_no=argument.lineno,
+                            end_col_offset=argument.end_col_offset,
+                            source=self.namespace.source,
+                            keyword_doc=self.current_kw_doc,
+                        )
+                        self._results[argument.value] = arg_def
+
+            except VariableError:
+                pass
 
+
+class OnlyArgumentsVisitor(VariableVisitorBase):
     def get(self, model: ast.AST) -> List[VariableDefinition]:
         self._results = {}
 
@@ -211,9 +278,11 @@ class BlockVariableVisitor(Visitor):
             super().visit(node)
 
     def visit_Keyword(self, node: ast.AST) -> None:  # noqa: N802
+        self.current_kw = cast(Keyword, node)
         try:
             self.generic_visit(node)
         finally:
+            self.current_kw = None
             self.current_kw_doc = None
 
     def visit_KeywordName(self, node: Statement) -> None:  # noqa: N802
@@ -248,53 +317,15 @@ class BlockVariableVisitor(Visitor):
                         keyword_doc=self.current_kw_doc,
                     )
 
-    def get_variable_token(self, token: Token) -> Optional[Token]:
-        return next(
-            (
-                v
-                for v in itertools.dropwhile(
-                    lambda t: t.type in Token.NON_DATA_TOKENS,
-                    tokenize_variables(token, ignore_errors=True, extra_types={Token.VARIABLE}),
-                )
-                if v.type == Token.VARIABLE
-            ),
-            None,
-        )
-
-    def visit_Arguments(self, node: Statement) -> None:  # noqa: N802
-        args: List[str] = []
+            if self.current_kw is not None:
+                args = ArgumentVisitor(
+                    self.namespace, self.nodes, self.position, self.in_args, self.current_kw_doc
+                ).get(self.current_kw)
+                if args:
+                    self._results.update(args)
 
-        arguments = node.get_tokens(Token.ARGUMENT)
 
-        for argument_token in arguments:
-            try:
-                argument = self.get_variable_token(argument_token)
-
-                if argument is not None and argument.value != "@{}":
-                    if (
-                        self.in_args
-                        and self.position is not None
-                        and self.position in range_from_token(argument_token)
-                        and self.position > range_from_token(argument).end
-                    ):
-                        break
-
-                    if argument.value not in args:
-                        args.append(argument.value)
-                        arg_def = ArgumentDefinition(
-                            name=argument.value,
-                            name_token=strip_variable_token(argument),
-                            line_no=argument.lineno,
-                            col_offset=argument.col_offset,
-                            end_line_no=argument.lineno,
-                            end_col_offset=argument.end_col_offset,
-                            source=self.namespace.source,
-                            keyword_doc=self.current_kw_doc,
-                        )
-                        self._results[argument.value] = arg_def
-
-            except VariableError:
-                pass
+class BlockVariableVisitor(OnlyArgumentsVisitor):
 
     def visit_ExceptHeader(self, node: Statement) -> None:  # noqa: N802
         variables = node.get_tokens(Token.VARIABLE)[:1]
@@ -990,10 +1021,12 @@ class Namespace:
         nodes: Optional[List[ast.AST]] = None,
         position: Optional[Position] = None,
         skip_commandline_variables: bool = False,
+        skip_local_variables: bool = False,
     ) -> Iterator[Tuple[VariableMatcher, VariableDefinition]]:
         yielded: Dict[VariableMatcher, VariableDefinition] = {}
 
         test_or_keyword = None
+        test_or_keyword_nodes = None
 
         if nodes:
             test_or_keyword_nodes = list(
@@ -1004,18 +1037,23 @@ class Namespace:
             )
             test_or_keyword = test_or_keyword_nodes[0] if test_or_keyword_nodes else None
 
+        in_args = isinstance(test_or_keyword_nodes[-1], Arguments) if test_or_keyword_nodes else False
+        only_args = (
+            isinstance(test_or_keyword_nodes[-1], (Arguments, Fixture, Timeout)) if test_or_keyword_nodes else False
+        )
+
         for var in chain(
             *[
                 (
                     (
-                        BlockVariableVisitor(
+                        (OnlyArgumentsVisitor if only_args else BlockVariableVisitor)(
                             self,
                             nodes,
                             position,
-                            isinstance(test_or_keyword_nodes[-1], Arguments) if nodes else False,
+                            in_args,
                         ).get(test_or_keyword)
                     )
-                    if test_or_keyword is not None
+                    if test_or_keyword is not None and not skip_local_variables
                     else []
                 )
             ],
@@ -1081,6 +1119,7 @@ class Namespace:
         nodes: Optional[List[ast.AST]] = None,
         position: Optional[Position] = None,
         skip_commandline_variables: bool = False,
+        skip_local_variables: bool = False,
         ignore_error: bool = False,
     ) -> Optional[VariableDefinition]:
         self.ensure_initialized()
@@ -1105,6 +1144,7 @@ class Namespace:
                 nodes,
                 position,
                 skip_commandline_variables=skip_commandline_variables,
+                skip_local_variables=skip_local_variables,
             ):
                 if matcher == m:
                     return v

robotcode/robot/diagnostics/namespace_analyzer.py

@@ -12,12 +12,14 @@ from robot.parsing.lexer.tokens import Token
 from robot.parsing.model.blocks import Keyword, TestCase
 from robot.parsing.model.statements import (
     Arguments,
+    DefaultTags,
     DocumentationOrMetadata,
     Fixture,
     KeywordCall,
     LibraryImport,
     ResourceImport,
     Statement,
+    Tags,
     Template,
     TemplateArguments,
     TestCaseName,
@@ -70,9 +72,35 @@ from .namespace import KeywordFinder, Namespace
 
 if get_robot_version() < (7, 0):
     from robot.variables.search import VariableIterator
+
+    VARIABLE_NOT_FOUND_HINT_TYPES: Tuple[Any, ...] = (
+        DocumentationOrMetadata,
+        TestCaseName,
+        Tags,
+        robot.parsing.model.statements.ForceTags,
+        DefaultTags,
+    )
+
+    IN_SETTING_TYPES: Tuple[Any, ...] = (
+        DocumentationOrMetadata,
+        Tags,
+        robot.parsing.model.statements.ForceTags,
+        DefaultTags,
+        Template,
+    )
 else:
     from robot.variables.search import VariableMatches
 
+    VARIABLE_NOT_FOUND_HINT_TYPES = (
+        DocumentationOrMetadata,
+        TestCaseName,
+        Tags,
+        robot.parsing.model.statements.TestTags,
+        DefaultTags,
+    )
+
+    IN_SETTING_TYPES = (DocumentationOrMetadata, Tags, robot.parsing.model.statements.TestTags, DefaultTags, Template)
+
 
 @dataclass
 class AnalyzerResult:
@@ -266,10 +294,10 @@ class NamespaceAnalyzer(Visitor, ModelHelper):
 
         self.node_stack.append(node)
         try:
+            in_setting = isinstance(node, IN_SETTING_TYPES)
+
             severity = (
-                DiagnosticSeverity.HINT
-                if isinstance(node, (DocumentationOrMetadata, TestCaseName))
-                else DiagnosticSeverity.ERROR
+                DiagnosticSeverity.HINT if isinstance(node, VARIABLE_NOT_FOUND_HINT_TYPES) else DiagnosticSeverity.ERROR
             )
 
             if isinstance(node, KeywordCall) and node.keyword:
@@ -295,6 +323,7 @@ class NamespaceAnalyzer(Visitor, ModelHelper):
                             self.node_stack,
                             range_from_token(token).start,
                             skip_commandline_variables=False,
+                            skip_local_variables=in_setting,
                             return_not_found=True,
                         ):
                             if isinstance(var, VariableNotFoundDefinition):
@@ -380,6 +409,7 @@ class NamespaceAnalyzer(Visitor, ModelHelper):
                     self.node_stack,
                     range_from_token(token).start,
                     skip_commandline_variables=False,
+                    skip_local_variables=in_setting,
                     return_not_found=True,
                 ):
                     if isinstance(var, VariableNotFoundDefinition):

{robotcode_robot-0.82.0.dist-info → robotcode_robot-0.82.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: robotcode-robot
-Version: 0.82.0
+Version: 0.82.2
 Summary: Support classes for RobotCode for handling Robot Framework projects.
 Project-URL: Homepage, https://robotcode.io
 Project-URL: Donate, https://opencollective.com/robotcode
@@ -26,7 +26,7 @@ Classifier: Topic :: Utilities
 Classifier: Typing :: Typed
 Requires-Python: >=3.8
 Requires-Dist: platformdirs<4.2.0,>=3.2.0
-Requires-Dist: robotcode-core==0.82.0
+Requires-Dist: robotcode-core==0.82.2
 Requires-Dist: robotframework>=4.1.0
 Requires-Dist: tomli>=1.1.0; python_version < '3.11'
 Description-Content-Type: text/markdown

{robotcode_robot-0.82.0.dist-info → robotcode_robot-0.82.2.dist-info}/RECORD

@@ -1,5 +1,5 @@
 robotcode/robot/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-robotcode/robot/__version__.py,sha256=3E7u1ATSjtoHo9Pq3zuZICzm_iPH64qoru2bGBZuNbw,23
+robotcode/robot/__version__.py,sha256=RTYfZgQQ2I7IHQ7CnMLKfzWJLJYwV_HoEeJ8bvw2WlY,23
 robotcode/robot/py.typed,sha256=bWew9mHgMy8LqMu7RuqQXFXLBxh2CRx0dUbSx-3wE48,27
 robotcode/robot/config/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 robotcode/robot/config/loader.py,sha256=LpGqJAdysvVSZpccW-Il52xn9RMBBb9X94emlBY7zCc,6077
@@ -11,9 +11,9 @@ robotcode/robot/diagnostics/entities.py,sha256=CrjhLHHwKCCE3YI_dcjoZADJz1urm0VGb
 robotcode/robot/diagnostics/errors.py,sha256=VavgWYuHoW5sTT16j2rl9hxMhWxBKNSFsNmHWPzARQQ,1413
 robotcode/robot/diagnostics/imports_manager.py,sha256=qE__lm0Hsj3R0gUauXRmbWJPYihNjk3O-c5hcICGQjc,56251
 robotcode/robot/diagnostics/library_doc.py,sha256=sLENYhMoq5IvYgd0jUhVpRkRTvY9IfTotlthLUUO0vY,97124
-robotcode/robot/diagnostics/model_helper.py,sha256=3-6lZluPQaT9KstHLfV0jsRUAjTfrkUOomSXiMPKoQg,29868
-robotcode/robot/diagnostics/namespace.py,sha256=yerha8VmX_01h8Ga7MaH4wRVk1waVQ4pJfPu55ohcYU,89792
-robotcode/robot/diagnostics/namespace_analyzer.py,sha256=tElI2NzcCRnsvPqv7wYaV2p5E22VyqEwQQ90FOOjadI,50086
+robotcode/robot/diagnostics/model_helper.py,sha256=_5ixKKMrb-nY-uvV8_WjJ1rlNlz7gT7kHM5NYi_hjVg,30232
+robotcode/robot/diagnostics/namespace.py,sha256=OlhhK1RH4ZFGXZEerFQIzye0ddAq8V-W0cCsxYg6tdc,91315
+robotcode/robot/diagnostics/namespace_analyzer.py,sha256=PWZjpLdn-9GCpp4L4dKxVF2B1hxzSEdDzBNUw2Lp51A,50937
 robotcode/robot/diagnostics/workspace_config.py,sha256=lWNq1KmGGJ9cHFNHn0QTCBHHzgz4AewTw0l-W4TKrj0,1803
 robotcode/robot/utils/__init__.py,sha256=OjNPMn_XSnfaMCyKd8Kmq6vlRt6mIGlzW4qiiD3ykUg,447
 robotcode/robot/utils/ast.py,sha256=N7PobxXjpPWwv6UBa-GBn5wn6RAsiRm8unP6UZt0P6M,10193
@@ -23,7 +23,7 @@ robotcode/robot/utils/robot_path.py,sha256=qKBh1cEnReBBLKkWu4gB9EzM-scAwE4xJc1m6
 robotcode/robot/utils/stubs.py,sha256=6-DMI_CQVJHDgG13t-zINKGCRb_Q7MQPm0_AkfhAEvE,748
 robotcode/robot/utils/variables.py,sha256=fEl8S37lb_mD4hn2MZRAlkiuLGBjAOeZVK0r2o2CfPw,742
 robotcode/robot/utils/visitor.py,sha256=uYLqEhGPmzWKWI3SSrmCaYMwtKvNShvbiPZ4b3FavX8,3241
-robotcode_robot-0.82.0.dist-info/METADATA,sha256=tolbKF_209BXd1oEnPJYGb3EXqx4n9bvKBp-lWuu-gI,2240
-robotcode_robot-0.82.0.dist-info/WHEEL,sha256=zEMcRr9Kr03x1ozGwg5v9NQBKn3kndp6LSoSlVg-jhU,87
-robotcode_robot-0.82.0.dist-info/licenses/LICENSE.txt,sha256=B05uMshqTA74s-0ltyHKI6yoPfJ3zYgQbvcXfDVGFf8,10280
-robotcode_robot-0.82.0.dist-info/RECORD,,
+robotcode_robot-0.82.2.dist-info/METADATA,sha256=pRwfD5QyBb_O5ISAAeBTS_ERjDt3ksXL7ac5FqxT2cg,2240
+robotcode_robot-0.82.2.dist-info/WHEEL,sha256=zEMcRr9Kr03x1ozGwg5v9NQBKn3kndp6LSoSlVg-jhU,87
+robotcode_robot-0.82.2.dist-info/licenses/LICENSE.txt,sha256=B05uMshqTA74s-0ltyHKI6yoPfJ3zYgQbvcXfDVGFf8,10280
+robotcode_robot-0.82.2.dist-info/RECORD,,