WIP

app/controllers/admin/editions_controller.rb

@@ -118,12 +118,6 @@ def save_and_publish
         flash[:alert] = @edition.errors.full_messages.join(", ")
       end
 
-      # PublishRequest.create!(
-      #   request_id: govuk_request_id,
-      #   edition_id: @edition.id,
-      #   country_slug: @edition.country_slug,
-      # )
-
       redirect_to admin_country_path(@edition.country_slug), notice: "#{@edition.title} published."
     else
       flash[:alert] = "We had some problems publishing: #{@edition.errors.full_messages.join(', ')}."

spec/controllers/admin/editions_controller_spec.rb

@@ -225,52 +225,6 @@
 
         expect(PublishingApiWorker.jobs.size).to eq(1)
       end
-
-      # it "creates a PublishRequest for that edition" do
-      #   request_id = "123456"
-      #   allow(GdsApi::GovukHeaders).to receive(:headers).and_return(govuk_request_id: request_id)
-      #   post :update, params: { id: draft.to_param, edition: {}, commit: "Save & Publish" }
-      #   publish_request = PublishRequest.last
-      #   expect(publish_request.edition_id).to eq(draft.id)
-      #   expect(publish_request.request_id).to eq(request_id)
-      # end
-
-      # context "when this is the first edition for the country" do
-      #   it "creates a PublishRequest for the email signup page" do
-      #     Sidekiq::Worker.clear_all
-
-      #     post :update, params: { id: draft.to_param, edition: {}, commit: "Save & Publish" }
-
-      #     expect(PublishingApiWorker.jobs.size).to eq(1)
-      #     actions = PublishingApiWorker.jobs[0]["args"][0]
-      #     expect(actions.count).to eq(6)
-      #     signup_email = actions.detect do |_, _, details|
-      #       details["base_path"] == "/foreign-travel-advice/aruba/email-signup"
-      #     end
-      #     expect(signup_email).not_to be_nil
-      #   end
-      # end
-
-      # context "when previous edition exist for the country" do
-      #   before do
-      #     create(:published_travel_advice_edition, country_slug: "aruba")
-      #     draft.update!(version_number: 2)
-      #   end
-
-      #   it "doesn't create a PublishRequest for the email signup page" do
-      #     Sidekiq::Worker.clear_all
-
-      #     post :update, params: { id: draft.to_param, edition: {}, commit: "Save & Publish" }
-
-      #     expect(PublishingApiWorker.jobs.size).to eq(1)
-      #     actions = PublishingApiWorker.jobs[0]["args"][0]
-      #     expect(actions.count).to eq(4)
-      #     signup_email = actions.detect do |_, _, details|
-      #       details["base_path"] == "/foreign-travel-advice/aruba/email-signup"
-      #     end
-      #     expect(signup_email).to be_nil
-      #   end
-      # end
     end
   end