Fix issues with the notification manager and tests.

Review Request #8083 — Created March 29, 2016 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x
9ec2e5b...

Reviewers

The cleanup change to notification manager seems to have been committed in a
weird state. The "moved" files just ended up adding the new versions without
deleting the old, which meant the old tests were still running. Additionally,
the implementation of the notify method was missing a local variable
assignment that was needed.

Ran js-tests.

    Loading...