fix: ensure session is registered with Hotswapper

mcollovati commented 1 week ago

Description

VaadinSession references are registered in Hotswapper in a session init listener. However, the listener might not always been invoked, for example if the HTTP session is serialized and deserialized on server restart. This change registers a UI init listener to make sure VaadinSession reference is also registered for existing sessions.

Fixes #19973

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.
[ ] New and existing tests are passing locally with my change.
[ ] I have performed self-review and corrected misspellings.

Additional for `Feature` type of change

[ ] Enhancement / new feature was discussed in a corresponding GitHub issue and Acceptance Criteria were created.

github-actions[bot] commented 1 week ago

Test Results

1 137 files ± 0 1 137 suites ±0 1h 9m 58s :stopwatch: - 11m 35s 7 399 tests ± 0 7 349 :white_check_mark: ± 0 50 :zzz: ±0 0 :x: ±0 7 759 runs +39 7 699 :white_check_mark: +39 60 :zzz: ±0 0 :x: ±0

Results for commit adbfe547. ± Comparison against base commit bf420c43.

:recycle: This comment has been updated with latest results.