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.

Sign up for GitHub

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

Jump to bottom

Add tests beyond linting #31

Open
joeyparrish opened this issue Feb 12, 2022 · 0 comments
Open

Add tests beyond linting #31

joeyparrish opened this issue Feb 12, 2022 · 0 comments
Labels
priority: P3 Useful but not urgent type: CI An issue with our continuous integration tests
Milestone
Backlog

Comments

@joeyparrish
Copy link
Member

Neither the base package nor any of the backends have unit or integration tests. The only thing even approaching testing is the use of eslint.

We need to decide the best way to test our testing tools.
@joeyparrish joeyparrish added the type: CI An issue with our continuous integration tests label Feb 12, 2022
@joeyparrish joeyparrish mentioned this issue Feb 12, 2022
Merged
joeyparrish added a commit that referenced this issue Feb 14, 2022
@joeyparrish
ci: Add test workflow (#32)
a40257e 
Although there aren't any real tests (see #31), we can run the linter
on new PRs.  When we add tests later, the workflow will activate them
through npm test.
@github-actions github-actions bot added this to the Backlog milestone Feb 24, 2022
@joeyparrish joeyparrish added the priority: P3 Useful but not urgent label May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority: P3 Useful but not urgent type: CI An issue with our continuous integration tests
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
1 participant
@joeyparrish