[openstreetmap/openstreetmap-website] More user factory replacements (#1481)

[Andy Allan]

I realise the display_name is ignored in all cases, but I've been trying to keep the tests reasonably sensible i.e. using the display_name of the owner of the trace. The only tricky bit is dealing with these tests that refer to non-existent traces, since they have no owner.

I've just pushed a change to make them all consistent, i.e. using a fresh create(:user) as the display_name whenever we're referring to a non-existent trace.

-- 
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/1481#issuecomment-285373860
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170309/38e29aac/attachment.html>