Why do we have to sudo?

[zhouzhuojie]

sudo ./run_nightwatch.sh

and

sudo npm install nightwatch@0.5.3 in launch_nightwatch_from_package.sh.

Can we just use npm install -g nightwatch and avoid using sudo? It feels like a security issue here.

[awatson1978]

Feel free to try removing either one of them. You'll find that errors get created with each. Remove the sudo from ./run_nightwatch.sh and the script doesn't have permission to write reports to disk. Remove the sudo from npm install -g nightwatch and nightwatch can't get installed.

I'm not against tweaking the file permissions to get the script to run without sudo references. But the Selenium server is a separate binary that needs access to write files to disk and launch separate processes. So it's not immediately obvious how to go about removing the sudo references and keep everything working.

I'd consider a pull-request (on a separate feature-branch) if you've got some ideas on how to keep all the functionality while removing the sudo references.