Promote a Task from Delayed back to To-do

To make the page more interactive, add a “Promote” button to move Tasks that have been marked as “Delayed” back to “Todo”.

Instead of acting as a bulk action, the PromotionsController#create operates on a single Task record.


      app/controllers/promotions_controller.rb

diff --git a/app/controllers/promotions_controller.rb b/app/controllers/promotions_controller.rb
new file mode 100644
index 0000000..1e71b02
--- /dev/null
+++ b/app/controllers/promotions_controller.rb
@@ -0,0 +1,9 @@
+class PromotionsController < ApplicationController
+  def create
+    task = Task.find(params[:task_id])
+
+    task.promote!
+
+    redirect_back fallback_location: root_url
+  end
+end


      app/models/task.rb

diff --git a/app/models/task.rb b/app/models/task.rb
index 1a4c8cc..0cd29c5 100644
--- a/app/models/task.rb
+++ b/app/models/task.rb
@@ -10,4 +10,8 @@ class Task < ApplicationRecord
   def delay!
     update!(delayed_at: Time.current)
   end
+
+  def promote!
+    update!(delayed_at: nil)
+  end
 end


      app/views/tasks/index.html.erb

diff --git a/app/views/tasks/index.html.erb b/app/views/tasks/index.html.erb
index 03191a3..1d027a8 100644
--- a/app/views/tasks/index.html.erb
+++ b/app/views/tasks/index.html.erb
@@ -42,6 +42,15 @@
     <% tasks.delayed.each do |task| %>
       <tr>
         <td><%= task.name %></td>
+
+        <td>
+          <%= form_with(scope: :event, url: task_promotions_path(task)) do |form| %>
+            <%= form.button id: dom_id(task, :promote) do %>
+              <%= translate(:promote, scope: [:helpers, :submit, :event]) %>
+              <%= task.name %>
+            <% end %>
+          <% end %>
+        </td>
       </tr>
     <% end %>
   </table>


      config/locales/en.yml

diff --git a/config/locales/en.yml b/config/locales/en.yml
index c42c105..401fade 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -35,6 +35,7 @@ en:
       event:
         complete: "Complete"
         delay: "Delay"
+        promote: "Promote"
 
   tasks:
     index:


      config/routes.rb

diff --git a/config/routes.rb b/config/routes.rb
index ddb5c00..e003fb8 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -3,5 +3,9 @@ Rails.application.routes.draw do
   resources :completions, only: :create
   resources :delays, only: :create
 
+  resources :tasks, only: [] do
+    resources :promotions, only: :create
+  end
+
   root to: "tasks#index"
 end


      test/controllers/promotions_controller_test.rb

diff --git a/test/controllers/promotions_controller_test.rb b/test/controllers/promotions_controller_test.rb
new file mode 100644
index 0000000..8f89b89
--- /dev/null
+++ b/test/controllers/promotions_controller_test.rb
@@ -0,0 +1,12 @@
+require "test_helper"
+
+class PromotionsControllerTest < ActionDispatch::IntegrationTest
+  test "#create marks a Task's delayed_at to nil" do
+    do_the_homework = tasks(:do_the_homework)
+    do_the_homework.delay!
+
+    post task_promotions_path(do_the_homework)
+
+    assert_nil do_the_homework.reload.delayed_at
+  end
+end


      test/controllers/tasks_controller_test.rb

diff --git a/test/controllers/tasks_controller_test.rb b/test/controllers/tasks_controller_test.rb
index 158392b..70f1e3c 100644
--- a/test/controllers/tasks_controller_test.rb
+++ b/test/controllers/tasks_controller_test.rb
@@ -25,6 +25,14 @@ class TasksControllerTest < ActionDispatch::IntegrationTest
     assert_no_section :delayed
   end
 
+  test "#index omits the Todo list when there are no Task records to be worked on" do
+    Task.todo.update_all(completed_at: Time.current)
+
+    get root_path
+
+    assert_no_section :todo
+  end
+
   def assert_no_section(i18n_key)
     title = translate(i18n_key, scope: [:tasks, :index])


      test/models/task_test.rb

diff --git a/test/models/task_test.rb b/test/models/task_test.rb
index 602ecc4..6cac316 100644
--- a/test/models/task_test.rb
+++ b/test/models/task_test.rb
@@ -58,4 +58,15 @@ class TaskTest < ActiveSupport::TestCase
       end
     end
   end
+
+  test "#promote! marks a Task's delayed_at to nil" do
+    freeze_time do
+      do_the_homework = tasks(:do_the_homework)
+      do_the_homework.delay!
+
+      assert_changes -> { do_the_homework.delayed_at }, from: Time.current, to: nil do
+        do_the_homework.promote!
+      end
+    end
+  end
 end


      test/system/tasks_test.rb

diff --git a/test/system/tasks_test.rb b/test/system/tasks_test.rb
index 0f3eaa1..3acda03 100644
--- a/test/system/tasks_test.rb
+++ b/test/system/tasks_test.rb
@@ -33,6 +33,19 @@ class TasksTest < ApplicationSystemTestCase
     end
   end
 
+  test "promotes delayed Task back to Todo" do
+    do_the_homework = tasks(:do_the_homework)
+    do_the_homework.delay!
+
+    visit root_path
+    within_task(do_the_homework.name) { click_on submit(:event, :promote) }
+
+    assert_no_section :delayed
+    within_section :todo do
+      assert_text do_the_homework.name
+    end
+  end
+
   def assert_no_section(i18n_key)
     assert_no_text translate(i18n_key, scope: [:tasks, :index])
   end
@@ -43,6 +56,10 @@ class TasksTest < ApplicationSystemTestCase
     within("section", text: title, &block)
   end
 
+  def within_task(text, &block)
+    within("tr", text: text, &block)
+  end
+
   def submit(i18n_key, action)
     translate(action, scope: [:helpers, :submit, i18n_key])
   end