Fix WebHook dispatch for local sites

Review Request #8229 — Created June 10, 2016 and submitted — Latest diff uploaded

Information

Review Board
release-2.5.x
2e2c3c4...

Reviewers

WebHooks dispatched from events triggered from local sites would fail
because the WebAPIEncoder would not have access to the local site to
complete local_site_reverse calls. We now pass in the name of the
local site to our FakeHTTPRequest class and store it on the instance
where local_site_reverse can find it. New unit tests have been added
to test each signal that triggers WebHook dispatch with a local site.

  • Saw that WebHooks fired when creating and updating review requests for
    local sites.
  • Ran unit tests.
    Loading...