What exactly was the discussion you wanted to have? Seems like a pretty simple change. Or were you simply asking to have it pulled in (in which case, it looks like it just was)?
You are correct that the spec folder wasn't being used, it was just legacy code. If anyone is curious (and I suppose I'll update the jquery-rails readme with this info), it's currently tested in two ways. 1. The jquery-ujs adapter that jquery-rails bundles is tested with qunit. 2. And jquery-rails itself has an integration test suite within a basic rails app, which allows me to catch any potential issues pertaining to changes in the rails api. For example, if links started being generated with `data-ajax=true` instead of `data-remote`, this would catch it and let me know the gem needs to be updated. Or, an example that actually did happen. when the behavior with the csrf token changed at one point. #1 can be found within the rails/jquery-ujs project. For #2, I've been meaning to pull this into the jquery-rails repo, but at the moment, it's still under my personal repo: https://github.com/JangoSteve/Rails-jQuery-Demo -- Steve Schwartz On Sunday, April 15, 2012 at 5:29 PM, Rafael Fernández López wrote: > Hi there, > > I don't know if this is the right place to discuss such a topic. I've > suggested to remove jquery-rails spec folder, since it hasn't got any tests, > they are not adapted to RSpec2 and not even to latest rails. They are one > year old and it contains only one empty test. > > I have an application that runs all tests on all engines, being jquery-rails > one of them. I have to explicitly skip this engine since tests are not > passing, and given that they aren't useful at all I didn't even work on > fixing them. > > https://github.com/rails/jquery-rails/issues/55 > > What do you think ? > > And sorry in the first place if this isn't the place to discuss this. If this > isn't the place I'd be glad to be redirected to the right place :) > > > Best regards, > Rafael Fernández López. > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Core" group. > To post to this group, send email to [email protected] > (mailto:[email protected]). > To unsubscribe from this group, send email to > [email protected] > (mailto:[email protected]). > For more options, visit this group at > http://groups.google.com/group/rubyonrails-core?hl=en. -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en.
