guard-phpunit2 0.2.6 → 0.2.7

data/spec/guard/phpunit2/logreader_spec.rb

@@ -0,0 +1,62 @@
+require 'spec_helper'
+
+describe Guard::PHPUnit2::LogReader do
+
+  describe '.parse_output' do
+    context 'when all tests pass' do
+      it 'returns a hash containing the results' do
+        output = load_phpunit_output('passing-json')
+        subject.parse_output(output).should == {
+          :tests => 2, :failures => 0,
+          :errors => 0, :pending => 0,
+          :duration => [0, 'seconds']
+        }
+      end
+    end
+
+    context 'when all tests fail' do
+      it 'returns a hash containing the tests result' do
+        output = load_phpunit_output('failing-json')
+        subject.parse_output(output).should == {
+          :tests  => 2, :failures => 2,
+          :errors => 0, :pending  => 0,
+          :duration => [0, "seconds"]
+        }
+      end
+    end
+
+    context 'when tests are skipped or incomplete' do
+      it 'returns a hash containing the tests result' do
+        output = load_phpunit_output('skipped_and_incomplete-json')
+        subject.parse_output(output).should == {
+          :tests  => 3, :failures => 0,
+          :errors => 0, :pending  => 3,
+          :duration => [0, "seconds"]
+        }
+      end
+    end
+
+    context 'when tests have mixed statuses' do
+      it 'returns a hash containing the tests result' do
+        output = load_phpunit_output('mixed-json')
+        subject.parse_output(output).should == {
+          :tests  => 5, :failures => 1,
+          :errors => 1, :pending  => 2,
+          :duration => [0, "seconds"]
+        }
+      end
+    end
+
+    context 'multiple test classes' do
+      it 'retuns the correct total sum of tests ran' do
+        output = load_phpunit_output('multiple-test-cases-json')
+        subject.parse_output(output).should == {
+          :tests => 14, :failures => 4,
+          :errors => 1, :pending => 5,
+          :duration => [0, 'seconds']
+        }
+      end
+    end
+
+  end
+end

data/spec/guard/phpunit2/realtime_runner_spec.rb

@@ -0,0 +1,167 @@
+require 'spec_helper'
+
+describe Guard::PHPUnit2::RealtimeRunner do
+
+  let(:formatter) { Guard::PHPUnit2::Formatter }
+  let(:logreader) { Guard::PHPUnit2::LogReader }
+  let(:notifier)  { Guard::PHPUnit2::Notifier  }
+  let(:ui)        { Guard::UI                 }
+
+  describe '#run' do
+    before do
+      FileUtils.stub(:ln_s)
+      FileUtils.stub(:mkdir_p)
+
+      subject.stub(:execute_command)
+      subject.stub(:phpunit_exists?).and_return(true)
+      notifier.stub(:notify_results)
+
+      $?.stub(:success?).and_return(true)
+    end
+
+    context 'when passed an empty paths list' do
+      it 'returns false' do
+        subject.run([]).should be_false
+      end
+    end
+
+    shared_examples_for 'paths list not empty' do
+      it 'checks that phpunit is installed' do
+        subject.should_receive(:phpunit_exists?)
+        subject.run( ['tests'] )
+      end
+
+      it 'displays an error when phpunit is not installed' do
+        subject.stub(:phpunit_exists?).and_return(false)
+        ui.should_receive(:error).with('the provided php unit command is invalid or phpunit is not installed on your machine.', anything)
+
+        subject.run( ['tests'] )
+      end
+
+      it 'notifies about running the tests' do
+        subject.should_receive(:notify_start).with( ['tests'], anything )
+        subject.run( ['tests'] )
+      end
+
+      it 'runs phpunit tests' do
+        subject.should_receive(:execute_command).with(
+          %r{^phpunit .+$}
+        ).and_return(true)
+        subject.run( ['tests'] )
+      end
+
+      it 'runs phpunit tests with provided command' do
+        formatter_path = @project_path.join('lib', 'guard', 'phpunit', 'formatters', 'PHPUnit-Progress')
+        subject.should_receive(:execute_command).with(
+          %r{^/usr/local/bin/phpunit --include-path #{formatter_path} --printer PHPUnit_Extensions_Progress_ResultPrinter .+$}
+        ).and_return(true)
+        subject.run( ['tests'] , {:command => '/usr/local/bin/phpunit'} )
+      end
+
+      context 'when PHPUnit executes the tests' do
+        it 'parses the tests log output' do
+          log    = load_phpunit_output('passing-json')
+          subject.stub(:execute_phpunit).and_return(log)
+
+          logreader.should_receive(:parse_output).with(log)
+
+          subject.run( ['tests'] )
+        end
+
+        it 'notifies about the tests output' do
+          log = load_phpunit_output('passing-json')
+          subject.stub(:execute_phpunit).and_return(log)
+          subject.should_receive(:notify_results).with(log, anything())
+
+          subject.run( ['tests'] )
+        end
+
+        it 'notifies about the tests output even when they contain failures' do
+          $?.stub(:success? => false, :exitstatus => 1)
+
+          subject.stub(:execute_command)
+          subject.should_receive(:notify_results).with(anything(), anything())
+
+          subject.run( ['tests'] )
+        end
+
+        it 'notifies about the tests output even when they contain errors' do
+          $?.stub(:success? => false, :exitstatus => 2)
+
+          subject.stub(:execute_command)
+          subject.should_receive(:notify_results).with(anything(), anything())
+
+          subject.run( ['tests'] )
+        end
+
+        it 'does not notify about failures' do
+          subject.should_receive(:execute_command)
+          subject.should_not_receive(:notify_failure)
+          subject.run( ['tests'] )
+        end
+      end
+
+      context 'when PHPUnit fails to execute' do
+        before do
+          $?.stub(:success? => false, :exitstatus => 255)
+          notifier.stub(:notify)
+        end
+
+        it 'notifies about the failure' do
+          subject.should_receive(:execute_command)
+          subject.should_receive(:notify_failure)
+          subject.run( ['tests'] )
+        end
+
+        it 'does not notify about the tests output' do
+          subject.should_not_receive(:notify_results)
+          subject.run( ['tests'] )
+        end
+      end
+
+      describe 'options' do
+        describe ':cli' do
+          it 'runs with CLI options passed to PHPUnit' do
+            cli_options = '--colors --verbose'
+            subject.should_receive(:execute_command).with(
+              %r{^phpunit --include-path .* --printer .* #{cli_options} .+$}
+            ).and_return(true)
+            subject.run( ['tests'], :cli => cli_options )
+          end
+        end
+
+        describe ':notification' do
+          it 'does not notify about tests output with notification option set to false' do
+            formatter.should_not_receive(:notify)
+            subject.run( ['tests'], :notification => false )
+          end
+        end
+      end
+    end
+
+    context 'when passed one path' do
+      it_should_behave_like 'paths list not empty'
+
+      it 'should not create a test folder' do
+        Dir.should_not_receive(:mktmpdir)
+        subject.run( ['spec/fixtures/sampleTest.php'] )
+      end
+    end
+
+    context 'when passed multiple paths' do
+      it_should_behave_like 'paths list not empty'
+
+      it 'creates a tests folder (tmpdir)' do
+        subject.should_receive(:create_tests_folder_for).with(instance_of(Array))
+        subject.stub(:execute_phpunit).and_return(load_phpunit_output('passing-json'))
+        subject.stub(:notify_results)
+        subject.run( ['spec/fixtures/sampleTest.php', 'spec/fixtures/emptyTest.php'] )
+      end
+
+      it 'symlinks passed paths to the tests folder' do
+        subject.should_receive(:symlink_paths_to_tests_folder).with(anything(), anything())
+        subject.run( ['spec/fixtures/sampleTest.php', 'spec/fixtures/emptyTest.php'] )
+      end
+    end
+  end
+end

data/spec/guard/phpunit2_spec.rb

@@ -27,6 +27,10 @@ describe Guard::PHPUnit2 do
       it 'sets a default :notification option' do
         subject.options[:notification].should be_true
       end
+
+      it 'sets a default :realtime option' do
+        subject.options[:realtime].should be_false
+      end
     end
 
     context 'when other options are provided' do
@@ -35,7 +39,8 @@ describe Guard::PHPUnit2 do
                                           :all_after_pass => false,
                                           :keep_failed    => false,
                                           :cli            => '--colors',
-                                          :tests_path     => 'tests'     }) }
+                                          :tests_path     => 'tests',
+                                          :realtime       => true}) }
 
       it 'sets :all_on_start with the provided option' do
         subject.options[:all_on_start].should be_false
@@ -56,6 +61,10 @@ describe Guard::PHPUnit2 do
       it 'sets :tests_path with the provided option' do
         subject.options[:tests_path].should == 'tests'
       end
+
+      it 'sets :realtime with the provided option' do
+        subject.options[:realtime].should be_true
+      end
     end
 
     it 'sets the tests path for the inspector' do
@@ -99,6 +108,25 @@ describe Guard::PHPUnit2 do
     end
   end
 
+  describe 'realtime handling' do
+    describe 'realtime endabled' do
+      let(:guard) { Guard::PHPUnit2.new(nil, {:realtime => true }) }
+      it 'should call run on the realtimerunner' do
+        Guard::PHPUnit2::RealtimeRunner.should_receive(:run).and_return(true)
+        guard.run_on_changes ['tests/firstTest.php']
+      end
+    end
+
+    describe 'realtime disabled' do
+      let(:guard) { Guard::PHPUnit2.new(nil, {:realtime => false }) }
+
+      it 'should call run on the Runner class' do
+        Guard::PHPUnit2::Runner.should_receive(:run).and_return(true)
+        guard.run_on_changes ['tests/firstTest.php']
+      end
+    end
+  end
+
   describe '#run_on_changes' do
     before do
       inspector.stub(:clean).and_return { |paths| paths }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: guard-phpunit2
 version: !ruby/object:Gem::Version
-  version: 0.2.6
+  version: 0.2.7
 platform: ruby
 authors:
 - Maher Sallam
@@ -12,9 +12,9 @@ bindir: bin
 cert_chain:
 - |
   -----BEGIN CERTIFICATE-----
-  MIIDfDCCAmSgAwIBAgIBATANBgkqhkiG9w0BAQUFADBCMQ4wDAYDVQQDDAVyYW1v
+  MIIDODCCAiCgAwIBAgIBAjANBgkqhkiG9w0BAQUFADBCMQ4wDAYDVQQDDAVyYW1v
   bjEbMBkGCgmSJomT8ixkARkWC2NvZGVjcmFmdDYzMRMwEQYKCZImiZPyLGQBGRYD
-  Y29tMB4XDTEzMDMyNzE3NDEyOVoXDTE0MDMyNzE3NDEyOVowQjEOMAwGA1UEAwwF
+  Y29tMB4XDTE0MDQwMjEzMzEwOVoXDTE1MDQwMjEzMzEwOVowQjEOMAwGA1UEAwwF
   cmFtb24xGzAZBgoJkiaJk/IsZAEZFgtjb2RlY3JhZnQ2MzETMBEGCgmSJomT8ixk
   ARkWA2NvbTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBANMe6/bxj0lU
   QJBEAPZg6YDvjo/by2I62ww7wUgd9Yh5gOFe0uxhrr4bOCRADhhC4icYow1NZ8Ju
@@ -22,17 +22,16 @@ cert_chain:
   XBfffGikJEwclkpdr4mRgm4TaBzkZUF029axX6YM7G5AbBcYfbHwSss+fLsXB1t9
   bU07BDLV6KSnFjn/6Gj9CrZpw3G0TCunTvB6UjcZIoBqXeWz6z9XJX+GhyaZGCjQ
   R1rXqqEfSRwsG31ibcTTFrqtoyTkzmz33h5LbMZZdLoMpeEFggkY0cQBUY4sGYK/
-  jrbnzdwgwtsCAwEAAaN9MHswCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAwHQYDVR0O
-  BBYEFIcHXF/pGBCu96efKVZ3eH10IgSxMCAGA1UdEQQZMBeBFXJhbW9uQGNvZGVj
-  cmFmdDYzLmNvbTAgBgNVHRIEGTAXgRVyYW1vbkBjb2RlY3JhZnQ2My5jb20wDQYJ
-  KoZIhvcNAQEFBQADggEBAJ1v3i31c1ADsx4eWX+RPYLbZK0y9je4EDbswsE7owWJ
-  gDmVZFa9LYzLTQf0ouR4yb3W5nwEkPl6VDGkSSwmGeGzAj+Z+MQgLRz3PP1IRKji
-  +UZWMqM5Mv3aIkTEmAGitRjSfi6RpQiBG0L24hJ66dw5KrQp2dUOdUf3tR+KWv90
-  Uoz3nIfPvMYkUOYxNWdC/uxQf7VNR3eXerjGLKskvrjBgP3kjytvmlEvido8OwKm
-  tfg0X0xrfUfUjJj8Ttd2n6fUss+S/H631xMHWA19shwEvey1SKOdCCvMjpV1rwup
-  7g4EOpFUXKG2E+5vow3VDEqcOhfQYvQ8UB3Of3FOp2o=
+  jrbnzdwgwtsCAwEAAaM5MDcwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAwHQYDVR0O
+  BBYEFIcHXF/pGBCu96efKVZ3eH10IgSxMA0GCSqGSIb3DQEBBQUAA4IBAQBVmXZf
+  1PL29cv73uxKHrIXMPIiUHwKW+8twzorMX+3BlYsQSq0OEOloHV4qHpEYtdOKTwt
+  FKqX3EPTy7HVQpeQjTJRo8qeqD5xMnJAFve6deF4wH5vTtjpF6BQBN7OEFSlyywK
+  q5TzllwektfgZuo6o6+TEZQRaTAuVioHFCyJ1SwmyNIHH5whXUlCWrurt7csbfnJ
+  J1k3vWJjQrItZ60kgoi2rBCPx7qAJG/lZ8UQoCMJMAAXGzarUrWvK6CKd2d1phth
+  tUGlQK5jGose4s/n5BjifxE3pKoirkrICQy813CvapcJDj5qAIFt3ZCEL/qiwrzY
+  VGMazdO/EkS6KnJc
   -----END CERTIFICATE-----
