Skip to content

Commit 1bfbba7

Browse files
committed
correctly roll specs while only the minimal (rack) dependencies are set-up
1 parent 03414a7 commit 1bfbba7

File tree

5 files changed

+45
-43
lines changed

5 files changed

+45
-43
lines changed

src/spec/ruby/action_controller/session/java_servlet_store_spec.rb

Lines changed: 16 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,22 @@
1-
#--
2-
# Copyright (c) 2010-2012 Engine Yard, Inc.
3-
# Copyright (c) 2007-2009 Sun Microsystems, Inc.
4-
# This source code is available under the MIT license.
5-
# See the file LICENSE.txt for details.
6-
#++
7-
8-
require File.expand_path('spec_helper', File.dirname(__FILE__) + '/../..')
9-
10-
require 'active_support'
11-
require 'action_controller'
12-
begin # help Rails 3.0 up
13-
require 'action_dispatch/middleware/session/abstract_store'
14-
rescue LoadError
15-
end
16-
begin # a Rails 2.3 require
17-
require 'action_controller/session/abstract_store'
18-
rescue LoadError
19-
end
20-
21-
require 'jruby/rack/session_store'
1+
require File.expand_path('../../spec_helper', File.dirname(__FILE__))
222

233
describe "ActionController::Session::JavaServletStore" do
244

255
before :all do
6+
7+
require 'active_support'
8+
require 'action_controller'
9+
begin # help Rails 3.0 up
10+
require 'action_dispatch/middleware/session/abstract_store'
11+
rescue LoadError
12+
end
13+
begin # a Rails 2.3 require
14+
require 'action_controller/session/abstract_store'
15+
rescue LoadError
16+
end
17+
18+
require 'jruby/rack/session_store'
19+
2620
require 'action_controller/session/java_servlet_store'
2721
end
2822

@@ -322,4 +316,4 @@ def new_session_hash(*args)
322316
::JRuby::Rack::Session::SessionHash.new(store, env)
323317
end
324318

325-
end
319+
end if defined? Rails

src/spec/ruby/jruby/rack/queues/activemq_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
# See the file LICENSE.txt for details.
66
#++
77

8-
require File.expand_path('spec_helper', File.dirname(__FILE__) + '/../../..')
8+
require File.expand_path('../../../spec_helper', File.dirname(__FILE__))
99
require 'jruby/rack/queues/activemq'
1010

1111
describe JRuby::Rack::Queues::ActiveMQ do

src/spec/ruby/jruby/rack/queues/pubsub_spec.rb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,7 @@
55
# See the file LICENSE.txt for details.
66
#++
77

8-
require File.expand_path('spec_helper', File.dirname(__FILE__) + '/../../..')
9-
require 'action_controller'
10-
require 'active_record'
8+
require File.expand_path('../../../spec_helper', File.dirname(__FILE__))
119
require 'jruby/rack/queues'
1210

1311
describe JRuby::Rack::Queues::MessagePublisher do

src/spec/ruby/jruby/rack/queues_spec.rb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,7 @@
55
# See the file LICENSE.txt for details.
66
#++
77

8-
require File.expand_path('spec_helper', File.dirname(__FILE__) + '/../..')
9-
require 'action_controller'
10-
require 'active_record'
8+
require File.expand_path('../../spec_helper', File.dirname(__FILE__))
119
require 'jruby/rack/queues'
1210

1311
describe JRuby::Rack::Queues do

src/spec/ruby/jruby/rack/rails_booter_spec.rb

Lines changed: 26 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -5,16 +5,22 @@
55
# See the file LICENSE.txt for details.
66
#++
77

8-
require File.expand_path('spec_helper', File.dirname(__FILE__) + '/../..')
8+
require File.expand_path('../../spec_helper', File.dirname(__FILE__))
99
require 'jruby/rack/rails_booter'
10-
require 'jruby/rack/rails/extensions'
11-
12-
require 'active_support'
13-
require 'cgi/session/java_servlet_store'
14-
class ::CGI::Session::PStore; end
1510

1611
describe JRuby::Rack::RailsBooter do
1712

13+
before :all do
14+
15+
require 'jruby/rack/rails/extensions'
16+
17+
require 'active_support'
18+
19+
require 'cgi/session/java_servlet_store'
20+
class ::CGI::Session::PStore; end
21+
22+
end
23+
1824
let(:booter) do
1925
JRuby::Rack::RailsBooter.new JRuby::Rack.context = @rack_context
2026
end
@@ -101,7 +107,7 @@ class ::CGI::Session::PStore; end
101107
booter.logger.instance_variable_get(:@logdev).write "hello"
102108
end
103109

104-
describe "Rails 2 environment", :lib => :stub do
110+
describe "Rails 2.3", :lib => :stub do
105111

106112
before do
107113
booter.stub(:rails2?).and_return true
@@ -198,7 +204,7 @@ class ::CGI::Session::PStore; end
198204
end
199205

200206
# NOTE: specs currently only test with a stubbed Rails::Railtie
201-
describe "Rails 3 environment", :lib => :stub do
207+
describe "Rails 3.x", :lib => :stub do
202208

203209
before :each do
204210
$servlet_context = @servlet_context
@@ -304,9 +310,9 @@ def initialize(logger); @logger = logger end
304310

305311
private
306312

307-
def log_initializer
308-
Rails::Railtie.__initializer.detect { |i| i[0] =~ /log/ }
309-
end
313+
def log_initializer
314+
Rails::Railtie.__initializer.detect { |i| i[0] =~ /log/ }
315+
end
310316

311317
end
312318

@@ -329,7 +335,7 @@ def log_initializer
329335
end
330336

331337
# NOTE: specs currently only test with a stubbed Rails::Railtie
332-
describe "Rails 3.1 environment", :lib => [ :stub ] do
338+
describe "Rails 3.1", :lib => [ :stub ] do
333339

334340
before :each do
335341
$servlet_context = @servlet_context
@@ -366,9 +372,14 @@ def log_initializer
366372
end
367373
end
368374

369-
end
375+
end if defined? Rails
370376

371377
describe JRuby::Rack, "Rails controller extensions" do
378+
379+
before :all do
380+
require 'action_controller'
381+
end
382+
372383
before :each do
373384
@controller = ActionController::Base.new
374385
@controller.stub(:request).and_return(request = double("request"))
@@ -389,4 +400,5 @@ def log_initializer
389400

390401
@controller.forward_to "/forward.jsp"
391402
end
392-
end
403+
404+
end if defined? Rails

0 commit comments

Comments
 (0)