Check port before connecting, not on client creation
Add BZ link catalog item duplicate test
Fixed login tests

Major change is the logical OR-ing of locators in the flash region;
the login page flash_div was never being looked for based on my
selenium logging.

Minor changes:
- Login and logout are combined into one test
- Bad password test now expects the error being raised by a failed login
- user dropdown moved to dashboard, since it isn't visible on the login page
- dashboard gets an identifying_loc
Always take a browser screenshot

Many tests are failing because they aren't marked to use the browser,
despite doing so in the course of their tests.

So, rather than inspecting the fixtures to decide whether or not to take
a screenshot, we'll just go ahead and take one. There's enough info in
the artifactor report now that we can easily decide if the screenshot is
useful.
Handling glitch from BZ#1112574
Split actions policy fixture

* Policy create relied on being able to assign to a provider.
* If assignment failed, policy would be recreated with the same
  name/description leading to failure.
* Modifications remove the assignment from the fixture and place
  in a new fixture.
Tree can read itself; pytest_selenium shortcut text() can pass stuff through
Disable legacy tree support
We can now read legacy trees
fix 'typo'
browse func
Update ScriptBox to use JS method for CodeMirror

* Previous implementation was broken with new move_to_element additions.
* Pure JS implementation is faster.
Flakey
Merge pull request #899 from seandst/login-catch-assert

Fixed login tests
Merge pull request #904 from psav/fix_script_box

Update ScriptBox to use JS method for CodeMirror
Merge pull request #905 from psav/flake

Flakey
Merge pull request #901 from mfalesni/glitch_1112574

Handling glitch from BZ#1112574
Merge pull request #903 from psav/actions_split_fixture

Split actions policy fixture
Handy comment!
Merge pull request #906 from seandst/always-take-browsershot

Always take a browser screenshot