Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

Commit f3938f9

Browse files
sptqcnishina
authored andcommitted
fix(jvmArgs): fixes jvmArgs launching selenium from config (#3698)
closes #3697
1 parent 21d534f commit f3938f9

File tree

2 files changed

+12
-1
lines changed

2 files changed

+12
-1
lines changed

lib/config.ts

+8
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,13 @@ export interface Config {
4646
* seleniumArgs: ['-browserTimeout=60']
4747
*/
4848
args?: any;
49+
50+
/**
51+
* Additional command line jvm options to pass to selenium. For example,
52+
* if you need to change the browser driver, use
53+
* jvmArgs: ['-Dwebdriver.ie.driver=IEDriverServer_Win32_2.53.1.exe']
54+
*/
55+
jvmArgs?: string[];
4956
};
5057
/**
5158
* ChromeDriver location is used to help find the chromedriver binary.
@@ -588,6 +595,7 @@ export interface Config {
588595
ng12Hybrid?: boolean;
589596

590597
seleniumArgs?: Array<any>;
598+
jvmArgs?: string[];
591599
configDir?: string;
592600
troubleshoot?: boolean;
593601
seleniumPort?: number;

lib/driverProviders/local.ts

+4-1
Original file line numberDiff line numberDiff line change
@@ -89,13 +89,16 @@ export class Local extends DriverProvider {
8989
if (serverConf.args === undefined) {
9090
serverConf.args = this.config_.seleniumArgs || [];
9191
}
92+
if (serverConf.jvmArgs === undefined) {
93+
serverConf.jvmArgs = this.config_.jvmArgs || [];
94+
}
9295
if (serverConf.port === undefined) {
9396
serverConf.port = this.config_.seleniumPort;
9497
}
9598

9699
// configure server
97100
if (this.config_.chromeDriver) {
98-
serverConf.args.push('-Dwebdriver.chrome.driver=' + this.config_.chromeDriver);
101+
serverConf.jvmArgs.push('-Dwebdriver.chrome.driver=' + this.config_.chromeDriver);
99102
}
100103

101104
this.server_ = new remote.SeleniumServer(this.config_.seleniumServerJar, serverConf);

0 commit comments

Comments
 (0)