fudge 0.6.1 → 0.6.3

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: d8267a0d4afa0b74d964694a5871e21528b9d2ae
+  data.tar.gz: 2406d80644f4ab0355491eafafb1f4699a48e468
+SHA512:
+  metadata.gz: 449dee87fe9c1b006a198bf5a4ca2e10842085f517fc91185a2c078ccc6814b4210b414dd2e39822f4568d18e1c69db7fa261677b5a942db7e1de03e90c5e796
+  data.tar.gz: 7ecf7d08cfbb75aadd7e388184f2b180632d059594e9e2e710c1c7bdb675e412dd6765853ad8ecb2a78a055007e5cc010e8da5601978d77e380eeafd1e1e8096

data/bin/fudge

@@ -6,4 +6,3 @@ rescue Fudge::Exceptions::Base => e
   puts e.message
   exit 1
 end
-

data/lib/fudge/build.rb

@@ -25,8 +25,6 @@ module Fudge
       success = super
       if callbacks
         return false unless run_callbacks(success)
-      else
-        message "Skipping callbacks..."
       end
       report_time(start_time, time)
 

data/lib/fudge/tasks.rb

@@ -33,5 +33,6 @@ module Fudge
     require 'fudge/tasks/flay'
     require 'fudge/tasks/brakeman'
     require 'fudge/tasks/sub_process'
+    require 'fudge/tasks/cucumber'
   end
 end

data/lib/fudge/tasks/cucumber.rb

@@ -0,0 +1,58 @@
+module Fudge
+  module Tasks
+    # Allow use of Cucumber as a task
+    class Cucumber < Shell
+      include Helpers::BundleAware
+
+      private
+
+      def check_for
+        if coverage
+          [check_regex, method(:coverage_checker)]
+        else
+          super
+        end
+      end
+
+      def check_regex
+        /((\d+\.\d+)%\) covered)/
+      end
+
+      def coverage
+        options[:coverage]
+      end
+
+      # checks if the expected coverage is met
+      def coverage_checker(matches)
+        matches = matches.to_s
+        test_coverage_threshold(matches)
+      end
+
+      # checks the matched string from the console output,
+      # to see if the number for the coverage is greater or
+      # equal than the expected coverage
+      def test_coverage_threshold(matches)
+        if matches.to_f >= coverage
+          true
+        else
+          'Insufficient Coverage.'
+        end
+      end
+
+      def color_options
+        ' --color' unless color == false
+      end
+
+      def cmd(options={})
+        self.arguments = 'features/' if (arguments.nil? || arguments.empty?)
+        bundle_cmd("cucumber#{color_options} #{arguments}", options)
+      end
+
+      def color
+        options[:color]
+      end
+    end
+
+    register Cucumber
+  end
+end

data/lib/fudge/version.rb

@@ -1,4 +1,4 @@
 module Fudge
   # Define gem version
-  VERSION = '0.6.1'
+  VERSION = '0.6.3'
 end

data/spec/lib/fudge/build_spec.rb

@@ -3,20 +3,18 @@ require 'spec_helper'
 describe Fudge::Build do
   it { is_expected.to be_a Fudge::Tasks::CompositeTask }
 
-  describe "#run" do
-
-    context "when provided an output" do
+  describe '#run' do
+    context 'when provided an output' do
       let(:stdout) { StringIO.new }
       let(:formatter) { Fudge::Formatters::Simple.new(stdout) }
 
-      it "prints messages to the formatter instead of default" do
-        subject.run :formatter => formatter
+      it 'prints messages to the formatter instead of default' do
+        subject.run formatter: formatter
 
-        expect(stdout.string).not_to be_empty
-        expect(stdout.string).to include "Skipping callbacks..."
+        expect(stdout.string).to be_empty
       end
 
-      context "when there are callback hooks" do
+      context 'when there are callback hooks' do
         let(:hook) { double(:Hook) }
 
         before :each do
@@ -24,24 +22,24 @@ describe Fudge::Build do
           subject.success_hooks << hook
         end
 
-        it "passesformatter down to the hook run" do
-          expect(hook).to receive(:run).with(:formatter =>formatter).and_return(true)
-          subject.run :formatter => formatter
+        it 'passesformatter down to the hook run' do
+          expect(hook).to receive(:run).with(formatter: formatter).and_return(true)
+          subject.run formatter: formatter
         end
       end
 
-      context "when the `time` flag is set" do
+      context 'when the `time` flag is set' do
         before do
           subject.callbacks = true
           allow(subject).to receive(:run_callbacks).and_return(true)
           subject.time = true
         end
 
-        it "should print out the time of the build" do
+        it 'should print out the time of the build' do
           expect(subject).to receive(:message) do |message|
-            expect(message).to match /Finished in \d+.\d\d seconds./
+            expect(message).to match(/Finished in \d+.\d\d seconds./)
           end
-          subject.run :formatter => formatter
+          subject.run formatter: formatter
         end
       end
     end

data/spec/lib/fudge/cli_spec.rb

@@ -5,7 +5,7 @@ class AnotherDummyTask < DummyTask
     :another_dummy
   end
 
-  def run(options={})
+  def run(_options = {})
     self.class.ran = true
   end
 end
@@ -18,7 +18,7 @@ describe Fudge::Cli do
     AnotherDummyTask.ran = false
   end
 
-  describe ".build" do
+  describe '.build' do
     before :each do
       contents = <<-RUBY
           build :default do
@@ -33,15 +33,15 @@ describe Fudge::Cli do
       allow(File).to receive(:open) { |&block| block.yield file }
     end
 
-    context "when not given a build name" do
-      it "runs the default build" do
+    context 'when not given a build name' do
+      it 'runs the default build' do
         subject.build 'default'
         expect(DummyTask.ran).to be_truthy
       end
     end
 
-    context "when given a build name" do
-      it "runs the only the named build" do
+    context 'when given a build name' do
+      it 'runs the only the named build' do
         subject.build 'other'
         expect(DummyTask.ran).to be_falsey
         expect(AnotherDummyTask.ran).to be_truthy
@@ -49,13 +49,13 @@ describe Fudge::Cli do
     end
   end
 
-  describe ".init" do
-    let (:file_state) { { exists: false, content: '' } }
-    let (:override_file_state) do
-      ->(exists, content) {
+  describe '.init' do
+    let(:file_state) { { exists: false, content: '' } }
+    let(:override_file_state) do
+      lambda do |exists, content|
         file_state[:exists] = exists
         file_state[:content] = content
-      }
+      end
     end
 
     before :each do
@@ -74,8 +74,8 @@ describe Fudge::Cli do
       allow(subject).to receive(:shell).and_return(shell)
     end
 
-    context "when a Fudgefile does not exist in the current directory" do
-      it "creates a new FudgeFile" do
+    context 'when a Fudgefile does not exist in the current directory' do
+      it 'creates a new FudgeFile' do
         expect(File).not_to be_exists('Fudgefile')
 
         subject.init
@@ -83,7 +83,7 @@ describe Fudge::Cli do
         expect(File).to be_exists('Fudgefile')
       end
 
-      it "writes a default build into the new Fudgefile" do
+      it 'writes a default build into the new Fudgefile' do
         subject.init
 
         File.open('Fudgefile', 'r') do |f|
@@ -91,33 +91,33 @@ describe Fudge::Cli do
         end
       end
 
-      it "outputs a success message" do
+      it 'outputs a success message' do
         subject.init
         expect(@output).to eq 'Fudgefile created.'
       end
     end
 
-    context "when a Fudgefile already exists in the current directory" do
+    context 'when a Fudgefile already exists in the current directory' do
       before :each do
-        override_file_state.(true, 'foo')
+        override_file_state.call(true, 'foo')
       end
 
-      it "does not modify the existing Fudgefile" do
+      it 'does not modify the existing Fudgefile' do
         expect(File).to be_exists('Fudgefile')
 
         subject.init
 
-        expect(File.open('Fudgefile') { |f| f.read }).to eql 'foo'
+        expect(File.open('Fudgefile', &:read)).to eql 'foo'
       end
 
-      it "outputs a failure message" do
+      it 'outputs a failure message' do
         subject.init
         expect(@output).to eq 'Fudgefile already exists.'
       end
     end
   end
 
-  describe ".list" do
+  describe '.list' do
     before :each do
       contents = <<-RUBY
           build :default do
