[java] implement custom method to toggle downloads enabled

Author: titusfortner

java/src/org/openqa/selenium/remote/AbstractDriverOptions.java

@@ -19,6 +19,7 @@
 
 import static org.openqa.selenium.remote.CapabilityType.ACCEPT_INSECURE_CERTS;
 import static org.openqa.selenium.remote.CapabilityType.BROWSER_VERSION;
+import static org.openqa.selenium.remote.CapabilityType.ENABLE_DOWNLOADS;
 import static org.openqa.selenium.remote.CapabilityType.PAGE_LOAD_STRATEGY;
 import static org.openqa.selenium.remote.CapabilityType.PLATFORM_NAME;
 import static org.openqa.selenium.remote.CapabilityType.PROXY;
@@ -101,6 +102,11 @@ public DO setProxy(Proxy proxy) {
     return (DO) this;
   }
 
+  public DO setEnableDownloads(boolean enableDownloads) {
+    setCapability(ENABLE_DOWNLOADS, enableDownloads);
+    return (DO) this;
+  }
+
   @Override
   public Set<String> getCapabilityNames() {
     TreeSet<String> names = new TreeSet<>(super.getCapabilityNames());

java/src/org/openqa/selenium/remote/CapabilityType.java

@@ -30,4 +30,5 @@ public interface CapabilityType {
   String TIMEOUTS = "timeouts";
   String STRICT_FILE_INTERACTABILITY = "strictFileInteractability";
   String UNHANDLED_PROMPT_BEHAVIOUR = "unhandledPromptBehavior";
+  String ENABLE_DOWNLOADS = "se:downloadsEnabled";
 }

java/test/org/openqa/selenium/chrome/ChromeOptionsTest.java

@@ -84,6 +84,7 @@ void canAddW3CCompliantOptions() {
         .setAcceptInsecureCerts(true)
         .setPageLoadStrategy(PageLoadStrategy.EAGER)
         .setStrictFileInteractability(true)
+        .setEnableDownloads(true)
         .setImplicitWaitTimeout(Duration.ofSeconds(1))
         .setPageLoadTimeout(Duration.ofSeconds(2))
         .setScriptTimeout(Duration.ofSeconds(3));
@@ -96,6 +97,7 @@ void canAddW3CCompliantOptions() {
     assertThat(mappedOptions.get("acceptInsecureCerts")).isEqualTo(true);
     assertThat(mappedOptions.get("pageLoadStrategy")).hasToString("eager");
     assertThat(mappedOptions.get("strictFileInteractability")).isEqualTo(true);
+    assertThat(mappedOptions.get("se:downloadsEnabled")).isEqualTo(true);
 
     Map<String, Long> expectedTimeouts = new HashMap<>();
     expectedTimeouts.put("implicit", 1000L);

java/test/org/openqa/selenium/grid/router/RemoteWebDriverDownloadTest.java

@@ -42,6 +42,7 @@
 import org.openqa.selenium.PersistentCapabilities;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.WebDriverException;
+import org.openqa.selenium.chrome.ChromeOptions;
 import org.openqa.selenium.environment.webserver.NettyAppServer;
 import org.openqa.selenium.grid.config.TomlConfig;
 import org.openqa.selenium.grid.router.DeploymentTypes.Deployment;
@@ -67,6 +68,9 @@ class RemoteWebDriverDownloadTest {
   public void setupServers() {
     Browser browser = Browser.detect();
     assert browser != null;
+    ChromeOptions options = new ChromeOptions();
+    options.setEnableDownloads(true);
+
     capabilities =
         new PersistentCapabilities(browser.getCapabilities())
             .setCapability("se:downloadsEnabled", true);