From 03d4e67baf38eccdd154281056021682651a2605 Mon Sep 17 00:00:00 2001
From: kosiakkatrina <54268893+kosiakkatrina@users.noreply.github.com>
Date: Fri, 18 Aug 2023 09:12:52 +0100
Subject: [PATCH] mark owning organisatinon question as derived on prod (#1852)

---
 .../sales/questions/owning_organisation_id.rb  |  2 +-
 spec/requests/sales_logs_controller_spec.rb    | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/app/models/form/sales/questions/owning_organisation_id.rb b/app/models/form/sales/questions/owning_organisation_id.rb
index ec7c28a85..43e97add6 100644
--- a/app/models/form/sales/questions/owning_organisation_id.rb
+++ b/app/models/form/sales/questions/owning_organisation_id.rb
@@ -65,7 +65,7 @@ class Form::Sales::Questions::OwningOrganisationId < ::Form::Question
   end
 
   def derived?
-    true if FeatureToggle.merge_organisations_enabled?
+    true
   end
 
   def hidden_in_check_answers?(_log, user = nil)
diff --git a/spec/requests/sales_logs_controller_spec.rb b/spec/requests/sales_logs_controller_spec.rb
index 8a21f3fa4..919f70725 100644
--- a/spec/requests/sales_logs_controller_spec.rb
+++ b/spec/requests/sales_logs_controller_spec.rb
@@ -159,6 +159,24 @@ RSpec.describe SalesLogsController, type: :request do
               expect(sales_log.owning_organisation.name).to eq("User org")
             end
           end
+
+          context "when the user's org doesn't hold stock and merge_organisations_enabled is false" do
+            let(:organisation) { FactoryBot.create(:organisation, name: "User org", holds_own_stock: false) }
+            let(:user) { FactoryBot.create(:user, :data_coordinator, organisation:) }
+
+            before do
+              RequestHelper.stub_http_requests
+              sign_in user
+              allow(FeatureToggle).to receive(:merge_organisations_enabled?).and_return(false)
+              post "/sales-logs", headers:
+            end
+
+            it "sets the stock-owning org as user's org" do
+              created_id = response.location.match(/[0-9]+/)[0]
+              sales_log = SalesLog.find_by(id: created_id)
+              expect(sales_log.owning_organisation.name).to eq("User org")
+            end
+          end
         end
       end
     end