@@ -137,32 +137,31 @@ describe Fudge::Cli do
       allow(subject).to receive(:shell).and_return(shell)
     end
 
-    context "when not given a filter string" do
-      it "lists all the defined builds" do
+    context 'when not given a filter string' do
+      it 'lists all the defined builds' do
         subject.list
         expect(@output).to eql "default\t\nother\tnot the default"
       end
     end
 
-    context "when given a filter string" do
-      context "that matches one or builds" do
-        it "lists only the builds that match the filter" do
+    context 'when given a filter string' do
+      context 'that matches one or builds' do
+        it 'lists only the builds that match the filter' do
           subject.list 'oth'
           expect(@output).to eql "other\tnot the default"
         end
 
-        it "ignores the case of the filter" do
+        it 'ignores the case of the filter' do
           subject.list 'OTH'
           expect(@output).to eql "other\tnot the default"
         end
       end
 
-      context "that matches no builds" do
-        it "outputs nothing" do
+      context 'that matches no builds' do
+        it 'outputs nothing' do
           expect(@output).to eql ''
         end
       end
     end
   end
-
 end

data/spec/lib/fudge/description_spec.rb

@@ -8,7 +8,6 @@ describe Fudge::Description do
   let(:build) { subject.builds.values.first }
   let(:build_tasks) { build.tasks.dup }
 
-
   def make_build
     subject.build :default do
       subject.task :dummy
@@ -17,16 +16,16 @@ describe Fudge::Description do
     build.callbacks = callbacks
   end
 
-  describe "#initialize" do
+  describe '#initialize' do
     let(:input) { 'build :foo do; end' }
 
-    it "should add the builds in the given string" do
+    it 'should add the builds in the given string' do
       expect(subject.builds[:foo]).to be_a Fudge::Build
     end
   end
 
-  describe "#build" do
-    it "should create a new build and add it to the builds array" do
+  describe '#build' do
+    it 'should create a new build and add it to the builds array' do
       expect(subject.builds).to be_empty
 
       subject.build :some_branch do
@@ -37,8 +36,8 @@ describe Fudge::Description do
     end
   end
 
-  describe "#task" do
-    it "should add a task to the current scope" do
+  describe '#task' do
+    it 'should add a task to the current scope' do
       subject.build :default do
         subject.task :dummy
       end
@@ -47,7 +46,7 @@ describe Fudge::Description do
       expect(build_tasks.first).to be_a DummyTask
     end
 
-    it "should pass arguments to the initializer" do
+    it 'should pass arguments to the initializer' do
       subject.build :default do
         subject.task :dummy, :foo, :bar
       end
@@ -55,7 +54,7 @@ describe Fudge::Description do
       expect(build_tasks.first.args).to eq([:foo, :bar])
     end
 
-    it "should forward missing methods to task" do
+    it 'should forward missing methods to task' do
       subject.build :default do
         subject.dummy :foo, :bar
       end
@@ -63,11 +62,11 @@ describe Fudge::Description do
       expect(build_tasks.first.args).to eq([:foo, :bar])
     end
 
-    it "should super method_missing if no task found" do
+    it 'should super method_missing if no task found' do
       expect { subject.no_task :foo, :bar }.to raise_error(NoMethodError)
     end
 
-    it "should add tasks recursively to composite tasks" do
+    it 'should add tasks recursively to composite tasks' do
       subject.build :default do
         subject.dummy_composite do
           subject.dummy
@@ -78,8 +77,8 @@ describe Fudge::Description do
     end
   end
 
-  describe "#task_group" do
-    it "should add a task group and allow it to be used in a build" do
+  describe '#task_group' do
+    it 'should add a task group and allow it to be used in a build' do
       subject.task_group :group1 do
         subject.task :dummy
       end
@@ -92,7 +91,7 @@ describe Fudge::Description do
       expect(DummyTask.ran).to be_truthy
     end
 
-    it "should allow passing arguments to task groups" do
+    it 'should allow passing arguments to task groups' do
       subject.task_group :special_group do |which|
         subject.task which
       end
@@ -105,7 +104,7 @@ describe Fudge::Description do
       expect(DummyTask.ran).to be_truthy
     end
 
-    it "should raise an exception if task group not found" do
+    it 'should raise an exception if task group not found' do
       expect do
         subject.build :default do
           subject.task_group :unkown_group
