rake-funnel 0.3.2.pre → 0.4.0.pre

data/spec/rake/funnel/integration/teamcity/progress_report_spec.rb

@@ -1,174 +1,174 @@
-include Rake
-include Rake::Funnel::Integration
-include Rake::Funnel::Integration::TeamCity
-
-describe Rake::Funnel::Integration::TeamCity::ProgressReport do
-  include DSL
-
-  let(:teamcity_running?) { false }
-  let(:teamcity_rake_runner?) { false }
-
-  before {
-    allow(TeamCity).to receive(:running?).and_return(teamcity_running?)
-    allow(TeamCity).to receive(:rake_runner?).and_return(teamcity_rake_runner?)
-    allow(ServiceMessages).to receive(:block_opened)
-    allow(ServiceMessages).to receive(:block_closed)
-    allow(ServiceMessages).to receive(:progress_start)
-    allow(ServiceMessages).to receive(:progress_finish)
-    allow(ServiceMessages).to receive(:build_problem)
-
-    Task.clear
-  }
-
-  subject! {
-    described_class.new
-  }
-
-  after {
-    subject.disable!
-  }
-
-  shared_examples :block_report do
-    it 'should write block start' do
-      expect(ServiceMessages).to have_received(:block_opened).with(name: 'task')
-    end
-
-    it 'should write block end' do
-      expect(ServiceMessages).to have_received(:block_closed).with(name: 'task')
-    end
-  end
-
-  shared_examples :no_block_report do
-    it 'should not write block start' do
-      expect(ServiceMessages).not_to have_received(:block_opened)
-    end
-
-    it 'should not write block end' do
-      expect(ServiceMessages).not_to have_received(:block_closed)
-    end
-  end
-
-  context 'when task succeeds' do
-    before {
-      task :task
-
-      Task[:task].invoke
-    }
-
-    it 'should not report build problems' do
-      expect(ServiceMessages).to_not have_received(:build_problem)
-    end
-
-    context 'not on TeamCity' do
-      it_behaves_like :no_block_report
-    end
-
-    context 'on TeamCity' do
-      let(:teamcity_running?) { true }
-
-      context 'without rake runner' do
-        it_behaves_like :block_report
-      end
-
-      context 'with rake runner' do
-        let(:teamcity_rake_runner?) { true }
-
-        it_behaves_like :no_block_report
-      end
-    end
-  end
-
-  context 'when task fails' do
-    class SpecificError < StandardError; end
-
-    before {
-      module Rake
-        class ApplicationAbortedException < StandardError
-          attr_reader :inner_exception
-
-          def initialize(other_exception)
-            @inner_exception = other_exception
-          end
-        end
-      end
-    }
-
-    let(:error) { SpecificError.new('task error' * 4000) }
-
-    before {
-      task :task do
-        raise error
-      end
-
-      begin
-        Task[:task].invoke
-      rescue ApplicationAbortedException => e
-      rescue SpecificError => e
-        @raised_error = e
-      end
-    }
-
-    context 'not on TeamCity' do
-      it_behaves_like :no_block_report
-
-      it 'should not swallow the error' do
-        expect(@raised_error).to be_a_kind_of(SpecificError)
-      end
-    end
-
-    context 'on TeamCity' do
-      let(:teamcity_running?) { true }
-
-      describe 'build problems' do
-        it 'should report build problems' do
-          expect(ServiceMessages).to have_received(:build_problem)
-        end
-
-        it 'should report the error message' do
-          expect(ServiceMessages).to have_received(:build_problem).with(hash_including({ description: be_an_instance_of(String) }))
-        end
-
-        it 'should report the first 4000 characters of the error message' do
-          expect(ServiceMessages).to have_received(:build_problem).with(hash_including({ description: have(4000).items }))
-        end
-      end
-
-      context 'without rake runner' do
-        it_behaves_like :block_report
-
-        it 'should report the error as a build problem' do
-          expect(ServiceMessages).to have_received(:build_problem)
-        end
-      end
-
-      context 'with rake runner' do
-        let(:teamcity_rake_runner?) { true }
-        let(:error) {
-          ApplicationAbortedException.new(SpecificError.new('inner message'))
-        }
-
-        it 'should report the inner error as a build problem (as it will be wrapped in a ApplicationAbortedException)' do
-          expect(ServiceMessages).to have_received(:build_problem).with({ description: 'inner message' })
-        end
-
-        it_behaves_like :no_block_report
-      end
-    end
-  end
-
-  context 'when progess report was disabled' do
-    let(:teamcity_running?) { true }
-
-    before {
-      subject.disable!
-
-      task :task
-
-      Task[:task].invoke
-    }
-
-    it 'should not write' do
-      expect(ServiceMessages).to_not have_received(:block_opened)
-    end
-  end
-end
+include Rake
+include Rake::Funnel::Integration
+include Rake::Funnel::Integration::TeamCity
+
+describe Rake::Funnel::Integration::TeamCity::ProgressReport do
+  include DSL
+
+  let(:teamcity_running?) { false }
+  let(:teamcity_rake_runner?) { false }
+
+  before {
+    allow(TeamCity).to receive(:running?).and_return(teamcity_running?)
+    allow(TeamCity).to receive(:rake_runner?).and_return(teamcity_rake_runner?)
+    allow(ServiceMessages).to receive(:block_opened)
+    allow(ServiceMessages).to receive(:block_closed)
+    allow(ServiceMessages).to receive(:progress_start)
+    allow(ServiceMessages).to receive(:progress_finish)
+    allow(ServiceMessages).to receive(:build_problem)
+
+    Task.clear
+  }
+
+  subject! {
+    described_class.new
+  }
+
+  after {
+    subject.disable!
+  }
+
+  shared_examples :block_report do
+    it 'should write block start' do
+      expect(ServiceMessages).to have_received(:block_opened).with(name: 'task')
+    end
+
+    it 'should write block end' do
+      expect(ServiceMessages).to have_received(:block_closed).with(name: 'task')
+    end
+  end
+
+  shared_examples :no_block_report do
+    it 'should not write block start' do
+      expect(ServiceMessages).not_to have_received(:block_opened)
+    end
+
+    it 'should not write block end' do
+      expect(ServiceMessages).not_to have_received(:block_closed)
+    end
+  end
+
+  context 'when task succeeds' do
+    before {
+      task :task
+
+      Task[:task].invoke
+    }
+
+    it 'should not report build problems' do
+      expect(ServiceMessages).to_not have_received(:build_problem)
+    end
+
+    context 'not on TeamCity' do
+      it_behaves_like :no_block_report
+    end
+
+    context 'on TeamCity' do
+      let(:teamcity_running?) { true }
+
+      context 'without rake runner' do
+        it_behaves_like :block_report
+      end
+
+      context 'with rake runner' do
+        let(:teamcity_rake_runner?) { true }
+
+        it_behaves_like :no_block_report
+      end
+    end
+  end
+
+  context 'when task fails' do
+    class SpecificError < StandardError; end
+
+    before {
+      module Rake
+        class ApplicationAbortedException < StandardError
+          attr_reader :inner_exception
+
+          def initialize(other_exception)
+            @inner_exception = other_exception
+          end
+        end
+      end
+    }
+
+    let(:error) { SpecificError.new('task error' * 4000) }
+
+    before {
+      task :task do
+        raise error
+      end
+
+      begin
+        Task[:task].invoke
+      rescue ApplicationAbortedException => e
+      rescue SpecificError => e
+        @raised_error = e
+      end
+    }
+
+    context 'not on TeamCity' do
+      it_behaves_like :no_block_report
+
+      it 'should not swallow the error' do
+        expect(@raised_error).to be_a_kind_of(SpecificError)
+      end
+    end
+
+    context 'on TeamCity' do
+      let(:teamcity_running?) { true }
+
+      describe 'build problems' do
+        it 'should report build problems' do
+          expect(ServiceMessages).to have_received(:build_problem)
+        end
+
+        it 'should report the error message' do
+          expect(ServiceMessages).to have_received(:build_problem).with(hash_including({ description: be_an_instance_of(String) }))
+        end
+
+        it 'should report the first 4000 characters of the error message' do
+          expect(ServiceMessages).to have_received(:build_problem).with(hash_including({ description: have(4000).items }))
+        end
+      end
+
+      context 'without rake runner' do
+        it_behaves_like :block_report
+
+        it 'should report the error as a build problem' do
+          expect(ServiceMessages).to have_received(:build_problem)
+        end
+      end
+
+      context 'with rake runner' do
+        let(:teamcity_rake_runner?) { true }
+        let(:error) {
+          ApplicationAbortedException.new(SpecificError.new('inner message'))
+        }
+
+        it 'should report the inner error as a build problem (as it will be wrapped in a ApplicationAbortedException)' do
+          expect(ServiceMessages).to have_received(:build_problem).with({ description: 'inner message' })
+        end
+
+        it_behaves_like :no_block_report
+      end
+    end
+  end
+
+  context 'when progess report was disabled' do
+    let(:teamcity_running?) { true }
+
+    before {
+      subject.disable!
+
+      task :task
+
+      Task[:task].invoke
+    }
+
+    it 'should not write' do
+      expect(ServiceMessages).to_not have_received(:block_opened)
+    end
+  end
+end

