Fix compatibility of I Done This integration with released RB 3.0

Review Request #9466 — Created Dec. 31, 2017 and submitted

Michael Udaltsov

My last update to was done right
before some changes in the main RB repository caused an incompatibility:

This change addresses the breakage and a few more issues with unit tests:

  1. to_submitter_only renamed to to_owner_only to match RB 3.0 API.
  2. Avoid calling set(None) if get_user_team_ids fails, and instead copy
    the set before returning the cached result from get_user_team_ids.
    The exception was caught and ignored by the signal hook, so it did not show
    up in unit tests unless a failed test printed the logged error.
  3. Check for exceptions in more unit tests to avoid hiding the above issue.
  4. Add a new test for review request signal and error during team IDs request.
  5. Fake urlopen calls need **kwargs after a recent kgb update, otherwise
    the fake call raises an exception due to unhandled default arguments.

All unit tests pass.
Manually verified that posting to I Done This works as expected.

David Trowbridge
  1. Ship It!
Michael Udaltsov
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (f0053b8)