Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Move tests to jest

See original GitHub issue

What the issue is about

Right now we’re using karma for some tests and jest for the other ones. Jest is faster, so we should set up all our tests to use it

Where to start

We’re using nx, check out their docs for moving to jest Search for test in package json to see how we run tests now

Issue Analytics

State:
Created 4 years ago
Comments:7 (1 by maintainers)

Top GitHub Comments

1reaction

O-Muttcommented, Oct 23, 2019

Hi @kirjs, Looks like I worked on some tests which are already done in #1176. @Mutmatt, can you tell me if there are any tests remaining which need to be refactored to use Jest.

I primarily tried to get kirjs’s tests running and functional. If you look at any other project the tests may be less functional. All of the tests should now be running under the jest framework, though.

1reaction

NothingEverHappenscommented, Oct 19, 2019

Yes, all apps should use jest in the end. Breaking the work up per-app would make sense to me.

Also note that some tests are still broken, fixing them doesn’t have to be part of this task