foster/denoland-deno
0
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2025-02-10 08:02:18 -05:00
Code Issues Packages 1 Wiki Activity
denoland-deno/cli/tests/lsp/definition_request_asset.json

15 lines
217 B
JSON
Raw Normal View History

feat(lsp): add asset cache regression test When we migrated away from all the locks, there was a regression that was not caught immediately. The tsc::get_asset() would attempt to modify the snapshot, but the problem was that the snapshot was a .clone() of the inner language server's assets, which meant that modifications to that where lost. When we then attempted to do a hover on those assets, the inner language servers assets didn't have the retrieved asset, and therefore would throw an error.
2021-02-06 13:39:01 +01:00
{
"jsonrpc": "2.0",
"id": 4,
fix(lsp): handle data URLs properly (#9522) Fixes #9514 Co-authored-by: Ryan Dahl <ry@tinyclouds.org>
2021-02-18 15:37:05 +11:00
"method": "textDocument/definition",
feat(lsp): add asset cache regression test When we migrated away from all the locks, there was a regression that was not caught immediately. The tsc::get_asset() would attempt to modify the snapshot, but the problem was that the snapshot was a .clone() of the inner language server's assets, which meant that modifications to that where lost. When we then attempted to do a hover on those assets, the inner language servers assets didn't have the retrieved asset, and therefore would throw an error.
2021-02-06 13:39:01 +01:00
"params": {
"textDocument": {
"uri": "file:///a/file.ts"
},
"position": {
"line": 0,
fix(lsp): handle data URLs properly (#9522) Fixes #9514 Co-authored-by: Ryan Dahl <ry@tinyclouds.org>
2021-02-18 15:37:05 +11:00
"character": 14
feat(lsp): add asset cache regression test When we migrated away from all the locks, there was a regression that was not caught immediately. The tsc::get_asset() would attempt to modify the snapshot, but the problem was that the snapshot was a .clone() of the inner language server's assets, which meant that modifications to that where lost. When we then attempted to do a hover on those assets, the inner language servers assets didn't have the retrieved asset, and therefore would throw an error.
2021-02-06 13:39:01 +01:00
}
}
}
Copy permalink