fix: handle tooltip controller closing state properly

web-padawan commented 1 week ago

Description

These are two findings that I discovered while working on #8129

When close() is called twice e.g. on target mouseleave and then parent mouseover, the second call overwrites the __closeTimeout reference so one of the timeouts is not cleared properly on reopening and tooltip unexpectedly closes,
Also, aborting close only aborts __closeTimeout but the tooltip is not removed from the closing set - so before reopening, it is immediately closed by the logic that is supposed to flush other closing tooltips.

This unfortunately can't be easily reproduced with default tooltips so I had to add a test that checks the controller logic. IMO this is fine since the controller API + manual mode is what we use in vaadin-menu-bar so we need to test it.