fix: add support for java8 date time in signals

taefi commented 2 weeks ago

Description

This introduces endpointObjectMapper bean to be used in both the EndpointInvoker and in the Signals library. If the user overrides the JacksonObjectMapperFactory by the EndpointController.ENDPOINT_MAPPER_FACTORY_BEAN_QUALIFIER qualifier, it also by default affects the the object mapper in the signals library.

Fixes #2891

Type of change

[x] Bugfix
[ ] Feature

Checklist

[x] I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
[x] I have added a description following the guideline.
[x] The issue is created in the corresponding repository and I have referenced it.
[x] I have added tests to ensure my change is effective and works as intended.
[x] New and existing tests are passing locally with my change.
[x] I have performed self-review and corrected misspellings.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.70%. Comparing base (02ae959) to head (3f09442).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #2899 +/- ## ======================================= Coverage 92.70% 92.70% ======================================= Files 83 83 Lines 2851 2851 Branches 739 739 ======================================= Hits 2643 2643 Misses 157 157 Partials 51 51 ``` | [Flag](https://app.codecov.io/gh/vaadin/hilla/pull/2899/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vaadin) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/vaadin/hilla/pull/2899/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vaadin) | `92.70% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vaadin#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features: