ruby-progressbar 1.4.0 → 1.4.1

data/lib/ruby-progressbar/components/progressable.rb

@@ -36,13 +36,13 @@ class ProgressBar
       end
 
       def increment
-        STDOUT.puts "WARNING: Your progress bar is currently at #{progress} out of #{total} and cannot be incremented. In v2.0.0 this will become a ProgressBar::InvalidProgressError." if progress == total
+        warn "WARNING: Your progress bar is currently at #{progress} out of #{total} and cannot be incremented. In v2.0.0 this will become a ProgressBar::InvalidProgressError." if progress == total
 
         self.progress += 1 unless progress == total
       end
 
       def decrement
-        STDOUT.puts "WARNING: Your progress bar is currently at #{progress} out of #{total} and cannot be decremented. In v2.0.0 this will become a ProgressBar::InvalidProgressError." if progress == 0
+        warn "WARNING: Your progress bar is currently at #{progress} out of #{total} and cannot be decremented. In v2.0.0 this will become a ProgressBar::InvalidProgressError." if progress == 0
 
         self.progress -= 1 unless progress == 0
       end

data/lib/ruby-progressbar/components/timer.rb

@@ -11,6 +11,8 @@ class ProgressBar
       end
 
       def stop
+        return unless started?
+
         @stopped_at = now
       end
 

data/lib/ruby-progressbar/version.rb

@@ -1,3 +1,3 @@
 class ProgressBar
-  VERSION = '1.4.0'
+  VERSION = '1.4.1'
 end

data/spec/lib/ruby-progressbar/base_spec.rb

@@ -682,4 +682,26 @@ describe ProgressBar::Base do
       end
     end
   end
+
+  context 'when the bar is started after having total set to 0' do
+    let(:progressbar) { ProgressBar::Base.new(:output => output, :autostart => false) }
+
+    it 'does not throw an error' do
+      progressbar.total = 0
+
+      expect { progressbar.start }.not_to raise_error
+    end
+  end
+
+  context 'when the bar has no items to process' do
+    context 'and it has not been started' do
+      let(:progressbar) { ProgressBar::Base.new(:started_at => 0, :total => 0, :autostart => false, :smoothing => 0.0, :format => ' %c/%C |%w>%i| %e ', :output => output) }
+
+      it 'does not throw an error if told to stop' do
+        progressbar.stop
+
+        expect { progressbar.start }.not_to raise_error
+      end
+    end
+  end
 end

data/spec/lib/ruby-progressbar/components/elapsed_timer_spec.rb

@@ -74,5 +74,18 @@ describe ProgressBar::Components::ElapsedTimer do
         end
       end
     end
+
+    context "when the timer has yet to be started" do
+      it 'is false' do
+        @timer.should_not be_stopped
+      end
+    end
+
+    context "when the timer is stopped without having been started" do
+      before { @timer.stop }
+      it 'is false' do
+        @timer.should_not be_stopped
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruby-progressbar
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.4.1
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-28 00:00:00.000000000 Z
+date: 2014-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec