harvested2 5.0.3

data/spec/functional/tasks_spec.rb

@@ -0,0 +1,119 @@
+require 'spec_helper'
+
+describe 'harvest tasks' do
+  let(:harvest) { Harvest.client(access_token: 'mytoken', account_id: '123') }
+
+  describe 'tasks' do
+    let(:task) { create(:task) }
+
+    context 'allows to add invoice tasks' do
+      before do
+        allow(harvest.tasks).to receive(:create)
+          .and_return(task)
+      end
+
+      it 'returns true' do
+        expect(task.default_hourly_rate).to eql(120)
+        expect(task.name).to eql('A crud task')
+      end
+    end
+
+    context 'allows to update invoice tasks' do
+      before do
+        allow(harvest.tasks).to receive(:update)
+          .and_return(task)
+        task.default_hourly_rate = 140
+        task = harvest.tasks.update(task)
+      end
+
+      it 'returns true' do
+        expect(task.default_hourly_rate).to eql(140)
+      end
+    end
+
+    context 'allows to remove invoice tasks' do
+      before do
+        allow(harvest.tasks).to receive(:delete).and_return([])
+        allow(harvest.tasks).to receive(:all).and_return([])
+        harvest.tasks.delete(task)
+      end
+
+      it 'returns true' do
+        expect(harvest.tasks.all.select do |t|
+          t.name == 'A crud task'
+        end).to eql([])
+      end
+    end
+  end
+
+  describe 'task assignments' do
+    let(:project) { create(:project) }
+    let(:task) do
+      create(:task, name: 'A task for joe', default_hourly_rate: 120)
+    end
+    let(:task_assignment) do
+      create(:task_assignment, project: project, task: task)
+    end
+
+    context 'allows to add invoice task assignments' do
+      before do
+        allow(harvest.task_assignments).to receive(:create)
+          .and_return(task)
+      end
+
+      it 'returns true' do
+        expect(task_assignment.hourly_rate).to eql(120)
+      end
+    end
+
+    context 'allows to update invoice task assignments' do
+      before do
+        allow(harvest.task_assignments).to receive(:update)
+          .and_return(task)
+        task_assignment.hourly_rate = 100
+        task = harvest.task_assignments.update(task_assignment)
+      end
+
+      it 'returns true' do
+        expect(task_assignment.hourly_rate).to eql(100)
+      end
+    end
+
+    context 'allows to remove invoice task assignments' do
+      before do
+        allow(harvest.task_assignments).to receive(:delete)
+          .and_return([])
+        allow(harvest.task_assignments).to receive(:all)
+          .and_return([task_assignment])
+        harvest.task_assignments.delete(task_assignment)
+      end
+
+      it 'returns true' do
+        expect(harvest.task_assignments.all(project).count).to eql(1)
+      end
+    end
+  end
+
+  describe 'projects' do
+    let(:project) { create(:project) }
+    let(:task) do
+      create(:task, name: 'A task for joe', default_hourly_rate: 120)
+    end
+    let(:task_assignment) do
+      create(:task_assignment, project: project, task: task)
+    end
+
+    context 'allows to add project and task' do
+      before do
+        allow(harvest.projects).to receive(:create_task)
+          .and_return(task)
+          allow(harvest.task_assignments).to receive(:all)
+          .and_return([task_assignment])
+      end
+
+      it 'returns true' do
+        expect(harvest.task_assignments.all(project).size).to eql(1)
+      end
+    end
+  end
+end

