Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Enable cypress tests on CI#490

Merged
apoorv1316 merged 1 commit intodevelopfrom
enable-cypress-on-ci
Jul 7, 2022
Merged

Enable cypress tests on CI#490
apoorv1316 merged 1 commit intodevelopfrom
enable-cypress-on-ci

Conversation

@apoorv1316
Copy link
Collaborator

No description provided.

@github-actions
Copy link

Current Code Coverage Percent of this PR:

89.24 %

Files having coverage below 100%

Impacted Files Coverage
/lib/custom_failure.rb 80.0 %
/app/controllers/users/invitations_controller.rb 86.36 %
/app/controllers/users/sessions_controller.rb 85.71 %
/app/services/invoice_payment/pdf_generation.rb 70.97 %
/app/services/invoice_payment/checkout.rb 44.0 %
/app/controllers/internal_api/v1/payment_settings_controller.rb 93.33 %
/app/controllers/internal_api/v1/profile_controller.rb 96.88 %
/app/controllers/internal_api/v1/companies_controller.rb 95.45 %
/app/controllers/internal_api/v1/payments/providers_controller.rb 94.74 %
/app/controllers/internal_api/v1/wise/recipients_controller.rb 90.0 %
/lib/benchmarking/benchmarker.rb 0.0 %

@akhilgkrishnan
Copy link
Member

@apoorv1316 Please check why the CI is failing

@apoorv1316
Copy link
Collaborator Author

@akhilgkrishnan They are old tests and I am updating them in a separate PR. So we can ignore them in this PR.

Copy link
Contributor

@mohinid mohinid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

@apoorv1316 apoorv1316 reopened this Jul 7, 2022
@apoorv1316 apoorv1316 merged commit 65077b4 into develop Jul 7, 2022
@apoorv1316 apoorv1316 deleted the enable-cypress-on-ci branch July 7, 2022 08:35
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
enable cypress tests on ci

Co-authored-by: “Apoorv <“[email protected]”>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants