chore: update to 24.1.4, add GH action workflow

[manolo]

Fixes #53

[CLAassistant]

All committers have signed the CLA.

[github-actions]

Test Results

22 tests   20 ✔️  2m 5s ⏱️
12 suites    2 💤
12 files      0 ❌

Results for commit 071c6ca.

♻️ This comment has been updated with latest results.

[mcollovati]

An additional note: when running locally I noticed that SimpleCaseIT and WrongBrowserIT leave some browser windows open after the test execution is completed.
I don't know if it is something missing on this project tests (for SimpleCaseIT we should probably call driver.close()) or a potential bug in Testbench (WrongBrowserIT seems to keep the browser open if the test is skipped because of an assumption, like Assumptions.assumeTrue(....))

[manolo]

Added code for closing the browsers as well

[mcollovati]

ScreenshotIT is now failing in development mode because of the dev tools panel in the bottom right of the screen

Expected is

But the taken screenshot is

EDIT: this happened on my local because I had the screenshot in src/test/screenshots/oneplustwo_linux_chrome_115.png created by a previous run in production mode. Once deleted, the test passed.
However, if you then run the test in production mode, it fails for the same reason.
I wonder if we can add a dev/prod token in the reference screenshot name, to make it work correctly on both modes

EDIT 2: The following seems to work

    @Test
    public void testOnePlusTwo() throws Exception {
        ....
        String referenceId = "oneplustwo";
        if (Boolean.TRUE.equals(getCommandExecutor().executeScript("return Vaadin.developmentMode;"))) {
            referenceId += "-development";
        } else {
            referenceId += "-production";
        }
        generateReferenceIfNotFound(referenceId);
       ....
        assertTrue(
                "Screenshot comparison for 'oneplustwo' failed, see "
                        + Parameters.getScreenshotErrorDirectory()
                        + " for error images",
                testBench().compareScreen(referenceId));
    }

[mcollovati]

Other than that, LGTM

[manolo]

@mcollovati good point with the screenshots. Implemented.