From 005d95e882b51410c64e72799af4b4ebecaabe7c Mon Sep 17 00:00:00 2001
From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com>
Date: Sun, 21 May 2023 13:33:29 +0100
Subject: [PATCH] Fix "Resend Confirmation" link and add missing test coverage

---
 app/helpers/spree/admin/orders_helper.rb |  1 -
 config/routes/spree.rb                   |  2 +-
 spec/system/admin/order_spec.rb          | 18 ++++++++++++++++++
 3 files changed, 19 insertions(+), 2 deletions(-)

diff --git a/app/helpers/spree/admin/orders_helper.rb b/app/helpers/spree/admin/orders_helper.rb
index 9032f7b787..81f9066f90 100644
--- a/app/helpers/spree/admin/orders_helper.rb
+++ b/app/helpers/spree/admin/orders_helper.rb
@@ -62,7 +62,6 @@ module Spree
         { name: t(:resend_confirmation),
           url: spree.resend_admin_order_path(@order),
           icon: 'icon-email',
-          method: 'post',
           confirm: t(:confirm_resend_order_confirmation) }
       end
 
diff --git a/config/routes/spree.rb b/config/routes/spree.rb
index 6959454e22..28834db308 100644
--- a/config/routes/spree.rb
+++ b/config/routes/spree.rb
@@ -86,7 +86,7 @@ Spree::Core::Engine.routes.draw do
       member do
         put :fire
         get :fire
-        post :resend
+        get :resend
         get :invoice
         get :print
         get :distribution
diff --git a/spec/system/admin/order_spec.rb b/spec/system/admin/order_spec.rb
index 8bb0f3c1b9..366c2f2a9a 100644
--- a/spec/system/admin/order_spec.rb
+++ b/spec/system/admin/order_spec.rb
@@ -611,6 +611,24 @@ describe '
                                     href: spree.fire_admin_order_path(order, e: 'cancel')
         end
       end
+
+      context "Resending confirmation email" do
+        before do
+          visit spree.edit_admin_order_path(order)
+          find("#links-dropdown .ofn-drop-down").click
+        end
+
+        it "shows the link" do
+          expect(page).to have_link "Resend Confirmation", href: spree.resend_admin_order_path(order)
+        end
+
+        it "resends the confirmation email" do
+          accept_alert "Are you sure you want to resend the order confirmation email?" do
+            click_link "Resend Confirmation"
+          end
+          expect(page).to have_content "Order email has been resent"
+        end
+      end
       
       context "Check send/print invoice links" do