adding skips to stubbed tests in users
removing an unused import
travis wants 2 newlines, not 1
Merge pull request #458 from blrm/user-skips

adding skips to stubbed tests in users
Fixes and updates CLI tests for OS.

Basically our existing CLI tests for Operating System were either
completely outdated/broken or badly implemented. I have removed a few
redundant tests and improved the existing data-driven CRUD tests.

All tests are GREEN except for the test for an existing Bugzilla issue,
which is ON_QA but the fix has not yet landed on latest compose build.

```bash
nosetests -c robottelo.properties tests/cli/test_os.py
INFO ssh:61: Paramiko instance prepared (and would be reused): 0x104b76e90
@test: Update an Operating System's major version. ... ERROR
@test: Displays info for operating system ... ok
@test: Displays list for operating system ... ok
@test: Create Operating System using invalid names ... ok
@test: Create Operating System using invalid names ... ok
@test: Create Operating System using invalid names ... ok
@test: Create Operating System using invalid names ... ok
@test: Create Operating System using invalid names ... ok
@test: Create Operating System using invalid names ... ok
@test: Create Operating System using invalid names ... ok
@test: Not delete Operating System for invalid data ... ok
@test: Not delete Operating System for invalid data ... ok
@test: Not delete Operating System for invalid data ... ok
@test: Not delete Operating System for invalid data ... ok
@test: Not delete Operating System for invalid data ... ok
@test: Negative update of system name ... ok
@test: Negative update of system name ... ok
@test: Negative update of system name ... ok
@test: Negative update of system name ... ok
@test: Negative update of system name ... ok
@test: Negative update of system name ... ok
@test: Negative update of system name ... ok
@test: Create Operating System for all variations of name ... ok
@test: Create Operating System for all variations of name ... ok
@test: Create Operating System for all variations of name ... ok
@test: Create Operating System for all variations of name ... ok
@test: Create Operating System for all variations of name ... ok
@test: Create Operating System for all variations of name ... ok
@test: Successfully deletes Operating System ... ok
@test: Successfully deletes Operating System ... ok
@test: Successfully deletes Operating System ... ok
@test: Successfully deletes Operating System ... ok
@test: Successfully deletes Operating System ... ok
@test: Positive update of system name ... ok
@test: Positive update of system name ... ok
@test: Positive update of system name ... ok
@test: Positive update of system name ... ok
@test: Positive update of system name ... ok
@test: Positive update of system name ... ok
@test: Search for newly created OS by name ... SKIP: Test skipped due to Redmine issue #4547

======================================================================
ERROR: @test: Update an Operating System's major version.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/omaciel/hacking/robottelo/tests/cli/test_os.py", line 129, in test_bugzilla_1051557
    result = OperatingSys.info({'id': result.stdout['id']})
TypeError: list indices must be integers, not str

----------------------------------------------------------------------
XML: foreman-results.xml
----------------------------------------------------------------------
Ran 40 tests in 397.463s

FAILED (SKIP=1, errors=1)
```
Merge pull request #459 from omaciel/fix-cli-test-os

Fixes and updates CLI tests for OS.
Fixes CLI tests for hammer template command.

Fixed the existing tests but had to flag them to be skipped due to
Redmine issue 4560.
CLI create method should always return dictionary

For some resources, the create method for the CLI was returning a list
with the created object as a dictionary, but for some other resources
the result was a plain dictionary. This fix should guarantee that when
a resource is created via the CLI, stdout should always be a dictionary.
Merge pull request #461 from omaciel/fix-template-cli-tests

Fixes CLI tests for hammer template command.
content menu locator changed
Merge pull request #462 from omaciel/create-returns-dictionary

CLI create method should always return dictionary
IMPORTANT fix made:

fixed in ssh module the execute command to wait in a loop for
channel(s) readyness instead of the simple 'if'. It waits for
max 10 seconds.

Other fixes in computeresource module - all tests I got passed here.
Merge branch 'master' into fix-computeresource
flake8 happier - indention, new line issue
Fix make_user cli factory
Fix make_template cli factory
Merge pull request #468 from elyezer/fix-user-cli-factory

Fix make_user cli factory
Merge pull request #469 from elyezer/fix-template-cli-factory

Fix make_template cli factory
Merge pull request #463 from sghai/run-tests

Updated content-menu locator
change the constant to be counted by seconds