Merge pull request #1466 from TNG/bugfix/fix-selenium

Change order of Drivers to fix Selenium tests
TNG · Dec 8, 2023 · 2817a83 · 2817a83
2 parents a14e8fc + 2e4582a
commit 2817a83
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/jgiven_branch_and_pr_build.yml b/.github/workflows/jgiven_branch_and_pr_build.yml
@@ -12,8 +12,8 @@ jobs:
         java: [ '11', '17', '19']
         junit: ['4.13.2']
     steps:
-      - name: Ensure firefox is installed
-        run: sudo apt-get install firefox
+      - name: Ensure chrome is installed
+        run: wget https://dl.google.com/linux/direct/google-chrome-stable_current_amd64.deb && sudo apt install ./google-chrome-stable_current_amd64.deb && rm *.deb
       - uses: actions/checkout@v4.1.1
       - name: Set up JDK ${{matrix.java}}
         uses: actions/setup-java@v4.0.0

diff --git a/jgiven-tests/src/test/java/com/tngtech/jgiven/report/html5/WebdriverSelectionUtil.java b/jgiven-tests/src/test/java/com/tngtech/jgiven/report/html5/WebdriverSelectionUtil.java
@@ -19,8 +19,8 @@
 
 public class WebdriverSelectionUtil {
     private final Logger log = LoggerFactory.getLogger(WebdriverSelectionUtil.class);
-    private final List<Supplier<WebDriver>> configuredProvider = Arrays.asList( this::setupFirefoxDriver,
-            this::setupChromeDriver, this::setupEdgeDriver);
+    private final List<Supplier<WebDriver>> configuredProvider = Arrays.asList( this::setupChromeDriver,
+            this::setupEdgeDriver, this::setupFirefoxDriver);
 
     public WebDriver setupAnyWebDriver() {
         WebDriver driver = tryAllWebdrivers();