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: merge develop to feature/test-burn-in (do not squash) #28169

Merged
merged 84 commits into from Oct 30, 2023

Conversation

cacieprins
Copy link
Contributor

@cacieprins cacieprins commented Oct 27, 2023

  • Closes

Additional details

This gets the test burn in branch up to date with develop. Do not squash - we want to preserve this git history so the feature branch can merge nicely into develop.

Steps to test

How has the user experience changed?

PR Tasks

Crustum7 and others added 30 commits September 8, 2023 16:53
Co-authored-by: Simon Törnqvist <simon.thornqvist99@gmail.com>
)

* chore: get changelog ready for release and bump cypress version

* chore: fix typos
…7770)

* basic card visuals, working on click

* visual tweaks and i18n

* match other i18n

* redo duration format, should make common too

* basic debug, needs validation of relevant run

* refactor and move files

* working top level state

* working grouping, needs subcomponent

* switch click target

* remove dayjs change

* fix current commit

* revise commit for message

* working view runs button

* fix type check

* fix tests

* fix integration tests

* remove old test

* add layout

* add component test for new layout and cleanup

* add count check

* standardize cy tags

* revise count test

* fix typos

* fix padding

* start wrapping

* happy wrap, next trunc

* responsive if wonky

* clean up and titles

* magic resize, needs tests per breakpoint

* standardize the rollup

* working default rollup

* rollup spacing tweaks

* center elements

* update tests for multiple viewports

* add missing breakpoint tests

* fix debug tests for status and badge

* remove unneeded check

* add functional component

* fix check_ts

* update container test

* add changelog

* add tooltip, needs aria label and external link update

* fix overflow

* fix avatar and add aria label to count summary

* change to external href

* remove popup sizing

* add tests

* add missing data cy

* fix runcard test

* Truncate result count

* make truncate style conditional

* match design truncate position

* adddress PR comments

* chore:updating styles (#27798)

* add tooltip content tests and skeletons

* clean up tooltip validation

* fix runs selector

* Update packages/app/src/runs/RunsSkeletonRow.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* Update packages/app/src/runs/RunTagCount.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* Update packages/app/src/runs/RunsSkeleton.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* Update packages/app/src/runs/RunTagCount.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* Update packages/app/src/runs/RunsSkeleton.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* Update packages/app/src/runs/RunsSkeletonRow.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* Update packages/app/src/runs/RunTagCount.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* Update packages/app/src/runs/RunTagCount.vue

Co-authored-by: Stokes Player <stokes@cypress.io>

* remove redundant shrink-1s

* more shrinks

---------

Co-authored-by: Stokes Player <stokes@cypress.io>
…27821)

* chore: update chrome PR creation workflow to add to firewatch board

* chore: adding in ability to manually run job

* chore: changing how we add the chrome update PR to the firewatch board
…#27778)

* chore: update kpi report generation to include feature review metrics

* chore: fix typo
…27837)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Ben M <benm@cypress.io>
#27874)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Ben M <benm@cypress.io>
* docs: update installation requirements in contributing.md

* docs: update installation requirements in contributing.md

* docs: update installation requirements in contributing.md

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>

---------

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
https://lodash.com/docs/#inRange `end` is exclusive, so it needs to be set to 1000 to allow status code 999 to be included (as per documented).

Co-authored-by: Adam Stone-Lord <adams@cypress.io>
Co-authored-by: Ben M <benm@cypress.io>
ryanthemanuel and others added 22 commits October 17, 2023 19:53
Co-authored-by: Matt Schile <mschile@cypress.io>
Co-authored-by: Matt Schile <mschile@gmail.com>
Co-authored-by: Emily Rohrbough <emilyrohrbough@users.noreply.github.com>
…#28132)

Co-authored-by: Cacie Prins <cacieprins@users.noreply.github.com>
* chore: set up feature/test-burn-in feature branch

