-
Notifications
You must be signed in to change notification settings - Fork 1
test: migrate to jest #24
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
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates tests from Tape to Jest and updates associated configurations and documentation. Key changes include:
- Replacing Tape assertions with Jest expectations in test files.
- Updating documentation badges and workflow configurations.
- Removing an obsolete code coverage configuration file.
Reviewed Changes
Copilot reviewed 85 out of 85 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
tests/src/rules/aria-proptypes.spec.js | Converted a Tape test to use Jest’s expect API for validityCheck. |
tests/index.spec.js | Updated tests to replace Tape assertions with Jest syntax. |
README.md | Revised badges and header formatting to improve documentation clarity. |
.github/workflows/ci.yml | Adjusted Node version configuration from fixed version to LTS markers. |
.c8rc.json | Removed the code coverage configuration file. |
Comments suppressed due to low confidence (1)
.c8rc.json:1
- The removal of the .c8rc.json file may affect code coverage reporting. Ensure that an alternative configuration is in place or update the documentation accordingly.
{
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
commit: |
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment Thanks for integrating Codecov - We've got you covered ☂️ |
22a5a22
to
7507105
Compare
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesFootnotes
|
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesFootnotes
|
related #7
related #7