Merge pull request #67 from zopaUK/synchronize-nunit-failed-projects

Synchronize access to the failed_projects

lib/bozo/test_runners/nunit.rb

@@ -110,24 +110,29 @@ def runner_args(projects = nil, report_prefix = nil)
 
     def execute
       if @execute_in_parallel
-        failed_projects = []
+        failed_projects = Queue.new
         threads = []
 
         @projects.each do |project|
           t = Thread.new {
             begin
               execute_command :nunit, [runner_path] << runner_args([project], "#{project}-#{Time.now.to_i}")
             rescue
-              failed_projects << project
+              failed_projects.push(project)
             end
           }
           threads.push(t)
         end
 
         threads.each(&:join)
 
-        if failed_projects.length > 0
-          raise Bozo::ExecutionError.new(:nunit, [runner_path] << failed_projects, 1)
+        failed = []
+        until failed_projects.empty?
+          failed << failed_projects.pop
+        end
+
+        if failed.length > 0
+          raise Bozo::ExecutionError.new(:nunit, [runner_path] << failed, 1)
         end
       else
         execute_command :nunit, [runner_path] << runner_args