fix jshint warnings in automation-tests/scripts
grandfather use of foo["bar"] in lib
fix leak to global of wsapi
fix annoying Missing semicolon warnings
use strict equals
remove unused variables
hoist shared counter that is always reinitialized
don't reuse variable
the loopfunc warning is not useful here
grandfather use of foo["bar"] in ./pages
grandfather use of foo["bar"] in ./tests
fix annoying Missing semicolon warnings in tests
fix more annoying Missing semicolon warnings in tests
fix some warnings in tests/public-terminals.js which is pretty broken, but anyways
fix other warnings in tests/public-terminals.js, and etc.
remove an extra ; I added
remove more unused variables in tests/
remove commented-out code from back when, and functions it used to call; it's always in the commit log history
declare variable noAuthTestUser
Merge pull request #3751 from jrgm/autotest-jshint

Clean up jshint warnings in ./automation-tests

Thanks for the cleanup @jrgm. The tests all pass for me still, great stuff!

r+