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

chore: add retries for navigation in ssr-tests-v9 #25844

Merged
merged 3 commits into from
Dec 1, 2022

Conversation

layershifter
Copy link
Member

We have intermittent errors in CI related to ssr-tests-v9 and navigation. The error is related to .goto(), this PR decreases a timeout (that page should load in 10 seconds for sure) and adds retries. We had a similar problem before with a browser start process, so this change might solve it.

image


If the error will stay, we will need to enable debugging from Puppeteer and get more logs.

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 30, 2022

📊 Bundle size report

🤖 This report was generated against 5079ab52943dded035d7bf7348d1c2762271ae9d

@size-auditor
Copy link

size-auditor bot commented Nov 30, 2022

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 5079ab52943dded035d7bf7348d1c2762271ae9d (build)

@layershifter layershifter marked this pull request as ready for review November 30, 2022 13:37
@layershifter layershifter requested a review from a team as a code owner November 30, 2022 13:37
@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 30, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 5ba92ee:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@layershifter layershifter requested a review from Hotell November 30, 2022 13:40
@layershifter layershifter enabled auto-merge (squash) November 30, 2022 14:53
auto-merge was automatically disabled November 30, 2022 16:46

Pull request was closed

@layershifter layershifter reopened this Nov 30, 2022
@layershifter layershifter merged commit 316184c into microsoft:master Dec 1, 2022
@layershifter layershifter deleted the chore/ssr-retry branch December 1, 2022 10:08
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Dec 1, 2022
* master:
  BREAKING: `useTable` renamed to `useTableFeatures` (microsoft#25797)
  chore: add retries for navigation in ssr-tests-v9 (microsoft#25844)
  fix: Cell actions should have correct background when row focused within (microsoft#25790)
  applying package updates
  Disable 3 Avatar Converged active stories (microsoft#25765)
  chore: introduce TS path aliases for improved DX in v8 (microsoft#25778)
  chore: prepare release react-northstar 0.65.0 (microsoft#25446)
  refactor(scripts): encapsulate v0 and v8 tooling within its domain boundaries (microsoft#25738)
  Support single point in area chart (microsoft#25842)
  chore: enable isolateModules in all v8 packages (microsoft#25774)
  chore: refactor styles for Button (microsoft#25216)
  feat: Improve docs for `DataGrid`, export as unstable (microsoft#25805)
  applying package updates
  fix: Allow data-selection-disabled to be respected by DetailsRow (microsoft#25836)
  docs(rfcs): Update recipes rfc with chosen option and add more details (microsoft#25823)
  chore(react-textarea): migrate to new package structure (microsoft#25820)
  chore(react-switch): migrate to new package structure (microsoft#25819)
  fix(react-avatar): AvatarGroup's pie layout places inline items correctly in rtl (microsoft#25822)
  chore: add few improvements to toolbar stories (microsoft#25635)
Hotell added a commit to Hotell/fluentui that referenced this pull request Feb 9, 2023
* chore: add retries for navigation in ssr-tests-v9

* Apply suggestions from code review

Co-authored-by: Martin Hochel <[email protected]>

* fix snapshot

Co-authored-by: Martin Hochel <[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