From 4cd8688d9a90e4cb662a4e0fe61df82f5c6208ef Mon Sep 17 00:00:00 2001 From: Ian Young Date: Sat, 14 Sep 2013 22:11:23 -0700 Subject: [PATCH] Fix failing test As far as I can tell, this is how the test is intended to work, and how it works in v2.2.2. If a recurring todo set to show on the date is completed *on* the specified date, it will create a deferred one for the *next* month. --- test/controllers/todos_controller_test.rb | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/test/controllers/todos_controller_test.rb b/test/controllers/todos_controller_test.rb index e51773d6..6a4d5ca6 100644 --- a/test/controllers/todos_controller_test.rb +++ b/test/controllers/todos_controller_test.rb @@ -729,23 +729,11 @@ class TodosControllerTest < ActionController::TestCase 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. + # 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? - # locate the new todo. This todo is created from the adjusted recurring - # pattern defined in this test - new_todo = Todo.where(:recurring_todo_id => recurring_todo_1.id, :state => 'active').first - 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.where(:recurring_todo_id => recurring_todo_1.id, :state => 'deferred').first assert !new_todo.nil?