Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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

sonarcloud bot commented Feb 21, 2024

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>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants