Andreas/tracks
1
0
Fork 0
mirror of https://github.com/TracksApp/tracks.git synced 2026-03-14 00:16:15 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove extra whitespace

Browse source
This commit is contained in:
Matt Rogers 2013-04-26 22:30:40 -05:00
parent 036c34f266
commit f0d71bdfbb
1 changed files with 141 additions and 141 deletions
Download patch file Download diff file Expand all files Collapse all files

282
test/functional/todos_controller_test.rb
View file

@ -10,7 +10,7 @@ class TodosControllerTest < ActionController::TestCase
############################ ############################
# not done / deferred counts # not done / deferred counts
############################ ############################
def test_not_done_counts def test_not_done_counts
login_as(:admin_user) login_as(:admin_user)
get :index get :index
@ -26,7 +26,7 @@ class TodosControllerTest < ActionController::TestCase
assert_equal 3, assigns['context_not_done_counts'][contexts(:call).id] assert_equal 3, assigns['context_not_done_counts'][contexts(:call).id]
assert_equal 1, assigns['context_not_done_counts'][contexts(:lab).id] assert_equal 1, assigns['context_not_done_counts'][contexts(:lab).id]
end end
def test_cached_not_done_counts_after_hiding_project def test_cached_not_done_counts_after_hiding_project
p = Project.find(1) p = Project.find(1)
p.hide! p.hide!
@ -37,7 +37,7 @@ class TodosControllerTest < ActionController::TestCase
assert_equal 2, assigns['context_not_done_counts'][contexts(:call).id] assert_equal 2, assigns['context_not_done_counts'][contexts(:call).id]
assert_equal nil, assigns['context_not_done_counts'][contexts(:lab).id] assert_equal nil, assigns['context_not_done_counts'][contexts(:lab).id]
end end
def test_not_done_counts_after_hiding_project def test_not_done_counts_after_hiding_project
p = Project.find(1) p = Project.find(1)
p.hide! p.hide!
@ -48,7 +48,7 @@ class TodosControllerTest < ActionController::TestCase
assert_equal 2, contexts(:call).todos.active.count assert_equal 2, contexts(:call).todos.active.count
assert_equal 0, contexts(:lab).todos.active.count assert_equal 0, contexts(:lab).todos.active.count
end end
def test_not_done_counts_after_hiding_and_unhiding_project def test_not_done_counts_after_hiding_and_unhiding_project
p = Project.find(1) p = Project.find(1)
p.hide! p.hide!
@ -61,7 +61,7 @@ class TodosControllerTest < ActionController::TestCase
assert_equal 3, contexts(:call).todos.not_completed.count assert_equal 3, contexts(:call).todos.not_completed.count
assert_equal 1, contexts(:lab).todos.not_completed.count assert_equal 1, contexts(:lab).todos.not_completed.count
end end
def test_deferred_count_for_project_source_view def test_deferred_count_for_project_source_view
login_as(:admin_user) login_as(:admin_user)
xhr :post, :toggle_check, :id => 5, :_source_view => 'project' xhr :post, :toggle_check, :id => 5, :_source_view => 'project'
@ -69,11 +69,11 @@ class TodosControllerTest < ActionController::TestCase
xhr :post, :toggle_check, :id => 15, :_source_view => 'project' xhr :post, :toggle_check, :id => 15, :_source_view => 'project'
assert_equal 0, assigns['remaining_deferred_or_pending_count'] assert_equal 0, assigns['remaining_deferred_or_pending_count']
end end
######### #########
# tagging # tagging
######### #########
def test_tag_is_retrieved_properly def test_tag_is_retrieved_properly
login_as(:admin_user) login_as(:admin_user)
get :index get :index
@ -82,7 +82,7 @@ class TodosControllerTest < ActionController::TestCase
assert_equal 'foo', t.tags[0].name assert_equal 'foo', t.tags[0].name
assert !t.starred? assert !t.starred?
end end
def test_tagging_changes_to_tag_with_numbers def test_tagging_changes_to_tag_with_numbers
# by default has_many_polymorph searches for tags with given id if the tag is a number. we do not want that # by default has_many_polymorph searches for tags with given id if the tag is a number. we do not want that
login_as(:admin_user) login_as(:admin_user)
@ -96,7 +96,7 @@ class TodosControllerTest < ActionController::TestCase
assert_equal t.description, "test tags" assert_equal t.description, "test tags"
assert_equal 3, t.tags.count assert_equal 3, t.tags.count
end end
def test_tagging_changes_to_handle_empty_tags def test_tagging_changes_to_handle_empty_tags
# by default has_many_polymorph searches for tags with given id if the tag is a number. we do not want that # by default has_many_polymorph searches for tags with given id if the tag is a number. we do not want that
login_as(:admin_user) login_as(:admin_user)
@ -120,7 +120,7 @@ class TodosControllerTest < ActionController::TestCase
assert_response :success assert_response :success
assert_equal 3, @tagged assert_equal 3, @tagged
end end
def test_get_boolean_expression_from_parameters_of_tag_view_single_tag def test_get_boolean_expression_from_parameters_of_tag_view_single_tag
login_as(:admin_user) login_as(:admin_user)
get :tag, :name => "single" get :tag, :name => "single"
@ -128,21 +128,21 @@ class TodosControllerTest < ActionController::TestCase
assert_equal "single", assigns['tag_expr'][0][0], "should store the single tag" assert_equal "single", assigns['tag_expr'][0][0], "should store the single tag"
assert_equal "single", assigns['tag_name'], "should store the single tag name" assert_equal "single", assigns['tag_name'], "should store the single tag name"
end end
def test_get_boolean_expression_from_parameters_of_tag_view_multiple_tags def test_get_boolean_expression_from_parameters_of_tag_view_multiple_tags
login_as(:admin_user) login_as(:admin_user)
get :tag, :name => "multiple", :and => "tags", :and1 => "present", :and2 => "here" get :tag, :name => "multiple", :and => "tags", :and1 => "present", :and2 => "here"
assert_equal false, assigns['single_tag'], "should recognize it has multiple tags" assert_equal false, assigns['single_tag'], "should recognize it has multiple tags"
assert_equal 4, assigns['tag_expr'].size, "should have 4 AND expressions" assert_equal 4, assigns['tag_expr'].size, "should have 4 AND expressions"
end end
def test_get_boolean_expression_from_parameters_of_tag_view_multiple_tags_without_digitless_and def test_get_boolean_expression_from_parameters_of_tag_view_multiple_tags_without_digitless_and
login_as(:admin_user) login_as(:admin_user)
get :tag, :name => "multiple", :and1 => "tags", :and2 => "present", :and3 => "here" get :tag, :name => "multiple", :and1 => "tags", :and2 => "present", :and3 => "here"
assert_equal false, assigns['single_tag'], "should recognize it has multiple tags" assert_equal false, assigns['single_tag'], "should recognize it has multiple tags"
assert_equal 4, assigns['tag_expr'].size, "should have 4 AND expressions" assert_equal 4, assigns['tag_expr'].size, "should have 4 AND expressions"
end end
def test_get_boolean_expression_from_parameters_of_tag_view_multiple_ORs def test_get_boolean_expression_from_parameters_of_tag_view_multiple_ORs
login_as(:admin_user) login_as(:admin_user)
get :tag, :name => "multiple,tags,present" get :tag, :name => "multiple,tags,present"
@ -150,7 +150,7 @@ class TodosControllerTest < ActionController::TestCase
assert_equal 1, assigns['tag_expr'].size, "should have 1 expressions" assert_equal 1, assigns['tag_expr'].size, "should have 1 expressions"
assert_equal 3, assigns['tag_expr'][0].size, "should have 3 ORs in 1st expression" assert_equal 3, assigns['tag_expr'][0].size, "should have 3 ORs in 1st expression"
end end
def test_get_boolean_expression_from_parameters_of_tag_view_multiple_ORs_and_ANDS def test_get_boolean_expression_from_parameters_of_tag_view_multiple_ORs_and_ANDS
login_as(:admin_user) login_as(:admin_user)
get :tag, :name => "multiple,tags,present", :and => "here,is,two", :and1=>"and,three" get :tag, :name => "multiple,tags,present", :and => "here,is,two", :and1=>"and,three"
@ -160,36 +160,36 @@ class TodosControllerTest < ActionController::TestCase
assert_equal 3, assigns['tag_expr'][1].size, "should have 3 ORs in 2nd expression" assert_equal 3, assigns['tag_expr'][1].size, "should have 3 ORs in 2nd expression"
assert_equal 2, assigns['tag_expr'][2].size, "should have 2 ORs in 3rd expression" assert_equal 2, assigns['tag_expr'][2].size, "should have 2 ORs in 3rd expression"
end end
def test_set_right_title_tag_page def test_set_right_title_tag_page
login_as(:admin_user) login_as(:admin_user)
get :tag, :name => "foo" get :tag, :name => "foo"
assert_equal "foo", assigns['tag_title'] assert_equal "foo", assigns['tag_title']
get :tag, :name => "foo,bar", :and => "baz" get :tag, :name => "foo,bar", :and => "baz"
assert_equal "foo,bar AND baz", assigns['tag_title'] assert_equal "foo,bar AND baz", assigns['tag_title']
end end
def test_set_default_tag def test_set_default_tag
login_as(:admin_user) login_as(:admin_user)
get :tag, :name => "foo" get :tag, :name => "foo"
assert_equal "foo", assigns['initial_tags'] assert_equal "foo", assigns['initial_tags']
get :tag, :name => "foo,bar", :and => "baz" get :tag, :name => "foo,bar", :and => "baz"
assert_equal "foo", assigns['initial_tags'] assert_equal "foo", assigns['initial_tags']
end end
############### ###############
# creating todo # creating todo
############### ###############
def test_create_todo def test_create_todo
assert_difference 'Todo.count' do assert_difference 'Todo.count' do
login_as(:admin_user) login_as(:admin_user)
put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{"notes"=>"", "description"=>"Call Warren Buffet to find out how much he makes per day", "due"=>"30/11/2006"}, "tag_list"=>"foo bar" put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{"notes"=>"", "description"=>"Call Warren Buffet to find out how much he makes per day", "due"=>"30/11/2006"}, "tag_list"=>"foo bar"
end end
end end
def test_create_todo_via_xml def test_create_todo_via_xml
login_as(:admin_user) login_as(:admin_user)
assert_difference 'Todo.count' do assert_difference 'Todo.count' do
@ -197,7 +197,7 @@ class TodosControllerTest < ActionController::TestCase
assert_response 201 assert_response 201
end end
end end
def test_create_todo_via_xhr def test_create_todo_via_xhr
login_as(:admin_user) login_as(:admin_user)
assert_difference 'Todo.count' do assert_difference 'Todo.count' do
@ -205,7 +205,7 @@ class TodosControllerTest < ActionController::TestCase
assert_response 200 assert_response 200
end end
end end
def test_fail_to_create_todo_via_xml def test_fail_to_create_todo_via_xml
login_as(:admin_user) login_as(:admin_user)
# try to create with no context, which is not valid # try to create with no context, which is not valid
@ -217,78 +217,78 @@ class TodosControllerTest < ActionController::TestCase
assert_xml_select "error", "Context can't be blank" assert_xml_select "error", "Context can't be blank"
end end
end end
def test_create_deferred_todo def test_create_deferred_todo
original_todo_count = Todo.count original_todo_count = Todo.count
login_as(:admin_user) login_as(:admin_user)
put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{"notes"=>"", "description"=>"Call Warren Buffet to find out how much he makes per day", "due"=>"30/11/2026", 'show_from' => '30/10/2026'}, "tag_list"=>"foo bar" put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{"notes"=>"", "description"=>"Call Warren Buffet to find out how much he makes per day", "due"=>"30/11/2026", 'show_from' => '30/10/2026'}, "tag_list"=>"foo bar"
assert_equal original_todo_count + 1, Todo.count assert_equal original_todo_count + 1, Todo.count
end end
def test_add_multiple_todos def test_add_multiple_todos
login_as(:admin_user) login_as(:admin_user)
start_count = Todo.count start_count = Todo.count
put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{ put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{
:multiple_todos=>"a\nb\nmuch \"ado\" about \'nothing\'"} :multiple_todos=>"a\nb\nmuch \"ado\" about \'nothing\'"}
assert_equal start_count+3, Todo.count, "two todos should have been added" assert_equal start_count+3, Todo.count, "two todos should have been added"
end end
def test_add_multiple_todos_with_validation_error def test_add_multiple_todos_with_validation_error
login_as(:admin_user) login_as(:admin_user)
long_string = "a" * 500 long_string = "a" * 500
start_count = Todo.count start_count = Todo.count
put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{ put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{
:multiple_todos=>"a\nb\nmuch \"ado\" about \'nothing\'\n#{long_string}"} :multiple_todos=>"a\nb\nmuch \"ado\" about \'nothing\'\n#{long_string}"}
assert_equal start_count, Todo.count, "no todos should have been added" assert_equal start_count, Todo.count, "no todos should have been added"
end end
def test_add_multiple_dependent_todos def test_add_multiple_dependent_todos
login_as(:admin_user) login_as(:admin_user)
start_count = Todo.count start_count = Todo.count
put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{ put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{
:multiple_todos=>"a\nb"}, :todos_sequential => 'true' :multiple_todos=>"a\nb"}, :todos_sequential => 'true'
put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{ put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{
:multiple_todos=>"c\nd"}, :todos_sequential => 'false' :multiple_todos=>"c\nd"}, :todos_sequential => 'false'
assert_equal start_count+4, Todo.count, "four todos should have been added" assert_equal start_count+4, Todo.count, "four todos should have been added"
# find a,b,c and d # find a,b,c and d
%w{a b c d}.each do |todo| %w{a b c d}.each do |todo|
eval "@#{todo} = Todo.where(:description => '#{todo}').first" eval "@#{todo} = Todo.where(:description => '#{todo}').first"
eval "assert !@#{todo}.nil?, 'a todo with description \"#{todo}\" should just have been added'" eval "assert !@#{todo}.nil?, 'a todo with description \"#{todo}\" should just have been added'"
end end
assert @b.predecessors.include?(@a), "a should be a predeccesor of b" assert @b.predecessors.include?(@a), "a should be a predeccesor of b"
assert !@d.predecessors.include?(@c), "c should not be a predecessor of d" assert !@d.predecessors.include?(@c), "c should not be a predecessor of d"
end end
######### #########
# destroy # destroy
######### #########
def test_destroy_todo def test_destroy_todo
login_as(:admin_user) login_as(:admin_user)
xhr :post, :destroy, :id => 1, :_source_view => 'todo' xhr :post, :destroy, :id => 1, :_source_view => 'todo'
todo = Todo.where(:id=>1).first todo = Todo.where(:id=>1).first
assert_nil todo assert_nil todo
end end
############### ###############
# edit / update # edit / update
############### ###############
def test_get_edit_form_using_xhr def test_get_edit_form_using_xhr
login_as(:admin_user) login_as(:admin_user)
xhr :get, :edit, :id => todos(:call_bill).id xhr :get, :edit, :id => todos(:call_bill).id
assert_response 200 assert_response 200
end end
def test_update_todo_project def test_update_todo_project
t = Todo.find(1) t = Todo.find(1)
login_as(:admin_user) login_as(:admin_user)
@ -296,7 +296,7 @@ class TodosControllerTest < ActionController::TestCase
t = Todo.find(1) t = Todo.find(1)
assert_equal 1, t.project_id assert_equal 1, t.project_id
end end
def test_update_todo_project_to_none def test_update_todo_project_to_none
t = Todo.find(1) t = Todo.find(1)
login_as(:admin_user) login_as(:admin_user)
@ -304,7 +304,7 @@ class TodosControllerTest < ActionController::TestCase
t = Todo.find(1) t = Todo.find(1)
assert_nil t.project_id assert_nil t.project_id
end end
def test_update_todo_to_deferred_is_reflected_in_badge_count def test_update_todo_to_deferred_is_reflected_in_badge_count
login_as(:admin_user) login_as(:admin_user)
get :index get :index
@ -312,7 +312,7 @@ class TodosControllerTest < ActionController::TestCase
xhr :post, :update, :id => 1, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Make more money than Billy Gates", "todo"=>{"id"=>"1", "notes"=>"", "description"=>"Call Warren Buffet to find out how much he makes per day", "due"=>"30/11/2006", "show_from"=>"30/11/2030"}, "tag_list"=>"foo bar" xhr :post, :update, :id => 1, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Make more money than Billy Gates", "todo"=>{"id"=>"1", "notes"=>"", "description"=>"Call Warren Buffet to find out how much he makes per day", "due"=>"30/11/2006", "show_from"=>"30/11/2030"}, "tag_list"=>"foo bar"
assert_equal 10, assigns['down_count'] assert_equal 10, assigns['down_count']
end end
def test_update_todo def test_update_todo
t = Todo.find(1) t = Todo.find(1)
login_as(:admin_user) login_as(:admin_user)
@ -324,7 +324,7 @@ class TodosControllerTest < ActionController::TestCase
actual = t.due.to_date actual = t.due.to_date
assert_equal expected, actual, "Expected #{expected.to_s(:db)}, was #{actual.to_s(:db)}" assert_equal expected, actual, "Expected #{expected.to_s(:db)}, was #{actual.to_s(:db)}"
end end
def test_update_todos_with_blank_project_name def test_update_todos_with_blank_project_name
t = Todo.find(1) t = Todo.find(1)
login_as(:admin_user) login_as(:admin_user)
@ -332,7 +332,7 @@ class TodosControllerTest < ActionController::TestCase
t.reload t.reload
assert t.project.nil? assert t.project.nil?
end end
def test_update_todo_tags_to_none def test_update_todo_tags_to_none
t = Todo.find(1) t = Todo.find(1)
login_as(:admin_user) login_as(:admin_user)
@ -340,7 +340,7 @@ class TodosControllerTest < ActionController::TestCase
t = Todo.find(1) t = Todo.find(1)
assert_equal true, t.tag_list.empty? assert_equal true, t.tag_list.empty?
end end
def test_update_todo_tags_with_whitespace_and_dots def test_update_todo_tags_with_whitespace_and_dots
t = Todo.find(1) t = Todo.find(1)
login_as(:admin_user) login_as(:admin_user)
@ -349,21 +349,21 @@ class TodosControllerTest < ActionController::TestCase
t = Todo.find(1) t = Todo.find(1)
assert_equal "8.1.2, four, one, three, two, version1.5", t.tag_list assert_equal "8.1.2, four, one, three, two, version1.5", t.tag_list
end end
def test_removing_hidden_project_activates_todo def test_removing_hidden_project_activates_todo
login_as(:admin_user) login_as(:admin_user)
# get a project and hide it, todos in the project should be hidden # get a project and hide it, todos in the project should be hidden
p = projects(:timemachine) p = projects(:timemachine)
p.hide! p.hide!
assert p.reload().hidden? assert p.reload().hidden?
todo = p.todos.first todo = p.todos.first
assert todo.project_hidden?, "todo should be project_hidden" assert todo.project_hidden?, "todo should be project_hidden"
# clear project from todo: the todo should be unhidden # clear project from todo: the todo should be unhidden
xhr :post, :update, :id => todo.id, :_source_view => 'todo', "project_name"=>"None", "todo"=>{} xhr :post, :update, :id => todo.id, :_source_view => 'todo', "project_name"=>"None", "todo"=>{}
assert assigns['project_changed'], "the project of the todo should be changed" assert assigns['project_changed'], "the project of the todo should be changed"
assert todo.reload().active?, "todo should be active" assert todo.reload().active?, "todo should be active"
end end
@ -371,28 +371,28 @@ class TodosControllerTest < ActionController::TestCase
def test_change_context_of_todo def test_change_context_of_todo
# called by dragging a todo to another context container # called by dragging a todo to another context container
login_as(:admin_user) login_as(:admin_user)
todo = users(:admin_user).todos.active.first todo = users(:admin_user).todos.active.first
context = users(:admin_user).contexts.first context = users(:admin_user).contexts.first
assert_not_equal todo.context.id, context.id assert_not_equal todo.context.id, context.id
xhr :post, :change_context, :id => todo.id, :todo=>{:context_id => context.id}, :_source_view=>"todo" xhr :post, :change_context, :id => todo.id, :todo=>{:context_id => context.id}, :_source_view=>"todo"
assert assigns['context_changed'], "context should have changed" assert assigns['context_changed'], "context should have changed"
assert_equal todo.id, assigns['todo'].id, 'correct todo should have been found' assert_equal todo.id, assigns['todo'].id, 'correct todo should have been found'
assert_equal context.id, todo.reload.context.id, 'context of todo should be changed' assert_equal context.id, todo.reload.context.id, 'context of todo should be changed'
end end
####### #######
# feeds # feeds
####### #######
def test_rss_feed def test_rss_feed
login_as(:admin_user) login_as(:admin_user)
get :index, { :format => "rss" } get :index, { :format => "rss" }
assert_equal 'application/rss+xml', @response.content_type assert_equal 'application/rss+xml', @response.content_type
# puts @response.body # puts @response.body
assert_xml_select 'rss[version="2.0"]' do assert_xml_select 'rss[version="2.0"]' do
assert_select 'channel' do assert_select 'channel' do
assert_select '>title', 'Tracks Actions' assert_select '>title', 'Tracks Actions'
@ -409,11 +409,11 @@ class TodosControllerTest < ActionController::TestCase
end end
end end
end end
def test_rss_feed_with_limit def test_rss_feed_with_limit
login_as(:admin_user) login_as(:admin_user)
get :index, { :format => "rss", :limit => '5' } get :index, { :format => "rss", :limit => '5' }
assert_xml_select 'rss[version="2.0"]' do assert_xml_select 'rss[version="2.0"]' do
assert_select 'channel' do assert_select 'channel' do
assert_select '>title', 'Tracks Actions' assert_select '>title', 'Tracks Actions'
@ -425,31 +425,31 @@ class TodosControllerTest < ActionController::TestCase
end end
end end
end end
def test_rss_feed_not_accessible_to_anonymous_user_without_token def test_rss_feed_not_accessible_to_anonymous_user_without_token
login_as nil login_as nil
get :index, { :format => "rss" } get :index, { :format => "rss" }
assert_response 401 assert_response 401
end end
def test_rss_feed_not_accessible_to_anonymous_user_with_invalid_token def test_rss_feed_not_accessible_to_anonymous_user_with_invalid_token
login_as nil login_as nil
get :index, { :format => "rss", :token => 'foo' } get :index, { :format => "rss", :token => 'foo' }
assert_response 401 assert_response 401
end end
def test_rss_feed_accessible_to_anonymous_user_with_valid_token def test_rss_feed_accessible_to_anonymous_user_with_valid_token
login_as nil login_as nil
get :index, { :format => "rss", :token => users(:admin_user).token } get :index, { :format => "rss", :token => users(:admin_user).token }
assert_response :ok assert_response :ok
end end
def test_atom_feed_content def test_atom_feed_content
login_as :admin_user login_as :admin_user
get :index, { :format => "atom" } get :index, { :format => "atom" }
assert_equal 'application/atom+xml', @response.content_type assert_equal 'application/atom+xml', @response.content_type
# #puts @response.body # #puts @response.body
assert_xml_select 'feed[xmlns="http://www.w3.org/2005/Atom"]' do assert_xml_select 'feed[xmlns="http://www.w3.org/2005/Atom"]' do
assert_xml_select '>title', 'Tracks Actions' assert_xml_select '>title', 'Tracks Actions'
assert_xml_select '>subtitle', "Actions for #{users(:admin_user).display_name}" assert_xml_select '>subtitle', "Actions for #{users(:admin_user).display_name}"
@ -460,25 +460,25 @@ class TodosControllerTest < ActionController::TestCase
end end
end end
end end
def test_atom_feed_not_accessible_to_anonymous_user_without_token def test_atom_feed_not_accessible_to_anonymous_user_without_token
login_as nil login_as nil
get :index, { :format => "atom" } get :index, { :format => "atom" }
assert_response 401 assert_response 401
end end
def test_atom_feed_not_accessible_to_anonymous_user_with_invalid_token def test_atom_feed_not_accessible_to_anonymous_user_with_invalid_token
login_as nil login_as nil
get :index, { :format => "atom", :token => 'foo' } get :index, { :format => "atom", :token => 'foo' }
assert_response 401 assert_response 401
end end
def test_atom_feed_accessible_to_anonymous_user_with_valid_token def test_atom_feed_accessible_to_anonymous_user_with_valid_token
login_as nil login_as nil
get :index, { :format => "atom", :token => users(:admin_user).token } get :index, { :format => "atom", :token => users(:admin_user).token }
assert_response :ok assert_response :ok
end end
def test_text_feed_content def test_text_feed_content
login_as(:admin_user) login_as(:admin_user)
get :index, { :format => "txt" } get :index, { :format => "txt" }
@ -486,25 +486,25 @@ class TodosControllerTest < ActionController::TestCase
assert !(/&nbsp;/.match(@response.body)) assert !(/&nbsp;/.match(@response.body))
# #puts @response.body # #puts @response.body
end end
def test_text_feed_not_accessible_to_anonymous_user_without_token def test_text_feed_not_accessible_to_anonymous_user_without_token
login_as nil login_as nil
get :index, { :format => "txt" } get :index, { :format => "txt" }
assert_response 401 assert_response 401
end end
def test_text_feed_not_accessible_to_anonymous_user_with_invalid_token def test_text_feed_not_accessible_to_anonymous_user_with_invalid_token
login_as nil login_as nil
get :index, { :format => "txt", :token => 'foo' } get :index, { :format => "txt", :token => 'foo' }
assert_response 401 assert_response 401
end end
def test_text_feed_accessible_to_anonymous_user_with_valid_token def test_text_feed_accessible_to_anonymous_user_with_valid_token
login_as nil login_as nil
get :index, { :format => "txt", :token => users(:admin_user).token } get :index, { :format => "txt", :token => users(:admin_user).token }
assert_response :ok assert_response :ok
end end
def test_ical_feed_content def test_ical_feed_content
login_as :admin_user login_as :admin_user
get :index, { :format => "ics" } get :index, { :format => "ics" }
@ -512,38 +512,38 @@ class TodosControllerTest < ActionController::TestCase
assert !(/&nbsp;/.match(@response.body)) assert !(/&nbsp;/.match(@response.body))
# #puts @response.body # #puts @response.body
end end
def test_tag_text_feed_not_accessible_to_anonymous_user_without_token def test_tag_text_feed_not_accessible_to_anonymous_user_without_token
login_as nil login_as nil
get :tag, {:name => "foo", :format => "txt" } get :tag, {:name => "foo", :format => "txt" }
assert_response 401 assert_response 401
end end
############## ##############
# mobile index # mobile index
############## ##############
def test_mobile_index_uses_text_html_content_type def test_mobile_index_uses_text_html_content_type
login_as(:admin_user) login_as(:admin_user)
get :index, { :format => "m" } get :index, { :format => "m" }
assert_equal 'text/html', @response.content_type assert_equal 'text/html', @response.content_type
end end
def test_mobile_index_assigns_down_count def test_mobile_index_assigns_down_count
login_as(:admin_user) login_as(:admin_user)
get :index, { :format => "m" } get :index, { :format => "m" }
assert_equal 11, assigns['down_count'] assert_equal 11, assigns['down_count']
end end
def test_mobile_redirect_to_login def test_mobile_redirect_to_login
get :index, { :format => "m" } get :index, { :format => "m" }
assert_redirected_to login_url(:format => "m") assert_redirected_to login_url(:format => "m")
end end
############### ###############
# mobile create # mobile create
############### ###############
def test_mobile_create_action_creates_a_new_todo def test_mobile_create_action_creates_a_new_todo
login_as(:admin_user) login_as(:admin_user)
post :create, {"format"=>"m", "todo"=>{"context_id"=>"2", post :create, {"format"=>"m", "todo"=>{"context_id"=>"2",
@ -560,7 +560,7 @@ class TodosControllerTest < ActionController::TestCase
assert_nil t.show_from assert_nil t.show_from
assert_equal Date.new(2007,1,2), t.due.to_date assert_equal Date.new(2007,1,2), t.due.to_date
end end
def test_mobile_create_action_redirects_to_mobile_home_page_when_successful def test_mobile_create_action_redirects_to_mobile_home_page_when_successful
login_as(:admin_user) login_as(:admin_user)
post :create, {"format"=>"m", "todo"=>{"context_id"=>"2", post :create, {"format"=>"m", "todo"=>{"context_id"=>"2",
@ -570,7 +570,7 @@ class TodosControllerTest < ActionController::TestCase
"notes"=>"test notes", "description"=>"test_mobile_create_action", "state"=>"0"}} "notes"=>"test notes", "description"=>"test_mobile_create_action", "state"=>"0"}}
assert_redirected_to '/mobile' assert_redirected_to '/mobile'
end end
def test_mobile_create_action_renders_new_template_when_save_fails def test_mobile_create_action_renders_new_template_when_save_fails
login_as(:admin_user) login_as(:admin_user)
post :create, {"format"=>"m", "todo"=>{"context_id"=>"2", post :create, {"format"=>"m", "todo"=>{"context_id"=>"2",
@ -580,33 +580,33 @@ class TodosControllerTest < ActionController::TestCase
"notes"=>"test notes"}, "tag_list"=>"test, test2"} "notes"=>"test notes"}, "tag_list"=>"test, test2"}
assert_template 'todos/new' assert_template 'todos/new'
end end
################ ################
# recurring todo # recurring todo
################ ################
def test_toggle_check_on_recurring_todo def test_toggle_check_on_recurring_todo
login_as(:admin_user) login_as(:admin_user)
# link todo_1 and recurring_todo_1 # link todo_1 and recurring_todo_1
recurring_todo_1 = RecurringTodo.find(1) recurring_todo_1 = RecurringTodo.find(1)
todo_1 = Todo.where(:recurring_todo_id => 1).first todo_1 = Todo.where(:recurring_todo_id => 1).first
# mark todo_1 as complete by toggle_check # mark todo_1 as complete by toggle_check
xhr :post, :toggle_check, :id => todo_1.id, :_source_view => 'todo' xhr :post, :toggle_check, :id => todo_1.id, :_source_view => 'todo'
todo_1.reload todo_1.reload
assert todo_1.completed? assert todo_1.completed?
# check that there is only one active todo belonging to recurring_todo # check that there is only one active todo belonging to recurring_todo
count = Todo.count(:all, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'}) count = Todo.count(:all, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'})
assert_equal 1, count assert_equal 1, count
# check there is a new todo linked to the recurring pattern # check there is a new todo linked to the recurring pattern
next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'}) next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'})
assert_equal "Call Bill Gates every day", next_todo.description assert_equal "Call Bill Gates every day", next_todo.description
# check that the new todo is not the same as todo_1 # check that the new todo is not the same as todo_1
assert_not_equal todo_1.id, next_todo.id assert_not_equal todo_1.id, next_todo.id
# change recurrence pattern to monthly and set show_from 2 days before due # change recurrence pattern to monthly and set show_from 2 days before due
# date this forces the next todo to be put in the tickler # date this forces the next todo to be put in the tickler
recurring_todo_1.show_from_delta = 2 recurring_todo_1.show_from_delta = 2
@ -620,17 +620,17 @@ class TodosControllerTest < ActionController::TestCase
# use assert to catch validation errors if present. we need to replace # use assert to catch validation errors if present. we need to replace
# this with a good factory implementation # this with a good factory implementation
assert recurring_todo_1.save assert recurring_todo_1.save
# mark next_todo as complete by toggle_check # mark next_todo as complete by toggle_check
xhr :post, :toggle_check, :id => next_todo.id, :_source_view => 'todo' xhr :post, :toggle_check, :id => next_todo.id, :_source_view => 'todo'
next_todo.reload next_todo.reload
assert next_todo.completed? assert next_todo.completed?
# check that there are three todos belonging to recurring_todo: two # check that there are three todos belonging to recurring_todo: two
# completed and one deferred # completed and one deferred
count = Todo.count(:all, :conditions => {:recurring_todo_id => recurring_todo_1.id}) count = Todo.count(:all, :conditions => {:recurring_todo_id => recurring_todo_1.id})
assert_equal 3, count assert_equal 3, count
# check there is a new todo linked to the recurring pattern in the tickler # check there is a new todo linked to the recurring pattern in the tickler
next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'}) next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'})
assert !next_todo.nil? assert !next_todo.nil?
@ -638,16 +638,16 @@ class TodosControllerTest < ActionController::TestCase
# check that the todo is in the tickler # check that the todo is in the tickler
assert !next_todo.show_from.nil? assert !next_todo.show_from.nil?
end end
def test_toggle_check_on_rec_todo_show_from_today def test_toggle_check_on_rec_todo_show_from_today
login_as(:admin_user) login_as(:admin_user)
# link todo_1 and recurring_todo_1 # link todo_1 and recurring_todo_1
recurring_todo_1 = RecurringTodo.find(1) recurring_todo_1 = RecurringTodo.find(1)
#set_user_to_current_time_zone(recurring_todo_1.user) #set_user_to_current_time_zone(recurring_todo_1.user)
todo_1 = Todo.where(:recurring_todo_id => 1).first todo_1 = Todo.where(:recurring_todo_id => 1).first
today = Time.zone.now.at_midnight today = Time.zone.now.at_midnight
# change recurrence pattern to monthly and set show_from to today # change recurrence pattern to monthly and set show_from to today
recurring_todo_1.target = 'show_from_date' recurring_todo_1.target = 'show_from_date'
recurring_todo_1.recurring_period = 'monthly' recurring_todo_1.recurring_period = 'monthly'
@ -655,66 +655,66 @@ class TodosControllerTest < ActionController::TestCase
recurring_todo_1.every_other1 = today.day recurring_todo_1.every_other1 = today.day
recurring_todo_1.every_other2 = 1 recurring_todo_1.every_other2 = 1
assert recurring_todo_1.save assert recurring_todo_1.save
# mark todo_1 as complete by toggle_check, this gets rid of todo_1 that was # mark todo_1 as complete by toggle_check, this gets rid of todo_1 that was
# not correctly created from the adjusted recurring pattern we defined # not correctly created from the adjusted recurring pattern we defined
# above. # above.
xhr :post, :toggle_check, :id => todo_1.id, :_source_view => 'todo' xhr :post, :toggle_check, :id => todo_1.id, :_source_view => 'todo'
todo_1.reload todo_1.reload
assert todo_1.completed? assert todo_1.completed?
# locate the new todo. This todo is created from the adjusted recurring # locate the new todo. This todo is created from the adjusted recurring
# pattern defined in this test # pattern defined in this test
new_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'}) new_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'})
assert !new_todo.nil? assert !new_todo.nil?
# mark new_todo as complete by toggle_check # mark new_todo as complete by toggle_check
xhr :post, :toggle_check, :id => new_todo.id, :_source_view => 'todo' xhr :post, :toggle_check, :id => new_todo.id, :_source_view => 'todo'
new_todo.reload new_todo.reload
assert todo_1.completed? assert todo_1.completed?
# locate the new todo in tickler # locate the new todo in tickler
new_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'}) new_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'})
assert !new_todo.nil? assert !new_todo.nil?
assert_equal "Call Bill Gates every day", new_todo.description assert_equal "Call Bill Gates every day", new_todo.description
# check that the new todo is not the same as todo_1 # check that the new todo is not the same as todo_1
assert_not_equal todo_1.id, new_todo.id assert_not_equal todo_1.id, new_todo.id
# check that the new_todo is in the tickler to show next month # check that the new_todo is in the tickler to show next month
assert !new_todo.show_from.nil? assert !new_todo.show_from.nil?
# do not use today here. It somehow gets messed up with the timezone calculation. # do not use today here. It somehow gets messed up with the timezone calculation.
next_month = (Time.zone.now + 1.month).at_midnight next_month = (Time.zone.now + 1.month).at_midnight
assert_equal next_month.utc.to_date.to_s(:db), new_todo.show_from.utc.to_date.to_s(:db) assert_equal next_month.utc.to_date.to_s(:db), new_todo.show_from.utc.to_date.to_s(:db)
end end
def test_check_for_next_todo def test_check_for_next_todo
login_as :admin_user login_as :admin_user
recurring_todo_1 = RecurringTodo.find(5) recurring_todo_1 = RecurringTodo.find(5)
@todo = Todo.where(:recurring_todo_id => 1).first @todo = Todo.where(:recurring_todo_id => 1).first
assert @todo.from_recurring_todo? assert @todo.from_recurring_todo?
# rewire @todo to yearly recurring todo # rewire @todo to yearly recurring todo
@todo.recurring_todo_id = 5 @todo.recurring_todo_id = 5
# make todo due tomorrow and change recurring date also to tomorrow # make todo due tomorrow and change recurring date also to tomorrow
@todo.due = Time.zone.now + 1.day @todo.due = Time.zone.now + 1.day
@todo.save @todo.save
recurring_todo_1.every_other1 = @todo.due.day recurring_todo_1.every_other1 = @todo.due.day
recurring_todo_1.every_other2 = @todo.due.month recurring_todo_1.every_other2 = @todo.due.month
recurring_todo_1.save recurring_todo_1.save
# mark todo complete # mark todo complete
xhr :post, :toggle_check, :id => @todo.id, :_source_view => 'todo' xhr :post, :toggle_check, :id => @todo.id, :_source_view => 'todo'
@todo.reload @todo.reload
assert @todo.completed? assert @todo.completed?
# check that there is no active todo # check that there is no active todo
next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'}) next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'})
assert next_todo.nil? assert next_todo.nil?
# check for new deferred todo # check for new deferred todo
next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'}) next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'})
assert !next_todo.nil? assert !next_todo.nil?
@ -725,7 +725,7 @@ class TodosControllerTest < ActionController::TestCase
############ ############
# todo notes # todo notes
############ ############
def test_url_with_slash_in_query_string_are_parsed_correctly def test_url_with_slash_in_query_string_are_parsed_correctly
# See http://blog.swivel.com/code/2009/06/rails-auto_link-and-certain-query-strings.html # See http://blog.swivel.com/code/2009/06/rails-auto_link-and-certain-query-strings.html
login_as(:admin_user) login_as(:admin_user)
@ -736,7 +736,7 @@ class TodosControllerTest < ActionController::TestCase
get :index get :index
assert_select("a[href=#{url}]") assert_select("a[href=#{url}]")
end end
def test_format_note_normal def test_format_note_normal
login_as(:admin_user) login_as(:admin_user)
todo = users(:admin_user).todos.first todo = users(:admin_user).todos.first
@ -745,7 +745,7 @@ class TodosControllerTest < ActionController::TestCase
get :index get :index
assert_select("div#notes_todo_#{todo.id}", "A normal description.") assert_select("div#notes_todo_#{todo.id}", "A normal description.")
end end
def test_format_note_textile def test_format_note_textile
login_as(:admin_user) login_as(:admin_user)
todo = users(:admin_user).todos.first todo = users(:admin_user).todos.first
@ -755,7 +755,7 @@ class TodosControllerTest < ActionController::TestCase
assert_select("div#notes_todo_#{todo.id}", "A bold description.") assert_select("div#notes_todo_#{todo.id}", "A bold description.")
assert_select("div#notes_todo_#{todo.id} strong", "bold description") assert_select("div#notes_todo_#{todo.id} strong", "bold description")
end end
def test_format_note_link def test_format_note_link
login_as(:admin_user) login_as(:admin_user)
todo = users(:admin_user).todos.first todo = users(:admin_user).todos.first
@ -765,7 +765,7 @@ class TodosControllerTest < ActionController::TestCase
assert_select("div#notes_todo_#{todo.id}", 'A link to http://github.com/.') assert_select("div#notes_todo_#{todo.id}", 'A link to http://github.com/.')
assert_select("div#notes_todo_#{todo.id} a[href=http://github.com/]", 'http://github.com/') assert_select("div#notes_todo_#{todo.id} a[href=http://github.com/]", 'http://github.com/')
end end
def test_format_note_link_message def test_format_note_link_message
login_as(:admin_user) login_as(:admin_user)
todo = users(:admin_user).todos.first todo = users(:admin_user).todos.first
@ -776,7 +776,7 @@ class TodosControllerTest < ActionController::TestCase
assert_select("div#notes_todo_#{todo.id} a", 'message://&lt;ABCDEF-GHADB-123455-FOO-BAR@example.com&gt;') assert_select("div#notes_todo_#{todo.id} a", 'message://&lt;ABCDEF-GHADB-123455-FOO-BAR@example.com&gt;')
assert_select("div#notes_todo_#{todo.id} a[href=message://&lt;ABCDEF-GHADB-123455-FOO-BAR@example.com&gt;]", 'message://&lt;ABCDEF-GHADB-123455-FOO-BAR@example.com&gt;') assert_select("div#notes_todo_#{todo.id} a[href=message://&lt;ABCDEF-GHADB-123455-FOO-BAR@example.com&gt;]", 'message://&lt;ABCDEF-GHADB-123455-FOO-BAR@example.com&gt;')
end end
def test_format_note_link_onenote def test_format_note_link_onenote
login_as(:admin_user) login_as(:admin_user)
todo = users(:admin_user).todos.first todo = users(:admin_user).todos.first
@ -791,111 +791,111 @@ class TodosControllerTest < ActionController::TestCase
############## ##############
# dependencies # dependencies
############## ##############
def test_make_todo_dependent def test_make_todo_dependent
login_as(:admin_user) login_as(:admin_user)
predecessor = todos(:call_bill) predecessor = todos(:call_bill)
successor = todos(:call_dino_ext) successor = todos(:call_dino_ext)
# no predecessors yet # no predecessors yet
assert_equal 0, successor.predecessors.size assert_equal 0, successor.predecessors.size
# add predecessor # add predecessor
put :add_predecessor, :predecessor=>predecessor.id, :successor=>successor.id put :add_predecessor, :predecessor=>predecessor.id, :successor=>successor.id
assert_equal 1, successor.predecessors.count assert_equal 1, successor.predecessors.count
assert_equal predecessor.id, successor.predecessors.first.id assert_equal predecessor.id, successor.predecessors.first.id
end end
def test_make_todo_with_dependencies_dependent def test_make_todo_with_dependencies_dependent
login_as(:admin_user) login_as(:admin_user)
predecessor = todos(:call_bill) predecessor = todos(:call_bill)
successor = todos(:call_dino_ext) successor = todos(:call_dino_ext)
other_todo = todos(:phone_grandfather) other_todo = todos(:phone_grandfather)
# predecessor -> successor # predecessor -> successor
put :add_predecessor, :predecessor=>predecessor.id, :successor=>successor.id put :add_predecessor, :predecessor=>predecessor.id, :successor=>successor.id
# other_todo -> predecessor -> successor # other_todo -> predecessor -> successor
put :add_predecessor, :predecessor=>other_todo.id, :successor=>predecessor.id put :add_predecessor, :predecessor=>other_todo.id, :successor=>predecessor.id
assert_equal 1, successor.predecessors(true).count assert_equal 1, successor.predecessors(true).count
assert_equal 0, other_todo.predecessors(true).count assert_equal 0, other_todo.predecessors(true).count
assert_equal 1, predecessor.predecessors(true).count assert_equal 1, predecessor.predecessors(true).count
assert_equal predecessor.id, successor.predecessors.first.id assert_equal predecessor.id, successor.predecessors.first.id
assert_equal other_todo.id, predecessor.predecessors.first.id assert_equal other_todo.id, predecessor.predecessors.first.id
end end
def test_mingle_dependent_todos_leave def test_mingle_dependent_todos_leave
# based on #1271 # based on #1271
login_as(:admin_user) login_as(:admin_user)
t1 = todos(:call_bill) t1 = todos(:call_bill)
t2 = todos(:call_dino_ext) t2 = todos(:call_dino_ext)
t3 = todos(:phone_grandfather) t3 = todos(:phone_grandfather)
t4 = todos(:construct_dilation_device) t4 = todos(:construct_dilation_device)
# t1 -> t2 # t1 -> t2
put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id
# t3 -> t4 # t3 -> t4
put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id
# t2 -> t4 # t2 -> t4
put :add_predecessor, :predecessor=>t2.id, :successor=>t4.id put :add_predecessor, :predecessor=>t2.id, :successor=>t4.id
# should be: t1 -> t2 -> t4 and t3 -> t4 # should be: t1 -> t2 -> t4 and t3 -> t4
assert t4.predecessors.map(&:id).include?(t2.id) assert t4.predecessors.map(&:id).include?(t2.id)
assert t4.predecessors.map(&:id).include?(t3.id) assert t4.predecessors.map(&:id).include?(t3.id)
assert t2.predecessors.map(&:id).include?(t1.id) assert t2.predecessors.map(&:id).include?(t1.id)
end end
def test_mingle_dependent_todos_root def test_mingle_dependent_todos_root
# based on #1271 # based on #1271
login_as(:admin_user) login_as(:admin_user)
t1 = todos(:call_bill) t1 = todos(:call_bill)
t2 = todos(:call_dino_ext) t2 = todos(:call_dino_ext)
t3 = todos(:phone_grandfather) t3 = todos(:phone_grandfather)
t4 = todos(:construct_dilation_device) t4 = todos(:construct_dilation_device)
# t1 -> t2 # t1 -> t2
put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id
# t3 -> t4 # t3 -> t4
put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id
# t3 -> t2 # t3 -> t2
put :add_predecessor, :predecessor=>t3.id, :successor=>t2.id put :add_predecessor, :predecessor=>t3.id, :successor=>t2.id
# should be: t1 -> t2 and t3 -> t4 & t2 # should be: t1 -> t2 and t3 -> t4 & t2
assert t3.successors.map(&:id).include?(t4.id) assert t3.successors.map(&:id).include?(t4.id)
assert t3.successors.map(&:id).include?(t2.id) assert t3.successors.map(&:id).include?(t2.id)
assert t2.predecessors.map(&:id).include?(t1.id) assert t2.predecessors.map(&:id).include?(t1.id)
assert t2.predecessors.map(&:id).include?(t3.id) assert t2.predecessors.map(&:id).include?(t3.id)
end end
def test_unmingle_dependent_todos def test_unmingle_dependent_todos
# based on #1271 # based on #1271
login_as(:admin_user) login_as(:admin_user)
t1 = todos(:call_bill) t1 = todos(:call_bill)
t2 = todos(:call_dino_ext) t2 = todos(:call_dino_ext)
t3 = todos(:phone_grandfather) t3 = todos(:phone_grandfather)
t4 = todos(:construct_dilation_device) t4 = todos(:construct_dilation_device)
# create same dependency tree as previous test # create same dependency tree as previous test
# should be: t1 -> t2 -> t4 and t3 -> t4 # should be: t1 -> t2 -> t4 and t3 -> t4
put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id
put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id
put :add_predecessor, :predecessor=>t2.id, :successor=>t4.id put :add_predecessor, :predecessor=>t2.id, :successor=>t4.id
# removing t4 as successor of t2 should leave t4 blocked with t3 as predecessor # removing t4 as successor of t2 should leave t4 blocked with t3 as predecessor
put :remove_predecessor, :predecessor=>t2.id, :id=>t4.id put :remove_predecessor, :predecessor=>t2.id, :id=>t4.id
t4.reload t4.reload
assert t4.pending?, "t4 should remain pending" assert t4.pending?, "t4 should remain pending"
assert t4.predecessors.map(&:id).include?(t3.id) assert t4.predecessors.map(&:id).include?(t3.id)
end end
end end