Extract organisation fetcher

app/controllers/publishers/ats_api/v1/create_vacancy_service.rb

@@ -2,6 +2,8 @@ module Publishers
   module AtsApi
     module V1
       class CreateVacancyService
+        extend OrganisationFetcher
+
         InvalidOrganisationError = Class.new(StandardError)
 
         class << self
@@ -26,20 +28,7 @@ def sanitised_params(params)
             raise InvalidOrganisationError, "No valid organisations found" if organisations.blank?
 
             params[:publish_on] ||= Time.zone.today.to_s
-            params[:working_patterns] ||= []
-            params[:phases] ||= []
-
-            params.except(:schools, :trust_uid).merge(organisations: organisations)
-          end
-
-          def fetch_organisations(school_params)
-            return [] unless school_params
-
-            if school_params[:trust_uid].present?
-              SchoolGroup.trusts.find_by(uid: school_params[:trust_uid])&.schools&.where(urn: school_params[:school_urns]) || []
-            else
-              School.where(urn: school_params[:school_urns])
-            end
+            params.except(:schools).merge(organisations: organisations)
           end
 
           def conflict_vacancy(vacancy)

app/controllers/publishers/ats_api/v1/organisation_fetcher.rb

@@ -0,0 +1,17 @@
+module Publishers
+  module AtsApi
+    module V1
+      module OrganisationFetcher
+        def fetch_organisations(school_params)
+          return [] unless school_params
+
+          if school_params[:trust_uid].present?
+            SchoolGroup.trusts.find_by(uid: school_params[:trust_uid]).schools&.where(urn: school_params[:school_urns]) || []
+          else
+            School.where(urn: school_params[:school_urns])
+          end
+        end
+      end
+    end
+  end
+end

app/controllers/publishers/ats_api/v1/update_vacancy_service.rb

@@ -2,6 +2,8 @@ module Publishers
   module AtsApi
     module V1
       class UpdateVacancyService
+        extend OrganisationFetcher
+
         class << self
           def call(vacancy, params)
             if vacancy.update(sanitised_params(params))
@@ -19,17 +21,7 @@ def sanitised_params(params)
             organisations = fetch_organisations(params[:schools])
             raise ActiveRecord::RecordNotFound, "No valid organisations found" if organisations.blank?
 
-            params.except(:schools, :trust_uid).merge(organisations: organisations)
-          end
-
-          def fetch_organisations(school_params)
-            return [] unless school_params
-
-            if school_params[:trust_uid].present?
-              SchoolGroup.trusts.find_by(uid: school_params[:trust_uid]).schools.where(urn: school_params[:school_urns])
-            else
-              School.where(urn: school_params[:school_urns])
-            end
+            params.except(:schools).merge(organisations: organisations)
           end
 
           def format_errors(errors)

spec/requests/publishers/ats_api/v1/create_vacancy_service_spec.rb

@@ -63,11 +63,10 @@
       let(:school_urns) { { school_urns: [9999] } }
 
       it "raises ActiveRecord::RecordNotFound" do
-        expect { create_vacancy_service }
-          .to raise_error(
-            Publishers::AtsApi::V1::CreateVacancyService::InvalidOrganisationError,
-            "No valid organisations found",
-          )
+        expect { create_vacancy_service }.to raise_error(
+          Publishers::AtsApi::V1::CreateVacancyService::InvalidOrganisationError,
+          "No valid organisations found",
+        )
       end
     end