Skip to content

Increasing timeout for Cypress workflow#4089

Merged
olemartinorg merged 1 commit intomainfrom
chore/cypress-increased-timeout
Mar 31, 2026
Merged

Increasing timeout for Cypress workflow#4089
olemartinorg merged 1 commit intomainfrom
chore/cypress-increased-timeout

Conversation

@olemartinorg
Copy link
Copy Markdown
Contributor

@olemartinorg olemartinorg commented Mar 31, 2026

Description

This should make it possible to run the full test suite within the time limit even while cypress cloud disallows us parallel runs

Related Issue(s)

  • closes #{issue number}

Verification/QA

  • Manual functionality testing
    • I have tested these changes manually
    • Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
    • No testing done/necessary
  • Automated tests
    • Unit test(s) have been added/updated
    • Cypress E2E test(s) have been added/updated
    • No automatic tests are needed here (no functional changes/additions)
    • I want someone to help me make some tests
  • UU/WCAG (follow these guidelines until we have our own)
    • I have tested with a screen reader/keyboard navigation/automated wcag validator
    • No testing done/necessary (no DOM/visual changes)
    • I want someone to help me perform accessibility testing
  • User documentation @ altinn-studio-docs
    • Has been added/updated
    • No functionality has been changed/added, so no documentation is needed
    • I will do that later/have created an issue
  • Support in Altinn Studio
    • Issue(s) created for support in Studio
    • This change/feature does not require any changes to Altinn Studio
  • Sprint board
    • The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
    • I don't have permissions to do that, please help me out
  • Labels
    • I have added a kind/* and backport* label to this PR for proper release notes grouping
    • I don't have permissions to add labels, please help me out

Summary by CodeRabbit

  • Chores
    • Extended test job timeouts to improve test reliability and accommodate longer test runs.

This should make it possible to run the full test suite within the time limit even while cypress cloud disallows us parallel runs
@olemartinorg olemartinorg added ignore-for-release Pull requests to be ignored in release notes backport-ignore This PR is a new feature and should not be cherry-picked onto release branches labels Mar 31, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 20258305-309a-4880-a7d9-e6443b8bbe5d

📥 Commits

Reviewing files that changed from the base of the PR and between 63ce678 and 783580c.

📒 Files selected for processing (1)
  • .github/workflows/cypress.yml

📝 Walkthrough

Walkthrough

The change extends CI/CD workflow timeout configurations in the Cypress test job definitions, increasing the timeout for the internal job from 60 to 120 minutes and establishing a 120-minute timeout for the external job.

Changes

Cohort / File(s) Summary
CI/CD Workflow Configuration
.github/workflows/cypress.yml
Extended job timeout limits for cypress-run-internal (60→120 minutes) and added timeout specification for cypress-run-external (now 120 minutes).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: increasing the timeout for the Cypress workflow jobs in the CI configuration.
Description check ✅ Passed The description includes the required template structure with all sections present, though the related issue reference is left as a placeholder and most verification checkboxes are unchecked.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/cypress-increased-timeout

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@olemartinorg olemartinorg merged commit ea5783a into main Mar 31, 2026
6 of 9 checks passed
@olemartinorg olemartinorg deleted the chore/cypress-increased-timeout branch March 31, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-ignore This PR is a new feature and should not be cherry-picked onto release branches ignore-for-release Pull requests to be ignored in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants