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

Unit tests #98

Closed
wants to merge 7 commits into from
Closed

Unit tests #98

wants to merge 7 commits into from

Conversation

mfleader
Copy link
Contributor

@mfleader mfleader commented Oct 6, 2023 •
edited
Loading

Walking Skeleton

  • add a small quantity of unit tests that pass locally
  • use ci to enforce 100% passing unit tests
  • use ci upload coverage report to job

The Rest

  • increase code coverage to some reasonable proportion
  • potentially: refactor source code to use dependency injection to increase observability of code behavior with tests
  • pin Node version to 18

@mfleader mfleader added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 6, 2023
@mfleader mfleader added this to the CDM CI milestone Oct 6, 2023
@mfleader mfleader self-assigned this Oct 6, 2023
@mfleader mfleader linked an issue Oct 6, 2023 that may be closed by this pull request
JavaScript Unit Tests #95
Open
@mfleader mfleader closed this Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mfleader mfleader

Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
CDM CI
Development

Successfully merging this pull request may close these issues.

JavaScript Unit Tests
1 participant
@mfleader