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

Tom Hughes notifications at github.com
Thu Mar 9 14:15:48 UTC 2017


Well it's true that those whole tests are of dubious validity since the username used in the URL doesn't actually mean anything as you say.

That's a "white box testing" argument of course, where you as tester know what is really going on, while the "black box" argument would be that you don't know that and you're testing whether it behaves differently when logged in as the alleged user. Of course in that case you can then argue for a third test to check what happens when logged in as a different user ;-)

What is definitely true is that the test is not doing the same thing as before, and different tests with essentially the same logic have been converted in inconsistent ways.

-- 
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-285362167
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170309/401ec7fa/attachment.html>


More information about the rails-dev mailing list