tree d4ec7268a263efae65a5f01b8701d2863e3a5d5e
parent b5c1125221ff90aaffd6b01cb299ace9ce81d0c4
author Oliver Newman <olivernewman@google.com> 1579112024 -0800
committer Oliver Newman <olivernewman@google.com> 1579116144 -0800

[fuchsia_cipd_roller] Clean up tests

The tests for this recipe were already quite verbose and made minimal
use of helper functions, presumably due to copy-pasting as more
functionality and tests were added. On top of that, formatting with YAPF
made them completely unreadable.

This continues some work I started in http://fxr/354728:
- make use of helper functions to make the test cases more concise
- override yapf formatting to more clearly distinguish separate test
  data objects
- rename many of the mocks to make it more clear what's happening (at
  the expense of having realistic mock data, which I think is a
  worthwhile tradeoff)
- use the status_check recipe module to make assertions about the
  overall recipe status/failure for each test case

Change-Id: I1d9a2be712a6578520ce7c910ef8a52eef38af67