one_inch_punch 0.3.0 → 0.3.1

data/History.txt

@@ -1,3 +1,8 @@
+== 0.3.1 2008-04-06
+
+* 1 tiny bugfix enhancement:
+  * Total no longer erroring on non-existent parent project
+
 == 0.3.0 2008-04-05
 
 * 1 minor enhancement:

data/lib/punch.rb

@@ -154,6 +154,7 @@ class Punch
     end
     
     def do_total_time(list_data, options)
+      return nil unless list_data
       total = list_data.collect { |t|  ((t['out'] || Time.now) - t['in']).to_i }.inject(0) { |sum, t|  sum + t }
       return total unless options[:format]
       total.elapsed_time

data/lib/punch/version.rb

@@ -2,7 +2,7 @@ class Punch
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 3
-    TINY  = 0
+    TINY  = 1
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

data/spec/punch_spec.rb

@@ -1145,7 +1145,29 @@ describe Punch do
         Punch.total(@projects['parent'], :after => @now - 21).should == total_data
       end
       
-      describe 'when do project is given' do
+      it 'should handle a non-existent parent project' do
+        @data.delete(@projects['parent'])
+        Punch.data = @data
+        total_data = { @projects['parent'] => nil, @projects['child'] => 10 }
+        Punch.total(@projects['parent']).should == total_data
+      end
+      
+      it 'should handle an empty parent project' do
+        @data[@projects['parent']] = []
+        Punch.data = @data
+        total_data = { @projects['parent'] => 0, @projects['child'] => 10 }
+        Punch.total(@projects['parent']).should == total_data
+      end
+      
+      it 'should handle an empty child project' do
+        @projects['other_child'] = @projects['parent'] + '/button'
+        @data[@projects['other_child']] = []
+        Punch.data = @data
+        total_data = { @projects['parent'] => 50, @projects['child'] => 10, @projects['other_child'] => 0 }
+        Punch.total(@projects['parent']).should == total_data
+      end
+      
+      describe 'when no project is given' do
         before do
           @extra_projects = ['test project', 'out project', 'other project']
           @data[@extra_projects[0]] = [ {'in' => @now - 50, 'out' => @now - 25} ]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: one_inch_punch
 version: !ruby/object:Gem::Version 
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors: 
 - Yossef Mendelssohn
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-04-05 00:00:00 -05:00
+date: 2009-04-06 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency