Permalink
Browse files

CORES needs to be a integer

  • Loading branch information...
1 parent f945d15 commit 453188568dc31b29aa2592517ebecb90ecfab681 @rafaelfranca rafaelfranca committed Jun 20, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 activesupport/lib/active_support/testing/isolation.rb
@@ -39,7 +39,7 @@ class ParallelEach
include Enumerable
# default to 2 cores
- CORES = ENV['TEST_CORES'] || 2
+ CORES = ENV['TEST_CORES'].to_i || 2
def initialize list
@list = list

3 comments on commit 4531885

The logic here is faulty, as nil.to_i == 0 so CORES will default to 0; you want CORES = (ENV['TEST_CORES'] || 2).to_i

Owner

It was already fixed at 39dec69

Ah didn't see the other commit!

Please sign in to comment.