xrono 1.0.0

data/spec/models/contact_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Contact do
+  let(:contact) { Contact.make }
+  let(:contact_2) { Contact.make }
+  let(:contact_2_receives_email) { Contact.make(:receives_email => true) }
+  let(:contact_3) { Contact.make }
+
+  it { should belong_to :client }
+  it { should validate_presence_of :first_name }
+  it { should validate_presence_of :last_name }
+  it { should validate_presence_of :email_address }
+
+  describe 'for_client' do
+    it 'should return the proper list of contacts for the client' do
+      client = contact_2.client
+      Contact.for_client(client).include?(contact_2).should be_true
+      Contact.for_client(client).include?(contact_3).should be_false
+    end
+  end
+
+  describe 'receives_email' do
+    it 'should return the proper list of contacts that can receive email' do
+      client = contact_2_receives_email.client
+      Contact.for_client(client).receives_email.include?(contact_2_receives_email).should be_true
+      Contact.for_client(client).receives_email.include?(contact_3).should_not be_true
+    end
+  end
+end

data/spec/models/data_vault_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe DataVault do
+  it { should belong_to :data_vaultable }
+end

data/spec/models/project_spec.rb

@@ -0,0 +1,115 @@
+require 'spec_helper'
+
+describe Project do
+  let(:project) { Project.make(:name => 'New Project') }
+  let(:ticket_1) { Ticket.make(:project => project) }
+  let(:ticket_2) { Ticket.make(:project => project) }
+
+  it { should belong_to :client }
+  it { should have_many :tickets }
+  it { should have_many :comments }
+  it { should have_many :file_attachments }
+
+  it { should validate_presence_of :name }
+  it { should validate_presence_of :client_id }
+  it { project; should validate_uniqueness_of(:name).scoped_to(:client_id) }
+
+  describe '#to_s' do
+    subject { project.to_s }
+
+    it 'returns the name of the project as a string' do
+      should == 'New Project'
+    end
+  end
+
+  describe '#hours' do
+    it 'returns total number of hours from all tickets on the project' do
+      WorkUnit.make(:ticket => ticket_1)
+      WorkUnit.make(:ticket => ticket_2)
+      project.hours.should == ticket_1.work_units.first.effective_hours + ticket_2.work_units.first.effective_hours
+    end
+  end
+
+  describe '#uninvoiced_hours' do
+    it "returns the sum of hours on all the client's work units" do
+      work_unit_1 = WorkUnit.make
+      ticket = work_unit_1.ticket
+      project = work_unit_1.project
+      work_unit_2 = WorkUnit.make(:ticket => ticket)
+      work_unit_3 = WorkUnit.make(:ticket => ticket, :invoiced => 'Invoiced', :invoiced_at => Time.current)
+      total_hours = work_unit_1.effective_hours + work_unit_2.effective_hours
+
+      project.uninvoiced_hours.should == total_hours
+    end
+  end
+
+  describe '.for_user' do
+    context 'when a user is assigned to a project' do
+      it 'returns the projects to which the user is assigned' do
+        user = User.make
+        project1 = Project.make
+        project2 = Project.make
+        user.has_no_roles_for!(project2)
+        user.has_role!(:developer, project1)
+        Project.for_user(user).include?(project1).should be_true
+        Project.for_user(user).include?(project2).should be_false
+      end
+    end
+  end
+
+  describe '#work_units' do
+    it 'returns the children work units for that project' do
+      work_unit = WorkUnit.make
+      project = work_unit.ticket.project
+      project.work_units.should == [work_unit]
+    end
+  end
+
+  describe "#allows_access?" do
+    let(:project) { Project.make }
+    let(:user_admin) {
+      _user = User.make
+      _user.roles.create(:name => 'admin')
+      _user
+    }
+    let(:user_non_admin) { User.make }
+    let(:user_with_role) {
+      _user = User.make
+      _user.roles.create(:name => "client", :authorizable => project)
+      _user
+    }
+
+    it "returns true if you have access" do
+      project.allows_access?(user_admin).should be_true
+      project.allows_access?(user_non_admin).should be_false
+      project.allows_access?(user_with_role).should be_true
+    end
+  end
+
+  describe "Project#for_user_and_role" do
+    it "returns all projects with user and role" do
+      project = Project.make
+      user    = User.make
+      user.roles.create(:name => "client", :authorizable => project)
+
+      Project.for_user_and_role(user, "client").should == [project]
+    end
+  end
+
+  describe '#files_and_comments' do
+    it 'lists all files and comments' do
+      project = Project.make
+      comment = project.comments.create(:title => "test", :comment => "test")
+      File.open("tmp/tmp.txt", "w") {|f| f.write "test"}
+      fa = project.file_attachments.create(:attachment_file => File.open("tmp/tmp.txt","r"))
+      project.files_and_comments.should == [comment,fa]
+      File.delete("tmp/tmp.txt")
+    end
+  end
+
+  describe 'tagging' do
+    it 'is taggable' do
+      Project.new.is_taggable?.should be_true
+    end
+  end
+end

data/spec/models/ticket_spec.rb

@@ -0,0 +1,223 @@
+require 'spec_helper'
+
+describe Ticket do
+  let(:ticket) { Ticket.make(:name => 'New Ticket') }
+
+  it { should belong_to :project }
+  it { should have_many :work_units }
+  it { should have_many :file_attachments }
+
+  it { should validate_presence_of :project_id }
+  it { should validate_presence_of :name }
+
+  describe '#to_s' do
+    subject { ticket.to_s }
+
+    it 'returns the name of the ticket as a string' do
+      should == 'New Ticket'
+    end
+  end
+
+  describe '#client' do
+    context 'when a ticket exists' do
+      it 'returns the client' do
+        ticket.client.should == ticket.project.client
+      end
+    end
+  end
+
+  describe '#unpaid_hours' do
+    context 'when there are unpaid work units on a ticket' do
+      it 'totals the unpaid hours for that ticket' do
+        work_unit_1 = WorkUnit.make(:ticket => ticket)
+        work_unit_2 = WorkUnit.make(:ticket => ticket)
+        work_unit_3 = WorkUnit.make(:ticket => ticket, :paid => 'paid on 2010-10-25')
+        unpaid_hours = work_unit_1.effective_hours + work_unit_2.effective_hours
+
+        ticket.unpaid_hours.should == unpaid_hours
+      end
+    end
+  end
+
+  describe '#uninvoiced_hours' do
+    context 'when there are uninvoiced work units on a ticket' do
+      it 'returns the total number of uninvoiced work units for that ticket' do
+        ticket = Ticket.make
+        work_unit_1 = WorkUnit.make(:ticket => ticket)
+        work_unit_2 = WorkUnit.make(:ticket => ticket)
+        work_unit_3 = WorkUnit.make(:ticket => ticket, :invoiced => 'invoiced on 2010-10-25')
+        uninvoiced_hours = work_unit_1.effective_hours + work_unit_2.effective_hours
+
+        ticket.uninvoiced_hours.should == uninvoiced_hours
+      end
+    end
+  end
+
+  describe '#long_name' do
+    it 'returns a descriptive string with the ticket id, project, and client' do
+      long_name = "Ticket: [#{ticket.id}] - #{ticket.project.name} - #{ticket.name} ticket for #{ticket.project.client.name}"
+
+      ticket.long_name.should == long_name
+    end
+  end
+
+  describe '#allows_access?' do
+    let(:project) { Project.make }
+    let(:ticket) { Ticket.make(:project => project) }
+    let(:user) { User.make }
+
+    it 'returns false if the user does not have access to the parent project' do
+      ticket.allows_access?(user).should be_false
+    end
+
+    it 'returns true if the user has access to the parent project' do
+      user.has_role!(:developer, project)
+      ticket.allows_access?(user).should be_true
+    end
+  end
+
+  describe '.for_user_role' do
+    context 'when a user has access to a project' do
+      let(:project_1) { Project.make }
+      let(:project_2) { Project.make }
+      let(:user) do
+        _user = User.make
+        _user.has_role!(:developer, project_1)
+        _user.has_role!(:client, project_2)
+        _user
+      end
+      let!(:ticket_1) { Ticket.make(:project => project_1) }
+      let!(:ticket_2) { Ticket.make(:project => project_1) }
+      let!(:ticket_3) { Ticket.make(:project => project_2) }
+
+      it 'returns a collection of tickets for all the projects to which the user is assigned' do
+        Ticket.for_user_and_role(user, "developer").include?(ticket_1).should be_true
+        Ticket.for_user_and_role(user, "developer").include?(ticket_2).should be_true
+        Ticket.for_user_and_role(user, "developer").include?(ticket_3).should be_false
+      end
+    end
+  end
+
+  describe '.for_user' do
+    context 'when a user has access to a project' do
+      let(:project_1) { Project.make }
+      let(:project_2) { Project.make }
+      let(:user) do
+        _user = User.make
+        _user.has_role!(:developer, project_1)
+        _user
+      end
+      let!(:ticket_1) { Ticket.make(:project => project_1) }
+      let!(:ticket_2) { Ticket.make(:project => project_1) }
+      let!(:ticket_3) { Ticket.make(:project => project_2) }
+
+      it 'returns a collection of tickets for all the projects to which the user is assigned' do
+        Ticket.for_user(user).include?(ticket_1).should be_true
+        Ticket.for_user(user).include?(ticket_2).should be_true
+        Ticket.for_user(user).include?(ticket_3).should be_false
+      end
+    end
+  end
+
+  describe 'returns a list of tickets using' do
+    let(:project) { Project.make }
+
+    let!(:ticket) do
+      _ticket = project.tickets.make
+      _ticket.update_attributes(git_branch: 'feature/test')
+      _ticket
+    end
+
+    it ".for_repo_and_branch" do
+      project.update_attributes(git_repo_name: 'test')
+      Ticket.for_repo_and_branch("test", "feature/test").should include(ticket)
+    end
+
+    it ".for_repo_url_and_branch" do
+      project.update_attributes(git_repo_url: 'test')
+      Ticket.for_repo_url_and_branch("test", "feature/test").should include(ticket)
+    end
+  end
+
+  describe '' do
+    let(:ticket) do
+      _ticket = Ticket.make
+      _ticket.update_attributes(estimated_hours: 10)
+      _ticket
+    end
+
+    let(:work_unit_is_under) do
+      wu = ticket.work_units.make
+      wu.update_column(:effective_hours, 5)
+      wu
+    end
+
+    describe '#remaining_hours' do
+      let(:work_unit_is_over) do
+        wu = ticket.work_units.make
+        wu.update_column(:effective_hours, 11)
+        wu
+      end
+
+      it 'returns the remaining hours when there are remaining hours' do
+        work_unit_is_under
+
+        ticket.remaining_hours.should == 5
+      end
+
+      it 'returns 0 when there are more effective hours than estimated hours' do
+        work_unit_is_over
+
+        ticket.remaining_hours.should == 0
+      end
+    end
+
+    describe '#percentage_complete' do
+      let(:ticket_invalid) { Ticket.make }
+
+      it 'calculates the percentage left on the ticket' do
+        work_unit_is_under
+
+        ticket.percentage_complete.should == 50.0
+      end
+
+      it 'returns 0 when there is an error' do
+        ticket_invalid.percentage_complete.should == 0
+      end
+    end
+  end
+
+  describe '#hours' do
+    context 'when there are normal work units with hours' do
+      let(:ticket_2) { Ticket.make }
+
+      it 'returns the correct sum of hours for those work units' do
+        work_unit_1 = WorkUnit.make(:ticket => ticket, :hours => '1.0', :hours_type => 'Normal')
+        work_unit_2 = WorkUnit.make(:ticket => ticket, :hours => '1.0', :hours_type => 'Normal')
+        work_unit_3 = WorkUnit.make(:ticket => ticket_2, :hours => '1.0', :hours_type => 'Normal')
+
+        ticket.hours.should == 2.0
+      end
+    end
+  end
+
+  describe 'while being created' do
+    it 'creates a new ticket from the blueprint' do
+      lambda do
+        Ticket.make
+      end.should change(Ticket, :count).by(1)
+    end
+  end
+
+  describe '#files_and_comments' do
+    it 'lists all files and comments' do
+      ticket = Ticket.make
+      comment = ticket.comments.create(:title => "test", :comment => "test")
+      File.open("tmp/tmp.txt", "w") {|f| f.write "test"}
+      fa = ticket.file_attachments.create(:attachment_file => File.open("tmp/tmp.txt","r"))
+      ticket.files_and_comments.should == [comment,fa]
+      File.delete("tmp/tmp.txt")
+    end
+  end
+
+end

data/spec/models/user_spec.rb

@@ -0,0 +1,258 @@
+require 'spec_helper'
+
+describe User do
+  let(:user) { User.make }
+  let(:user_2) { User.make }
+  let(:work_unit1) { WorkUnit.make(:user => user) }
+  let(:work_unit2) { WorkUnit.make(:user => user) }
+  let(:work_unit3) { WorkUnit.make(:user => user) }
+
+  it { should validate_presence_of :first_name }
+  it { should validate_presence_of :last_name }
+  describe '.with_unpaid_work_units' do
+    subject { User.with_unpaid_work_units }
+
+    before do
+      work_unit1.update_attributes(:paid => nil, :paid_at => nil)
+      work_unit2.update_attributes(:paid => nil, :paid_at => nil)
+    end
+
+    it 'returns a collection of users who have unpaid work units' do
+      should == [user]
+    end
+  end
+
+  describe '.unlocked' do
+    subject { User.unlocked }
+
+    before { user.unlock_access! }
+
+    it 'returns a collection of users who are not locked' do
+      should == [user]
+    end
+  end
+
+  describe '.sort_by_name' do
+    subject { User.sort_by_name }
+
+    before do
+      user.update_attributes(first_name: 'Aaron')
+      user_2.update_attributes(first_name: 'Zed')
+    end
+
+    it 'returns a collection of users sorted by first name' do
+      should == [user, user_2]
+    end
+  end
+
+  describe '#initials' do
+    subject { user.initials }
+
+    before { user.update_attributes(:first_name => 'Aaron', :middle_initial => 'B', :last_name => 'Crenshaw') }
+
+    it 'returns the initials of the user' do
+      should == 'ABC'
+    end
+  end
+
+  describe '#work_units_for_day' do
+
+    before do
+      work_unit1.update_attributes(:scheduled_at => Time.now)
+      work_unit2.update_attributes(:scheduled_at => 1.days.ago)
+    end
+
+    it 'returns a collection of work units for the user scheduled on a given day' do
+      user.work_units_for_day(Time.now).should == [work_unit1]
+    end
+
+    it "sums up all work units' hours  for the user scheduled on a given day" do
+      work_unit1.update_attributes(:hours => 3)
+      work_unit2.update_attributes(:hours => 1, :scheduled_at => Time.now)
+      user.hours_entered_for_day(Time.now).should == 4
+    end
+  end
+
+  describe '.clients_for_day' do
+    subject { user.clients_for_day(Time.now) }
+
+    before do
+      work_unit1.update_attributes(:scheduled_at => Time.now)
+      work_unit2.update_attributes(:scheduled_at => Time.now - 1.day)
+    end
+
+    context 'when the user has work units scheduled for a given day' do
+      it 'returns a collection of clients for those work units' do
+        should == [work_unit1.client]
+      end
+    end
+  end
+
+  describe '#work_units_for_week' do
+    subject { user.work_units_for_week(Time.now) }
+
+    before do
+      work_unit1.update_attributes(:scheduled_at => Time.now)
+      work_unit2.update_attributes(:scheduled_at => Time.now - 1.week)
+    end
+
+    it 'returns a collection of work units for the user scheduled during the week of a given day' do
+      should == [work_unit1]
+    end
+  end
+
+  describe '#unpaid_work_units' do
+    subject { user.unpaid_work_units }
+
+    before do
+      work_unit1.update_attributes(:paid => nil, :paid_at => nil)
+      work_unit2.update_attributes(:paid => 'True', :paid_at => Time.now)
+      work_unit3.update_attributes(:user => user_2)
+    end
+
+    it 'returns a collection of work units which are unpaid and belong to the user' do
+      should == [work_unit1]
+    end
+  end
+
+  describe '.for_project' do
+    let(:project) { Project.make }
+    before(:each) { user.has_role!("developer", project) }
+
+    it 'returns all the users listed on a project' do
+      User.for_project(project).should == [user]
+    end
+  end
+
+  describe '#to_s' do
+    subject { user.to_s }
+
+    before { user.update_attributes(:first_name => 'Aaron', :middle_initial => 'B', :last_name => 'Crenshaw') }
+
+    it 'returns the first name, middle initial, and last name for the user' do
+      should == 'Aaron B Crenshaw'
+    end
+  end
+
+  describe '#admin?' do
+    subject { user.admin? }
+
+    context 'when the user has an admin role' do
+      before { user.has_role!(:admin) }
+      it { should be_true }
+    end
+
+    context 'when the user does not have an admin role' do
+      before { user.has_no_role!(:admin) }
+      it { should be_false }
+    end
+  end
+
+  describe '#locked' do
+    subject { user.locked }
+
+    context 'when the user is locked' do
+      before { user.lock_access! }
+      it { should be_true }
+    end
+
+    context 'when the user is unlocked' do
+      before { user.unlock_access! }
+      it { should be_false }
+    end
+  end
+
+  describe '#pto_hours_left' do
+    subject { user.pto_hours_left(Date.today.end_of_year) }
+
+    let(:site_settings) { SiteSettings.make }
+
+    before do
+      site_settings.update_attributes(:total_yearly_pto_per_user => 40)
+      work_unit1.update_attributes(:hours => 2, :hours_type => 'PTO', :scheduled_at => Date.today)
+      work_unit2.update_attributes(:hours => 3, :hours_type => 'PTO', :scheduled_at => 1.days.from_now)
+      work_unit3.update_attributes(:hours => 5, :hours_type => 'PTO', :scheduled_at => '2010-12-31')
+    end
+
+    it 'returns the number of PTO hours left for the given year as of the passed date' do
+      should == 35
+    end
+  end
+
+  describe 'while being created' do
+    it 'creates a new user from the blueprint' do
+      lambda do
+        User.make
+      end.should change(User, :count).by(1)
+    end
+  end
+
+  describe 'target_hours_offset' do
+    subject { user.target_hours_offset(Date.current) }
+
+    it 'raises an error on non-date objects' do
+      lambda{ user.target_hours_offset(Time.now) }.should raise_error(RuntimeError)
+    end
+
+    it 'calculates the hours a user needs to meet their daily target hours' do
+      should == 0.0
+    end
+  end
+
+  describe 'expected_hours' do
+    subject { user.expected_hours(Date.current).to_s }
+
+    before do
+      work_unit1.update_attributes(:hours => 2, :hours_type => 'Normal', :scheduled_at => Date.yesterday)
+    end
+
+    it 'raises an error if not passed a date' do
+      lambda do
+        user.expected_hours(Time.now)
+      end.should raise_error(RuntimeError)
+    end
+
+    it 'calculates the expected hours for a user' do
+      should =~ /\d+/
+    end
+
+    context 'user has been here all year' do
+      before do
+        work_unit1.update_attributes(:hours => 2, :hours_type => 'Normal', :scheduled_at => Date.current.years_ago(1))
+      end
+      it { should =~ /\d+/}
+    end
+  end
+
+  describe 'percentage_work_for' do
+    subject { user.percentage_work_for(Client.new, Date.yesterday, Date.current) }
+
+    it 'raises an exception if the client is not a Client' do
+      lambda do
+        user.percentage_work_for(User.new, Date.yesterday, Date.current)
+      end.should raise_error(RuntimeError)
+    end
+
+    it 'raises an exception if the start_date is not a Date' do
+      lambda do
+        user.percentage_work_for(Client.new, 1.days.ago, Date.current)
+      end.should raise_error(RuntimeError)
+    end
+
+    it 'raises an exception if the end_date is not a Date' do
+      lambda do
+        user.percentage_work_for(Client.new, Date.yesterday, Time.now)
+      end.should raise_error(RuntimeError)
+    end
+
+    it 'does not raise an exception if given the right parameter types' do
+      lambda do
+        user.percentage_work_for(client, start_date, end_date)
+      end.should_not raise_error(RuntimeError)
+    end
+
+    it 'calculates the percentage of hours worked for a client as a percentage of all work done' do
+      should == 0
+    end
+  end
+end