Skip to content
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.

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

Tests for OSX just skip if the setup does not complete #46

Open
adiclepcea opened this issue Oct 8, 2017 · 3 comments
Open

Tests for OSX just skip if the setup does not complete #46

adiclepcea opened this issue Oct 8, 2017 · 3 comments

Comments

@adiclepcea
Copy link
Collaborator

In #44 the tests for OSX just skip if the setup fails.
This happens here:
https://github.com/cristianoliveira/ergo/blob/master/tests/ergo_run_test.go#L362
and here:
https://github.com/cristianoliveira/ergo/blob/master/tests/ergo_run_test.go#L508

There is also a mention in the comments there.
They should fail if setup fails just like Linux and Windows tests do.

@adiclepcea
Copy link
Collaborator Author

adiclepcea commented Oct 13, 2017

@cristianoliveira
I had access to a desktop mac today. It seems that the problems comes from the fact that the code is dependent on the Wi-Fi network. If I replaced the Wi-Fi with Ethernet , than it worked ok.
see https://github.com/cristianoliveira/ergo/blob/master/commands/setup.go#L52
and
https://github.com/cristianoliveira/ergo/blob/master/commands/setup.go#L54
Another option would be to replace the line with something like:

sudo networksetup -setautoproxyurl "$(networksetup -listallnetworkservices | grep -v '*' | head 1)" "` + proxyURL + `"

This should take the first network found.
For me it is hard to test it though.

@cristianoliveira
Copy link
Owner

Hey @adiclepcea

I think that works. Another option is setting this for every network like is done in https://github.com/cristianoliveira/ergo/blob/master/resources/osx-setup-runner.sh

I will take a look on this later

@adiclepcea
Copy link
Collaborator Author

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants