Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pure] [lsp] Try to recover unit testing of pure interface #403

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ejgallego
Copy link
Collaborator

PR #289 silently removed important unit tests in the Pure module.

We try to recover them, but indeed, they don't work, as it seems the
library handling API introduced in #289 is lacking in terms of how it
can be used. There seem to be many problems with the new system in
terms of what a language server needs, in particular regarding
assumptions about the underlying file system.

I'm afraid this seems to need significant work.

@ejgallego
Copy link
Collaborator Author

In general it would be nice if future PRs don't remove important tests without fixing them.

@ejgallego
Copy link
Collaborator Author

@vycastor , I think that some of the refactoring we need for the server should also fix this; in particular the way state is handling needs to be become a bit more abstract as to allow for example file-less parsing.

@ejgallego ejgallego force-pushed the inline_test+recover branch from 26be259 to 368e29a Compare May 27, 2020 08:35
@fblanqui fblanqui mentioned this pull request Dec 16, 2021
@ejgallego ejgallego force-pushed the inline_test+recover branch 2 times, most recently from abbaa09 to b33eb64 Compare October 18, 2022 10:01
PR Deducteam#289 silently removed important unit tests in the Pure module.

We try to recover them, but indeed, they don't work, as it seems the
library handling API introduced in Deducteam#289 is lacking in terms of how it
can be used. There seem to be many problems with the new system in
terms of what a language server needs, in particular regarding
assumptions about the underlying file system.
@ejgallego ejgallego force-pushed the inline_test+recover branch from b33eb64 to eceff9a Compare October 18, 2022 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant