Skip to content

add rand port functionality #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
roethke wants to merge 7 commits into
base: master
Choose a base branch
from
Open

add rand port functionality #13

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
0537c88
add random port
suomynona Jun 22, 2017
ac8db2a
fix port
suomynona Jun 22, 2017
b796007
test port
suomynona Jun 22, 2017
04418f3
tmp fix
suomynona Jun 22, 2017
34d490b
add rand port
suomynona Jun 23, 2017
b12abc6
fix
suomynona Jun 23, 2017
2a5b0c2
fix
suomynona Jun 23, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion chromedriver.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ import (
"os/exec"
"strconv"
"time"

"github.com/phayes/freeport"
)

type ChromeSwitches map[string]interface{}
Expand All @@ -36,14 +38,17 @@ type ChromeDriver struct {
logFile *os.File
}

var rand_port = freeport.GetPort()

//create a new service using chromedriver.
//function returns an error if not supported switches are passed. Actual content
//of valid-named switches is not validate and is passed as it is.
//switch silent is removed (output is needed to check if chromedriver started correctly)
func NewChromeDriver(path string) *ChromeDriver {
d := &ChromeDriver{}
d.path = path
d.Port = 9515
// d.Port = 50386
d.Port = rand_port
d.BaseUrl = ""
d.Threads = 4
d.LogPath = "chromedriver.log"
Expand Down