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

test: cleanup and fix helper focus tests #2256

Merged
merged 1 commit into from
Jul 21, 2021
Merged

Conversation

web-padawan
Copy link
Member

Description

  1. Updated all the tests for helper focus removal to use spy and avoid relying on focused attribute.
  2. Minor cleanup: removed nested components in slotted helper tests, removed empty newlines.

See #2255

@sonarcloud
Copy link

sonarcloud bot commented Jul 21, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@alvarezguille alvarezguille merged commit 4923156 into master Jul 21, 2021
@alvarezguille alvarezguille deleted the test/helper-focus branch July 21, 2021 07:34
alvarezguille added a commit to vaadin/vaadin-checkbox that referenced this pull request Jul 21, 2021
web-padawan added a commit that referenced this pull request Jul 21, 2021
alvarezguille added a commit to vaadin/vaadin-text-field that referenced this pull request Jul 21, 2021
alvarezguille added a commit to vaadin/vaadin-checkbox that referenced this pull request Jul 21, 2021
alvarezguille added a commit to vaadin/vaadin-text-field that referenced this pull request Jul 21, 2021
* fix: do not focus the field on helper element click

cherry pick of vaadin/web-components#2232 for v14

Fixes vaadin/flow-components#955

* chore: move helper tests with own fixture into own suite

also pick vaadin/web-components#2251 for v14
also pick vaadin/web-components#2256 for v14
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.

3 participants