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

fix: fix date/time/datetime picker tester validation #1845

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

mcollovati
Copy link
Contributor

Validation API of Date/Time/DateTime picker has been changed since Vaadin 24.5, and the UI Unit testers are failing with recent Vaadin version. This change makes the testers compatible with all Vaadin 24 series.

Fixes #1840

Validation API of Date/Time/DateTime picker has been changed since Vaadin 24.5,
and the UI Unit testers are failing with recent Vaadin version.
This change makes the testers compatible with all Vaadin 24 series.

Fixes #1840
@mshabarov mshabarov requested a review from tepi January 13, 2025 12:38
@mcollovati
Copy link
Contributor Author

Tested the change on 24.5, 24.6 and 24.7 and validation passed (#1846).

@mcollovati mcollovati merged commit 23de5be into main Jan 13, 2025
2 checks passed
@mcollovati mcollovati deleted the issues/1840-fix_date_time_pickers_validation branch January 13, 2025 14:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DatePickerTester and DateTimePickerTester try to find isInvalid
2 participants