Remove workaround for canonicalize function in tests

Author: fhammerschmidt

CHANGELOG.md

@@ -17,6 +17,8 @@
 - Fix highlighting of other languages being affected by rescript-vscode. https://github.com/rescript-lang/rescript-vscode/pull/973
 - Use canonicalized URIs/paths for jump to definition. https://github.com/rescript-lang/rescript-vscode/pull/982
 - Fix JSX prop special case in end of JSX element. https://github.com/rescript-lang/rescript-vscode/pull/984
+- preserve URI format in canonicalizeUri. https://github.com/rescript-lang/rescript-vscode/pull/990
+- Remove workaround for canonicalize function in tests https://github.com/rescript-lang/rescript-vscode/pull/992
 
 #### :nail_care: Polish
 

analysis/src/Cfg.ml

@@ -1,5 +1,3 @@
 let debugFollowCtxPath = ref false
 
 let isDocGenFromCompiler = ref false
-
-let isTestMode = ref false

analysis/src/Files.ml

@@ -104,7 +104,5 @@ let classifySourceFile path =
   else Other
 
 let canonicalizeUri uri =
-  if Cfg.isTestMode.contents then uri |> Uri.toString
-  else
-    let path = Uri.toPath uri in
-    path |> Unix.realpath |> Uri.fromPath |> Uri.toString
+  let path = Uri.toPath uri in
+  path |> Unix.realpath |> Uri.fromPath |> Uri.toString