Skip to content

Commit

Permalink
fixed issue with certain browsers, when defining no proxy configuration
Browse files Browse the repository at this point in the history
  • Loading branch information
slu-it committed Mar 15, 2017
1 parent c85f38f commit 43c9f8a
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
import info.novatec.testit.webtester.browser.Browser;
import info.novatec.testit.webtester.browser.BrowserFactory;
import info.novatec.testit.webtester.browser.WebDriverBrowser;
import info.novatec.testit.webtester.browser.proxy.NoProxyConfiguration;
import info.novatec.testit.webtester.browser.proxy.ProxyConfiguration;


Expand Down Expand Up @@ -61,7 +62,7 @@ public Browser createBrowser(WebDriver webDriver) {
}

protected void setOptionalProxyConfiguration(DesiredCapabilities capabilities) {
if (proxyConfiguration != null) {
if (proxyConfiguration != null && !(proxyConfiguration instanceof NoProxyConfiguration)) {
Proxy proxy = new Proxy();
proxyConfiguration.configureProxy(proxy);
capabilities.setCapability(CapabilityType.PROXY, proxy);
Expand Down

0 comments on commit 43c9f8a

Please sign in to comment.