pablobm left a comment (openstreetmap/openstreetmap-website#6564)
Ah, you are right about the provenance of `@user`, I got a bit mixed up when
putting together the writeup. But I also missed to include a detail: it's
`current_user&.administrator?` that is the problem. In this test, the
administrator is the current user. The user is suspended, so it's not
`visible?` and only an administrator can see the heatmap, therefore a
`current_user` is required, missing, and `@heatmap_data` is not set.
Also, either way the controller allows for `@heatmap_data` not to be set.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6564#issuecomment-3612351066
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6564/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