-date: 2014-02-01 00:00:00.000000000 Z
+date: 2014-04-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: guard
@@ -173,20 +172,33 @@ files:
 - lib/guard/phpunit2.rb
 - lib/guard/phpunit2/formatter.rb
 - lib/guard/phpunit2/inspector.rb
+- lib/guard/phpunit2/logreader.rb
 - lib/guard/phpunit2/notifier.rb
+- lib/guard/phpunit2/realtime_runner.rb
 - lib/guard/phpunit2/runner.rb
 - lib/guard/phpunit2/templates/Guardfile
 - lib/guard/phpunit2/version.rb
 - spec/fixtures/emptyTest.php
+- spec/fixtures/failureSampleTest.php
+- spec/fixtures/mixedSampleTest.php
+- spec/fixtures/passingSampleTest.php
 - spec/fixtures/results/errors.txt
+- spec/fixtures/results/failing-json.txt
 - spec/fixtures/results/failing.txt
+- spec/fixtures/results/mixed-json.txt
 - spec/fixtures/results/mixed.txt
+- spec/fixtures/results/multiple-test-cases-json.txt
+- spec/fixtures/results/passing-json.txt
 - spec/fixtures/results/passing.txt
+- spec/fixtures/results/skipped_and_incomplete-json.txt
 - spec/fixtures/results/skipped_and_incomplete.txt
 - spec/fixtures/sampleTest.php
+- spec/fixtures/skippedSampleTest.php
 - spec/guard/phpunit2/formatter_spec.rb
 - spec/guard/phpunit2/inspector_spec.rb
+- spec/guard/phpunit2/logreader_spec.rb
 - spec/guard/phpunit2/notifier_spec.rb
+- spec/guard/phpunit2/realtime_runner_spec.rb
 - spec/guard/phpunit2/runner_spec.rb
 - spec/guard/phpunit2_spec.rb
 - spec/spec_helper.rb
@@ -210,21 +222,32 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.6
 requirements: []
 rubyforge_project: guard-phpunit2
-rubygems_version: 2.2.0
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Guard gem for PHPUnit
 test_files:
 - spec/fixtures/emptyTest.php
+- spec/fixtures/failureSampleTest.php
+- spec/fixtures/mixedSampleTest.php
+- spec/fixtures/passingSampleTest.php
 - spec/fixtures/results/errors.txt
+- spec/fixtures/results/failing-json.txt
 - spec/fixtures/results/failing.txt
+- spec/fixtures/results/mixed-json.txt
 - spec/fixtures/results/mixed.txt
+- spec/fixtures/results/multiple-test-cases-json.txt
+- spec/fixtures/results/passing-json.txt
 - spec/fixtures/results/passing.txt
+- spec/fixtures/results/skipped_and_incomplete-json.txt
 - spec/fixtures/results/skipped_and_incomplete.txt
 - spec/fixtures/sampleTest.php
+- spec/fixtures/skippedSampleTest.php
 - spec/guard/phpunit2/formatter_spec.rb
 - spec/guard/phpunit2/inspector_spec.rb
+- spec/guard/phpunit2/logreader_spec.rb
 - spec/guard/phpunit2/notifier_spec.rb
+- spec/guard/phpunit2/realtime_runner_spec.rb
 - spec/guard/phpunit2/runner_spec.rb
 - spec/guard/phpunit2_spec.rb
 - spec/spec_helper.rb