Fix tests on Windows

`lsp_types::Url::from_file_path()` returns `None` if not a valid path.
Path should start with e.g. `C:\` on Windows.

Author: lionel-

crates/ark/src/lsp/definitions.rs

@@ -87,6 +87,17 @@ mod tests {
     use crate::lsp::documents::Document;
     use crate::lsp::indexer;
 
+    fn test_path() -> (PathBuf, Url) {
+        let path = if cfg!(windows) {
+            PathBuf::from(r"C:\test.R")
+        } else {
+            PathBuf::from("/test.R")
+        };
+        let uri = Url::from_file_path(&path).unwrap();
+
+        (path, uri)
+    }
+
     #[test]
     fn test_goto_definition() {
         let _guard = indexer::ResetIndexerGuard;
@@ -96,15 +107,13 @@ foo <- 42
 print(foo)
 "#;
         let doc = Document::new(code, None);
-        let path = PathBuf::from("/foo/test.R");
+        let (path, uri) = test_path();
 
         indexer::update(&doc, &path).unwrap();
 
         let params = GotoDefinitionParams {
             text_document_position_params: lsp_types::TextDocumentPositionParams {
-                text_document: lsp_types::TextDocumentIdentifier {
-                    uri: Url::from_file_path(&path).unwrap(),
-                },
+                text_document: lsp_types::TextDocumentIdentifier { uri },
                 position: lsp_types::Position::new(2, 7),
             },
             work_done_progress_params: Default::default(),
@@ -114,9 +123,6 @@ print(foo)
         assert_matches!(
             goto_definition(&doc, params).unwrap(),
             Some(GotoDefinitionResponse::Link(ref links)) => {
-                assert!(!links.is_empty());
-                assert_eq!(links[0].target_uri, Url::from_file_path(&path).unwrap());
-
                 assert_eq!(
                     links[0].target_range,
                     lsp_types::Range {
@@ -138,14 +144,13 @@ foo <- 1
 print(foo)
 "#;
         let doc = Document::new(code, None);
-        let path = PathBuf::from("/foo/section_test.R");
+        let (path, uri) = test_path();
+
         indexer::update(&doc, &path).unwrap();
 
         let params = lsp_types::GotoDefinitionParams {
             text_document_position_params: lsp_types::TextDocumentPositionParams {
-                text_document: lsp_types::TextDocumentIdentifier {
-                    uri: lsp_types::Url::from_file_path(&path).unwrap(),
-                },
+                text_document: lsp_types::TextDocumentIdentifier { uri },
                 position: lsp_types::Position::new(3, 7),
             },
             work_done_progress_params: Default::default(),
@@ -155,14 +160,6 @@ print(foo)
         assert_matches!(
             goto_definition(&doc, params).unwrap(),
             Some(lsp_types::GotoDefinitionResponse::Link(ref links)) => {
-                assert!(!links.is_empty());
-
-                let link = &links[0];
-                assert_eq!(
-                    link.target_uri,
-                    lsp_types::Url::from_file_path(&path).unwrap()
-                );
-
                 // The section should is not the target, the variable has priority
                 assert_eq!(
                     links[0].target_range,