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

feat(component-testing): Component testing runner splash screen #15091

Merged
merged 10 commits into from
Feb 15, 2021

Conversation

dmtrKovalenko
Copy link
Contributor

@dmtrKovalenko dmtrKovalenko commented Feb 15, 2021

Here what the user will see once component testing is open:
image

This PR also includes a lot of small UI bugfixes that I found, you can review them one-by-one by following the commits:

  • Fix scroll bars that always displaying
  • Add background to resizer in order to not display 2 px white line
  • Fix keyboard navigation when specs are filtered

@dmtrKovalenko dmtrKovalenko requested a review from a team February 15, 2021 13:08
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Feb 15, 2021

Thanks for taking the time to open a PR!

className={cs('reporter-pane', { 'is-reporter-resizing': isResizing })}
>
<div>
{state.spec && (
<div style={{ height: '100%' }}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

A utility class like full-height will be perfect here, but I don't like global styles :) so instead of making another specs-list-runner-container inlining the style makes it more clear what we need right here

@cypress
Copy link

cypress bot commented Feb 15, 2021



Test summary

9323 0 118 3Flakiness 1


Run details

Project cypress
Status Passed
Commit 12bded1
Started Feb 15, 2021 4:09 PM
Ended Feb 15, 2021 4:19 PM
Duration 10:29 💡
OS Linux Debian - 10.5
Browser Multiple

View run in Cypress Dashboard ➡️


Flakiness

cypress/integration/retries.ui.spec.js Flakiness
1 runner/cypress retries.ui.spec > opens attempt on each attempt failure for the screenshot, and closes after test passes

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@dmtrKovalenko dmtrKovalenko merged commit 1d878d2 into develop Feb 15, 2021
@lmiller1990 lmiller1990 deleted the feat/splash-screen branch February 15, 2021 22:57
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.

2 participants