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: fix kitchensink windows #30909

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

AtofStryker
Copy link
Contributor

@AtofStryker AtofStryker commented Jan 17, 2025

  • Closes N/A

Additional details

fixes the issues with wait-on in the windows-test-binary-against-kitchensink-chrome. wait-on isn't actually installed by the kitchen sink repo, so we need to install it before invoking it

Steps to test

How has the user experience changed?

PR Tasks

@AtofStryker AtofStryker marked this pull request as ready for review January 17, 2025 19:17
Copy link

cypress bot commented Jan 17, 2025

cypress    Run #59951

Run Properties:  status check passed Passed #59951  •  git commit 7b8e6d3586: chore: fix kitchensink windows
Project cypress
Branch Review chore/fix_windows_kitchensink
Run status status check passed Passed #59951
Run duration 18m 14s
Commit git commit 7b8e6d3586: chore: fix kitchensink windows
Committer AtofStryker
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 9
Tests that did not run due to a developer annotating a test with .skip  Pending 1099
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 26477
View all changes introduced in this branch ↗︎
UI Coverage  45.43%
  Untested elements 192  
  Tested elements 164  
Accessibility  92.55%
  Failed rules  3 critical   8 serious   2 moderate   2 minor
  Failed elements 888  

@AtofStryker AtofStryker force-pushed the chore/fix_windows_kitchensink branch 3 times, most recently from 50b0525 to adea4eb Compare January 22, 2025 02:30
@@ -2769,7 +2779,6 @@ jobs:
- test-binary-against-rwa:
repo: cypress-realworld-app
browser: chrome
wait-on: http://localhost:3000
Copy link
Contributor Author

Choose a reason for hiding this comment

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

wasn't actually doing anything as wait-on wasn't defined

@AtofStryker AtofStryker force-pushed the chore/fix_windows_kitchensink branch from adea4eb to 7b8e6d3 Compare January 22, 2025 15:21
@@ -908,10 +908,6 @@ commands:
description: Pull request number to check out before installing and testing
type: integer
default: 0
wait-on:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

param defined but isn't actually used

@AtofStryker AtofStryker self-assigned this Jan 22, 2025
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.

1 participant