<p>I finally got around to looking at this - we really do need to get some testing going on our JS code - and it all looked pretty good to me so I have merged it and fixed up the tests to work with the current code.</p>
<p>It's a bit fiddly to get poltergeist and phantomjs working on Fedora at the moment but hopefully it should be easier on other platforms.</p>
<p>That said, it's not clear to me how much of our JS we will be able to test this was - most of <code>osm.js.erb</code> is now covered but most of the rest of it relies on interacting with the pages built by rails. Maybe we should look at using capybara directly for those?</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/105#issuecomment-36292797'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/1419053__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwOTA2ODgyMywiZGF0YSI6eyJpZCI6Mjc2NjQ5OH19--f62f6b7bcd80c06f9096171fa059e46aa19d6565.gif' height='1' width='1'></p>