@@ -113,14 +112,14 @@ describe Fudge::Description do
       end.to raise_error Fudge::Exceptions::TaskGroupNotFound
     end
 
-    context "grouped tasks" do
+    context 'grouped tasks' do
       before :each do
         subject.task_group :group1 do
           subject.task :dummy
         end
       end
 
-      it "allows group task reuse in composite tasks" do
+      it 'allows group task reuse in composite tasks' do
         subject.build :default do
           subject.task :dummy_composite do
             subject.task_group :group1
@@ -131,9 +130,9 @@ describe Fudge::Description do
         expect(DummyTask.ran).to be_truthy
       end
 
-      it "supports when options are given" do
+      it 'supports when options are given' do
         subject.build :default do
-          subject.task :dummy_composite, :hello, :foobar => true do
+          subject.task :dummy_composite, :hello, foobar: true do
             subject.task_group :group1
           end
         end
@@ -146,10 +145,9 @@ describe Fudge::Description do
         expect(DummyTask.ran).to be_truthy
       end
     end
-
   end
 
-  describe "Callback Hooks" do
+  describe 'Callback Hooks' do
     before :each do
       @ran = []
       allow_any_instance_of(Fudge::Tasks::Shell).to receive(:run_command) do |cmd|
@@ -158,21 +156,21 @@ describe Fudge::Description do
       end
     end
 
-    describe "#on_success" do
-      context "when callbacks is set to true" do
+    describe '#on_success' do
+      context 'when callbacks is set to true' do
         let(:callbacks) { true }
 
-        it "should add success hooks that run after the build is successful" do
+        it 'should add success hooks that run after the build is successful' do
           make_build do
             subject.on_success { subject.shell 'FOO' }
             subject.on_success { subject.shell 'BAR' }
           end
 
           expect(build.run).to be_truthy
-          expect(@ran).to eq(['FOO', 'BAR'])
+          expect(@ran).to eq(%w(FOO BAR))
         end
 
-        it "fails the build HARD when hooks fail" do
+        it 'fails the build HARD when hooks fail' do
           make_build do
             subject.on_success { subject.shell 'fail'; subject.shell 'FOO' }
             subject.on_success { subject.shell 'BAR' }
@@ -183,10 +181,10 @@ describe Fudge::Description do
         end
       end
 
-      context "when callbacks is set to false" do
+      context 'when callbacks is set to false' do
         let(:callbacks) { false }
 
-        it "should not run the callbacks if the build succeeds" do
+        it 'should not run the callbacks if the build succeeds' do
           make_build do
             subject.on_success { subject.shell 'echo "WOOP"' }
           end
@@ -197,27 +195,30 @@ describe Fudge::Description do
       end
     end
 
-    describe "#on_failure" do
+    describe '#on_failure' do
       before :each do
         allow_any_instance_of(DummyTask).to receive(:run).and_return(false)
       end
 
-      context "when callbacks is set to true" do
+      context 'when callbacks is set to true' do
         let(:callbacks) { true }
 
-        it "should add failure hooks that run after the build fails" do
+        it 'should add failure hooks that run after the build fails' do
           make_build do
             subject.on_failure { subject.shell 'WOOP' }
             subject.on_failure { subject.shell 'BAR' }
           end
 
           expect(build.run).to be_falsey
-          expect(@ran).to eq(['WOOP', 'BAR'])
+          expect(@ran).to eq(%w(WOOP BAR))
         end
 
-        it "fails the build HARD when hooks fail" do
+        it 'fails the build HARD when hooks fail' do
           make_build do
-            subject.on_failure { subject.shell 'fail'; subject.shell 'FOO' }
+            subject.on_failure do
+              subject.shell 'fail'
+              subject.shell 'FOO'
+            end
             subject.on_failure { subject.shell 'BAR' }
           end
 
@@ -226,10 +227,10 @@ describe Fudge::Description do
         end
       end
 
-      context "when callbacks is set to false" do
+      context 'when callbacks is set to false' do
         let(:callbacks) { false }
 
-        it "should not run the callbacks if the build fails" do
+        it 'should not run the callbacks if the build fails' do
           make_build do
             subject.on_failure { subject.shell 'WOOP' }
           end