watch_tower 0.0.0.1 → 0.0.1.beta1

data/spec/watch_tower/file_tree_spec.rb

@@ -0,0 +1,156 @@
+require 'spec_helper'
+
+describe FileTree do
+  before(:each) do
+    @base_path = "/project"
+    @simple_files = [
+      { path: "#{@base_path}/file1.rb", elapsed_time: 3600 },
+      { path: "#{@base_path}/file2.rb", elapsed_time: 1800 },
+    ]
+    @files = @simple_files.dup
+    @files << { path: "#{@base_path}/folder/file_under_folder1.rb", elapsed_time: 1800 }
+    @files << { path: "#{@base_path}/folder/file_under_folder2.rb", elapsed_time: 900 }
+
+    @simple_files_elapsed_times = {
+      @base_path => 5400
+    }
+
+    @files_elapsed_times = {
+      @base_path => 8100,
+      "#{@base_path}/folder" => 2700
+    }
+  end
+
+  describe "#remove_base_path_from_files" do
+    subject { FileTree.new(@base_path, @files) }
+
+    it { should respond_to :remove_base_path_from_files }
+
+    it "should remove the base_path from all the files in the @files array" do
+      files = subject.send(:remove_base_path_from_files, @base_path, @simple_files)
+
+      files.each do |f|
+        f[:path].should_not match %r(#{@base_path}/)
+      end
+    end
+  end
+
+  describe "#process" do
+    subject { FileTree.new(@base_path, @files) }
+
+    it { should respond_to :process }
+
+    it "should be called on initialize" do
+      FileTree.any_instance.expects(:process).once
+
+      FileTree.new(@base_path, @files)
+    end
+
+    it "should call parse_files" do
+      FileTree.any_instance.expects(:parse_files).once
+      FileTree.any_instance.stubs(:parse_folders)
+
+      FileTree.new(@base_path, @files)
+    end
+
+    it "should call parse_folders" do
+      FileTree.any_instance.stubs(:parse_files)
+      FileTree.any_instance.expects(:parse_folders).once
+
+      FileTree.new(@base_path, @files)
+    end
+  end
+
+  describe "simple files without folders" do
+    subject { FileTree.new(@base_path, @simple_files) }
+
+    describe "#files" do
+      it { should respond_to :files }
+
+      it "should return an array" do
+        subject.files.should be_instance_of Array
+      end
+
+      it "should include all the files with their elapsed times" do
+        subject.files.each do |f|
+          f.should be_instance_of Hash
+          f[:path].should =~ /^file(1|2)\.rb$/
+          f[:elapsed_time].to_s.should =~ /^(3600|1800)$/
+        end
+      end
+    end
+
+    describe "#elapsed_time" do
+      it { should respond_to :elapsed_time }
+
+      it "should return elapsed_time" do
+        subject.elapsed_time.should == @simple_files_elapsed_times[@base_path]
+      end
+    end
+
+    describe "#nested_tree" do
+      it { should respond_to :nested_tree }
+
+      it "should have an empty nested_tree" do
+        subject.nested_tree.should be_empty
+      end
+
+    end
+  end
+
+  describe "files with folders" do
+    subject { FileTree.new(@base_path, @files) }
+
+    describe "#files" do
+      it { should respond_to :files }
+
+      it "should return an array" do
+        subject.files.should be_instance_of Array
+      end
+
+      it "should include all the files with their elapsed times" do
+        subject.files.each do |f|
+          f.should be_instance_of Hash
+          f[:path].should =~ /^file(1|2)\.rb$/
+          f[:elapsed_time].to_s.should =~ /^(3600|1800)$/
+        end
+      end
+    end
+
+    describe "#elapsed_time" do
+      it { should respond_to :elapsed_time }
+
+      it "should return elapsed_time" do
+        subject.elapsed_time.should == @files_elapsed_times[@base_path]
+      end
+    end
+
+    describe "#nested_tree" do
+      it { should respond_to :nested_tree }
+
+      it "should not have an empty nested_tree" do
+        subject.nested_tree.should_not be_empty
+      end
+
+      it "should be a Hash" do
+        subject.nested_tree.should be_instance_of Hash
+      end
+
+      it "should return a FileTree for each element of the hash" do
+        subject.nested_tree['folder'].should be_instance_of FileTree
+      end
+
+      it "should include all the files under the folder" do
+        subject.nested_tree['folder'].files.each do |f|
+          f.should be_instance_of Hash
+          f[:path].should =~ /^file_under_folder(1|2)\.rb$/
+          f[:elapsed_time].to_s.should =~ /^(1800|900)$/
+        end
+      end
+
+      it "should return the elapsed time of the folder" do
+        subject.nested_tree['folder'].elapsed_time.should == @files_elapsed_times["#{@base_path}/folder"]
+      end
+    end
+  end
+end

data/spec/watch_tower/server/app_spec.rb

@@ -9,13 +9,18 @@ module Server
       }
 
       @projects[:not_empty][:project] = FactoryGirl.create(:project)
