Skip to content

Test collecting lsp4mp extensions#7

Merged
xorye merged 1 commit intoredhat-developer:masterfrom
rzgry:test-collect-lsp4mp-extensions
Jul 21, 2020
Merged

Test collecting lsp4mp extensions#7
xorye merged 1 commit intoredhat-developer:masterfrom
rzgry:test-collect-lsp4mp-extensions

Conversation

@rzgry
Copy link
Copy Markdown
Collaborator

@rzgry rzgry commented Jul 20, 2020

Adds test for collecting lsp4mp extensions and java version requirements.

Switches mocha interface from tdd to bdd so we don't need to import describe / it in test files.

Also removes the vscodeTests directory since vscode-microprofile has no vscodeUITests folder like vscode-quarkus so the extra vscodeTests folder is not needed.

Signed-off-by: Ryan Zegray ryan.zegray@ibm.com

@rzgry rzgry force-pushed the test-collect-lsp4mp-extensions branch from 17cadf5 to 455051e Compare July 20, 2020 14:52
@@ -0,0 +1,47 @@
import * as fs from "fs";
Copy link
Copy Markdown

@xorye xorye Jul 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fbricon do we need a special header in the new files this PR creates?

@rzgry
Copy link
Copy Markdown
Collaborator Author

rzgry commented Jul 20, 2020

FYI if you merge #8 first. I can rebase and update the tests to include document selectors as well.

Comment thread .vscode/launch.json Outdated
Signed-off-by: Ryan Zegray <ryan.zegray@ibm.com>
@rzgry rzgry force-pushed the test-collect-lsp4mp-extensions branch from 455051e to 3e037a7 Compare July 21, 2020 16:57
@xorye xorye merged commit 973ba01 into redhat-developer:master Jul 21, 2020
@xorye
Copy link
Copy Markdown

xorye commented Jul 21, 2020

Thanks @rzgry!

@angelozerr angelozerr added this to the 0.1.0 milestone Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants