freshtrack 0.4.1 → 0.4.1.1

data/History.txt

@@ -1,3 +1,7 @@
+== 0.4.1.1 2009-07-31
+
+* Maybe adding the invoice files to the Manifest is a good idea.
+
 == 0.4.1 2009-07-31
 
 * 1 tiny enhancement:

data/Manifest.txt

@@ -17,6 +17,7 @@ lib/freshtrack/time_collectors/punch.rb
 lib/freshtrack/time_collectors/punchy_template.rb
 lib/freshbooks/extensions.rb
 lib/freshbooks/extensions/base_object.rb
+lib/freshbooks/extensions/invoice.rb
 lib/freshbooks/extensions/project.rb
 lib/freshbooks/extensions/task.rb
 lib/freshbooks/extensions/time_entry.rb
@@ -30,6 +31,7 @@ spec/core_ext/time_spec.rb
 spec/time_collectors/one_inch_punch_spec.rb
 spec/time_collectors/punch_spec.rb
 spec/freshbooks/base_object_spec.rb
+spec/freshbooks/invoice_spec.rb
 spec/freshbooks/project_spec.rb
 spec/freshbooks/task_spec.rb
 spec/freshbooks/time_entry_spec.rb

data/lib/freshbooks/extensions/invoice.rb

@@ -0,0 +1,40 @@
+module FreshBooks
+  class Invoice
+    TYPE_MAPPINGS['date'] = Date
+    
+    def open?
+      !%w[draft paid].include?(status)
+    end
+    
+    def client
+      Client.get(client_id)
+    end
+    
+    attr_accessor :number
+    
+    alias_method :old_brackets, :[]
+    def [](m)
+      if m.to_s == 'number'
+        self.number
+      else
+        old_brackets(m)
+      end
+    end
+
+    alias_method :old_brackets_equal, :[]=
+    def []=(m, v)
+      if m.to_s == 'number'
+        self.number = v 
+      else
+        old_brackets_equal(m, v)
+      end
+    end
+    
+    class << self
+      alias_method :old_members, :members
+      def members
+        old_members + ['number']
+      end
+    end
+  end
+end

data/lib/freshtrack/version.rb

@@ -3,7 +3,8 @@ module Freshtrack #:nodoc:
     MAJOR = 0
     MINOR = 4
     TINY  = 1
+    FUCKUP = 1
 
-    STRING = [MAJOR, MINOR, TINY].join('.')
+    STRING = [MAJOR, MINOR, TINY, FUCKUP].join('.')
   end
 end

data/spec/freshbooks/invoice_spec.rb

@@ -0,0 +1,136 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper.rb')
+
+describe FreshBooks::Invoice do
+  before :each do
+    @invoice = FreshBooks::Invoice.new
+  end
+  
+  describe 'attributes' do
+    it 'should have an invoice_id' do
+      @invoice.should respond_to(:invoice_id)
+    end
+    
+    it 'should have a client_id' do
+      @invoice.should respond_to(:client_id)
+    end
+    
+    it 'should have a date' do
+      @invoice.should respond_to(:date)
+    end
+    
+    it 'should have a status' do
+      @invoice.should respond_to(:status)
+    end
+  end
+  
+  describe 'type mappings' do
+    before :each do
+      @mapping = FreshBooks::Invoice::TYPE_MAPPINGS
+    end
+    
+    it 'should map client_id to Fixnum' do
+      @mapping['client_id'].should == Fixnum
+    end
+    
+    it 'should map date to Date' do
+      @mapping['date'].should == Date
+    end
+  end
+  
+  it 'should indicate open status' do
+    @invoice.should respond_to(:open?)
+  end
+  
+  describe 'indicating open status' do
+    it "should be false if the status is 'draft'" do
+      @invoice.status = 'draft'
+      @invoice.should_not be_open
+    end
+    
+    it "should be true if the status is 'sent'" do
+      @invoice.status = 'sent'
+      @invoice.should be_open
+    end
+    
+    it "should be true if the status is 'viewed'" do
+      @invoice.status = 'viewed'
+      @invoice.should be_open
+    end
+    
+    it "should be false if the status is 'paid'" do
+      @invoice.status = 'paid'
+      @invoice.should_not be_open
+    end
+    
+    it "should be true if the status is 'partial'" do
+      @invoice.status = 'partial'
+      @invoice.should be_open
+    end
+  end
+  
+  it 'should have a client' do
+    @invoice.should respond_to(:client)
+  end
+  
+  describe 'client' do
+    it 'should find client based on client_id' do
+      client_id = stub('client ID')
+      @invoice.stubs(:client_id).returns(client_id)
+      FreshBooks::Client.expects(:get).with(client_id)
+      @invoice.client
+    end
+    
+    it 'should return found project' do
+      client = stub('client')
+      client_id = stub('client ID')
+      @invoice.stubs(:client_id).returns(client_id)
+      FreshBooks::Client.expects(:get).with(client_id).returns(client)
+      @invoice.client.should == client
+    end
+  end
+  
+  describe 'number' do
+    it 'should be settable and gettable as an accessor' do
+      @invoice.number = '1234'
+      @invoice.number.should == '1234'
+    end
+    
+    it 'should be settable using []' do
+      @invoice['number'] = '1234'
+      @invoice.number.should == '1234'
+    end
+    
+    it 'should be gettable using []' do
+      @invoice.number = '1234'
+      @invoice['number'].should == '1234'
+    end
+    
+    it 'should show up in the members list' do
+      FreshBooks::Invoice.members.should include('number')
+    end
+  end
+  
+  it 'should still have important core behavior' do
+    FreshBooks::Invoice.should respond_to(:list)
+  end
+  
+  it 'should still be a type of BaseObject' do
+    FreshBooks::Invoice.should < FreshBooks::BaseObject
+  end
+  
+  it 'should still have other fields in the members list' do
+    members = FreshBooks::Invoice.members
+    members.should include('invoice_id')
+    members.should include('client_id')
+    members.should include('status')
+    members.should include('date')
+  end
+  
+  it 'should still allow other fields to be set and get using []' do
+    @invoice['status'] = 'paid'
+    @invoice.status.should == 'paid'
+    
+    @invoice.client_id = 3
+    @invoice['client_id'].should == 3
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: freshtrack
 version: !ruby/object:Gem::Version 
-  version: 0.4.1
+  version: 0.4.1.1
 platform: ruby
 authors: 
 - Yossef Mendelssohn
@@ -83,6 +83,7 @@ files:
 - lib/freshtrack/time_collectors/punchy_template.rb
 - lib/freshbooks/extensions.rb
 - lib/freshbooks/extensions/base_object.rb
+- lib/freshbooks/extensions/invoice.rb
 - lib/freshbooks/extensions/project.rb
 - lib/freshbooks/extensions/task.rb
 - lib/freshbooks/extensions/time_entry.rb
@@ -96,6 +97,7 @@ files:
 - spec/time_collectors/one_inch_punch_spec.rb
 - spec/time_collectors/punch_spec.rb
 - spec/freshbooks/base_object_spec.rb
+- spec/freshbooks/invoice_spec.rb
 - spec/freshbooks/project_spec.rb
 - spec/freshbooks/task_spec.rb
 - spec/freshbooks/time_entry_spec.rb