Fix JavaScript test regressions from recently-merged changes.

Review Request #9120 — Created Aug. 4, 2017 and submitted

Christian Hammond
Review Board

Our JavaScript unit tests had a few regressions due to changes made
against Review Board 2.0/2.5 that made use of Jasmine functionality and
Review Board semantics that were no longer correct on 3.0. There were
three breakages:

  1. RB.DnDUploader on 3.0 no longer allows multiple instances to be
    created, and we weren't resetting this.

  2. Jasmine's andCallFake is now and.callFake.

  3. The syntax for asynchronous tests in Jasmine changed, breaking the new
    XSS tests.

These issues have all been fixed.

Unit tests pass.

  • 0
  • 0
  • 2
  • 0
  • 2
Description From Last Updated
David Trowbridge
  2. Typo in summary: changed -> changes

  3. Typo in description: asyncronous -> asynchronous

Christian Hammond
David Trowbridge
  1. Ship It!
Christian Hammond
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (3761759)