Skip to content
This repository has been archived by the owner on Jun 18, 2022. It is now read-only.

Notes on replacing Jasmine #1

Closed
Arcanemagus opened this issue Jul 25, 2016 · 3 comments
Closed

Notes on replacing Jasmine #1

Arcanemagus opened this issue Jul 25, 2016 · 3 comments
Labels
🕸️ area/tests This affects tests 💪 phase/solved Post is done 🦋 type/enhancement This is great to have

Comments

@Arcanemagus
Copy link

Since you have a request about it in the readme, you can see where custom test framework support was added to Atom here, with a Mocha runner implemented here.
@wooorm
Copy link
Member

wooorm commented Jul 25, 2016

Oh cool! Thanks @Arcanemagus! 💃

@Arcanemagus
Copy link
Author

Looks like it's been in Atom since v1.2.0.

Also I haven't seen anything about coverage testing, but if you find something I'd be interested to!

@wooorm
Copy link
Member

wooorm commented Jul 25, 2016

👍 I tried using nyc, but couldn’t get it to work, as it uses eval for some reason, and atom’s security settings don’t allow that... I believe the reason for that, in nyc, was to get a global scope or something, which should probably be fixable with a PR to nyc. Anyway, too much work for now! 😬

@wooorm wooorm closed this as completed in a0582cb Aug 3, 2016
@wooorm wooorm added the 💪 phase/solved Post is done label May 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🕸️ area/tests This affects tests 💪 phase/solved Post is done 🦋 type/enhancement This is great to have
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Arcanemagus @wooorm