specjour 0.2.5 → 0.3.0.rc1

data/lib/specjour.rb

@@ -7,32 +7,35 @@ autoload :Timeout, 'timeout'
 autoload :Benchmark, 'benchmark'
 autoload :Logger, 'logger'
 autoload :Socket, 'socket'
+autoload :StringIO, 'stringio'
 
 module Specjour
+  autoload :CLI, 'specjour/cli'
   autoload :CPU, 'specjour/cpu'
+  autoload :Configuration, 'specjour/configuration'
   autoload :Connection, 'specjour/connection'
+  autoload :DbScrub, 'specjour/db_scrub'
   autoload :Dispatcher, 'specjour/dispatcher'
   autoload :Manager, 'specjour/manager'
   autoload :OpenStruct, 'ostruct'
   autoload :Printer, 'specjour/printer'
   autoload :Protocol, 'specjour/protocol'
+  autoload :QuietFork, 'specjour/quiet_fork'
   autoload :RsyncDaemon, 'specjour/rsync_daemon'
-  autoload :SocketHelpers, 'specjour/socket_helpers'
+  autoload :SocketHelper, 'specjour/socket_helper'
   autoload :Worker, 'specjour/worker'
 
   autoload :Cucumber, 'specjour/cucumber'
   autoload :Rspec, 'specjour/rspec'
 
-  VERSION = "0.2.5".freeze
-
-  class Error < StandardError; end
+  VERSION = "0.3.0.rc1".freeze
 
   def self.logger
     @logger ||= new_logger
   end
 
   def self.new_logger(level = Logger::UNKNOWN)
-    @logger = Logger.new $stdout
+    @logger = Logger.new $stderr
     @logger.level = level
     @logger
   end
@@ -40,4 +43,9 @@ module Specjour
   def self.log?
     logger.level != Logger::UNKNOWN
   end
+
+  Error = Class.new(StandardError)
+  PROGRAM_NAME = $PROGRAM_NAME # keep a reference of the original program name
+
+  GC.copy_on_write_friendly = true if GC.respond_to?(:copy_on_write_friendly=)
 end

data/spec/spec_helper.rb

@@ -3,6 +3,18 @@ $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 require 'specjour'
 require 'spec/autorun'
 
+class NullObject
+  def method_missing(name, *args)
+    self
+  end
+end
+
+begin
+  Specjour::DbScrub
+rescue LoadError
+  $stderr.puts "DbScrub failed to load properly, that's okay though"
+end
+
 Spec::Runner.configure do |config|
   config.mock_with :rr
 end

data/spec/specjour/cli_spec.rb

@@ -0,0 +1,104 @@
+require 'spec_helper'
+
+describe Specjour::CLI do
+  let(:fake_pid) { 100_000_000 }
+  before do
+    stub(Specjour::CPU).cores.returns(27)
+    stub(Specjour::Dispatcher).new.returns(stub!)
+    stub(Specjour::Manager).new.returns(stub!)
+    stub(Specjour::Worker).new.returns(stub!)
+  end
+
+  describe "#listen" do
+    let(:manager) { NullObject.new }
+
+    before do
+      stub(Dir).pwd { '/home/someone/foo-bar' }
+    end
+
+    def manager_receives_options(options)
+      expected_options = hash_including(options)
+      mock(Specjour::Manager).new(expected_options).returns(manager)
+    end
+
+    it "defaults workers to system cores" do
+      manager_receives_options("worker_size" => 27)
+      Specjour::CLI.start %w(listen -p none)
+    end
+
+    it "accepts an array of projects to listen to" do
+      manager_receives_options("registered_projects" => %w(one two three))
+      Specjour::CLI.start %w(listen --projects one two three)
+    end
+
+    it "listens to the current path by default" do
+      manager_receives_options("registered_projects" => %w(foo-bar))
+      Specjour::CLI.start %w(listen)
+    end
+  end
+
+  describe "#dispatch" do
+    let(:dispatcher) { NullObject.new }
+
+    def dispatcher_receives_options(options)
+      expected_options = hash_including(options)
+      mock(Specjour::Dispatcher).new(expected_options).returns(dispatcher)
+    end
+
+    it "defaults path to the current directory" do
+      stub(Dir).pwd.returns("eh")
+      dispatcher_receives_options("project_path" => "eh")
+      Specjour::CLI.start %w(dispatch)
+    end
+
+    it "defaults workers to system cores" do
+      dispatcher_receives_options("worker_size" => 27)
+      Specjour::CLI.start %w(dispatch)
+    end
+
+    it "accepts a project alias" do
+      dispatcher_receives_options("project_alias" => "eh")
+      Specjour::CLI.start %w(dispatch --alias eh)
+    end
+  end
+
+  describe "#work" do
+    it "starts a worker with the required parameters" do
+      worker = NullObject.new
+      args = {'project_path' => "eh", 'printer_uri' => "specjour://1.1.1.1:12345", 'number' => 1, 'task' => 'run_tests'}
+      mock(Specjour::Worker).new(hash_including(args)).returns(worker)
+      Specjour::CLI.start %w(work --project-path eh --printer-uri specjour://1.1.1.1:12345 --number 1 --task run_tests)
+    end
+  end
+
+  describe "#handle_logging" do
+    before do
+      stub(subject).options.returns({})
+    end
+
+    it "enables logging" do
+      subject.options['log'] = true
+      mock(Specjour).new_logger(Logger::DEBUG).returns(stub!)
+      subject.send(:handle_logging)
+    end
+
+    it "doesn't enable logging" do
+      dont_allow(Specjour).new_logger
+      subject.send(:handle_logging)
+    end
+  end
+
+  describe "#prepare" do
+    let(:dispatcher) { NullObject.new }
+
+    def dispatcher_receives_options(options)
+      expected_options = hash_including(options)
+      mock(Specjour::Dispatcher).new(expected_options).returns(dispatcher)
+    end
+
+    it "sets the worker task to 'prepare'" do
+      dispatcher_receives_options("worker_task" => "prepare")
+      Specjour::CLI.start %w(prepare)
+    end
+  end
+end

data/spec/specjour/configuration_spec.rb

@@ -0,0 +1,112 @@
+require 'spec_helper'
+
+module RailsAndActiveRecordDefined
+  def self.extended(base)
+    base.class_eval do
+      before do
+        Object.const_set(:Rails, Module.new)
+        Object.const_set(:ActiveRecord, Module.new)
+        ActiveRecord.const_set(:Base, Class.new)
+      end
+
+      after do
+        Object.send(:remove_const, :ActiveRecord)
+        Object.send(:remove_const, :Rails)
+      end
+    end
+  end
+end
+
+describe Specjour::Configuration do
+  subject do
+    Specjour::Configuration
+  end
+
+  before { subject.reset }
+
+  describe "#before_fork" do
+    context "default proc" do
+      context "ActiveRecord defined" do
+        extend RailsAndActiveRecordDefined
+        it "disconnects from the database" do
+          mock(ActiveRecord::Base).remove_connection
+          subject.before_fork.call
+        end
+      end
+
+      context "bundler installed" do
+        before do
+          mock(subject).system("which bundle") { true }
+        end
+
+        context "when gems not satisfied" do
+          before do
+            mock(subject).system("bundle check") { false }
+          end
+
+          it "runs 'bundle install'" do
+            mock(subject).system('bundle install')
+            subject.before_fork.call
+          end
+        end
+
+        context "when gems are satisfied" do
+          before do
+            mock(subject).system("bundle check") { true }
+          end
+
+          it "doesn't run 'bundle install'" do
+            dont_allow(subject).system('bundle install')
+            subject.before_fork.call
+          end
+        end
+      end
+    end
+
+    context "custom proc" do
+      it "runs block" do
+        subject.before_fork = lambda { :custom_before }
+        subject.before_fork.call.should == :custom_before
+      end
+    end
+  end
+
+  describe "#after_fork" do
+    it "defaults to nothing" do
+      subject.after_fork.call.should be_nil
+    end
+
+    it "runs the block" do
+      subject.after_fork = lambda { :custom_after }
+      subject.after_fork.call.should == :custom_after
+    end
+
+    context "ActiveRecord defined" do
+      extend RailsAndActiveRecordDefined
+      it "scrubs the db" do
+        mock(Specjour::DbScrub).scrub
+        subject.after_fork.call
+      end
+    end
+  end
+
+  describe "#prepare" do
+    it "defaults to nothing" do
+      subject.prepare.call.should be_nil
+    end
+
+    it "runs the block" do
+      subject.prepare = lambda { :custom_preparation }
+      subject.prepare.call.should == :custom_preparation
+    end
+
+    context "ActiveRecord defined" do
+      extend RailsAndActiveRecordDefined
+      it "drops then scrubs the db" do
+        mock(Specjour::DbScrub).drop
+        mock(Specjour::DbScrub).scrub
+        subject.prepare.call
+      end
+    end
+  end
+end

data/spec/{manager_spec.rb → specjour/manager_spec.rb}

@@ -2,8 +2,8 @@ require 'spec_helper'
 
 describe Specjour::Manager do
   describe "#available_for?" do
-    it "is available for all projects by default" do
-      subject.available_for?(rand.to_s).should be_true
+    it "isn't available for all projects by default" do
+      subject.available_for?(rand.to_s).should be_false
     end
 
     it "is available for one project" do

data/spec/specjour/printer_spec.rb

@@ -0,0 +1,101 @@
+require 'spec_helper'
+
+describe Specjour::Printer do
+  subject do
+    Specjour::Printer.new []
+  end
+
+  describe "#exit_status" do
+    let(:rspec_report) { Object.new }
+    let(:cucumber_report) { Object.new }
+
+    context "when cucumber_report is nil" do
+      context "and rspec_report has true exit status" do
+        before do
+          stub(rspec_report).exit_status { true }
+          subject.instance_variable_set(:@rspec_report, rspec_report)
+        end
+
+        it "has a true exit status" do
+          subject.exit_status.should be_true
+        end
+      end
+
+      context "and rspec_report has false exit status" do
+        before do
+          stub(rspec_report).exit_status { false }
+          subject.instance_variable_set(:@rspec_report, rspec_report)
+        end
+
+        it "has a true exit status" do
+          subject.exit_status.should be_false
+        end
+      end
+    end
+
+    context "when rspec report is nil" do
+      context "and cucumber_report has true exit status" do
+        before do
+          stub(cucumber_report).exit_status { true }
+          subject.instance_variable_set(:@cucumber_report, cucumber_report)
+        end
+
+        it "has a true exit status" do
+          subject.exit_status.should be_true
+        end
+      end
+
+      context "and cucumber_report has false exit status" do
+        before do
+          stub(cucumber_report).exit_status { false }
+          subject.instance_variable_set(:@cucumber_report, cucumber_report)
+        end
+
+        it "has a true exit status" do
+          subject.exit_status.should be_false
+        end
+      end
+    end
+
+    context "when both rspec and cucumber reports exists" do
+      context "and rspec exit status is false" do
+        before do
+          stub(rspec_report).exit_status { false }
+          stub(cucumber_report).exit_status { true }
+          subject.instance_variable_set(:@cucumber_report, cucumber_report)
+          subject.instance_variable_set(:@rspec_report, rspec_report)
+        end
+
+        it "returns false" do
+          subject.exit_status.should be_false
+        end
+      end
+
+      context "and cucumber exit status is false" do
+        before do
+          stub(rspec_report).exit_status { true }
+          stub(cucumber_report).exit_status { false }
+          subject.instance_variable_set(:@cucumber_report, cucumber_report)
+          subject.instance_variable_set(:@rspec_report, rspec_report)
+        end
+
+        it "returns false" do
+          subject.exit_status.should be_false
+        end
+      end
+
+      context "both cucumber and rspec exit status are true" do
+        before do
+          stub(rspec_report).exit_status { true }
+          stub(cucumber_report).exit_status { true }
+          subject.instance_variable_set(:@cucumber_report, cucumber_report)
+          subject.instance_variable_set(:@rspec_report, rspec_report)
+        end
+
+        it "returns false" do
+          subject.exit_status.should be_true
+        end
+      end
+    end
+  end
+end

data/spec/{rsync_daemon_spec.rb → specjour/rsync_daemon_spec.rb}

@@ -11,6 +11,8 @@ describe Specjour::RsyncDaemon do
     stub(Dir).chdir
     stub(File).open
     stub(File).read
+    stub(FileUtils).rm
+    stub(Process).kill
   end
 
   specify { subject.config_directory.should == '/tmp/seasonal/.specjour' }

data/spec/specjour_spec.rb

@@ -1,5 +1,16 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
+
+class CustomException < RuntimeError
+end
+
+def boo
+  raise CustomException, 'fails'
+end
 
 describe Specjour do
-  it "lives"
+  it "pends as an example"
+
+  it "fails as an example" do
+    boo
+  end
 end

data/specjour.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{specjour}
-  s.version = "0.2.5"
+  s.version = "0.3.0.rc1"
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.required_rubygems_version = Gem::Requirement.new("> 1.3.1") if s.respond_to? :required_rubygems_version=
   s.authors = ["Sandro Turriate"]
-  s.date = %q{2010-05-13}
+  s.date = %q{2010-07-12}
   s.default_executable = %q{specjour}
   s.description = %q{Distribute your spec suite amongst your LAN via Bonjour.}
   s.email = %q{sandro.turriate@gmail.com}
@@ -28,48 +28,52 @@ Gem::Specification.new do |s|
      "VERSION",
      "bin/specjour",
      "lib/specjour.rb",
+     "lib/specjour/cli.rb",
+     "lib/specjour/configuration.rb",
      "lib/specjour/connection.rb",
      "lib/specjour/cpu.rb",
      "lib/specjour/cucumber.rb",
-     "lib/specjour/cucumber/dispatcher.rb",
      "lib/specjour/cucumber/distributed_formatter.rb",
      "lib/specjour/cucumber/final_report.rb",
-     "lib/specjour/cucumber/printer.rb",
+     "lib/specjour/cucumber/preloader.rb",
      "lib/specjour/db_scrub.rb",
      "lib/specjour/dispatcher.rb",
      "lib/specjour/manager.rb",
      "lib/specjour/printer.rb",
      "lib/specjour/protocol.rb",
+     "lib/specjour/quiet_fork.rb",
      "lib/specjour/rspec.rb",
      "lib/specjour/rspec/distributed_formatter.rb",
      "lib/specjour/rspec/final_report.rb",
      "lib/specjour/rspec/marshalable_rspec_failure.rb",
+     "lib/specjour/rspec/preloader.rb",
      "lib/specjour/rsync_daemon.rb",
-     "lib/specjour/socket_helpers.rb",
-     "lib/specjour/tasks/dispatch.rake",
-     "lib/specjour/tasks/specjour.rb",
+     "lib/specjour/socket_helper.rb",
      "lib/specjour/worker.rb",
-     "rails/init.rb",
-     "spec/cpu_spec.rb",
-     "spec/lib/specjour/worker_spec.rb",
-     "spec/manager_spec.rb",
-     "spec/rsync_daemon_spec.rb",
      "spec/spec.opts",
      "spec/spec_helper.rb",
+     "spec/specjour/cli_spec.rb",
+     "spec/specjour/configuration_spec.rb",
+     "spec/specjour/cpu_spec.rb",
+     "spec/specjour/manager_spec.rb",
+     "spec/specjour/printer_spec.rb",
+     "spec/specjour/rsync_daemon_spec.rb",
      "spec/specjour_spec.rb",
      "specjour.gemspec"
   ]
   s.homepage = %q{http://github.com/sandro/specjour}
   s.rdoc_options = ["--charset=UTF-8"]
   s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.3.6}
+  s.rubygems_version = %q{1.3.7}
   s.summary = %q{Distribute your spec suite amongst your LAN via Bonjour.}
   s.test_files = [
-    "spec/cpu_spec.rb",
-     "spec/lib/specjour/worker_spec.rb",
-     "spec/manager_spec.rb",
-     "spec/rsync_daemon_spec.rb",
-     "spec/spec_helper.rb",
+    "spec/spec_helper.rb",
+     "spec/specjour/cli_spec.rb",
+     "spec/specjour/configuration_spec.rb",
+     "spec/specjour/cpu_spec.rb",
+     "spec/specjour/manager_spec.rb",
+     "spec/specjour/printer_spec.rb",
+     "spec/specjour/rsync_daemon_spec.rb",
      "spec/specjour_spec.rb"
   ]
 
@@ -77,25 +81,28 @@ Gem::Specification.new do |s|
     current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
     s.specification_version = 3
 
-    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<dnssd>, ["= 1.3.1"])
       s.add_runtime_dependency(%q<rspec>, [">= 0"])
+      s.add_runtime_dependency(%q<thor>, ["= 0.13.6"])
       s.add_development_dependency(%q<rspec>, ["= 1.3.0"])
-      s.add_development_dependency(%q<rr>, ["= 0.10.11"])
-      s.add_development_dependency(%q<yard>, ["= 0.5.3"])
+      s.add_development_dependency(%q<rr>, [">= 0.10.11"])
+      s.add_development_dependency(%q<yard>, [">= 0.5.3"])
     else
       s.add_dependency(%q<dnssd>, ["= 1.3.1"])
       s.add_dependency(%q<rspec>, [">= 0"])
+      s.add_dependency(%q<thor>, ["= 0.13.6"])
       s.add_dependency(%q<rspec>, ["= 1.3.0"])
-      s.add_dependency(%q<rr>, ["= 0.10.11"])
-      s.add_dependency(%q<yard>, ["= 0.5.3"])
+      s.add_dependency(%q<rr>, [">= 0.10.11"])
+      s.add_dependency(%q<yard>, [">= 0.5.3"])
     end
   else
     s.add_dependency(%q<dnssd>, ["= 1.3.1"])
     s.add_dependency(%q<rspec>, [">= 0"])
+    s.add_dependency(%q<thor>, ["= 0.13.6"])
     s.add_dependency(%q<rspec>, ["= 1.3.0"])
-    s.add_dependency(%q<rr>, ["= 0.10.11"])
-    s.add_dependency(%q<yard>, ["= 0.5.3"])
+    s.add_dependency(%q<rr>, [">= 0.10.11"])
+    s.add_dependency(%q<yard>, [">= 0.5.3"])
   end
 end