fixing 2 cli activation key tests
Blocked tests with Bz id
Merge pull request #1191 from sghai/block-template-test

Blocked template tests with Bz id
Merge pull request #1189 from alda519/cli_fix_actkey

fixing 2 cli activation key tests
Update LifecycleEnvironment field.

`entities.LifecycleEnvironment` has a `prior` foreignkey that points to
another instance of a `entities.LifecycleEnvironment`, which in turn is
renamed to `prior_id` before any actions can be performed. However, the
`Katello` API expects that this attribute will remain unchanged, so this
PR makes sure that `prior` will remain this named to avoid issues.
Merge pull request #1192 from omaciel/lifecycle-prior

Update LifecycleEnvironment field.
update/delete role tests via api
Merge pull request #1194 from sghai/roles-api2

update/delete role tests via api
Revert changes to smoke test suite.

Just recently I updated the smoke test suite to go from
'Default_Organization' and 'Default_Location' to 'Default Organization'
and 'Default Location' respectively (note the absence of the
underscore). Well, turns out that only upstream `Katello` code applies
to this change but not downstream code. So sadly, this PR reverts these
changes and we will handle the upstream tests differently.
Merge pull request #1196 from omaciel/back-to-zero

Revert changes to smoke test suite.
Add tests targeting BZ 1105773

The test fails, indicating that the bug still needs more work.
Merge pull request #1195 from Ichimonji10/bz-1105773

Add tests targeting BZ 1105773
Do not use the `fields` build() and create() arg

Remove all usages of the `field` argument. For example, this:

    SomeEntity().create(fields={'field': value})

Is now written as this:

    SomeEntity(field=value).create()

The `fields` argument to `build()` and `create()` should not be used, as it is
highly error-prone.
Merge pull request #1198 from Ichimonji10/dont-use-fields

Do not use the `fields` build() and create() arg
Remove arg from Factory.{attributes,build,create}

Remove the `fields` argument from methods `Factory.attributes`, `Factory.build`
and `Factory.create`. These three methods largely duplicated the functionality
provided by entities. This is bad in an of itself, because it meant that the
API-related code was not orthogonal:

    SomeEntity(field=value).create()
    SomeEntity().create(fields={'field': value})

However, the arg is also worth dropping because it makes the user vulnerable to
trivial errors. For example, consider these two statements:

    Organization(not_a_field=value).create()
    Organization().create(fields={'not_a_field': value})

Only one of them will raise an error, though both are clearly wrong.
Merge pull request #1199 from Ichimonji10/remove-fields

Remove arg from Factory.{attributes,build,create}