You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.
We can consider making a CI script to test all of the functions on every commit similar to how @web4er does manual QA.
Perhaps this GitHub action can open a new issue and do a routine. Perhaps it can also tag the title with the commit hash so that it is easier to identify which commit the tests correspond to.
A minimum goal to start would be some end-to-end tests that run all commands and check their responses.
A lengthy goal would be to also include unit tests with mocked github events and other mocked infrastructure dependencies like logger, database, etc leading to the possibility of high coverage and test-driven development style.
It wouldn't be just CI step, yarn test will also help developers to get quick feedback if they broke something with new changes.
@web4er Could you elaborate on the difference between these 2 issues: #634 and #635?
From my point of view they're basically the same because implementing #634 implies mocking 3rd party dependencies (logger, supabase DB, etc...) so I don't really understand why we need this issue. Perhaps we should close this one?
We can consider making a CI script to test all of the functions on every commit similar to how @web4er does manual QA.
Perhaps this GitHub action can open a new issue and do a routine. Perhaps it can also tag the title with the commit hash so that it is easier to identify which commit the tests correspond to.
context: #621 (review)
Task list
The text was updated successfully, but these errors were encountered: