-
Notifications
You must be signed in to change notification settings - Fork 219
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Parallelize test execution #298
Comments
I expect that github's test-queue rspec runner would make the tests run faster. |
it looks like test-queue works for mini-test and rspec. The rspec tests are not the bottleneck in this case. Parallel tests may help though since it works with cucumber. |
test-queue has a cucumber runner |
regular cucumber is
I had to modify start_app to work in parallel properly. $ git diff
diff --git a/features/sample-app/sample_app.rb b/features/sample-app/sample_app.rb
index 2d8beda..7646102 100755
--- a/features/sample-app/sample_app.rb
+++ b/features/sample-app/sample_app.rb
@@ -6,11 +6,16 @@ require 'webrick'
class SampleApp
def self.start(host, port)
- Rack::Handler::WEBrick.run new,
+
+ begin
+ @sample_app ||= Rack::Handler::WEBrick.run new,
:Host => host,
:Port => port,
:Logger => ::WEBrick::Log.new(RUBY_PLATFORM =~ /mswin|mingw/ ? 'NUL:' : '/dev/null'),
:AccessLog => [nil, nil]
+ rescue Errno::EADDRINUSE
+ puts "Server already started!"
+ end
end
def initialize |
Sauce allows more VMs for open source projects. I think combined with parallel_tests, we'd be able to reduce the current 3 hours it takes to run the suite.
The text was updated successfully, but these errors were encountered: