@@ -49,7 +49,7 @@ class Punch stats = {} projects.each { |project| stats[project] = status(project, options) } if options[:short] - stats.reject! { |k, v| v == 'out' } + stats.reject! { |k, v| v != 'in' } stats = 'out' if stats.empty? end return stats lib/punch.rb @@ -278,6 +278,16 @@ describe Punch do } end + it 'should not include empty projects' do + @data['empty_project'] = [] + Punch.data = @data + + Punch.status(:short => true).should == { + @projects['in'] => 'in', + @projects['in2'] => 'in' + } + end + it "should return 'out' if all projects are punched out" do @data.delete(@projects['in']) @data.delete(@projects['in2']) @@ -285,6 +295,15 @@ describe Punch do Punch.status(:short => true).should == 'out' end + + it "should return 'out' if all projects are punched out or empty" do + @data.delete(@projects['in']) + @data.delete(@projects['in2']) + @data['empty_project'] = [] + Punch.data = @data + + Punch.status(:short => true).should == 'out' + end end end end spec/punch_spec.rb de5db99209adfe3c003c0cf509f6d4c43ddc9d26 Yossef Mendelssohn ymendel@pobox.com http://github.com/ymendel/one_inch_punch/commit/a42d85bcaa54185dcd741a88658f1f540603fd53 a42d85bcaa54185dcd741a88658f1f540603fd53 2009-06-25T15:16:26-07:00 2009-06-25T15:16:26-07:00 Excluding empty projects from short status. I don't want to remove all projects that will say they're out, but all projects that won't say they're in. nil-status projects don't interest me that much. 8ed800908f9f0c0f3f3694797ba79fc21a1486c2 Yossef Mendelssohn ymendel@pobox.com