Skip to content

Remove unused webdriver.chrome.driver property and related configuration from pom.xml #62

Closed as not planned

Description

@coderabbitai

Summary

The webdriver.chrome.driver property and related configuration in pom.xml appear to be unused and can be safely removed.

Details

Tests use WebDriverManager.chromedriver().setup() to manage the Chrome driver and create instances with new ChromeDriver(). The property defined in the <properties> block and passed as a system property in the integration-tests profile via <systemPropertyVariables> is never read by test code. Additionally, the drivers directory referenced by this property does not exist in the repository.

Proposed Cleanup

Remove the following from pom.xml:

  1. The property definition:
<webdriver.chrome.driver>${drivers.dir}/chromedriver</webdriver.chrome.driver>
  1. The system property variable block in the integration-tests profile:
<systemPropertyVariables>
    <!-- Pass location of downloaded webdrivers to the tests -->
    <webdriver.chrome.driver>${webdriver.chrome.driver}</webdriver.chrome.driver>
</systemPropertyVariables>

References

Activity

  1. javier-godoy commented on Apr 28, 2026

    @javier-godoy
    Contributor
    1. The drivers directory is not commited to the repository:

      drivers

    2. Drivers are managed through this dependency

      ChatAssistant/pom.xml

      Lines 167 to 171 in 8bfc64e

      <dependency>
      <groupId>io.github.bonigarcia</groupId>
      <artifactId>webdrivermanager</artifactId>
      <version>${webdrivermanager.version}</version>
      <scope>test</scope>

    3. The driver executable must be included in the operating system PATH or be given to the test using a driver-specific system Java property: Google Chrome: webdriver.chrome.driver (from https://vaadin.com/docs/latest/flow/testing/end-to-end/installing-webdrivers#adding-webdriver-to-system-path)

  2. moved this from Inbox (needs triage) to Done in Flowing Code Addonson May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

invalidThis doesn't seem right

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions