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

Upgrade to Jasmine 3 #1103

Merged
merged 4 commits into from
Jan 12, 2021
Merged

Conversation

illright
Copy link
Contributor

Description of the Change

This will replace the default Atom test runner powered by the long outdated Jasmine 1.3 with a new custom test runner powered by the latest Jasmine 3.6.

Benefits

The documentation of Jasmine 1.3 is very poor, so upgrading the version will allow developers to write tests more efficiently with the new Jasmine docs. It also improves the developer experience because Jasmine 3 is more pleasant to work with.

Possible Drawbacks

Not that I can think of any.

@illright
Copy link
Contributor Author

This will also take care of #1095 as well as other security vulnerabilities that were present in the package dependencies

Co-authored-by: Amin Yahyaabadi <aminyahyaabadi74@gmail.com>
@sadick254 sadick254 merged commit ed12ea7 into atom:master Jan 12, 2021
@illright illright deleted the tests/upgrade-jasmine3 branch January 14, 2021 13:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants