mirror of
https://github.com/openfoodfoundation/openfoodnetwork
synced 2026-02-04 22:16:08 +00:00
Remove order link in non-user order cancellation email
This commit is contained in:
@@ -3,9 +3,10 @@
|
||||
|
||||
%p.callout
|
||||
= t("email_so_confirmation_explainer_html")
|
||||
= t("email_so_edit_false_html",
|
||||
orders_close_at: l(@order.order_cycle.orders_close_at, format: mail_long_datetime_format),
|
||||
order_url: spree.order_url(@order))
|
||||
- if @order.user.present?
|
||||
= t("email_so_edit_false_html",
|
||||
orders_close_at: l(@order.order_cycle.orders_close_at, format: mail_long_datetime_format),
|
||||
order_url: spree.order_url(@order))
|
||||
= t("email_so_contact_distributor_html", distributor: @order.distributor.name, email: @order.distributor.contact.email)
|
||||
|
||||
%p
|
||||
|
||||
@@ -93,7 +93,8 @@ describe SubscriptionMailer do
|
||||
end
|
||||
|
||||
describe "order confirmation" do
|
||||
let(:subscription) { create(:subscription, with_items: true) }
|
||||
let(:customer) { create(:customer) }
|
||||
let(:subscription) { create(:subscription, customer: customer, with_items: true) }
|
||||
let(:proxy_order) { create(:proxy_order, subscription: subscription) }
|
||||
let!(:order) { proxy_order.initialise_order! }
|
||||
|
||||
@@ -106,7 +107,29 @@ describe SubscriptionMailer do
|
||||
it "sends the email" do
|
||||
body = SubscriptionMailer.deliveries.last.body.encoded
|
||||
expect(body).to include "This order was automatically placed for you"
|
||||
expect(body).to include "href=\"#{spree.order_url(order)}\""
|
||||
end
|
||||
|
||||
describe "linking to order page" do
|
||||
let(:order_link_href) { "href=\"#{spree.order_url(order)}\"" }
|
||||
|
||||
let(:email) { SubscriptionMailer.deliveries.last }
|
||||
let(:body) { email.body.encoded }
|
||||
|
||||
context "when the customer has a user account" do
|
||||
let(:customer) { create(:customer) }
|
||||
|
||||
it "provides link to view details" do
|
||||
expect(body).to match /#{order_link_href}/
|
||||
end
|
||||
end
|
||||
|
||||
context "when the customer has no user account" do
|
||||
let(:customer) { create(:customer, user: nil) }
|
||||
|
||||
it "does not provide link" do
|
||||
expect(body).to_not match /#{order_link_href}/
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
|
||||
Reference in New Issue
Block a user