From 8bb92833b5f89c2d685b7d89b8a6a59dd333e5b6 Mon Sep 17 00:00:00 2001
From: Matt Rogers <codemattr@gmail.com>
Date: Wed, 18 Feb 2015 22:05:05 -0600
Subject: [PATCH] Use rails helpers for the delete dependency menu item

---
 app/helpers/todos_helper.rb       | 2 +-
 test/helpers/todo_helpers_test.rb | 7 +++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/app/helpers/todos_helper.rb b/app/helpers/todos_helper.rb
index 8431f6bf..2035f3fb 100644
--- a/app/helpers/todos_helper.rb
+++ b/app/helpers/todos_helper.rb
@@ -191,7 +191,7 @@ module TodosHelper
   def remote_delete_dependency(todo, predecessor)
     link_to(
       image_tag("blank.png", :title => t('todos.remove_dependency'), :align => "absmiddle", :class => "delete_item"),
-      url_for({:controller => 'todos', :action => 'remove_predecessor', :id => todo.id}),
+      remove_predecessor_todo_path(todo),
       {:class => "delete_dependency_button", :x_predecessors_id => predecessor.id}
     )
   end
diff --git a/test/helpers/todo_helpers_test.rb b/test/helpers/todo_helpers_test.rb
index 5db3203d..a800aca4 100644
--- a/test/helpers/todo_helpers_test.rb
+++ b/test/helpers/todo_helpers_test.rb
@@ -12,4 +12,11 @@ class TodosHelpersTest < ActionView::TestCase
     html = remote_delete_menu_item(todos(:call_bill))
     assert_equal "<a class=\"icon_delete_item\" href=\"/todos/1\" id=\"delete_todo_1\" title=\"Delete action\" x_confirm_message=\"Are you sure that you want to delete the action &#39;Call Bill Gates to find out how much he makes per day&#39;?\">Delete</a>", html
   end
+
+  test "remote_delete_dependency" do
+    todo = todos(:call_bill_gates_every_day)
+    predecessor = todos(:call_bill)
+    html = remote_delete_dependency(todo, predecessor)
+    assert_equal "<a class=\"delete_dependency_button\" href=\"/todos/18/remove_predecessor\" x_predecessors_id=\"1\"><img align=\"absmiddle\" alt=\"Blank\" class=\"delete_item\" src=\"/images/blank.png\" title=\"Remove dependency (does not delete the action)\" /></a>", html
+  end
 end