Open nihal467 opened 17 hours ago
[!IMPORTANT]
Review skipped
Draft detected.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The pull request introduces significant modifications to the Cypress workflow configuration and updates the package.json
file. The workflow changes include the removal of separate steps for running Cypress tests based on pull request origins, replaced by a single parallel execution step. Additionally, artifact upload steps have been renamed and consolidated. New steps for starting the frontend application and ensuring server readiness have been added. The package.json
updates include new scripts for running tests and a new dependency for parallel test execution.
File | Change Summary |
---|---|
.github/workflows/cypress.yaml |
- Removed the strategy section and separate steps for non-forked and forked PRs.- Introduced "Run Cypress Tests in Parallel". - Renamed upload steps for screenshots and videos. - Added steps for starting the frontend application and waiting for the server to be ready. - Added debugging steps for files, logs, and Docker containers. |
package.json |
- Added new scripts: "cy:run" and "cy:parallel" .- Added new dependency: "cypress-parallel": "^0.14.0" in devDependencies . |
cypress.config.ts |
- Added dotenv support for environment variables.- Retained existing task for reading files and conditional checks. |
needs review
, tested
🐰 In the garden where tests do play,
Cypress hops in a brand new way.
With artifacts gathered, all in one place,
Parallel runs quicken the race.
So let’s cheer for the code we embrace! 🎉
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Name | Link |
---|---|
Latest commit | dff56ccadf681d5858d7894fddb9952d6527ec67 |
Latest deploy log | https://app.netlify.com/sites/care-ohc/deploys/67405f7eba4a5800077f5635 |
Deploy Preview | https://deploy-preview-9178--care-ohc.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Latest commit: |
dff56cc
|
Status: | ✅ Deploy successful! |
Preview URL: | https://a75da2cd.care-fe.pages.dev |
Branch Preview URL: | https://cypress-parallel-test.care-fe.pages.dev |
todo: add weights to the run file so that the parallel split is optimized
Proposed Changes
Pro's
Con's
@ohcnetwork/care-fe-code-reviewers
Merge Checklist
Summary by CodeRabbit
New Features
cy:run
andcy:parallel
.Bug Fixes
Chores