Downvote to HAML. It's totally unknown outside of ruby-land and would significantly cut down on the number of possible contributors to the theme layer, nixing the two or three people I know who are working on patches.<div>
<br></div><div>Downvote to selenium; MapBox used it on TileMill and it was a headache in every way - setup, maintenance, false-positives (and negatives). The interface level of OSM is not large enough nor complex enough to warrant that kind of testing - it's more important to get the Rails-level tests working well.<br>
<br><div class="gmail_quote">On Fri, Aug 31, 2012 at 9:28 AM, Shaun McDonald <span dir="ltr"><<a href="mailto:shaun@shaunmcdonald.me.uk" target="_blank">shaun@shaunmcdonald.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On 31 Aug 2012, at 10:31, Andy Allan <<a href="mailto:gravitystorm@gmail.com">gravitystorm@gmail.com</a>> wrote:<br>
<br>
> On 30 August 2012 22:49, Tom Hughes <<a href="mailto:tom@compton.nu">tom@compton.nu</a>> wrote:<br>
>> I commented in one of the pull requests I merged a few days ago that I had<br>
>> chosen not to merge a commit that added some tests because it introduced<br>
>> some new testing technology and I thought we should discuss that before<br>
>> making a decision.<br>
><br>
> It's a long time since I've looked at the rails-port tests. There's a<br>
> lot of things I'd like to change, but I'm wary about changing for<br>
> change's sake - given that I know how long it took Shaun to write most<br>
> of them originally, and that I'm unlikely to be spending a similar<br>
> amount of time to re-implement things! So take what I say with a pinch<br>
> of salt.<br>
><br>
>> The specific technology in question was the capybara testing framework along<br>
>> with the selenium driver.<br>
>><br>
>> Personally I don't see any problem with capybara, though maybe some of the<br>
>> other rails people (Shaun? Andy?) here have preferences for something else?<br>
><br>
> I use capybara for testing pages, I don't have any issues with that. I<br>
> don't, however, use any of the javascript drivers, so I can't comment<br>
> on selenium vs webkit. When I've tried getting selenium working<br>
> before, it's been a bit of a nightmare.<br>
<br>
</div>selenium is quite difficult to get working, though does give the advantage of being able to test the js on many browsers, though I'm not sure how best to test that the maps have loaded correctly.<br>
<br>
Webkit is more reliable in getting working, though a year or two ago I came across and issue where someone in the team couldn't get it to work.<br>
<br>
The browser based testing technology is quite fast moving, though I think the technology choices are starting to settle down compared to a couple of years ago.<br>
<div class="im"><br>
><br>
>> Anyway, my more general question is what thoughts any of the rails experts<br>
>> here have on what new technologies we should be considering employing if<br>
>> any...<br>
>><br>
>> That includes both things like testing tools, which I know Shaun has<br>
>> expressed opinions about in the past, and things like alternative template<br>
>> languages - I think Andy has suggested HAML in the past for example.<br>
><br>
> Yep, in summary:<br>
><br>
> * FactoryGirl as a replacement for fixtures<br>
</div>+1 this will also speed the tests up.<br>
<br>
> * shoulda-matchers<br>
No opinion<br>
<div class="im"><br>
> * rspec instead of test::unit, but I doubt it's actually worth the hassle.<br>
</div>Doesn't make much difference. It's just a different way of doing things. test::unit is supposed to be slightly more efficient as it doesn't use the missing_method stuff. I wouldn't say it's worth the change.<br>

<div class="im"><br>
> * haml instead of erb<br>
</div>+1<br>
<div class="im"><br>
> * devise<br>
> * declarative_authorization instead of scattered authorisation around<br>
> controller actions<br>
</div>+1 it does take a bit to get your head around that way of doing things, however makes things much more consistent.<br>
<span class="HOEnZb"><font color="#888888"><br>
Shaun<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> Cheers,<br>
> Andy<br>
><br>
> _______________________________________________<br>
> rails-dev mailing list<br>
> <a href="mailto:rails-dev@openstreetmap.org">rails-dev@openstreetmap.org</a><br>
> <a href="http://lists.openstreetmap.org/listinfo/rails-dev" target="_blank">http://lists.openstreetmap.org/listinfo/rails-dev</a><br>
<br>
<br>
_______________________________________________<br>
rails-dev mailing list<br>
<a href="mailto:rails-dev@openstreetmap.org">rails-dev@openstreetmap.org</a><br>
<a href="http://lists.openstreetmap.org/listinfo/rails-dev" target="_blank">http://lists.openstreetmap.org/listinfo/rails-dev</a><br>
</div></div></blockquote></div><br></div>