Skip to content
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

Unit and Integration Tests Infrastructure #178

Closed
7 tasks done
zainfathoni opened this issue Jul 19, 2021 · 4 comments · Fixed by #358
Closed
7 tasks done

Unit and Integration Tests Infrastructure #178

zainfathoni opened this issue Jul 19, 2021 · 4 comments · Fixed by #358
Assignees
Labels
ci-cd Continuous Integration & Continuous Delivery enhancement New feature or request epic A master issue thread which contains other smaller issues
Milestone

Comments

@zainfathoni
Copy link
Member

zainfathoni commented Jul 19, 2021

Overview

Thanks to our fellow contributors, this project grows fast in the past few days.
It's time to introduce a proper infrastructure for unit and integration tests.

This epic is blocking #294.

References

You can refer to this project as an example.

Current Tasks

@zainfathoni zainfathoni added the epic A master issue thread which contains other smaller issues label Jul 19, 2021
@zainfathoni zainfathoni added this to the Further Optimizations milestone Jul 19, 2021
@zainfathoni zainfathoni self-assigned this Jul 19, 2021
@zainfathoni zainfathoni added the enhancement New feature or request label Jul 19, 2021
@resir014
Copy link
Member

@zainfathoni We should potentially also include Typescript + ESLint in the CI to catch any type/lint errors as well alongside unit tests.

@zainfathoni
Copy link
Member Author

zainfathoni commented Jul 19, 2021

Shall we create another epic for the .github/workflows/test.yml file to contain everything needs to be done on the CI validation step @resir014? We can track some of these issues in that epic as well.

UPDATE:
The epic has been created on #186.

@mazipan mazipan added the ci-cd Continuous Integration & Continuous Delivery label Jul 21, 2021
@resir014
Copy link
Member

resir014 commented Jul 23, 2021

@zainfathoni I think we should expedite this issue.

Since we're getting a lot more tasks which need unit/integration tests available (e.g. #197, #294) we should make this high-priority and shouldn't delay any further in implementing unit/integration tests.

@zainfathoni
Copy link
Member Author

Agreed.
I'll work on it tonight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd Continuous Integration & Continuous Delivery enhancement New feature or request epic A master issue thread which contains other smaller issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants