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.

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

feat(ci): add cypress tests to github ci #5183

Merged
merged 11 commits into from
Sep 30, 2024
Merged

feat(ci): add cypress tests to github ci #5183

merged 11 commits into from
Sep 30, 2024

Conversation

pixincreate
Copy link
Member

@pixincreate pixincreate commented Jul 3, 2024

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

This PR makes Cypress tests to run against PRs.
At present, it runs against Stripe.

Jobs Time taken to build router Time taken to run Cypress Overall time consumption
4 18m 02s 07m 18s 26m 53s
6 16m 08s 07m 12s 26m 21s
6 16m 11s 07m 26s 26m 16s
8 15m 47s 07m 13s 24m 44s
8 24m 36s 07m 09s 36m 10s

6 core should be fine for now. Will be monitoring this over a week.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

NIL

How did you test it?

CI should run against PRs successfully: https://github.com/juspay/hyperswitch/actions/runs/11089890539/job/30811578393?pr=5183

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@pixincreate pixincreate added the A-CI-CD Area: Continuous Integration/Deployment label Jul 3, 2024
@pixincreate pixincreate added this to the July 2024 Release milestone Jul 3, 2024
@pixincreate pixincreate self-assigned this Jul 3, 2024
@pixincreate pixincreate force-pushed the add-cypress-ci branch 6 times, most recently from 1aecfc9 to f74d05c Compare July 3, 2024 14:54
@pixincreate pixincreate force-pushed the add-cypress-ci branch 3 times, most recently from e90f4d9 to 21a73c9 Compare July 3, 2024 16:52
@pixincreate pixincreate marked this pull request as ready for review July 3, 2024 17:25
@pixincreate pixincreate requested a review from a team as a code owner July 3, 2024 17:25
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Outdated Show resolved Hide resolved
.github/workflows/cypress-tests-runner.yml Show resolved Hide resolved
@pixincreate pixincreate requested review from a team as code owners July 4, 2024 08:25
@hyperswitch-bot hyperswitch-bot bot added the M-api-contract-changes Metadata: This PR involves API contract changes label Jul 4, 2024
@hyperswitch-bot hyperswitch-bot bot removed the M-api-contract-changes Metadata: This PR involves API contract changes label Jul 4, 2024
@pixincreate pixincreate removed request for a team July 4, 2024 09:44
@pixincreate pixincreate force-pushed the add-cypress-ci branch 2 times, most recently from 515d584 to cab647b Compare July 4, 2024 17:20
@pixincreate pixincreate marked this pull request as ready for review September 27, 2024 10:15
@likhinbopanna likhinbopanna added this pull request to the merge queue Sep 30, 2024
Merged via the queue into main with commit e4a35d3 Sep 30, 2024
21 checks passed
@likhinbopanna likhinbopanna deleted the add-cypress-ci branch September 30, 2024 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI-CD Area: Continuous Integration/Deployment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants