feat: Forward-port Liferay support to Vaadin 23.1

[mshabarov]

Description

Co-authored-by: Mikael Grankvist mgrankvi@vaadin.com

Related-to #221

Type of change

• [ ] Bugfix
• [x] 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.

Additional for Feature type of change

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

[mcollovati]

Running tests locally I got the following error in LiferayUploadIT

org.openqa.selenium.UnsupportedCommandException: unknown command: unknown command: session/f1cfdffffa23ffd675299c0f666f2534/se/file

It seems a know issue with Selenium 4, already fixed in TestBench https://github.com/vaadin/flow-components/issues/2633 https://github.com/SeleniumHQ/selenium/issues/10352

Directly using UploadElement.upload instead of manually handling input WebElement seems to fix the problem locally

UploadElement upload = getVaadinPortletRootElement()
        .$(UploadElement.class).waitForFirst();
/* ----> Current  Implementation <------
WebElement input = upload.$("*").id("fileInput");
setLocalFileDetector(input);
input.sendKeys(file.getAbsolutePath());
 */

upload.upload(file); // <---- Replaces current implementation

[vaadin-bot]

SonarQube analysis reported 7 issues

1. VaadinPortlet.java#L662: Remove these unused method parameters.
2. VaadinPortletResponse.java#L37: Make "HTTP_DATE_FORMAT" an instance variable.
3. VaadinPortletResponse.java#L130: Take the required action to fix the issue indicated by this comment.
4. PortletApplicationRouteRegistryUtil.java#L37: "context" is a method parameter, and should not be used for synchronization.
5. VaadinPortletException.java#L12: Remove this unused import 'com.vaadin.flow.component.HasValue'.
6. VaadinPortletService.java#L68: Rename this field "DEFAULT_HANDLER" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
7. VaadinPortletService.java#L247: Complete the task associated to this TODO comment.