Fix RB.DnDUploader state leakage between unit tests.

Review Request #9145 — Created Aug. 22, 2017 and submitted

Information

Review Board
release-3.0.x
034323a...

Reviewers

Many tests are at some point dependent on RB.DnDUploader. Some tests
initialize this, some clear it out, and some don't specifically deal
with it but views they set up do. Depending on which subsets of tests
are run, tests may break due to a lack of an available instance for the
uploader, or due to an instance not being cleaned up.

This change fixes up all those tests, ensuring that we always clear it
out in views that implicitly or explicitly deal with it, and create it
if necessary prior to running tests.

All unit tests pass.

Ran subsets of tests that were breaking before when run in isolation,
and saw that they now pass.

david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-3.0.x (528b268)