* feat: add burnIn Configuration option (currently a no-op) (#27377)

* feat: add the burnIn Configuration to the config package. Option
currently is a no-op

* chore: make burn in experimental

* chore: set experimentalBurnIn to false by default

* feat: add new experimental retries configuration (#27412)

* feat: implement the experimental retries configuration options to pair
with test burn in

* [run ci]

* fix cache invalidation [run ci]

* fix snapshot added in v13 for module api to include test burn in experimentalflag

* chore: fix merge conflict

* chore: add burnInTestAction capability (#27768)

* add burnInTestAction capability

* feat: add burn in capability for cloud

* chore: fix snapshot for record_spec

* feat: implement experimental retries (#27826)

* chore: format the retries/runner snapshot files to make diff easier

* feat: implement experimentalRetries strategies 'detect-flake-and-pass-on-threshold' and 'detect-flake-but-always-fail'. This should not be a breaking change, though it does modify mocha and the test object even when the experiment is not configured. This is to exercise the system and make sure things still work as expected even when we go GA. Test updates will follow in following commits.

* chore: update snapshots from system tests and cy-in-cy tests that now have the cypress test metadata property _cypressTestStatusInfo. tests have been added in the fail-with-[before|after]each specs to visually see the suite being skipped when developing.

* chore: add cy-in-cy tests to verify reporter behavior for pass/fail tests, as well as new mocha snapshots to verify attempts. New tests were needed for this as the 'retries' option in testConfigOverrides currently is and will be invalid for experiment and will function as an override. tests run in the cy-in-cy tests are using globally configured experimentalRetries for the given tested project, which showcases the different behavior between attempts/retries and pass/fail status.

* chore: add unit test like driver test to verify the test object in mocha is decorated/handled properly in calculateTestStatus

* chore: add sanity system tests to verify console reporter output for experimental retries logic. Currently there is a bug in the reporter where the logged status doesnt wait for the aftereach to complete, which impacts the total exitCode and printed status.

* fix: aftereach console output. make sure to fail the test in the appropriate spot in runner.ts and not prematurely, which in turn updates the snapshots for cy-in-cy as the fail event comes later."

* chore: address comments from code review

* fix: make sure hook failures print outer status + attempts when the error is the hook itself.

* chore: improve types within calculateTestStatus inside mocha.ts

* Revert "feat: add burnIn Configuration option (currently a no-op) (#27377)"

This reverts commit c428443.

* Revert "chore: add burnInTestAction capability (#27768)"

This reverts commit ae3df1a.

* chore: run snapshot and binary jobs against experimental retries feature branch

* chore: add changelog entry (wip)

* Revert "fix snapshot added in v13 for module api to include test burn in experimentalflag"

This reverts commit bb5046c.

* Fix system tests

* Clear CircleCI cache

* Normalize retries config for test execution

* Fixed some unit tests

* update snapshots for newer test metadata

* Fix cy-in-cy snapshots

* update snapshots

* bump cache version

* chore: ensure legacy retry overrides work; reject exp. retries overrides (#28045)

* update changelog

* flip if statement in experimental retries option validation

* refactor invalid experimental retry override for more useful error msg

* revert testConfigOverrides snapshot

* update snapshots for test override sys test

* Update packages/config/src/validation.ts

Co-authored-by: Chris Breiding <chrisbreiding@users.noreply.github.com>

* succinct changelog entry; links to docs for details

* testConfigOverride system test snapshots

* Update .github/workflows/update_v8_snapshot_cache.yml

Co-authored-by: Ryan Manuel <ryanm@cypress.io>

* Update cli/CHANGELOG.md

Co-authored-by: Ryan Manuel <ryanm@cypress.io>

* Update packages/driver/src/cypress.ts

Co-authored-by: Ryan Manuel <ryanm@cypress.io>

* updating cache-version

* improve typescript usage when appending experimental retry options to experiments in Experimenets.vue

* Revert "improve typescript usage when appending experimental retry options to experiments in Experimenets.vue"

This reverts commit b459aba.

* refactor test config override validation for experimental retry subkeys

* account for error throw differences in browsers in system tests

* bump circle cache

* bump circle cache again

---------

Co-authored-by: astone123 <adams@cypress.io>
Co-authored-by: mabela416 <mabel@cypress.io>
Co-authored-by: Muaz Othman <muaz@cypress.io>
Co-authored-by: Muaz Othman <muazweb@gmail.com>
Co-authored-by: Cacie Prins <cacie@cypress.io>
Co-authored-by: Cacie Prins <cacieprins@users.noreply.github.com>
Co-authored-by: Chris Breiding <chrisbreiding@users.noreply.github.com>
Co-authored-by: Ryan Manuel <ryanm@cypress.io>
Co-authored-by: Matthew Schile <mschile@cypress.io>
Co-authored-by: Matt Schile <mschile@cypress.io>
@cypress
Copy link

cypress bot commented Oct 27, 2023

4 failed and 9 flaky tests on run #51942 ↗︎

4 751 40 0 Flakiness 9

Details:

chore: fixes changelog whitespace (#28168)
Project: cypress Commit: 08d73d4a53
Status: Failed Duration: 19:20 💡
Started: Oct 27, 2023 8:24 PM Ended: Oct 27, 2023 8:43 PM
Failed  cypress\e2e\scaffold-component-testing.cy.ts • 1 failed test • launchpad-e2e

View Output Video

Test Artifacts
scaffolding component testing > react-vite-ts-unconfigured > detects react dependency even if `package.json` is not declared in `exports` Test Replay Output Screenshots
Failed  cypress\e2e\runs.cy.ts • 3 failed tests • app-e2e

View Output Video

Test Artifacts
... > displays a list of recorded runs if a run has been recorded Test Replay Output Screenshots
... > displays each run with correct information Test Replay Output Screenshots
... > opens the run page if a run is clicked Test Replay Output Screenshots
Flakiness  cypress\e2e\scaffold-component-testing.cy.ts • 2 flaky tests • launchpad-e2e

View Output Video

Test Artifacts
scaffolding component testing > vuecli4vue2 > scaffolds component testing for Vue CLI 4 w/ Vue 2 project Test Replay Output Screenshots
scaffolding component testing > angular-cli-unconfigured > scaffolds component testing for Angular Test Replay Output Screenshots
Flakiness  cypress\e2e\runs.cy.ts • 1 flaky test • app-e2e

View Output Video

Test Artifacts
App: Runs > Runs - No Runs > displays a copy button and copies correct command in E2E Test Replay Output Screenshots
Flakiness  cypress\e2e\runner\ct-framework-errors.cy.ts • 2 flaky tests • app-e2e

View Output Video

Test Artifacts
Nuxt > error conditions Test Replay Output Screenshots
Angular 14 > error conditions Test Replay Output Screenshots
Flakiness  cypress\e2e\create-from-component.cy.ts • 1 flaky test • app-e2e

View Output Video

Test Artifacts
... > Shows create from component card for React projects Test Replay Output Screenshots
Flakiness  cypress\e2e\specs.cy.ts • 1 flaky test • app-e2e

View Output Video

Test Artifacts
... > shows No Specs page with specPattern from config Test Replay Output Screenshots

The first 5 flaky specs are shown, see all 6 specs in Cypress Cloud.

Review all test suite changes for PR #28169 ↗︎

@cacieprins cacieprins marked this pull request as ready for review October 27, 2023 19:57
@cacieprins cacieprins changed the title Merge/develop to test burn in chore: merge develop to feature/test-burn-in (do not squash) Oct 30, 2023
@ryanthemanuel ryanthemanuel merged commit 3225c00 into feature/test-burn-in Oct 30, 2023
7 of 10 checks passed
@ryanthemanuel ryanthemanuel deleted the merge/develop-to-test-burn-in branch October 30, 2023 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet