Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Migrate from mocha to ava with better test coverage #78

Closed
darahayes opened this issue Jul 1, 2019 · 2 comments
Closed

Migrate from mocha to ava with better test coverage #78

darahayes opened this issue Jul 1, 2019 · 2 comments
Labels
Milestone

Comments

@darahayes
Copy link
Contributor

We're making some really great progress with Offix, however I think some work needs to be done around testing. I'd like this issue to focus solely on unit tests. There is plenty of scope for integration tests also, but I think we can start simple and cover those in a separate issue.

There are several components in Offix that perform some complex and tricky logic. Cache operations, data structure manipulations. Let's try to collaborate and identify the areas where we will get the most value out of unit tests.

I see this initiative being tackled as lots of small PRs. Contributions in the form of PRs, and suggestions/discussion are welcome. Myself and other collaborators are here to help!

@darahayes darahayes added enhancement New feature or request good first issue Good for newcomers help-wanted labels Jul 1, 2019
@wtrocki wtrocki changed the title Improve unit test coverage Migrate from mocha to ava with better test coverage Jul 2, 2019
@wtrocki wtrocki added this to the 0.7.0 milestone Jul 3, 2019
@wtrocki
Copy link
Contributor

wtrocki commented Sep 24, 2019

@darahayes This aged badly. How about we will use jest for both integration and unit tests.
Unit tests now could be much easier but we still need to have integration tests focused testing due to the complexity of the flow.

@darahayes
Copy link
Contributor Author

Closing in favour of #192

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants