From fd5062429ae8dd0561d882e63d55bcff978342b0 Mon Sep 17 00:00:00 2001 From: Luis Ramos Date: Tue, 9 Jun 2020 11:11:48 +0100 Subject: [PATCH] Add context to delete specs so we can add other specs --- .../admin/shipping_methods_controller_spec.rb | 32 ++++++++++--------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/spec/controllers/spree/admin/shipping_methods_controller_spec.rb b/spec/controllers/spree/admin/shipping_methods_controller_spec.rb index d705972c87..c8fb13f686 100644 --- a/spec/controllers/spree/admin/shipping_methods_controller_spec.rb +++ b/spec/controllers/spree/admin/shipping_methods_controller_spec.rb @@ -3,29 +3,31 @@ require 'spec_helper' describe Spree::Admin::ShippingMethodsController, type: :controller do include AuthenticationWorkflow - describe "shipping method not referenced by order" do - let(:shipping_method) { create(:shipping_method) } + describe "#delete" do + describe "shipping method not referenced by order" do + let(:shipping_method) { create(:shipping_method) } - scenario "is soft deleted" do - login_as_admin - expect(shipping_method.deleted_at).to be_nil + scenario "is soft deleted" do + login_as_admin + expect(shipping_method.deleted_at).to be_nil - spree_delete :destroy, "id" => shipping_method.id + spree_delete :destroy, "id" => shipping_method.id - expect(shipping_method.reload.deleted_at).not_to be_nil + expect(shipping_method.reload.deleted_at).not_to be_nil + end end - end - describe "shipping method referenced by order" do - let(:order) { create(:order_with_line_items) } + describe "shipping method referenced by order" do + let(:order) { create(:order_with_line_items) } - scenario "is not soft deleted" do - login_as_admin - expect(order.shipping_method.deleted_at).to be_nil + scenario "is not soft deleted" do + login_as_admin + expect(order.shipping_method.deleted_at).to be_nil - spree_delete :destroy, "id" => order.shipping_method.id + spree_delete :destroy, "id" => order.shipping_method.id - expect(order.shipping_method.reload.deleted_at).to be_nil + expect(order.shipping_method.reload.deleted_at).to be_nil + end end end end