From f0d71bdfbb29a3ef7e5ee69e0b3adf1847586466 Mon Sep 17 00:00:00 2001 From: Matt Rogers Date: Fri, 26 Apr 2013 22:30:40 -0500 Subject: [PATCH] Remove extra whitespace --- test/functional/todos_controller_test.rb | 282 +++++++++++------------ 1 file changed, 141 insertions(+), 141 deletions(-) diff --git a/test/functional/todos_controller_test.rb b/test/functional/todos_controller_test.rb index 2f118357..1a3f92fc 100644 --- a/test/functional/todos_controller_test.rb +++ b/test/functional/todos_controller_test.rb @@ -10,7 +10,7 @@ class TodosControllerTest < ActionController::TestCase ############################ # not done / deferred counts ############################ - + def test_not_done_counts login_as(:admin_user) get :index @@ -26,7 +26,7 @@ class TodosControllerTest < ActionController::TestCase assert_equal 3, assigns['context_not_done_counts'][contexts(:call).id] assert_equal 1, assigns['context_not_done_counts'][contexts(:lab).id] end - + def test_cached_not_done_counts_after_hiding_project p = Project.find(1) p.hide! @@ -37,7 +37,7 @@ class TodosControllerTest < ActionController::TestCase assert_equal 2, assigns['context_not_done_counts'][contexts(:call).id] assert_equal nil, assigns['context_not_done_counts'][contexts(:lab).id] end - + def test_not_done_counts_after_hiding_project p = Project.find(1) p.hide! @@ -48,7 +48,7 @@ class TodosControllerTest < ActionController::TestCase assert_equal 2, contexts(:call).todos.active.count assert_equal 0, contexts(:lab).todos.active.count end - + def test_not_done_counts_after_hiding_and_unhiding_project p = Project.find(1) p.hide! @@ -61,7 +61,7 @@ class TodosControllerTest < ActionController::TestCase assert_equal 3, contexts(:call).todos.not_completed.count assert_equal 1, contexts(:lab).todos.not_completed.count end - + def test_deferred_count_for_project_source_view login_as(:admin_user) 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' assert_equal 0, assigns['remaining_deferred_or_pending_count'] end - + ######### # tagging ######### - + def test_tag_is_retrieved_properly login_as(:admin_user) get :index @@ -82,7 +82,7 @@ class TodosControllerTest < ActionController::TestCase assert_equal 'foo', t.tags[0].name assert !t.starred? end - + 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 login_as(:admin_user) @@ -96,7 +96,7 @@ class TodosControllerTest < ActionController::TestCase assert_equal t.description, "test tags" assert_equal 3, t.tags.count end - + 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 login_as(:admin_user) @@ -120,7 +120,7 @@ class TodosControllerTest < ActionController::TestCase assert_response :success assert_equal 3, @tagged end - + def test_get_boolean_expression_from_parameters_of_tag_view_single_tag login_as(:admin_user) 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_name'], "should store the single tag name" end - + def test_get_boolean_expression_from_parameters_of_tag_view_multiple_tags login_as(:admin_user) get :tag, :name => "multiple", :and => "tags", :and1 => "present", :and2 => "here" assert_equal false, assigns['single_tag'], "should recognize it has multiple tags" assert_equal 4, assigns['tag_expr'].size, "should have 4 AND expressions" end - + def test_get_boolean_expression_from_parameters_of_tag_view_multiple_tags_without_digitless_and login_as(:admin_user) get :tag, :name => "multiple", :and1 => "tags", :and2 => "present", :and3 => "here" assert_equal false, assigns['single_tag'], "should recognize it has multiple tags" assert_equal 4, assigns['tag_expr'].size, "should have 4 AND expressions" end - + def test_get_boolean_expression_from_parameters_of_tag_view_multiple_ORs login_as(:admin_user) 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 3, assigns['tag_expr'][0].size, "should have 3 ORs in 1st expression" end - + def test_get_boolean_expression_from_parameters_of_tag_view_multiple_ORs_and_ANDS login_as(:admin_user) 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 2, assigns['tag_expr'][2].size, "should have 2 ORs in 3rd expression" end - + def test_set_right_title_tag_page login_as(:admin_user) - + get :tag, :name => "foo" assert_equal "foo", assigns['tag_title'] get :tag, :name => "foo,bar", :and => "baz" assert_equal "foo,bar AND baz", assigns['tag_title'] end - + def test_set_default_tag login_as(:admin_user) - + get :tag, :name => "foo" assert_equal "foo", assigns['initial_tags'] get :tag, :name => "foo,bar", :and => "baz" assert_equal "foo", assigns['initial_tags'] end - + ############### # creating todo ############### - + def test_create_todo assert_difference 'Todo.count' do 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" end end - + def test_create_todo_via_xml login_as(:admin_user) assert_difference 'Todo.count' do @@ -197,7 +197,7 @@ class TodosControllerTest < ActionController::TestCase assert_response 201 end end - + def test_create_todo_via_xhr login_as(:admin_user) assert_difference 'Todo.count' do @@ -205,7 +205,7 @@ class TodosControllerTest < ActionController::TestCase assert_response 200 end end - + def test_fail_to_create_todo_via_xml login_as(:admin_user) # 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" end end - + def test_create_deferred_todo original_todo_count = Todo.count 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" assert_equal original_todo_count + 1, Todo.count end - + def test_add_multiple_todos login_as(:admin_user) - + start_count = Todo.count put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{ :multiple_todos=>"a\nb\nmuch \"ado\" about \'nothing\'"} - + assert_equal start_count+3, Todo.count, "two todos should have been added" end - + def test_add_multiple_todos_with_validation_error login_as(:admin_user) - + long_string = "a" * 500 - + start_count = Todo.count 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}"} - + assert_equal start_count, Todo.count, "no todos should have been added" end - + def test_add_multiple_dependent_todos login_as(:admin_user) - + start_count = Todo.count put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{ :multiple_todos=>"a\nb"}, :todos_sequential => 'true' put :create, :_source_view => 'todo', "context_name"=>"library", "project_name"=>"Build a working time machine", "todo"=>{ :multiple_todos=>"c\nd"}, :todos_sequential => 'false' - + assert_equal start_count+4, Todo.count, "four todos should have been added" - + # find a,b,c and d %w{a b c d}.each do |todo| eval "@#{todo} = Todo.where(:description => '#{todo}').first" eval "assert !@#{todo}.nil?, 'a todo with description \"#{todo}\" should just have been added'" end - + 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" end - + ######### # destroy ######### - + def test_destroy_todo login_as(:admin_user) xhr :post, :destroy, :id => 1, :_source_view => 'todo' todo = Todo.where(:id=>1).first assert_nil todo end - + ############### # edit / update ############### - + def test_get_edit_form_using_xhr login_as(:admin_user) xhr :get, :edit, :id => todos(:call_bill).id assert_response 200 end - + def test_update_todo_project t = Todo.find(1) login_as(:admin_user) @@ -296,7 +296,7 @@ class TodosControllerTest < ActionController::TestCase t = Todo.find(1) assert_equal 1, t.project_id end - + def test_update_todo_project_to_none t = Todo.find(1) login_as(:admin_user) @@ -304,7 +304,7 @@ class TodosControllerTest < ActionController::TestCase t = Todo.find(1) assert_nil t.project_id end - + def test_update_todo_to_deferred_is_reflected_in_badge_count login_as(:admin_user) 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" assert_equal 10, assigns['down_count'] end - + def test_update_todo t = Todo.find(1) login_as(:admin_user) @@ -324,7 +324,7 @@ class TodosControllerTest < ActionController::TestCase actual = t.due.to_date assert_equal expected, actual, "Expected #{expected.to_s(:db)}, was #{actual.to_s(:db)}" end - + def test_update_todos_with_blank_project_name t = Todo.find(1) login_as(:admin_user) @@ -332,7 +332,7 @@ class TodosControllerTest < ActionController::TestCase t.reload assert t.project.nil? end - + def test_update_todo_tags_to_none t = Todo.find(1) login_as(:admin_user) @@ -340,7 +340,7 @@ class TodosControllerTest < ActionController::TestCase t = Todo.find(1) assert_equal true, t.tag_list.empty? end - + def test_update_todo_tags_with_whitespace_and_dots t = Todo.find(1) login_as(:admin_user) @@ -349,21 +349,21 @@ class TodosControllerTest < ActionController::TestCase t = Todo.find(1) assert_equal "8.1.2, four, one, three, two, version1.5", t.tag_list end - + def test_removing_hidden_project_activates_todo login_as(:admin_user) - + # get a project and hide it, todos in the project should be hidden p = projects(:timemachine) p.hide! assert p.reload().hidden? todo = p.todos.first - + assert todo.project_hidden?, "todo should be project_hidden" - + # clear project from todo: the todo should be unhidden 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 todo.reload().active?, "todo should be active" end @@ -371,28 +371,28 @@ class TodosControllerTest < ActionController::TestCase def test_change_context_of_todo # called by dragging a todo to another context container login_as(:admin_user) - + todo = users(:admin_user).todos.active.first context = users(:admin_user).contexts.first - + assert_not_equal todo.context.id, context.id - + xhr :post, :change_context, :id => todo.id, :todo=>{:context_id => context.id}, :_source_view=>"todo" assert assigns['context_changed'], "context should have changed" 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' end - + ####### # feeds ####### - + def test_rss_feed login_as(:admin_user) get :index, { :format => "rss" } assert_equal 'application/rss+xml', @response.content_type # puts @response.body - + assert_xml_select 'rss[version="2.0"]' do assert_select 'channel' do assert_select '>title', 'Tracks Actions' @@ -409,11 +409,11 @@ class TodosControllerTest < ActionController::TestCase end end end - + def test_rss_feed_with_limit login_as(:admin_user) get :index, { :format => "rss", :limit => '5' } - + assert_xml_select 'rss[version="2.0"]' do assert_select 'channel' do assert_select '>title', 'Tracks Actions' @@ -425,31 +425,31 @@ class TodosControllerTest < ActionController::TestCase end end end - + def test_rss_feed_not_accessible_to_anonymous_user_without_token login_as nil get :index, { :format => "rss" } assert_response 401 end - + def test_rss_feed_not_accessible_to_anonymous_user_with_invalid_token login_as nil get :index, { :format => "rss", :token => 'foo' } assert_response 401 end - + def test_rss_feed_accessible_to_anonymous_user_with_valid_token login_as nil get :index, { :format => "rss", :token => users(:admin_user).token } assert_response :ok end - + def test_atom_feed_content login_as :admin_user get :index, { :format => "atom" } assert_equal 'application/atom+xml', @response.content_type # #puts @response.body - + assert_xml_select 'feed[xmlns="http://www.w3.org/2005/Atom"]' do assert_xml_select '>title', 'Tracks Actions' assert_xml_select '>subtitle', "Actions for #{users(:admin_user).display_name}" @@ -460,25 +460,25 @@ class TodosControllerTest < ActionController::TestCase end end end - + def test_atom_feed_not_accessible_to_anonymous_user_without_token login_as nil get :index, { :format => "atom" } assert_response 401 end - + def test_atom_feed_not_accessible_to_anonymous_user_with_invalid_token login_as nil get :index, { :format => "atom", :token => 'foo' } assert_response 401 end - + def test_atom_feed_accessible_to_anonymous_user_with_valid_token login_as nil get :index, { :format => "atom", :token => users(:admin_user).token } assert_response :ok end - + def test_text_feed_content login_as(:admin_user) get :index, { :format => "txt" } @@ -486,25 +486,25 @@ class TodosControllerTest < ActionController::TestCase assert !(/ /.match(@response.body)) # #puts @response.body end - + def test_text_feed_not_accessible_to_anonymous_user_without_token login_as nil get :index, { :format => "txt" } assert_response 401 end - + def test_text_feed_not_accessible_to_anonymous_user_with_invalid_token login_as nil get :index, { :format => "txt", :token => 'foo' } assert_response 401 end - + def test_text_feed_accessible_to_anonymous_user_with_valid_token login_as nil get :index, { :format => "txt", :token => users(:admin_user).token } assert_response :ok end - + def test_ical_feed_content login_as :admin_user get :index, { :format => "ics" } @@ -512,38 +512,38 @@ class TodosControllerTest < ActionController::TestCase assert !(/ /.match(@response.body)) # #puts @response.body end - + def test_tag_text_feed_not_accessible_to_anonymous_user_without_token login_as nil get :tag, {:name => "foo", :format => "txt" } assert_response 401 end - + ############## # mobile index ############## - + def test_mobile_index_uses_text_html_content_type login_as(:admin_user) get :index, { :format => "m" } assert_equal 'text/html', @response.content_type end - + def test_mobile_index_assigns_down_count login_as(:admin_user) get :index, { :format => "m" } assert_equal 11, assigns['down_count'] end - + def test_mobile_redirect_to_login get :index, { :format => "m" } assert_redirected_to login_url(:format => "m") end - + ############### # mobile create ############### - + def test_mobile_create_action_creates_a_new_todo login_as(:admin_user) post :create, {"format"=>"m", "todo"=>{"context_id"=>"2", @@ -560,7 +560,7 @@ class TodosControllerTest < ActionController::TestCase assert_nil t.show_from assert_equal Date.new(2007,1,2), t.due.to_date end - + def test_mobile_create_action_redirects_to_mobile_home_page_when_successful login_as(:admin_user) 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"}} assert_redirected_to '/mobile' end - + def test_mobile_create_action_renders_new_template_when_save_fails login_as(:admin_user) post :create, {"format"=>"m", "todo"=>{"context_id"=>"2", @@ -580,33 +580,33 @@ class TodosControllerTest < ActionController::TestCase "notes"=>"test notes"}, "tag_list"=>"test, test2"} assert_template 'todos/new' end - + ################ # recurring todo ################ - + def test_toggle_check_on_recurring_todo login_as(:admin_user) - + # link todo_1 and recurring_todo_1 recurring_todo_1 = RecurringTodo.find(1) todo_1 = Todo.where(:recurring_todo_id => 1).first - + # mark todo_1 as complete by toggle_check xhr :post, :toggle_check, :id => todo_1.id, :_source_view => 'todo' todo_1.reload assert todo_1.completed? - + # 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'}) assert_equal 1, count - + # 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'}) assert_equal "Call Bill Gates every day", next_todo.description # check that the new todo is not the same as todo_1 assert_not_equal todo_1.id, next_todo.id - + # 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 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 # this with a good factory implementation assert recurring_todo_1.save - + # mark next_todo as complete by toggle_check xhr :post, :toggle_check, :id => next_todo.id, :_source_view => 'todo' next_todo.reload assert next_todo.completed? - + # check that there are three todos belonging to recurring_todo: two # completed and one deferred count = Todo.count(:all, :conditions => {:recurring_todo_id => recurring_todo_1.id}) assert_equal 3, count - + # 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'}) assert !next_todo.nil? @@ -638,16 +638,16 @@ class TodosControllerTest < ActionController::TestCase # check that the todo is in the tickler assert !next_todo.show_from.nil? end - + def test_toggle_check_on_rec_todo_show_from_today login_as(:admin_user) - + # link todo_1 and recurring_todo_1 recurring_todo_1 = RecurringTodo.find(1) #set_user_to_current_time_zone(recurring_todo_1.user) todo_1 = Todo.where(:recurring_todo_id => 1).first today = Time.zone.now.at_midnight - + # change recurrence pattern to monthly and set show_from to today recurring_todo_1.target = 'show_from_date' 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_other2 = 1 assert recurring_todo_1.save - + # 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 # above. xhr :post, :toggle_check, :id => todo_1.id, :_source_view => 'todo' todo_1.reload assert todo_1.completed? - + # locate the new todo. This todo is created from the adjusted recurring # pattern defined in this test new_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'}) assert !new_todo.nil? - + # mark new_todo as complete by toggle_check xhr :post, :toggle_check, :id => new_todo.id, :_source_view => 'todo' new_todo.reload assert todo_1.completed? - + # locate the new todo in tickler new_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'}) assert !new_todo.nil? - + assert_equal "Call Bill Gates every day", new_todo.description # check that the new todo is not the same as todo_1 assert_not_equal todo_1.id, new_todo.id - + # check that the new_todo is in the tickler to show next month assert !new_todo.show_from.nil? - + # do not use today here. It somehow gets messed up with the timezone calculation. 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) end - + def test_check_for_next_todo login_as :admin_user - + recurring_todo_1 = RecurringTodo.find(5) @todo = Todo.where(:recurring_todo_id => 1).first assert @todo.from_recurring_todo? # rewire @todo to yearly recurring todo @todo.recurring_todo_id = 5 - + # make todo due tomorrow and change recurring date also to tomorrow @todo.due = Time.zone.now + 1.day @todo.save recurring_todo_1.every_other1 = @todo.due.day recurring_todo_1.every_other2 = @todo.due.month recurring_todo_1.save - + # mark todo complete xhr :post, :toggle_check, :id => @todo.id, :_source_view => 'todo' @todo.reload assert @todo.completed? - + # check that there is no active todo next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'active'}) assert next_todo.nil? - + # check for new deferred todo next_todo = Todo.find(:first, :conditions => {:recurring_todo_id => recurring_todo_1.id, :state => 'deferred'}) assert !next_todo.nil? @@ -725,7 +725,7 @@ class TodosControllerTest < ActionController::TestCase ############ # todo notes ############ - + 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 login_as(:admin_user) @@ -736,7 +736,7 @@ class TodosControllerTest < ActionController::TestCase get :index assert_select("a[href=#{url}]") end - + def test_format_note_normal login_as(:admin_user) todo = users(:admin_user).todos.first @@ -745,7 +745,7 @@ class TodosControllerTest < ActionController::TestCase get :index assert_select("div#notes_todo_#{todo.id}", "A normal description.") end - + def test_format_note_textile login_as(:admin_user) 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} strong", "bold description") end - + def test_format_note_link login_as(:admin_user) 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[href=http://github.com/]", 'http://github.com/') end - + def test_format_note_link_message login_as(:admin_user) todo = users(:admin_user).todos.first @@ -776,7 +776,7 @@ class TodosControllerTest < ActionController::TestCase assert_select("div#notes_todo_#{todo.id} a", 'message://<ABCDEF-GHADB-123455-FOO-BAR@example.com>') assert_select("div#notes_todo_#{todo.id} a[href=message://<ABCDEF-GHADB-123455-FOO-BAR@example.com>]", 'message://<ABCDEF-GHADB-123455-FOO-BAR@example.com>') end - + def test_format_note_link_onenote login_as(:admin_user) todo = users(:admin_user).todos.first @@ -791,111 +791,111 @@ class TodosControllerTest < ActionController::TestCase ############## # dependencies ############## - + def test_make_todo_dependent login_as(:admin_user) - + predecessor = todos(:call_bill) successor = todos(:call_dino_ext) - + # no predecessors yet assert_equal 0, successor.predecessors.size - + # add predecessor put :add_predecessor, :predecessor=>predecessor.id, :successor=>successor.id - + assert_equal 1, successor.predecessors.count assert_equal predecessor.id, successor.predecessors.first.id end - + def test_make_todo_with_dependencies_dependent login_as(:admin_user) - + predecessor = todos(:call_bill) successor = todos(:call_dino_ext) other_todo = todos(:phone_grandfather) - + # predecessor -> successor put :add_predecessor, :predecessor=>predecessor.id, :successor=>successor.id - + # other_todo -> predecessor -> successor put :add_predecessor, :predecessor=>other_todo.id, :successor=>predecessor.id - + assert_equal 1, successor.predecessors(true).count assert_equal 0, other_todo.predecessors(true).count assert_equal 1, predecessor.predecessors(true).count assert_equal predecessor.id, successor.predecessors.first.id assert_equal other_todo.id, predecessor.predecessors.first.id end - + def test_mingle_dependent_todos_leave # based on #1271 login_as(:admin_user) - + t1 = todos(:call_bill) t2 = todos(:call_dino_ext) t3 = todos(:phone_grandfather) t4 = todos(:construct_dilation_device) - + # t1 -> t2 put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id # t3 -> t4 put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id - + # t2 -> t4 put :add_predecessor, :predecessor=>t2.id, :successor=>t4.id - + # should be: t1 -> t2 -> t4 and t3 -> t4 assert t4.predecessors.map(&:id).include?(t2.id) assert t4.predecessors.map(&:id).include?(t3.id) assert t2.predecessors.map(&:id).include?(t1.id) end - + def test_mingle_dependent_todos_root # based on #1271 login_as(:admin_user) - + t1 = todos(:call_bill) t2 = todos(:call_dino_ext) t3 = todos(:phone_grandfather) t4 = todos(:construct_dilation_device) - + # t1 -> t2 put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id # t3 -> t4 put :add_predecessor, :predecessor=>t3.id, :successor=>t4.id - + # t3 -> t2 put :add_predecessor, :predecessor=>t3.id, :successor=>t2.id - + # should be: t1 -> t2 and t3 -> t4 & t2 assert t3.successors.map(&:id).include?(t4.id) assert t3.successors.map(&:id).include?(t2.id) assert t2.predecessors.map(&:id).include?(t1.id) assert t2.predecessors.map(&:id).include?(t3.id) end - + def test_unmingle_dependent_todos # based on #1271 login_as(:admin_user) - + t1 = todos(:call_bill) t2 = todos(:call_dino_ext) t3 = todos(:phone_grandfather) t4 = todos(:construct_dilation_device) - + # create same dependency tree as previous test # should be: t1 -> t2 -> t4 and t3 -> t4 put :add_predecessor, :predecessor=>t1.id, :successor=>t2.id put :add_predecessor, :predecessor=>t3.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 put :remove_predecessor, :predecessor=>t2.id, :id=>t4.id - + t4.reload assert t4.pending?, "t4 should remain pending" assert t4.predecessors.map(&:id).include?(t3.id) end - + end