+      Timecop.freeze(Time.now)
       2.times do
-        @projects[:not_empty][:files] << FactoryGirl.create(:file, project: @projects[:not_empty][:project])
-      end
-      5.times do
-        @projects[:not_empty][:time_entries] << FactoryGirl.create(:time_entry, file: @projects[:not_empty][:files].first)
+        2.times do
+          @projects[:not_empty][:files] << FactoryGirl.create(:file, project: @projects[:not_empty][:project])
+        end
+        5.times do
+          @projects[:not_empty][:time_entries] << FactoryGirl.create(:time_entry, file: @projects[:not_empty][:files].first)
+        end
+        @projects[:not_empty][:duration] = FactoryGirl.create(:duration, file: @projects[:not_empty][:files].first)
+        Timecop.freeze(Time.now + 2.days)
       end
-      @projects[:not_empty][:duration] = FactoryGirl.create(:duration, file: @projects[:not_empty][:files].first)
+      Timecop.return
 
       @projects[:empty][:project] = FactoryGirl.create(:project)
       2.times do
@@ -25,6 +30,21 @@ module Server
       @projects[:empty][:duration] = FactoryGirl.create(:duration, file: @projects[:empty][:files].first)
     end
 
+    after(:each) do
+      # Reset the from/to date
+      visit "/?from_date=&to_date="
+    end
+
+    describe "#layout" do
+      it "should show a datepicker" do
+        visit '/'
+        within '#main' do
+          page.should have_selector 'aside#date'
+          page.should have_selector 'aside#date input'
+        end
+      end
+    end
+
     describe "#index" do
       before(:each) do
         visit '/'
@@ -93,6 +113,24 @@ module Server
           page.should have_selector('a', href: "/project/#{@projects[:not_empty][:project].id}")
         end
       end
+
+      it "should display No projects available for the selected date range if there are no projects" do
+        Project.delete_all
+        visit '/'
+        page.should have_content "No projects available for the selected date range."
+      end
+
+      it "should display No projects available for the selected date range if projects exist but date range is way off" do
+        params = ['from_date=10/01/2001', 'to_date=10/10/2001']
+        visit "/?#{params.join('&')}"
+        page.should have_content "No projects available for the selected date range."
+      end
+
+      it "should display the project with elapsed time with the date range" do
+        params = [(Time.now + 1.day).strftime('%m/%d/%Y'), (Time.now + 3.days).strftime('%m/%d/%Y')]
+        visit "/?#{params.join('&')}"
+        page.should have_content '8 seconds'
+      end
     end
 
     describe "#project" do
@@ -146,41 +184,47 @@ module Server
         end
       end
 
-      it "should have a div for the file's path" do
-        within 'article#project section#files article.file' do
-          page.should have_selector 'div.path'
+      it "should have a span for the file's path" do
+        within 'article#project section#files li.file' do
+          page.should have_selector 'span.path'
         end
       end
 
       it "should display the file's path" do
-        within 'article#project section#files article.file div.path' do
-          page.should have_content @projects[:not_empty][:files].first.path
+        within 'article#project section#files li.file span.path' do
+          page.should have_content ::File.basename(@projects[:not_empty][:files].first.path)
         end
       end
 
-      it "should have a div for the file's elapsed time" do
-        within 'article#project section#files article.file' do
-          page.should have_selector 'div.elapsed'
+      it "should have a span for the file's elapsed time" do
+        within 'article#project section#files li.file' do
+          page.should have_selector 'span.elapsed_time'
         end
       end
 
       it "should display the file's elaped time" do
-        within 'article#project section#files article.file div.elapsed' do
+        within 'article#project section#files li.file span.elapsed_time' do
           page.should have_content @projects[:not_empty][:files].first.elapsed_time.to_s
         end
       end
 
-      it "should display an image under each file's name to categorize percentage" do
-        within 'article#project section#files article.file' do
-          page.should have_selector '.percentage_img_container > .percentage > img'
-        end
-      end
-
       it "should not display files having 0 seconds" do
         within 'article#project section#files' do
           page.should_not have_content @projects[:not_empty][:files].last.path
         end
       end
+
+      it "should display No files available for the selected date range if there are no files" do
+        @projects[:not_empty][:project].files.delete_all
+        visit "/project/#{@projects[:not_empty][:project].id}"
+        page.should have_content "No files available for the selected date range."
+      end
+
+      it "should display No files available for the selected date range if projects exist but date range is way off" do
+        params = ['from_date=10/01/2001', 'to_date=10/10/2001']
+        visit "/project/#{@projects[:not_empty][:project].id}?#{params.join('&')}"
+        page.should have_content "No files available for the selected date range."
+      end
     end
   end
 end

data/spec/watch_tower/server/models/file_spec.rb

@@ -229,44 +229,25 @@ module Server
         File.worked_on.should_not include(@files.last)
       end
     end
-  end
-
-  describe "Methods" do
-    before(:each) do
-      @project = FactoryGirl.create :project
-      @files = []
-      2.times do
-        @files << FactoryGirl.create(:file, project: @project)
-        2.times do
-          FactoryGirl.create :time_entry, file: @files.last
-        end
-      end
-    end
 
-    describe "one project" do
-      describe "#sum_elapsed_time" do
-        it "should return the sum of all elapsed times of all the files for the same project" do
-          @project.files.sum_elapsed_time.should == @files.inject(0) {|s, f| s += f.elapsed_time }
-        end
+    describe "Methods" do
+      it "should respond_to sum_elapsed_time" do
+        File.should respond_to(:sum_elapsed_time)
       end
 
-      describe "#percent" do
-        it "should return 50" do
-          @project.files.first.percent.should == 50
-        end
-      end
-    end
+      it { should respond_to :percent }
 
-    describe "two projects" do
-      before(:each) do
-        @other_files = []
-        2.times do
-          @other_files << FactoryGirl.create(:file)
+      describe "one project" do
+        before(:each) do
+          @project = FactoryGirl.create :project
+          @files = []
           2.times do
-            FactoryGirl.create :time_entry, file: @other_files.last
+            @files << FactoryGirl.create(:file, project: @project)
+            2.times do
+              FactoryGirl.create :time_entry, file: @files.last
+            end
           end
         end
-
         describe "#sum_elapsed_time" do
           it "should return the sum of all elapsed times of all the files for the same project" do
             @project.files.sum_elapsed_time.should == @files.inject(0) {|s, f| s += f.elapsed_time }
@@ -279,6 +260,87 @@ module Server
           end
         end
       end
+
+      describe "two projects" do
+        before(:each) do
+          @project = FactoryGirl.create :project
+          @files = []
+          2.times do
+            @files << FactoryGirl.create(:file, project: @project)
+            2.times do
+              FactoryGirl.create :time_entry, file: @files.last
+            end
+          end
+
+          @other_files = []
+          2.times do
+            @other_files << FactoryGirl.create(:file)
+            2.times do
+              FactoryGirl.create :time_entry, file: @other_files.last
+            end
+          end
+
+          describe "#sum_elapsed_time" do
+            it "should return the sum of all elapsed times of all the files for the same project" do
+              @project.files.sum_elapsed_time.should == @files.inject(0) {|s, f| s += f.elapsed_time }
+            end
+          end
+
+          describe "#percent" do
+            it "should return 50" do
+              @project.files.first.percent.should == 50
+            end
+          end
+        end
+      end
+
+      describe "#date_range" do
+        before(:each) do
+          @tw_files = []
+          @lw_files = []
+          @project = FactoryGirl.create(:project)
+          # Freeze the time to this week
+          Timecop.freeze(Time.now)
+          # Create a bunch of files
+          2.times do
+            @tw_files << FactoryGirl.create(:file, project: @project)
+            2.times do |n|
+              FactoryGirl.create(:time_entry, file: @tw_files.last)
+            end
+          end
+          # Freeze the time to last week
+          Timecop.freeze(Time.now - 7.days)
+          # Create a bunch of files
+          2.times do
+            @lw_files << FactoryGirl.create(:file, project: @project)
+            2.times do |n|
+              FactoryGirl.create(:time_entry, file: @lw_files.last)
+            end
+          end
+          # Return the frozen time to now
+          Timecop.freeze(Time.now + 7.days)
+        end
+
+        subject { File }
+
+        it { should respond_to :date_range }
+
+        it "should return an active relation object" do
+          from = (Time.now - 6.days).strftime '%m/%d/%Y'
+          to = Time.now.strftime '%m/%d/%Y'
+
+          q = subject.date_range from, to
+          q.should be_instance_of ActiveRecord::Relation
+        end
+
+        it "should not return project's of last week" do
+          from = (Time.now - 6.days).strftime '%m/%d/%Y'
+          to = Time.now.strftime '%m/%d/%Y'
+
+          q = subject.date_range from, to
+          @lw_files.each {|p| q.should_not include(p)}
+        end
+      end
     end
   end
 end

data/spec/watch_tower/server/models/project_spec.rb

@@ -8,10 +8,6 @@ module Server
       it { should respond_to :files_count }
     end
 
-    describe "Methods" do
-      it { should respond_to :elapsed_time }
-    end
-
     describe "Validations" do
       it { should_not be_valid }
 
@@ -46,20 +42,23 @@ module Server
         @project.reload.files_count.should == 10
       end
 
-      it "should have time_entries through files" # do
-       #        file1 = FactoryGirl.create :file, project: @project
-       #        file2 = FactoryGirl.create :file, project: @project
-       #
-       #        10.times do
-       #          FactoryGirl.create :time_entry, file: file1
-       #        end
-       #
-       #        10.times do
-       #          FactoryGirl.create :time_entry, file: file2
-       #        end
-       #
-       #        @project.reload.time_entries.size.should == 20
-       #      end
+      it "should have many time entries through files" do
+        f = FactoryGirl.create :file, project: @project
+        t = FactoryGirl.create :time_entry, file: f
+
+        @project.reload.time_entries.should include(t)
+      end
+
+      it "should have many durations through files" do
+        f = FactoryGirl.create :file, project: @project
+
+        2.times do
+          FactoryGirl.create :time_entry, file: f
+        end
+
+        @project.reload.durations.should_not be_empty
+        @project.reload.durations.should include(Duration.first)
+      end
     end
 
     describe "TimeEntries count" do
@@ -161,5 +160,135 @@ module Server
         Project.worked_on.should_not include(@projects.last)
       end
     end
+
+    describe "Methods" do
+      describe "#elapsed_time" do
+        before(:each) do
+          @projects = []
+          2.times do
+            @projects << FactoryGirl.create(:project)
+          end
+
+          @projects.each do |p|
+            2.times do
+              f = FactoryGirl.create(:file, project: p)
+              2.times do
+                FactoryGirl.create :time_entry, file: f
+              end
+            end
+          end
+        end
+
+        it { should respond_to :elapsed_time }
+
+        it "should have the right elapsed_time" do
+          @projects.first.elapsed_time.should == 4
+        end
+      end
+
+
+      describe "#sum_elapsed_time" do
+        before(:each) do
+          @projects = []
+          2.times do
+            @projects << FactoryGirl.create(:project)
+          end
+
+          @projects.each do |p|
+            2.times do
+              f = FactoryGirl.create(:file, project: p)
+              2.times do
+                FactoryGirl.create :time_entry, file: f
+              end
+            end
+          end
+        end
+
+        it "should respond_to sum_elapsed_time" do
+          Project.should respond_to(:sum_elapsed_time)
+        end
+
+        it "should return the sum of all elapsed times of all projects" do
+          Project.sum_elapsed_time.should == @projects.inject(0) {|s, p| s += p.elapsed_time }
+        end
+      end
+
+      describe "#date_range" do
+        before(:each) do
+          @tw_projects = []
+          @lw_projects = []
+          # Freeze the time to this week
+          Timecop.freeze(Time.now)
+          # Create a bunch of projects
+          2.times do
+            @tw_projects << FactoryGirl.create(:project)
+            2.times do
+              f = FactoryGirl.create(:file, project: @tw_projects.last)
+              2.times do |n|
+                FactoryGirl.create(:time_entry, file: f)
+              end
+            end
+          end
+          # Freeze the time to last week
+          Timecop.freeze(Time.now - 7.days)
+          # Create a bunch of projects
+          2.times do
+            @lw_projects << FactoryGirl.create(:project)
+            2.times do
+              f = FactoryGirl.create(:file, project: @lw_projects.last)
+              2.times do
+                FactoryGirl.create(:time_entry, file: f)
+              end
+            end
+          end
+          # Return the frozen time to now
+          Timecop.freeze(Time.now + 7.days)
+        end
+
+        subject { Project }
+
+        it { should respond_to :date_range }
+
+        it "should return an active relation object" do
+          from = (Time.now - 6.days).strftime '%m/%d/%Y'
+          to = Time.now.strftime '%m/%d/%Y'
+
+          q = subject.date_range from, to
+          q.should be_instance_of ActiveRecord::Relation
+        end
+
+        it "should not return project's of last week" do
+          from = (Time.now - 6.days).strftime '%m/%d/%Y'
+          to = Time.now.strftime '%m/%d/%Y'
+
+          q = subject.date_range from, to
+          @lw_projects.each {|p| q.should_not include(p)}
+        end
+      end
+
+      describe "#percent" do
+        before(:each) do
+          @projects = []
+          2.times do
+            @projects << FactoryGirl.create(:project)
+          end
+
+          @projects.each do |p|
+            2.times do
+              f = FactoryGirl.create(:file, project: p)
+              2.times do
+                FactoryGirl.create :time_entry, file: f
+              end
+            end
+          end
+        end
+
+        it { should respond_to :percent }
+
+        it "should return 50" do
+          @projects.first.percent.should == 50
+        end
+      end
+    end
   end
 end

data/spec/watch_tower/server/models/time_entry_spec.rb

@@ -19,6 +19,16 @@ module Server
         m.should_not be_valid
       end
 
+      it "should require an editor_name" do
+        m = FactoryGirl.build :time_entry, editor_name: nil
+        m.should_not be_valid
+      end
+
+      it "should require an editor_version" do
+        m = FactoryGirl.build :time_entry, editor_version: nil
+        m.should_not be_valid
+      end
+
       it "should be valid if attributes requirements are met" do
         t = FactoryGirl.build :time_entry
         t.should be_valid

data/spec/watch_tower/server/{decorator/project_decorator_spec.rb → presenters/application_presenter_spec.rb}

@@ -1,21 +1,18 @@
 require 'spec_helper'
 
 module Server
-  module Decorator
-    describe ProjectDecorator do
-      before(:each) do
-        @project = FactoryGirl.create :project
-      end
+  module Presenters
+    describe ApplicationPresenter do
 
-      subject { ProjectDecorator.find(@project.id) }
+      describe "Instance Methods" do
+        before(:each) do
+          @model = mock
+          @model.stubs(:id).returns(1)
+          @model.stubs(:elapsed_time).returns(1234)
+        end
 
-      it "should return a formatted elapsed time" do
-        time = 1.day + 2.hours + 3.minutes + 34.seconds
-        subject.stubs(:elapsed_time).returns(time)
-        subject.elapsed.should == '1 day, 2 hours, 3 minutes and 34 seconds'
-      end
+        subject { ApplicationPresenter.new(@model, nil) }
 
-      describe "Application Decorator" do
         describe "#humanize_time" do
           it "should return 1 second" do
             subject.send(:humanize_time, 1).should == '1 second'
@@ -55,6 +52,20 @@ module Server
           end
         end
       end
+
+      describe "Class Methods" do
+        subject { ApplicationPresenter }
+
+        describe "#presents" do
+          it { should respond_to :presents }
+
+          it "should create a method the same name as the argument passed to presents" do
+            Class.new(subject) do
+              presents :foo
+            end.new(@project, nil).should respond_to :foo
+          end
+        end
+      end
     end
   end
-end
+end

data/spec/watch_tower/server/presenters/file_presenter_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+
+module Server
+  module Presenters
+    describe FilePresenter do
+    end
+  end
+end