From a678f2f348361526fd95309204d5ac92cd262367 Mon Sep 17 00:00:00 2001
From: Greg Gauthier <gmgauthier@protonmail.com>
Date: Mon, 12 Oct 2020 10:08:01 +0100
Subject: [PATCH] make headless a local var and use it correctly

---
 src/main/java/tools/BrowserDriver.java | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/main/java/tools/BrowserDriver.java b/src/main/java/tools/BrowserDriver.java
index e4534a3..ca34aca 100644
--- a/src/main/java/tools/BrowserDriver.java
+++ b/src/main/java/tools/BrowserDriver.java
@@ -20,11 +20,9 @@ import java.util.List;
 public class BrowserDriver extends RemoteWebDriver {
     private final RemoteWebDriver driver;
     private final String browser;
-    private final Boolean headless;
 
     public BrowserDriver(final String browser, final Boolean headless){
         this.browser = browser;
-        this.headless = headless;
 
         switch (browser) {
             case "chrome":  this.driver = getChrome(headless);
@@ -47,7 +45,7 @@ public class BrowserDriver extends RemoteWebDriver {
         validateBrowser();
         WebDriverManager.chromedriver().setup();
         ChromeOptions options = new ChromeOptions();
-        options.setHeadless(true);
+        options.setHeadless(headless);
         options.addArguments("--ignore-certificate-errors"); // only for limited test envs
         return new ChromeDriver(options);
     }
@@ -56,7 +54,7 @@ public class BrowserDriver extends RemoteWebDriver {
         validateBrowser();
         WebDriverManager.firefoxdriver().setup();
         FirefoxOptions options = new FirefoxOptions();
-        options.setHeadless(true);
+        options.setHeadless(headless);
         options.addArguments("--ignore-certificate-errors");
         return new FirefoxDriver(options);
     }