backlog 0.13.0 → 0.13.1

data/test/functional/tasks_controller_test.rb

@@ -206,122 +206,6 @@ class TasksControllerTest < Test::Unit::TestCase
     assert_equal 3, subtask_after.position
   end
   
-  def test_move_to_next_period
-    before = Task.find(tasks(:another).id)
-    
-    post :move_to_next_period, :id => tasks(:another).id
-    
-    assert_response :success
-    
-    after = Task.find(tasks(:another).id)
-    assert_equal tasks(:another).period_id, after.period_id
-    
-    new_task = Task.find_by_period_id_and_previous_task_id(periods(:future).id, before.id)
-    assert_nil new_task.finished_at
-    assert_nil new_task.resolution
-    assert_equal 1, new_task.position
-  end
-  
-  def test_move_to_next_period_from_past
-    task = tasks(:first)
-    before = Task.find(task.id)
-    
-    post :move_to_next_period, :id => task.id
-    
-    assert_response :success
-    
-    after = Task.find(task.id)
-    assert_equal task.period_id, after.period_id
-    
-    new_task = Task.find_by_period_id_and_previous_task_id(periods(:active).id, task.id)
-    assert_nil new_task.finished_at
-    assert_nil new_task.resolution
-    assert_equal 1, new_task.position
-  end
-  
-  def test_move_to_next_period_at_end
-    before = Task.find(tasks(:in_last_period).id)
-    assert_equal tasks(:in_last_period).period_id, before.period_id
-    
-    post :move_to_next_period, :id => tasks(:in_last_period).id
-    
-    assert_response :success
-    
-    before.reload
-    assert_equal tasks(:in_last_period).period_id, before.period_id
-    
-    after = Task.find(tasks(:in_last_period).id)
-    assert_equal before.period_id, after.period_id
-  end
-  
-  def test_move_to_next_period_from_ancient
-    before = tasks(:in_ancient)
-    previous_last_id = Task.find(:first, :order => 'id DESC').id
-    
-    post :move_to_next_period, :id => before.id
-    
-    assert_response :success
-    
-    after = Task.find(before.id)
-    assert_equal before.period_id, after.period_id
-    assert_equal Task::POSTPONED, after.resolution
-    assert_not_nil after.finished_at
-    
-    new = Task.find(:first, :order => 'id DESC', :conditions => "id > #{previous_last_id}")
-    assert_not_nil new
-    assert_equal before.id, new.previous_task_id
-    assert_equal 1, new.position
-  end
-  
-  def test_move_to_period
-    highest = Task.find(:first, :order => 'id DESC')
-    
-    post :move_to_period, :id => "task_#{tasks(:another).id}",
-        :detour => {:action => 'show_no_layout', :id => periods(:active).id, :controller => 'periods'}, 
-        :period_id => periods(:future).id
-    
-    new_task = Task.find(:first, :order => 'id DESC')
-    assert new_task.id != highest.id
-    assert_equal periods(:future).id, new_task.period_id
-  end
-  
-  def test_move_to_period_past
-    post :move_to_period, :id => "task_#{tasks(:another).id}",
-        :detour => {:action => 'show_no_layout', :id => periods(:active).id, :controller => 'periods'}, 
-        :period_id => periods(:past).id
-    assert_equal periods(:active).id, tasks(:another).period_id
-  end
-  
-  def test_move_to_period_back_to_earlier_period
-    task_to_move = tasks(:in_last_period)
-    target_task = tasks(:postponed)
-    target_period = periods(:active)
-    
-    post :move_to_period, {"id"=> task_to_move.id,
-                          "detour"=>{"action"=>"edit_no_layout", "id"=>"39", "controller"=>"backlogs"}, 
-                          "period_id"=> target_period.id
-    }
-    
-    old_task = Task.find(task_to_move.id)
-    assert_not_nil old_task.finished_at
-    assert_equal Task::POSTPONED, old_task.resolution
-    assert_nil old_task.position
-    
-    new_task = Task.find(target_task.id)
-    new_task.reload
-    assert_nil new_task.finished_at
-    assert_nil new_task.resolution
-    assert_equal 1, new_task.position
-  end
-
-  def test_abort
-    post :abort, :id => tasks(:first).id
-    
-    after = Task.find(tasks(:first).id)
-    assert_not_nil after.finished_at
-    assert_equal Task::ABORTED, after.resolution
-  end
-  
   def test_start_work
     task_id = tasks(:first).id
     

data/test/functional/works_controller_test.rb

@@ -80,6 +80,14 @@ class WorksControllerTest < Test::Unit::TestCase
     assert_redirected_to :controller => 'periods', :action => 'show', :id => periods(:past).id
   end
   
+  def test_update_time
+    post :update_time, :id => 1, :work => {"started_at_time"=>"8:15"}
+    assert_response :success
+    now = Time.now
+    expected_time = Time.local(now.year , now.month, now.day, 8, 15)
+    assert_equal expected_time.iso8601, assigns(:work).started_at.iso8601
+  end
+  
   def test_destroy
     assert_not_nil Work.find(1)
     
@@ -106,7 +114,7 @@ class WorksControllerTest < Test::Unit::TestCase
     assert_equal num_tasks, Task.count
     assert_equal num_open_tasks - 1, Task.find_open.size
   end
-
+  
   def test_weekly_work_sheet
     get :weekly_work_sheet, :id => 24
   end

data/test/performance/test_threaded.rb

@@ -1,27 +1,45 @@
 require File.dirname(__FILE__) + '/../test_helper'
 
-# TODO (uwe): Start application for test.  Don't use production environment...
-
 class BacklogsControllerTest < Test::Unit::TestCase
+  PORT = 4000
+  
+  def setup
+    @server_thread = IO.popen("script/server -e test -p #{PORT}")
+    Thread.start do
+      loop do
+        lines = @server_thread.readline
+        #puts lines
+      end
+    end
+    sleep 4
+  end
+  
+  def teardown
+    if @server_thread
+      Process.kill("TERM", @server_thread.pid)
+    end
+  end
+  
   def test_lots_of_requests
-    runs = 5
-    no_of_threads = 10
-    hits_per_thread = 20
+    runs = 3
+    no_of_threads = 2
+    hits_per_thread = 5
     
-    duration = with_timing("First") {hit}
-    assert duration <= 5
+    duration = with_timing("First") {get_burn_down_chart}
+    first_hit_limit = 12
+    assert duration <= first_hit_limit, "Request took #{duration} seconds.  Should be less than #{first_hit_limit} seconds."
     
     runs.times do |run_no|
-      threads = (1..no_of_threads).map do |i| 
-        Thread.new do 
-          hits_per_thread.times {hit}
-        end
-      end
       duration = with_timing((run_no+1)*no_of_threads*hits_per_thread, no_of_threads*hits_per_thread) do
-        threads.each {|thread| thread.run}
+        threads = (1..no_of_threads).map do |i| 
+          Thread.new do 
+            hits_per_thread.times {get_burn_down_chart}
+          end
+        end
         threads.each {|thread| thread.join}
       end
-      assert duration <= 90
+      maximum_duration = 6.0 * no_of_threads * hits_per_thread
+      assert duration <= maximum_duration, "Performance to low.  Expected #{maximum_duration} or less, took #{duration}."
     end
   end
   
@@ -36,10 +54,12 @@ class BacklogsControllerTest < Test::Unit::TestCase
     duration
   end
   
-  def hit
-    path = 'parties/burn_down_chart/1'
-    system 'GET http://localhost:3000/#{path} >/dev/null'
-    #system 'GET http://localhost:8080/backlog/#{path} >/dev/null'
+  def get_burn_down_chart
+    path = '/parties/burn_down_chart/1'
+    Net::HTTP::start('localhost', port = PORT) do |http|
+      response = http.get path
+      assert response.body =~ /^.PNG/
+    end
   end
   
 end

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.9.4
 specification_version: 1
 name: backlog
 version: !ruby/object:Gem::Version 
-  version: 0.13.0
-date: 2007-11-14 00:00:00 +01:00
+  version: 0.13.1
+date: 2007-11-17 00:00:00 +01:00
 summary: Application to aid collecting, processing, organizing, reviewing and doing tasks.
 require_paths: 
 - lib
@@ -433,16 +433,18 @@ files:
 - app/views/layouts/_notice.rhtml
 - app/views/layouts/_headers.rhtml
 - app/views/backlogs
+- app/views/backlogs/move_task_to_period.rjs
 - app/views/backlogs/_name_list.rhtml
 - app/views/backlogs/list.rhtml
 - app/views/backlogs/_buttons.rhtml
+- app/views/backlogs/finish_task.rjs
 - app/views/backlogs/show.rhtml
 - app/views/backlogs/_form.rhtml
 - app/views/backlogs/new.rhtml
 - app/views/backlogs/edit.rhtml
 - app/views/backlogs/_tasks.rhtml
+- app/views/backlogs/reopen_task.rjs
 - app/views/estimates
-- app/views/estimates/create_ajax.rjs
 - app/views/groups
 - app/views/groups/list.rhtml
 - app/views/groups/_form.rhtml
@@ -461,17 +463,15 @@ files:
 - app/views/tasks
 - app/views/tasks/_backlog_header.rhtml
 - app/views/tasks/list.rhtml
-- app/views/tasks/move_to_period.rjs
 - app/views/tasks/_form.rhtml
 - app/views/tasks/_period_header.rhtml
-- app/views/tasks/finish.rjs
+- app/views/tasks/update_estimate.rjs
 - app/views/tasks/new.rhtml
 - app/views/tasks/start_work.rjs
 - app/views/tasks/_fields_header.rhtml
 - app/views/tasks/_completed.rhtml
 - app/views/tasks/edit.rhtml
 - app/views/tasks/list_started.rhtml
-- app/views/tasks/reopen.rjs
 - app/views/tasks/_task.rhtml
 - app/views/customers
 - app/views/customers/list.rhtml
@@ -503,6 +503,8 @@ files:
 - app/views/user_notify/signup_en.rhtml
 - app/views/redirect.rjs
 - app/views/periods
+- app/views/periods/move_task_to_period.rjs
+- app/views/periods/finish_task.rjs
 - app/views/periods/show.rhtml
 - app/views/periods/_show_active.rhtml
 - app/views/periods/_form.rhtml
@@ -511,6 +513,7 @@ files:
 - app/views/periods/_title.rhtml
 - app/views/periods/_link.rhtml
 - app/views/periods/_burn_down_chart.rhtml
+- app/views/periods/reopen_task.rjs
 test_files: 
 - test/performance/test_threaded.rb
 - test/test_helper.rb

data/app/views/estimates/create_ajax.rjs

@@ -1,11 +0,0 @@
-page.replace_html :notice, @message
-page.visual_effect(:appear, :notice)
-if @success
-  if @task.completed?
-    page.visual_effect :fade, "task_#{@task.id}"
-    page.remove "task_#{@task.id}"
-    page.insert_html :bottom, :completed_tasks, :partial => '/tasks/task'
-    page.visual_effect :appear, "task_#{@task.id}"
-  end
-  update_burn_down_chart(page)
-end