Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove direct support for HtmlUnit #351

Merged
merged 1 commit into from
Sep 15, 2020

Conversation

klieber
Copy link
Collaborator

@klieber klieber commented Sep 14, 2020

HtmlUnit is no longer bundled with Selenium and it has not be the default WebDriver for a long time now so rather than including it as an extra dependency it makes more sense to just remove direct support for it. That being said it is still possible to configure your project to work with HtmlUnit just with a bit more configuration.

Here is an example:

<plugin>
  <groupId>com.github.searls</groupId>
  <artifactId>jasmine-maven-plugin</artifactId>
  <version>${jasmine-maven-plugin.version}</version>
  <executions>
    <execution>
      <goals>
        <goal>test</goal>
      </goals>
    </execution>
  </executions>
  <configuration>
    <webDriverClassName>org.openqa.selenium.htmlunit.HtmlUnitDriver</webDriverClassName>
    <webDriverCapabilities>
      <capability>
        <name>browserName</name>
        <value>htmlunit</value>
      </capability>
      <capability>
        <name>browserVersion</name>
        <value>firefox</value>
      </capability>
    </webDriverCapabilities>
  </configuration>
  <dependencies>
    <dependency>
      <groupId>org.seleniumhq.selenium</groupId>
      <artifactId>htmlunit-driver</artifactId>
      <version>2.43.1</version>
    </dependency>
  </dependencies>
</plugin>

@klieber klieber force-pushed the feature/remove-htmlunit branch 2 times, most recently from f8cc748 to a7d8a04 Compare September 15, 2020 02:33
@klieber klieber merged commit acce0d2 into searls:master Sep 15, 2020
@klieber klieber changed the title remove htmlunit Remove direct support for HtmlUnit Sep 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant