one_inch_punch 0.0.1 → 0.0.2

data/History.txt

@@ -1,3 +1,10 @@
+== 0.0.2 2008-08-22
+
+* 3 tiny enhancements:
+  * Punching in will now create a project if necessary
+  * Can now delete a project
+  * Can now list project data
+
 == 0.0.1 2008-08-18
 
 * 1 major enhancement:

data/README.txt

@@ -12,11 +12,12 @@ One-inch punch: Smaller, more effective
 == FEATURES/PROBLEMS:
 
 * Can load and write .punch.yml data compatibly with Ara's punch gem
-* Can punch in and out of projects
+* Can punch in and out of projects (including creating a project by punching in)
 * Can query project status
+* Can delete a project
+* Can list project data
 
 * Cannot give a total time (yet)
-* Cannot list time entries (yet)
 * Cannot be used command-line (yet)
 * More, since this is unfinished
 

data/lib/punch/version.rb

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

data/lib/punch.rb

@@ -53,6 +53,7 @@ module Punch
     
     def in(project)
       return false if in?(project)
+      data[project] ||= []
       data[project].push({'in' => Time.now})
       write
       true
@@ -64,5 +65,15 @@ module Punch
       write
       true
     end
+    
+    def delete(project)
+      return nil unless data.delete(project)
+      write
+      true
+    end
+    
+    def list(project)
+      data[project]
+    end
   end
 end

data/spec/punch_spec.rb

@@ -337,6 +337,42 @@ describe Punch do
         Punch.in(@project).should == true
       end
     end
+    
+    describe 'when the project does not yet exist' do
+      before :each do
+        @project = 'non-existent project'
+      end
+      
+      it 'should create the project' do
+        Punch.in(@project)
+        Punch.data.should include(@project)
+      end
+      
+      it 'should add a time entry to the project data' do
+        Punch.in(@project)
+        Punch.data[@project].length.should == 1
+      end
+      
+      it 'should use now for the punch-in time' do
+        Punch.in(@project)
+        Punch.data[@project].last['in'].should == @now
+      end
+      
+      it 'should use now for the punch-in time' do
+        Punch.in(@project)
+        Punch.data[@project].last['in'].should == @now
+      end
+      
+      it 'should write the data' do
+        @test.become('test')
+        Punch.expects(:write).when(@test.is('test'))
+        Punch.in(@project)
+      end
+      
+      it 'should return true' do
+        Punch.in(@project).should == true
+      end
+    end
   end
   
   it 'should punch a project out' do
@@ -421,4 +457,110 @@ describe Punch do
       end
     end
   end
+
+  it 'should delete a project' do
+    Punch.should respond_to(:delete)
+  end
+  
+  describe 'deleting a project' do
+    before :each do
+      @now = Time.now
+      @project = 'test project'
+      @data = { @project => [ {'in' => @now - 50, 'out' => @now - 25} ] }
+      
+      Punch.instance_eval do
+        class << self
+          public :data, :data=
+        end
+      end
+      Punch.data = @data
+      
+      @test = states('test').starts_as('setup')
+      Punch.stubs(:write).when(@test.is('setup'))
+    end
+    
+    it 'should accept a project name' do
+      lambda { Punch.delete('proj') }.should_not raise_error(ArgumentError)
+    end
+    
+    it 'should require a project name' do
+      lambda { Punch.delete }.should raise_error(ArgumentError)
+    end
+    
+    describe 'when the project exists' do
+      it 'should remove the project data' do
+        Punch.delete(@project)
+        Punch.data.should_not include(@project)
+      end
+      
+      it 'should write the data' do
+        @test.become('test')
+        Punch.expects(:write).when(@test.is('test'))
+        Punch.delete(@project)
+      end
+      
+      it 'should return true' do
+        Punch.delete(@project).should == true
+      end
+    end
+    
+    describe 'when the project does not exist' do
+      before :each do
+        @project = 'non-existent project'
+      end
+      
+      it 'should not write the data' do
+        @test.become('test')
+        Punch.expects(:write).never.when(@test.is('test'))
+        Punch.delete(@project)
+      end
+      
+      it 'should return nil' do
+        Punch.delete(@project).should be_nil
+      end
+    end
+  end
+  
+  it 'should list project data' do
+    Punch.should respond_to(:list)
+  end
+  
+  describe 'listing project data' do
+    before :each do
+      @now = Time.now
+      @project = 'test project'
+      @data = { @project => [ {'in' => @now - 50, 'out' => @now - 25} ] }
+      
+      Punch.instance_eval do
+        class << self
+          public :data, :data=
+        end
+      end
+      Punch.data = @data
+    end
+    
+    it 'should accept a project name' do
+      lambda { Punch.list('proj') }.should_not raise_error(ArgumentError)
+    end
+    
+    it 'should require a project name' do
+      lambda { Punch.list }.should raise_error(ArgumentError)
+    end
+    
+    describe 'when the project exists' do
+      it 'should return the project data' do
+        Punch.list(@project).should == Punch.data[@project]
+      end
+    end
+
+    describe 'when the project does not exist' do
+      before :each do
+        @project = 'non-existent project'
+      end
+      
+      it 'should return nil' do
+        Punch.list(@project).should be_nil
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: one_inch_punch
 version: !ruby/object:Gem::Version 
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors: 
 - Yossef Mendelssohn
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-08-18 00:00:00 -05:00
+date: 2008-08-22 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency