From 8fa9f69f4776627ff9b5934bcce82752e7e0d94c Mon Sep 17 00:00:00 2001 From: Alexei Vinogradov Date: Sat, 3 Dec 2016 03:04:17 +0100 Subject: [PATCH] Fix: to avoid using existing /Application/FirefoxNN.app as binary It may exist, but is directory, not binary. --- .../src/org/openqa/selenium/firefox/internal/Executable.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/java/client/src/org/openqa/selenium/firefox/internal/Executable.java b/java/client/src/org/openqa/selenium/firefox/internal/Executable.java index 3775a30f0bd79..838b7e90600ea 100644 --- a/java/client/src/org/openqa/selenium/firefox/internal/Executable.java +++ b/java/client/src/org/openqa/selenium/firefox/internal/Executable.java @@ -124,7 +124,7 @@ private static File locateFirefoxBinaryFromSystemProperty() { return null; File binary = new File(binaryName); - if (binary.exists()) + if (binary.exists() && !binary.isDirectory()) return binary; Platform current = Platform.getCurrent();