<!DOCTYPE html>

<html>

  <head>

</head>

<body style='font: 12pt Roboto, "DejaVu Sans", Arial; margin-left: 5px; margin-right: 5px;'>

<div id="entry">

<h1 style='font: 12pt Roboto, "DejaVu Sans", Arial; display:block; background: #c0cf88; border-bottom: solid 3px #573; padding: 5px; margin-top: 0px; color: #573;'><a href="https://kallithea-scm.org/repos/kallithea/changeset/ac5d236a995aed95fb28692b1b863cc746b86647" style="text-decoration: none; font-weight: bold; color: #573;">tests: improve robustness of notifications tests  The notifications tests make the assumption that there are no notifications at the start of the test, which...</a></h1>

<div id="body">

patrickdp committed on 2015-06-22 12:56:01<br />branch: default<br />tag: tip<br />changeset: <a href="https://kallithea-scm.org/repos/kallithea/changeset/ac5d236a995aed95fb28692b1b863cc746b86647">ac5d236a</a><pre>tests: improve robustness of notifications tests

The notifications tests make the assumption that there are no notifications

at the start of the test, which is explicitly asserted.

However, any other test, like the one introduced in commit 9a23b444a7fe

(pullrequests: detect invalid reviewers and raise HTTPBadRequest), could add

new notifications to the database, and thus fail these assertions.

Just like the notifications tests already cleaned all notifications at the

end of the test (tearDown), make sure to clean them at the start (setUp)

too.

 M kallithea/tests/models/test_notifications.py (3 lines added, 1 lines removed)</pre>

</div>

</div>

</div>

</body>

</html>