[openstreetmap/openstreetmap-website] Allow admins and moderators to delete traces (#1734)

Andy Allan notifications at github.com
Sun Feb 11 12:14:41 UTC 2018


It's hard to be sure since that's not a complete test case, but check here:

```
:params => { :display_name => administrator_user.display_name, :id => public_trace_file.id }
```
Is `public_trace_file` actually owned by `administrator_user`? I would expect the `display_name` to be a regular user here (e.g. `public_trace_file.user.display_name`). You are trying to log in as an administrator (the session part of the command) but the trace you are trying to delete should by owned by a normal user, for the test to be correct.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1734#issuecomment-364747036
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180211/30c3ddc0/attachment.html>


More information about the rails-dev mailing list