| Commit message (Collapse) | Author | Lines |
|
|
|
|
|
We do not wish to log 404 exceptions as those are expected, however,
if something else goes wrong, we shouldn't silence it.
This also removes the explicit None return as it only adds syntax
noise.
|
|
|
|
This will be useful for others tests as well.
|
|
|
|
For cleanliness, I've decided to make a separate class for each method.
Since most tests will want to have an `Incident` instance ready, they
can inherit the `setUp` from `TestIncidents`, which does not make any
assertions on its own.
|
|
Let's make it clear that this is our own mock. We also move the
definition to the top of the module.
|
|
No reason to build own MagicMocks as we already have helpers that
more accurately mimic the mocked behaviour.
|
|
|