#94 ✓wontfix

spec/spec_helper shouldn't require merb unless an option is passed instructing it to do so

Reported by rbpandey | December 16th, 2008 @ 12:49 PM

When I try to run the mechanize_session_spec.rb i get the error below, but if I comment out require "merb-core" and require "webrat/merb" of spec/spec_helper.rb all is well.

1164-lap:~/dev/git-projects/webrat rpandey$ spec spec/webrat/mechanize/mechanize_session_spec.rb
/opt/local/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:27:in gem_original_require': no such file to load -- merb-core (LoadError) from /opt/local/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:27:inrequire' from /Users/rpandey/dev/git-projects/webrat/spec/spec_helper.rb:10 from /opt/local/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:27:in gem_original_require' from /opt/local/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:27:inrequire' from ./spec/webrat/mechanize/mechanize_session_spec.rb:1 from /opt/local/lib/ruby/gems/1.8/gems/rspec-1.1.11/lib/spec/runner/example_group_runner.rb:14:in load' from /opt/local/lib/ruby/gems/1.8/gems/rspec-1.1.11/lib/spec/runner/example_group_runner.rb:14:inload_files' from /opt/local/lib/ruby/gems/1.8/gems/rspec-1.1.11/lib/spec/runner/example_group_runner.rb:13:in each' from /opt/local/lib/ruby/gems/1.8/gems/rspec-1.1.11/lib/spec/runner/example_group_runner.rb:13:inload_files' from /opt/local/lib/ruby/gems/1.8/gems/rspec-1.1.11/lib/spec/runner/options.rb:98:in run_examples' from /opt/local/lib/ruby/gems/1.8/gems/rspec-1.1.11/lib/spec/runner/command_line.rb:10:inrun' from /opt/local/lib/ruby/gems/1.8/gems/rspec-1.1.11/bin/spec:4 from /opt/local/bin/spec:19:in `load' from /opt/local/bin/spec:19

Comments and changes to this ticket

  • gaffo

    gaffo January 6th, 2009 @ 02:32 AM

    At the moment you pretty much have to install all of the dependencies to be able to run tests for webrat. Eg sinatra, merb-*, rspec, rails.

    The alternative is that people wouldn't run all of the tests such as the tinatra tests before they commit.

    Personally, I'd rather test to make sure that I didn't break something for another framework than have my patch rejected because it does.

    That said, it should likely be better segregated.

  • Bryan Helmkamp

    Bryan Helmkamp January 17th, 2009 @ 02:47 PM

    • State changed from “new” to “wontfix”

    In regards to the specific issue of the spec suite depending on sinatra, merb, rspec and rails, I think that's generally alright considering the alternatives.

    If there are specific ideas for how segregation could make things more maintainable, please open additional tickets and let's discuss them.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Ruby Acceptance Testing for Web applications.

Shared Ticket Bins

People watching this ticket