<p>Because of the international character of the project, I thought it did no harm to use the tester's locale.</p>
<p>From the reading I did, it seemed that a separate <code>helpers</code> directory was a recommended practice. Helper tests extend <code>ActiveView::TestCase</code> as opposed to <code>ActiveSupport::TestCase</code> so they seem like different beasts, and it raises awareness that there aren't other helper tests.</p>
<p>Using variables (and taking advantage of color coding) made it easier for me to see how the urls were being combined than simply hard-coding them.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/openstreetmap/openstreetmap-website/pull/223#issuecomment-15776984'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/uTRSc6ihLa7Shf84BpiOppw-DtWqir3Uqa5m5zJnEaf0XWJKLnFRuMwmWG1TuifM.gif' height='1' width='1'></p>