PulpQE / pulp-smash / 2600

63%

Pull #1056

travis-ci

Rewrite response handlers

Currently, API response handlers accept two arguments: a
`PulpSmashConfig` object and a `requests.Response` object. Here's a
typical method signature:

    pulp_smash.api.json_handler(cfg, response)

This mostly works fine. But it does mean that the response handlers lose
some context about the state of the calling `pulp_smash.api.Client`
objects. For example, response handlers don't know if the calling API
client has been instructed to poll a specific
`pulp_smash.config.PulpHost`, as defined by the parent's `pulp_host`
attribute.

Re-write the API response handlers, so that the API client object which
is calling them is now passed to them. As an example of how this is
useful, the `_handle_202` function now knows which Pulp host it should
communicate with when polling asynchronous tasks.

This improved level of access also means that response handlers have a
greater ability to screw things up. Rewrite and expand the docstring on
`pulp_smash.api.Client`, in an attempt to ensure that users know
everything they need to know.

8 of 8 new or added lines in 1 file covered. (100.0%)

698 of 1086 relevant lines covered (64.27%)

1.93 hits per line