Skip to content

Drop mocha as testing framework #10723

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

Closed
Tracked by #10724
lnicola opened this issue Nov 8, 2021 · 4 comments · Fixed by #10903
Closed
Tracked by #10724

Drop mocha as testing framework #10723

lnicola opened this issue Nov 8, 2021 · 4 comments · Fixed by #10903
Labels
A-vscode vscode plugin issues C-feature Category: feature request S-actionable Someone could pick this issue up and work on it right now

Comments

@lnicola
Copy link
Member

lnicola commented Nov 8, 2021

We currently have a couple of client-side test, using vscode-test and mocha. The latter seems popular, but looking over its dependency tree made me quite uncomfortable.

Let's try to implement simple a replacement for it, if possible.

@lnicola lnicola added A-vscode vscode plugin issues S-actionable Someone could pick this issue up and work on it right now C-feature Category: feature request labels Nov 8, 2021
@lnicola lnicola mentioned this issue Nov 8, 2021
3 tasks
@Aloso
Copy link

Aloso commented Nov 30, 2021

I wouldn't recommend writing a testing framework from scratch, that would be reinventing the wheel. There are testing libraries with no (or very few) dependencies, for example QUnit.

@lnicola
Copy link
Member Author

lnicola commented Dec 2, 2021

QUnit has no dependencies

I... don't even know what that means.

image

@Aloso
Copy link

Aloso commented Dec 2, 2021

@lnicola the devDependencies aren't installed when you use a NPM package, only the dependencies. So QUnit has 3 direct dependencies (5 in total). But QUnit was just an example, there are other options as well.

@flodiebold
Copy link
Member

I wouldn't recommend writing a testing framework from scratch, that would be reinventing the wheel.

It's not so much reinventing the wheel, as realizing we don't need a wheel because we're completely stationary 😉

@bors bors bot closed this as completed in 3b02aec Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-vscode vscode plugin issues C-feature Category: feature request S-actionable Someone could pick this issue up and work on it right now
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants