Thanks for pointing me to that file. I guess I should use require "activesupport/lib/active_support/testing/isolation" and use railties/ test/isolation/abstract_unit.rb as an example.
I was thinking of creating the file actionpack/test/dependency_test.rb that will test all dependencies for actionpack. This file would have tests like: test "require ActionDispatch::Integration::Session" do require 'action_dispatch/testing/integration' ActionDispatch::Integration::Session end Do you think that's the way to go? Jeroen On Feb 27, 8:52 pm, Yehuda Katz <[email protected]> wrote: > This is actually what the Isolated test case is for. You'll see that we use > it extensively in Railties specifically to isolate tests from added > dependencies. > > Yehuda Katz > Developer | Engine Yard > (ph) 718.877.1325 > > On Sat, Feb 27, 2010 at 11:49 AM, Jeroen van Dijk < > > [email protected]> wrote: > > When hacking on rails 3 I have discovered some missing requires. For > > example, when I wanted to use ActionDispatch::Integration::Session, it > > get an error because Test::Unit::Assertions is used but 'test/unit/ > > assertions' is not required. > > > So I have tried to build a patch for this situation. Well actually, > > the creation of the patch is simple, but how do you test (with test/ > > unit) that we can use this class without requiring its dependency > > 'test/unit/assertions'? To have a real test case I would need an > > isolated environment which would require the class, but would not > > require any other dependency. One thing that is hard, is that the > > rails test suite itself already loads a lot of necessary dependencies > > which would make it hard to write a failing test for these cases (see > > 'abstract_unit' in actionpack/lib/test). > > > Anyhow, when I wrote a first test case I got a chain of dependency > > issues so I think it is worth to investigate this, but I can't really > > think of a way to include it in the current test suite. > > > Any suggestions? > > > Jeroen > > > -- > > 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]<rubyonrails-core%[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.
