[PATCH 25/25] tests: Remove 'default_states' fixture requirement

Andy Doan andy.doan at linaro.org
Thu Jun 30 03:10:20 AEST 2016


On 06/23/2016 04:53 PM, Stephen Finucane wrote:
> The 'default_states' fixture is used in many tests. However, the
> 'default_states' are merely suggestions for various States, and the
> names themselves have no meaning from a patchwork perspective. In
> addition, these fixtures significantly increase run time of the tests.
>
> Resolve this by creating new State objects on demand, rather than
> using versions provided by the fixture.
>
> Signed-off-by: Stephen Finucane <stephen.finucane at intel.com>

Good idea. Now that REST api is merged you may have to make changes to 
test_rest_api before you can apply this.

Reviewed-by: Andy Doan <andy.doan at linaro.org>



More information about the Patchwork mailing list