Fix Sphinx warnings

* Do not reference the non-existent modules `robottelo.factory` and
  `tests.robottelo.test_factory`.
* Do not reference the non-existent method `robottelo.factory.Factory.build`.
* Correctly demarcate a code block with `::`.
* Change a mistaken reference from `gen_missing` to `create_missing`.
Merge pull request #1691 from Ichimonji10/sphinx

Fix Sphinx warnings
Make API tests inherit from APITestCase

Group similar behaviour under a common class.
Drop old APITestCase related code

Drop old APITestCase helper functions.
Merge pull request #1694 from elyezer/api-testcase

Make API tests inherit from APITestCase
Merge pull request #1695 from elyezer/drop-old-code

Drop old APITestCase related code
Fixes for UI timing issues  and adding of bugzilla id to data type.

cherry picked from commit f8ed08ef39e1063f21beeca881bd3570074f9f85
Also resolved for conflicts after cherry picking.

fixed for timing issues s/find_element/wait_until_element/ ui/login.py
----------------
tests.foreman.ui.test_domain.Domain.test_create_domain_1

Fixed for html  [ html chars vary, so not possible to test for 255 chars.]
--------------
tests.foreman.ui.test_domain.Domain.test_create_domain_2

fixed for UTF8 added 'bugzilla': 1144162
---------------
tests.foreman.ui.test_org.Org.test_add_user_2

fixed for UTF8 added 'bugzilla': 1164247
-------------
tests.foreman.ui.test_location.Location.test_remove_configtemplate_1

fixed for timing issues s/find_element/wait_until_element/
-----------------------
tests.foreman.ui.test_user.User.test_positive_create_user_10
tests.foreman.ui.test_user.User.test_positive_create_user_24
tests.foreman.ui.test_user.User.test_positive_create_user_25
tests.foreman.ui.test_user.User.test_positive_create_user_9

Conflicts:
	robottelo/ui/factory.py
	robottelo/ui/login.py
	tests/foreman/ui/test_location.py
	tests/foreman/ui/test_org.py
Merge pull request #1700 from kbidarkar/fixes-from-zsream

Fixes for UI timing issues  and adding of bugzilla id to data type.
Added test for add_repos to cv with names.
Merge pull request #1701 from kbidarkar/add-cv-tests

Added test for add_repos to cv with names.
Added RH content to test_cv_composite_create in ui/contentviews
Merge pull request #1703 from kbidarkar/rh-content-cmpcv

Added RH content to test_cv_composite_create in ui/contentviews
Fixed: test_negative_create_{0-2} org test by setting the edit variable value to False
Fixed: org, location, domain, config_group, medium tests by placing wait_for_ajax before calling search fn
Fixed: test_negative_create{1-4} location tests

(cherry picked from commit aa0b29c9ea8f791b5214ad83ab37183a2cbf4336)
Merge pull request #1724 from sghai/zstream_fixes

Fixed UI test failures - cherry-picked commit from zstream branch
Cleaning up cli code base
Fixed string formatting.
Substituted single double quotes for single.
Updated all docstring that were out of date or missing.
Merge pull request #1693 from JacobCallahan/clean-cli

Cleaning up cli code base
Deduplicate test data

Create function `_test_data` in module `tests.foreman.api.test_repository`. Use
this function to feed data to several DDT data decorators, like this:

    @ddt.data(*_test_data())
    def test_foo(self, data):
        ...

This allows test data to be written out once within the body of that function
and then re-used in several places.
Merge pull request #1734 from Ichimonji10/test-repository

Deduplicate test data
Update EntityDeleteMixin, change _poll_task args

Create method `EntityDeleteMixin.delete_raw`. This method is useful when a
client wishes to delete something and get the response object, which contains
information such as a status code and the raw byte response.

Make method `EntityDeleteMixin.delete` act more like other methods that deal
with foreman tasks. It now always returns a dict of data, rather than evaluating
some mildly complex logic and returning either a foreman task ID or None.

Change the arguments to the `robottelo.orm._poll_task` method. Make that method
use `None` as a default argument to several methods instead of concrete values.
This change makes it easier to override that method.

Update the unit tests for `robottelo.orm.EntityDeleteMixin.delete`.

    $ nosetests tests/robottelo/
    ......................................................................................................................................................
    ----------------------------------------------------------------------
    Ran 150 tests in 0.107s

    OK

Use method `delete_raw` instead of `client.delete`. Using `delete_raw` allows
for more compact code.

    $ nosetests tests/foreman/api/test_multiple_paths.py  -m test_delete_status_code
    ......................
    ----------------------------------------------------------------------
    Ran 22 tests in 31.614s

    OK
Merge pull request #1736 from Ichimonji10/delete-raw

Update EntityDeleteMixin, change _poll_task args