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(tests): add keyboard interactivity test for button #5037

Merged

Conversation

seanforyou23
Copy link
Collaborator

This PR integrates cypress-tab-plugin and extends integration tests for Button component to ensure keyboard interactions work as expected. It also fixes a broken link in the contributing guidelines page.

What: Steps toward closing #3953

fix broken link in contributing guidelines
@patternfly-build
Copy link
Contributor

patternfly-build commented Oct 20, 2020

@codecov-io
Copy link

Codecov Report

Merging #5037 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5037   +/-   ##
=======================================
  Coverage   52.17%   52.17%           
=======================================
  Files         530      530           
  Lines        9766     9766           
  Branches     3596     3596           
=======================================
  Hits         5095     5095           
  Misses       4010     4010           
  Partials      661      661           
Flag Coverage Δ
#patternfly4 52.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 53fc64a...059b6a8. Read the comment docs.

Copy link
Contributor

@jessiehuff jessiehuff left a comment

Choose a reason for hiding this comment

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

LGTM! 😄

@redallen redallen merged commit 8ccd1cb into patternfly:master Oct 20, 2020
@patternfly-build
Copy link
Contributor

Your changes have been released in:

Thanks for your contribution! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants