22706 cli testscontainers and test reporting improvements

[dcolina]

Proposed Changes

• Add a new testcontainers property to enable or disable this functionality
• Fixing some CLI integration tests.

Fixes #22706

🤖 Generated by Copilot at 4589d26

This pull request introduces several changes related to the integration testing of the api-data-model and cli modules of the dotCMS CLI tool. It enables and configures the Testcontainers library for both modules, renames some test and helper classes to improve consistency and clarity, modifies the Maven build and the GitHub workflow to handle test failures and generate the Allure test report, and removes some unused imports from the test classes.

🤖 Generated by Copilot at 4589d26

• Added the testcontainers.enabled property to the application.properties files of the api-data-model and cli modules, with a default value of true, to enable the Testcontainers usage by default for integration testing (link, link)
• Modified the testcontainers.dotcms.license.file property to include a default value of /path/to/license/file.dat in the application.properties files of the api-data-model and cli modules, to make the Testcontainers configuration more clear and avoid errors when the property is not set (link, link)
• Added the TESTCONTAINERS_ENABLED constant to the ContainerResource classes of the api-data-model and cli modules, to make the Testcontainers usage configurable and optional by reading the testcontainers.enabled property (link, link)
• Modified the startContainers and stopContainers methods of the ContainerResource classes of the api-data-model and cli modules, to check the value of the TESTCONTAINERS_ENABLED constant before starting or stopping the Docker containers, and to skip the Testcontainers startup or shutdown when the constant is false (link, link, link, link)
• Changed the <test.failure.ignore> property from true to false in the pom.xml files of the api-data-model and cli modules, to make the Maven build fail fast and report the test failures as soon as possible (link, link)
• Modified the ./mvnw clean verify command in the .github/workflows/cli-cicd-test.yml file, to add the -Dtest.failure.ignore=true option, to allow the Maven build to continue even if some tests fail and to enable the generation of the Allure test report (link)
• Added the .allure directories to the .gitignore file of the tools/dotcms-cli module, to prevent these directories from being tracked by Git and avoid unnecessary files in the repository (link)
• Renamed the FilesTestHelper class to FilesHelperIntegrationTest in the cli module, to make the name of the helper class more consistent and descriptive for integration testing purposes (link, link, link, link, link, link, link, link)
• Renamed the RemoteTraversalServiceTest class to RemoteTraversalServiceIntegrationTest in the cli module, to make the name of the test class more consistent and descriptive for integration testing purposes (link, link)
• Renamed the FilesPullCommandTest class to FilesPullCommandIntegrationTest in the cli module, to make the name of the test class more consistent and descriptive for integration testing purposes (link)
• Removed the unused imports of the Test and Logger classes from the ContainerTest classes of the api-data-model and cli modules, to clean up the code and follow the best practices of Java coding (link, link)

[nollymar]

why do we need these artifacts commented? shouldn't we just remove them?