Merge lp://staging/~fs-8/selenium-simple-test/sst-remote-fixes into lp://staging/selenium-simple-test

thanks for picking up that mistake.. you can see that sst.remote is neglected in terms of test coverage compared to the rest of sst :)

one small nitpick before merging this branch:
the way you have brackets/parens closed is not pep8 compliant, so it breaks our pep8 unit test.
can you fix up the closing brackets/parens and re-push to your branch?

it should be like:

    browser_factory = browsers.RemoteBrowserFactory(
        cmd_opts.webdriver_remote_url,
        {
            "browserName": cmd_opts.browser_type.lower(),
            "platform": cmd_opts.browser_platform.upper(),
            "version": cmd_opts.browser_version,
            "javascriptEnabled": not cmd_opts.javascript_disabled,
            "name": cmd_opts.session_name,
        },
    )

regards,
-Corey

Thank you for your review, please excuse my taking so long! I have gladly included your suggestion and re-pushed.

I used pip and installed sst version 0.2.4, but the problem still there, it has not been fixed.

I checked your code, and I think the fundamental problem is in 'browsers.py' line 79. Since selenium's remote webdriver is defined like "def __init__(self, command_executor='http://127.0.0.1:4444/wd/hub',desired_capabilities=None, browser_profile=None, proxy=None, keep_alive=False):" which means the "remote_url" should be the first argument and the second should be the "capabilities".