Add tests for Markdown/RST in comments

Author: taminomara

crates/emmylua_ls/src/handlers/completion/providers/desc_provider.rs

@@ -157,7 +157,6 @@ fn add_by_prefix(
 
         // Modules.
         add_modules(builder, &prefix, None);
-        dbg!(&seen_types);
         complete_types_by_prefix(builder, &prefix, Some(&seen_types));
     }
 

crates/emmylua_ls/src/handlers/test/definition_test.rs

@@ -1,6 +1,7 @@
 #[cfg(test)]
 mod tests {
     use crate::handlers::test_lib::{ProviderVirtualWorkspace, VirtualLocation, check};
+    use emmylua_code_analysis::{DocSyntax, Emmyrc};
     use googletest::prelude::*;
 
     type Expected = VirtualLocation;
@@ -193,11 +194,12 @@ mod tests {
                 local test = t.test
                 te<??>st()
             "#,
-            vec![Expected {
+            vec![VirtualLocation {
                 file: "test.lua".to_string(),
                 line: 1
             }]
         ));
+
         Ok(())
     }
 
@@ -328,4 +330,104 @@ mod tests {
         ));
         Ok(())
     }
+
+    #[gtest]
+    fn test_doc_resolve() -> Result<()> {
+        let mut ws = ProviderVirtualWorkspace::new();
+
+        let mut emmyrc = Emmyrc::default();
+        emmyrc.doc.syntax = DocSyntax::Myst;
+        ws.analysis.update_config(emmyrc.into());
+
+        ws.def_file(
+            "a.lua",
+            r#"
+                --- @class X
+                --- @field a string
+
+                --- @class ns.Y
+                --- @field b string
+            "#,
+        );
+
+        check!(ws.check_definition(
+            r#"
+                --- {lua:obj}`X<??>`
+            "#,
+            vec![Expected {
+                file: "a.lua".to_string(),
+                line: 1
+            }]
+        ));
+
+        check!(ws.check_definition(
+            r#"
+                --- {lua:obj}`X<??>.a`
+            "#,
+            vec![Expected {
+                file: "a.lua".to_string(),
+                line: 1
+            }]
+        ));
+
+        check!(ws.check_definition(
+            r#"
+                --- {lua:obj}`X.a<??>`
+            "#,
+            vec![Expected {
+                file: "a.lua".to_string(),
+                line: 2
+            }]
+        ));
+
+        check!(ws.check_definition(
+            r#"
+                --- @using ns
+
+                --- {lua:obj}`X<??>`
+            "#,
+            vec![Expected {
+                file: "a.lua".to_string(),
+                line: 1
+            }]
+        ));
+
+        check!(ws.check_definition(
+            r#"
+                --- @using ns
+
+                --- {lua:obj}`Y<??>`
+            "#,
+            vec![Expected {
+                file: "a.lua".to_string(),
+                line: 4
+            }]
+        ));
+
+        check!(ws.check_definition(
+            r#"
+                --- @using ns
+
+                --- {lua:obj}`ns.Y<??>`
+            "#,
+            vec![Expected {
+                file: "a.lua".to_string(),
+                line: 4
+            }]
+        ));
+
+        check!(ws.check_definition(
+            r#"
+                --- {lua:obj}`c<??>`
+                --- @class Z
+                --- @field c string
+            "#,
+            vec![Expected {
+                file: "".to_string(),
+                line: 3
+            }]
+        ));
+
+        Ok(())
+    }
 }

crates/emmylua_ls/src/handlers/test_lib/mod.rs

@@ -214,7 +214,7 @@ impl ProviderVirtualWorkspace {
     pub fn check_completion_with_kind(
         &mut self,
         block_str: &str,
-        expected: Vec<VirtualCompletionItem>,
+        mut expected: Vec<VirtualCompletionItem>,
         trigger_kind: CompletionTriggerKind,
     ) -> Result<()> {
         let (content, position) = Self::handle_file_content(block_str)?;
@@ -229,11 +229,14 @@ impl ProviderVirtualWorkspace {
         .ok_or("failed to get completion")
         .or_fail()?;
         // 对比
-        let items = match result {
+        let mut items = match result {
             CompletionResponse::Array(items) => items,
             CompletionResponse::List(list) => list.items,
         };
 
+        items.sort_by_key(|item| item.label.clone());
+        expected.sort_by_key(|item| item.label.clone());
+
         fn get_item_detail(i: &CompletionItem) -> Option<&String> {
             i.label_details.as_ref().and_then(|d| d.detail.as_ref())
         }