denoland / deno

A modern runtime for JavaScript and TypeScript.
https://deno.com
MIT License
93.06k stars 5.14k forks source link

chore(lsp): Fail test if error logged to stderr [WIP] #23742

Open nathanwhit opened 1 week ago

nathanwhit commented 1 week ago

Sometimes in the LSP we log errors and continue on, but our tests don't notice this and pass as long as all the assertions pass. I discovered at least one bug that we've missed due to this, so opened this PR with a hacky check just to see what fails.