data/spec/rake/funnel/integration/teamcity/service_messages_spec.rb

@@ -1,136 +1,136 @@
-include Rake::Funnel::Integration
-
-describe Rake::Funnel::Integration::TeamCity::ServiceMessages do
-  before {
-    allow(TeamCity).to receive(:running?).and_return(teamcity_running?)
-    allow($stdout).to receive(:puts)
-  }
-
-  context 'when running outside TeamCity' do
-    let(:teamcity_running?) { false }
-
-    it 'should not publish messages' do
-      described_class.progress_start 'foo'
-
-      expect($stdout).not_to have_received(:puts)
-    end
-  end
-
-  context 'when running inside TeamCity' do
-    let(:teamcity_running?) { true }
-
-    describe 'escaping' do
-      context 'when publishing messages without special characters' do
-        it 'should not escape' do
-          described_class.progress_start "the message"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart 'the message']")
-        end
-      end
-
-      context 'when publishing messages with special characters' do
-        it 'should escape apostrophes' do
-          described_class.progress_start "'"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|'']")
-        end
-
-        it 'should escape line feeds' do
-          described_class.progress_start "\n"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|n']")
-        end
-
-        it 'should escape carriage returns' do
-          described_class.progress_start "\r"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|r']")
-        end
-
-        it 'should escape next lines' do
-          described_class.progress_start "\u0085"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|x']")
-        end
-
-        it 'should escape line separators' do
-          described_class.progress_start "\u2028"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|l']")
-        end
-
-        it 'should escape paragraph separators' do
-          described_class.progress_start "\u2029"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|p']")
-        end
-
-        it 'should escape vertical bars' do
-          described_class.progress_start '|'
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '||']")
-        end
-
-        it 'should escape opening brackets' do
-          described_class.progress_start '['
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|[']")
-        end
-
-        it 'should escape closing brackets' do
-          described_class.progress_start ']'
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|]']")
-        end
-
-        it 'should escape all special characters in a string' do
-          described_class.progress_start "[\r|\n]"
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|[|r|||n|]']")
-        end
-      end
-    end
-
-    describe 'parameters' do
-      context 'when reporting a message without parameters' do
-        it 'should print the service message' do
-          described_class.enable_service_messages
-
-          expect($stdout).to have_received(:puts).with('##teamcity[enableServiceMessages]')
-        end
-      end
-
-      context 'when reporting a message with an unnamed parameter' do
-        it 'should print the service message' do
-          described_class.progress_message 'the message'
-
-          expect($stdout).to have_received(:puts).with("##teamcity[progressMessage 'the message']")
-        end
-      end
-
-      context 'when reporting a message with a named parameter' do
-        it 'should print the service message' do
-          described_class.block_opened({ name: 'block name' })
-
-          expect($stdout).to have_received(:puts).with("##teamcity[blockOpened name='block name']")
-        end
-      end
-
-      context 'when reporting a message with multiple named parameters' do
-        it 'should print the service message' do
-          described_class.test_started ({ name: 'test name', captureStandardOutput: true })
-
-          expect($stdout).to have_received(:puts).with("##teamcity[testStarted name='test name' captureStandardOutput='true']")
-        end
-      end
-
-      context 'when reporting a message with Ruby-style named parameters' do
-        it 'should print the service message' do
-          described_class.test_started ({ capture_standard_output: true })
-
-          expect($stdout).to have_received(:puts).with("##teamcity[testStarted captureStandardOutput='true']")
-        end
-      end
-    end
-  end
-end
+include Rake::Funnel::Integration
+
+describe Rake::Funnel::Integration::TeamCity::ServiceMessages do
+  before {
+    allow(TeamCity).to receive(:running?).and_return(teamcity_running?)
+    allow($stdout).to receive(:puts)
+  }
+
+  context 'when running outside TeamCity' do
+    let(:teamcity_running?) { false }
+
+    it 'should not publish messages' do
+      described_class.progress_start 'foo'
+
+      expect($stdout).not_to have_received(:puts)
+    end
+  end
+
+  context 'when running inside TeamCity' do
+    let(:teamcity_running?) { true }
+
+    describe 'escaping' do
+      context 'when publishing messages without special characters' do
+        it 'should not escape' do
+          described_class.progress_start "the message"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart 'the message']")
+        end
+      end
+
+      context 'when publishing messages with special characters' do
+        it 'should escape apostrophes' do
+          described_class.progress_start "'"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|'']")
+        end
+
+        it 'should escape line feeds' do
+          described_class.progress_start "\n"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|n']")
+        end
+
+        it 'should escape carriage returns' do
+          described_class.progress_start "\r"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|r']")
+        end
+
+        it 'should escape next lines' do
+          described_class.progress_start "\u0085"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|x']")
+        end
+
+        it 'should escape line separators' do
+          described_class.progress_start "\u2028"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|l']")
+        end
+
+        it 'should escape paragraph separators' do
+          described_class.progress_start "\u2029"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|p']")
+        end
+
+        it 'should escape vertical bars' do
+          described_class.progress_start '|'
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '||']")
+        end
+
+        it 'should escape opening brackets' do
+          described_class.progress_start '['
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|[']")
+        end
+
+        it 'should escape closing brackets' do
+          described_class.progress_start ']'
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|]']")
+        end
+
+        it 'should escape all special characters in a string' do
+          described_class.progress_start "[\r|\n]"
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressStart '|[|r|||n|]']")
+        end
+      end
+    end
+
+    describe 'parameters' do
+      context 'when reporting a message without parameters' do
+        it 'should print the service message' do
+          described_class.enable_service_messages
+
+          expect($stdout).to have_received(:puts).with('##teamcity[enableServiceMessages]')
+        end
+      end
+
+      context 'when reporting a message with an unnamed parameter' do
+        it 'should print the service message' do
+          described_class.progress_message 'the message'
+
+          expect($stdout).to have_received(:puts).with("##teamcity[progressMessage 'the message']")
+        end
+      end
+
+      context 'when reporting a message with a named parameter' do
+        it 'should print the service message' do
+          described_class.block_opened({ name: 'block name' })
+
+          expect($stdout).to have_received(:puts).with("##teamcity[blockOpened name='block name']")
+        end
+      end
+
+      context 'when reporting a message with multiple named parameters' do
+        it 'should print the service message' do
+          described_class.test_started ({ name: 'test name', captureStandardOutput: true })
+
+          expect($stdout).to have_received(:puts).with("##teamcity[testStarted name='test name' captureStandardOutput='true']")
+        end
+      end
+
+      context 'when reporting a message with Ruby-style named parameters' do
+        it 'should print the service message' do
+          described_class.test_started ({ capture_standard_output: true })
+
+          expect($stdout).to have_received(:puts).with("##teamcity[testStarted captureStandardOutput='true']")
+        end
+      end
+    end
+  end
+end