data/spec/functional/time_entries_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+
+describe 'harvest time tracking' do
+  let(:harvest) { Harvest.client(access_token: 'mytoken', account_id: '123') }
+
+  describe 'time entries' do
+    let(:time_entry) { create(:time_entry) }
+
+    context 'allows to add time entry' do
+      before do
+        allow(harvest.time_entries).to receive(:create)
+          .and_return(time_entry)
+      end
+
+      it 'returns true' do
+        expect(time_entry.notes).to eql('Test api support')
+      end
+    end
+
+    context 'allows to update time entry' do
+      before do
+        allow(harvest.time_entries).to receive(:update)
+          .and_return(time_entry)
+        time_entry.notes = 'New test api support'
+        time_entry = harvest.time_entries.update(time_entry)
+      end
+
+      it 'returns true' do
+        expect(time_entry.notes).to eql('New test api support')
+      end
+    end
+
+    context 'allows to remove time entry' do
+      before do
+        allow(harvest.time_entries).to receive(:delete).and_return([])
+        allow(harvest.time_entries).to receive(:all).and_return([])
+        harvest.time_entries.delete(time_entry)
+      end
+
+      it 'returns true' do
+        expect(harvest.time_entries.all.select do |t|
+          t.notes == 'New test api support'
+        end).to eql([])
+      end
+    end
+  end
+
+  describe 'toggle' do
+    let(:time_entry) { create(:time_entry, :started) }
+
+    before do
+      allow(harvest.time_entries).to receive(:toggle)
+        .and_return(time_entry)
+    end
+
+    it 'should be active' do
+      expect(time_entry.timer_started_at).not_to be_nil
+    end
+
+    it 'should be deactive' do
+      time_entry.timer_started_at = nil
+      time_entry = harvest.time_entries.toggle(time_entry)
+      expect(time_entry.timer_started_at).to be_nil
+    end
+
+    it 'should reactive a time_entry' do
+      time_entry = harvest.time_entries.toggle(time_entry)
+      expect(time_entry).not_to be_nil
+    end
+  end
+
+  describe 'projects' do
+    let(:user) { create(:user) }
+    let(:time_entry) { create(:time_entry) }
+
+    context 'allows to add project and task' do
+      before do
+        allow(harvest.time_entries).to receive(:trackable_projects)
+          .and_return([])
+      end
+
+      it 'returns true' do
+        expect(harvest.time_entries.trackable_projects(Time.now, user)).to eql([])
+      end
+    end
+  end
+end

data/spec/functional/users_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+describe 'harvest users' do
+  let(:harvest) { Harvest.client(access_token: 'mytoken', account_id: '123') }
+
+  describe 'users' do
+    let(:user) { create(:user) }
+
+    context 'allows to add users' do
+      before do
+        allow(harvest.users).to receive(:create).and_return(user)
+      end
+
+      it 'returns true' do
+        expect(user.id).not_to be_nil
+        expect(user.first_name).to eql('Edgar')
+        expect(user.last_name).to eql('Ruth')
+      end
+    end
+
+    context 'allows to update users' do
+      before do
+        allow(harvest.users).to receive(:update).and_return(user)
+        user.first_name = 'Joey'
+        user = harvest.users.update(user)
+      end
+
+      it 'returns true' do
+        expect(user.first_name).to eql('Joey')
+      end
+    end
+
+    context 'allows to remove users' do
+      before do
+        allow(harvest.users).to receive(:delete).and_return([])
+        allow(harvest.users).to receive(:all).and_return([])
+        harvest.users.delete(user)
+      end
+
+      it 'returns true' do
+        expect(harvest.users.all.select do |u|
+          u.first_name == 'Joey'
+        end).to eql([])
+      end
+    end
+
+    context 'allows activating and deactivating users' do
+      let(:user) { create(:user, :active) }
+
+      before do
+        allow(harvest.users).to receive(:update)
+          .and_return(user)
+      end
+
+      it 'should be active' do
+        expect(user).to be_active
+      end
+
+      it 'should be deactive' do
+        user.is_active = false
+        user = harvest.users.update(user)
+        expect(user).not_to be_active
+      end
+
+      it 'should reactive a user' do
+        user.is_active = true
+        user = harvest.users.update(user)
+        expect(user).to be_active
+      end
+    end
+  end
+end

data/spec/harvest/base_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe Harvest::Base do
+  describe 'username/password errors' do
+    it 'should raise error if missing credentials' do
+      lambda { Harvest::Base.new }
+        .should raise_error(/must provide either/i)
+    end
+  end
+end

data/spec/harvest/basic_auth_credentials_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe Harvest::BasicAuthCredentials do
+  context '#set_authentication' do
+    it 'should set the headers Authorization' do
+      credentials = Harvest::BasicAuthCredentials.new(access_token: 'mytoken', account_id: '1')
+      credentials.set_authentication(options = {})
+
+      expect(options[:headers]['Authorization']).to eql('Bearer mytoken')
+    end
+  end
+end

data/spec/harvest/expense_category_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::ExpenseCategory do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/expense_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe Harvest::Expense do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+
+  describe "#spent_date" do
+    it "should parse strings" do
+      date = RUBY_VERSION =~ /1.8/ ? "12/01/2009" : "01/12/2009"
+      expense = Harvest::Expense.new(:spent_date => date)
+      expense.spent_date.should == Date.parse(date)
+    end
+
+    it "should accept times" do
+      expense = Harvest::Expense.new(:spent_date => Time.utc(2009, 12, 1))
+      expense.spent_date.should == Date.new(2009, 12, 1)
+    end
+  end
+end

data/spec/harvest/invoice_message_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::InvoiceMessage do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/invoice_payment_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::InvoicePayment do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/invoice_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::Invoice do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/oauth_credentials_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe Harvest::OAuthCredentials do
+  context '#set_authentication' do
+    it "should set the access token in the query" do
+      credentials = Harvest::OAuthCredentials.new(access_token: 'theaccesstoken')
+      credentials.set_authentication(options = {})
+      options[:query]['access_token'].should == 'theaccesstoken'
+    end
+  end
+end

data/spec/harvest/project_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::Project do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/task_assignment_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::TaskAssignment do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/task_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::Task do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/time_entry_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe Harvest::TimeEntry do
+  context '#as_json' do
+    it 'builds a specialized hash' do
+      entry = Harvest::TimeEntry.new(:notes => 'the notes', :project_id => 'the project id', :task_id => 'the task id', :hours => 'the hours', :spent_date => Time.utc(2009, 12, 28))
+      entry.as_json.should == {"notes" => "the notes", "hours" => "the hours", "project_id" => "the project id", "task_id" => "the task id", "spent_date" => "2009-12-28"}
+    end
+  end
+
+  context "#spent_date" do
+    it "should parse strings" do
+      date = RUBY_VERSION =~ /1.8/ ? "12/01/2009" : "01/12/2009"
+      entry = Harvest::TimeEntry.new(:spent_date => date)
+      entry.spent_date.should == Date.parse(date)
+    end
+
+    it "should accept times" do
+      entry = Harvest::TimeEntry.new(:spent_date => Time.utc(2009, 12, 1))
+      entry.spent_date.should == Date.new(2009, 12, 1)
+    end
+  end
+end

data/spec/harvest/user_assignment_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Harvest::UserAssignment do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+end

data/spec/harvest/user_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe Harvest::User do
+  it_behaves_like 'a json sanitizer', %w(cache_version)
+  
+  describe "#timezone" do
+    it "should convert friendly timezone setting" do
+      p = Harvest::User.new(:timezone => :cst)
+      p.timezone.should == 'Central Time (US & Canada)'
+      
+      p = Harvest::User.new(:timezone => :est)
+      p.timezone.should == 'Eastern Time (US & Canada)'
+      
+      p = Harvest::User.new(:timezone => :mst)
+      p.timezone.should == 'Mountain Time (US & Canada)'
+      
+      p = Harvest::User.new(:timezone => :pst)
+      p.timezone.should == 'Pacific Time (US & Canada)'
+      
+      p = Harvest::User.new(:timezone => 'pst')
+      p.timezone.should == 'Pacific Time (US & Canada)'
+    end
+    
+    it "should convert standard zones" do
+      p = Harvest::User.new(:timezone => 'america/chicago')
+      p.timezone.should == 'Central Time (US & Canada)'
+    end
+    
+    it "should leave literal zones" do
+      p = Harvest::User.new(:timezone => 'Central Time (US & Canada)')
+      p.timezone.should == 'Central Time (US & Canada)'
+    end
+  end
+end