Skip to content

test(rename): expose Not_found on incomplete local module - #2033

Open
rgrinberg wants to merge 1 commit into
masterfrom
push-tuqoovrkqsnq
Open

test(rename): expose Not_found on incomplete local module#2033
rgrinberg wants to merge 1 commit into
masterfrom
push-tuqoovrkqsnq

Conversation

@rgrinberg

Copy link
Copy Markdown
Member

Adds a regression test for occurrence lookup on the incomplete local-module expression let module X.

textDocument/prepareRename at the valid EOF position currently leaks Not_found from Merlin_analysis.Occurrences.uid_and_loc_of_node as JSON-RPC InternalError. The test censors the unstable backtrace and preserves the exception and response code for a test-first fix.

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
@rgrinberg

Copy link
Copy Markdown
Member Author

Seems like a merlin issue @voodoos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant