From c3c396eaea75809147633a3944f0425f2ec22598 Mon Sep 17 00:00:00 2001
From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com>
Date: Fri, 30 Apr 2021 11:04:05 +0100
Subject: [PATCH] Add soft-deletion test coverage to Package#shipping_methods

---
 .../services/order_management/stock/package_spec.rb   | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/engines/order_management/spec/services/order_management/stock/package_spec.rb b/engines/order_management/spec/services/order_management/stock/package_spec.rb
index 1a7fbd017a..899b998ee5 100644
--- a/engines/order_management/spec/services/order_management/stock/package_spec.rb
+++ b/engines/order_management/spec/services/order_management/stock/package_spec.rb
@@ -158,11 +158,18 @@ module OrderManagement
 
         let(:shipping_method1) { create(:shipping_method, distributors: [enterprise]) }
         let(:shipping_method2) { create(:shipping_method, distributors: [other_enterprise]) }
+        let!(:shipping_method3) {
+          create(:shipping_method, distributors: [enterprise], deleted_at: Time.zone.now)
+        }
 
-        describe '#shipping_methods' do
-          it 'does not return shipping methods not used by the package\'s order distributor' do
+        describe "#shipping_methods" do
+          it "does not return shipping methods not used by the package's order distributor" do
             expect(package.shipping_methods).to eq [shipping_method1]
           end
+
+          it "does not return soft-deleted shipping methods" do
+            expect(package.shipping_methods).to_not include shipping_method3
+          end
         end
 
         describe '#shipping_categories' do