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: Add/fix tests for binding cross-validation #18770

Merged
merged 3 commits into from
Feb 21, 2024

Conversation

tepi
Copy link
Contributor

@tepi tepi commented Feb 21, 2024

Fixes and adds tests to ensure we don't get a regression like #18735.

Note to reviewer, if running these tests without this patch, the following test failures should happen:

[ERROR]   BinderTest.two_asRequired_fields_without_initial_values_setBean:1582 Age field should not be in error, since it was not modified.
[ERROR]   BinderTest.validated_and_asRequired_fields_without_initial_values_setBean:1644 Last name field should not be in error, since it was not modified.

Copy link

github-actions bot commented Feb 21, 2024

Test Results

1 094 files  ± 0  1 094 suites  ±0   1h 21m 21s ⏱️ + 3m 5s
6 920 tests + 2  6 872 ✅ + 2  48 💤 ±0  0 ❌ ±0 
7 276 runs  +11  7 216 ✅ +11  60 💤 ±0  0 ❌ ±0 

Results for commit e3e3816. ± Comparison against base commit 83c552a.

This pull request removes 1 and adds 3 tests. Note that renamed tests count towards both.
com.vaadin.flow.data.binder.BinderTest ‑ two_asRequired_fields_without_initial_values
com.vaadin.flow.data.binder.BinderTest ‑ two_asRequired_fields_without_initial_values_setBean
com.vaadin.flow.data.binder.BinderTest ‑ validated_and_asRequired_fields_without_initial_values_readBean
com.vaadin.flow.data.binder.BinderTest ‑ validated_and_asRequired_fields_without_initial_values_setBean

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@mcollovati mcollovati left a comment

Choose a reason for hiding this comment

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

Just a formatting nit, but LGTM

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@tepi tepi merged commit bddd220 into main Feb 21, 2024
26 checks passed
@tepi tepi deleted the test/fix-binding-cross-validation-tests branch February 21, 2024 13:23
vaadin-bot pushed a commit that referenced this pull request Feb 21, 2024
* test: Add/fix tests for binding cross-validation

* Fix unintentional formatting changes
vaadin-bot pushed a commit that referenced this pull request Feb 21, 2024
* test: Add/fix tests for binding cross-validation

* Fix unintentional formatting changes
vaadin-bot added a commit that referenced this pull request Feb 21, 2024
* test: Add/fix tests for binding cross-validation

* Fix unintentional formatting changes

Co-authored-by: Teppo Kurki <teppo.kurki@vaadin.com>
vaadin-bot added a commit that referenced this pull request Feb 21, 2024
* test: Add/fix tests for binding cross-validation

* Fix unintentional formatting changes

Co-authored-by: Teppo Kurki <teppo.kurki@vaadin.com>
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