Skip to content
This repository was archived by the owner on Feb 6, 2026. It is now read-only.

longer timeouts for get-tiles#7372

Merged
nickgerace merged 1 commit intomainfrom
wendy/longer-timeouts-get-tiles-test
Sep 29, 2025
Merged

longer timeouts for get-tiles#7372
nickgerace merged 1 commit intomainfrom
wendy/longer-timeouts-get-tiles-test

Conversation

@wendybujalski
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions github-actions bot added the A-web label Sep 29, 2025

// Find the URL for the synthetic workspace and go there
cy.get('a[href="' + AUTH_API_URL + '/workspaces/' + SI_WORKSPACE_ID + '/go"]')
cy.get('a[href="' + AUTH_API_URL + '/workspaces/' + SI_WORKSPACE_ID + '/go"]', { timeout: 60000 })
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The default timeout here was 4 seconds, this lengthens it to 1 minute.

// For virtual scrolling, we need to scroll through the entire list
// First scroll to top, then gradually scroll down to ensure all items are rendered
cy.get('[data-testid="explore-grid"]').parent().scrollTo("top");
cy.get('[data-testid="explore-grid"]', { timeout: 60000 }).parent().scrollTo("top");
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Same here. 1 minute might not be the exact length we want, but 4 seconds is definitely too short!

@nickgerace nickgerace added this pull request to the merge queue Sep 29, 2025
Merged via the queue into main with commit 6c02906 Sep 29, 2025
19 of 22 checks passed
@nickgerace nickgerace deleted the wendy/longer-timeouts-get-tiles-test branch September 29, 2025 23:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants