add more tests

Author: d-biehl

tests/robotcode/language_server/robotframework/parts/data/goto.robot

@@ -41,4 +41,28 @@ first
 
     firstresource.do something in a resource
 #                 ^^^^^^^^^^^^^^^^^^^^^^^^^^  Keyword from resource
-#   ^^^^^^^^^^^^^  Namespace from resource
+#   ^^^^^^^^^^^^^  Namespace from resource
+    a simple keyword
+#   ^^^^^^^^^^^^^^^^  call a simple keyword
+    an unknown keyword
+#   ^^^^^^^^^^^^^^^^^^  unknown keyword
+
+second
+    [Setup]    Log  hello setup
+#              ^^^ a keyword in setup
+    [Teardown]    BuiltIn.Log  hello teardown
+#                         ^^^ a keyword in teardown
+#                 ^^^^^^^ a namespace in teardown
+
+a templated Test
+    [Template]    BuiltIn.Log
+#                         ^^^ a keyword in template
+#                 ^^^^^^^ a namespace in template
+    hello
+    world
+
+
+*** Keywords ***
+a simple keyword
+#^^^^^^^^^^^^^^^ a simple keyword
+    Log    hello

tests/robotcode/language_server/robotframework/parts/data/hover.robot

@@ -21,8 +21,9 @@ ${A VAR}          i'm a var
 first
     [Setup]  Log    Hello ${A VAR}
 #            ^^^ Keyword in Setup
-    [Teardown]  Log    Hello ${A VAR}
-#               ^^^ Keyword in Teardown
+    [Teardown]  BuiltIn.Log    Hello ${A VAR}
+#                       ^^^ Keyword in Teardown
+#               ^^^^^^^ Namespace in Teardown
 
     Log    Hello ${A VAR}
 #   ^^^ Keyword from Library
@@ -51,6 +52,13 @@ first
 #                 ^^^^^^^^^^^^^^^^^^^^^^^^^^  KeywordCall from resource with Namespace
 #   ^^^^^^^^^^^^^  Namespace from resource
 
+second
+#^^^^^ Test Case
+    [Template]    BuiltIn.Log
+#                         ^^^ Keyword in Template
+#                 ^^^^^^^ Namespace in Template
+    hello
+    world
 
 *** Keywords ***
 a keyword

tests/robotcode/language_server/robotframework/parts/test_goto.py

@@ -18,6 +18,29 @@
 )
 
 
+def split(
+    result: Union[Location, LocationLink, List[Location], List[LocationLink], None]
+) -> Union[Location, LocationLink, List[Location], List[LocationLink], None]:
+    if result is None:
+        return None
+    if isinstance(result, Location):
+        return Location((Uri(result.uri).to_path().name), result.range)
+    if isinstance(result, LocationLink):
+        return LocationLink(
+            result.origin_selection_range,
+            (Uri(result.target_uri).to_path().name),
+            result.target_range,
+            result.target_selection_range,
+        )
+    if isinstance(result, list) and len(result) > 0 and isinstance(result[0], LocationLink):
+        return cast("List[LocationLink]", [split(v) for v in result])
+
+    if isinstance(result, list) and len(result) > 0 and isinstance(result[0], Location):
+        return cast("List[Location]", [split(v) for v in result])
+
+    return result
+
+
 @pytest.mark.parametrize(
     ("test_document", "data"),
     generate_tests_from_source_document(Path(Path(__file__).parent, "data/goto.robot")),
@@ -26,7 +49,7 @@
 )
 @pytest.mark.usefixtures("protocol")
 @pytest.mark.asyncio
-async def test(
+async def test_definition(
     data_regression: DataRegressionFixture,
     protocol: RobotLanguageServerProtocol,
     test_document: TextDocument,
@@ -39,26 +62,28 @@ async def test(
         Position(line=data.line, character=data.character),
     )
 
-    def split(
-        result: Union[Location, LocationLink, List[Location], List[LocationLink], None]
-    ) -> Union[Location, LocationLink, List[Location], List[LocationLink], None]:
-        if result is None:
-            return None
-        if isinstance(result, Location):
-            return Location((Uri(result.uri).to_path().name), result.range)
-        if isinstance(result, LocationLink):
-            return LocationLink(
-                result.origin_selection_range,
-                (Uri(result.target_uri).to_path().name),
-                result.target_range,
-                result.target_selection_range,
-            )
-        if isinstance(result, list) and len(result) > 0 and isinstance(result[0], LocationLink):
-            return cast("List[LocationLink]", [split(v) for v in result])
+    data_regression.check({"data": data, "result": split(result)})
 
-        if isinstance(result, list) and len(result) > 0 and isinstance(result[0], Location):
-            return cast("List[Location]", [split(v) for v in result])
 
-        return result
+@pytest.mark.parametrize(
+    ("test_document", "data"),
+    generate_tests_from_source_document(Path(Path(__file__).parent, "data/goto.robot")),
+    indirect=["test_document"],
+    ids=generate_test_id,
+)
+@pytest.mark.usefixtures("protocol")
+@pytest.mark.asyncio
+async def test_implementation(
+    data_regression: DataRegressionFixture,
+    protocol: RobotLanguageServerProtocol,
+    test_document: TextDocument,
+    data: GeneratedTestData,
+) -> None:
+
+    result = await protocol.robot_goto.collect_implementation(
+        protocol.robot_goto,
+        test_document,
+        Position(line=data.line, character=data.character),
+    )
 
     data_regression.check({"data": data, "result": split(result)})