 RUBYCRITIC
RUBYCRITIC
      
    require 'rails_helper'
RSpec.describe Admin::PetitionEmailsController, type: :controller, admin: true do
  let!(:petition) { FactoryBot.create(:open_petition) }
  describe 'not logged in' do  -       
let(:email) { FactoryBot.create(:petition_email, petition: petition) }
    describe 'GET /new' do
      it 'redirects to the login page' do
        get :new, petition_id: petition.id
        expect(response).to redirect_to('https://moderate.petition.parliament.uk/admin/login')
      end
    end
    describe 'POST /' do
      it 'redirects to the login page' do
        post :create, petition_id: petition.id
        expect(response).to redirect_to('https://moderate.petition.parliament.uk/admin/login')
      end
    end
    describe 'GET /:id/edit' do
      it 'redirects to the login page' do
        get :edit, petition_id: petition.id, id: email.id
        expect(response).to redirect_to('https://moderate.petition.parliament.uk/admin/login')
      end
    end
    describe 'PATCH /:id' do
      it 'redirects to the login page' do
        patch :update, petition_id: petition.id, id: email.id
        expect(response).to redirect_to('https://moderate.petition.parliament.uk/admin/login')
      end
    end
    describe 'DELETE /:id' do
      it 'redirects to the login page' do
        patch :destroy, petition_id: petition.id, id: email.id
        expect(response).to redirect_to('https://moderate.petition.parliament.uk/admin/login')
      end
    end
  end
  context 'logged in as moderator user but need to reset password' do  -       
let(:email) { FactoryBot.create(:petition_email, petition: petition) }
    let(:user) { FactoryBot.create(:moderator_user, force_password_reset: true) }
    before { login_as(user) }
    describe 'GET /new' do
      it 'redirects to edit profile page' do
        get :new, petition_id: petition.id
        expect(response).to redirect_to("https://moderate.petition.parliament.uk/admin/profile/#{user.id}/edit")
      end
    end
    describe 'POST /' do
      it 'redirects to edit profile page' do
        post :create, petition_id: petition.id
        expect(response).to redirect_to("https://moderate.petition.parliament.uk/admin/profile/#{user.id}/edit")
      end
    end
    describe 'GET /:id/edit' do
      it 'redirects to the login page' do
        get :edit, petition_id: petition.id, id: email.id
        expect(response).to redirect_to("https://moderate.petition.parliament.uk/admin/profile/#{user.id}/edit")
      end
    end
    describe 'PATCH /:id' do
      it 'redirects to the login page' do
        patch :update, petition_id: petition.id, id: email.id
        expect(response).to redirect_to("https://moderate.petition.parliament.uk/admin/profile/#{user.id}/edit")
      end
    end
    describe 'DELETE /:id' do
      it 'redirects to the login page' do
        patch :destroy, petition_id: petition.id, id: email.id
        expect(response).to redirect_to("https://moderate.petition.parliament.uk/admin/profile/#{user.id}/edit")
      end
    end
  end
  describe "logged in as moderator user" do
    let(:user) { FactoryBot.create(:moderator_user) }
    before { login_as(user) }
    describe 'GET /new' do
      describe 'for an open petition' do
        it 'fetches the requested petition' do
          get :new, petition_id: petition.id
          expect(assigns(:petition)).to eq petition
        end
        it 'responds successfully and renders the petitions/show template' do
          get :new, petition_id: petition.id
          expect(response).to be_success
          expect(response).to render_template('petitions/show')
        end
      end
      shared_examples_for 'trying to view the email petitioners form of a petition in the wrong state' do  -       
it 'raises a 404 error' do
          expect {
            get :new, petition_id: petition.id
          }.to raise_error ActiveRecord::RecordNotFound
        end
      end
      describe 'for a pending petition' do
        before { petition.update_column(:state, Petition::PENDING_STATE) }
        it_behaves_like 'trying to view the email petitioners form of a petition in the wrong state'
      end
      describe 'for a validated petition' do
        before { petition.update_column(:state, Petition::VALIDATED_STATE) }
        it_behaves_like 'trying to view the email petitioners form of a petition in the wrong state'
      end
      describe 'for a sponsored petition' do
        before { petition.update_column(:state, Petition::SPONSORED_STATE) }
        it_behaves_like 'trying to view the email petitioners form of a petition in the wrong state'
      end
    end
    describe 'POST /' do
      let(:petition_email_attributes) do
        {
          subject: "Petition email subject",
          body: "Petition email body"
        }
      end
      context 'when clicking the Email button' do
        def do_post(overrides = {})
          params = {
            petition_id: petition.id,
            petition_email: petition_email_attributes,
            save_and_email: "Email"
          }
          post :create, params.merge(overrides)
        end
        describe 'for an open petition' do
          it 'fetches the requested petition' do
            do_post
            expect(assigns(:petition)).to eq petition
          end
          describe 'with valid params' do  -       
it 'redirects to the petition show page' do
              do_post
              expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
            end
            it 'tells the moderator that their email will be sent overnight' do
              do_post
              expect(flash[:notice]).to eq 'Email will be sent overnight'
            end
            it 'stores the supplied email details in the db' do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::it#stores the supplied email details in the db has a flog score of 50            
do_post
              petition.reload
              email = petition.emails.last
              expect(email).to be_present
              expect(email.subject).to eq "Petition email subject"
              expect(email.body).to eq "Petition email body"
              expect(email.sent_by).to eq user.pretty_name
            end
            context "emails out the petition email" do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::context#emails out the petition email has a flog score of 37            
before do
                3.times do |i|
                  attributes = {
                    name: "Laura #{i}",
                    email: "laura_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Sarah #{i}",
                    email: "sarah_#{i}@example.com",
                    notify_by_email: false,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Brian #{i}",
                    email: "brian_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  FactoryBot.create(:pending_signature, attributes)
                end
                petition.reload
              end
              it "queues a job to process the emails" do
                assert_enqueued_jobs 1 do
                  do_post
                end
              end
              it "stamps the 'petition_email' email sent timestamp on each signature when the job runs" do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::context(emails out the petition email)::it#stamps the 'petition_email' email sent timestamp on each signature when the job runs has a flog score of 44            
perform_enqueued_jobs do
                  do_post
                  petition.reload
                  petition_timestamp = petition.get_email_requested_at_for('petition_email')
                  expect(petition_timestamp).not_to be_nil
                  petition.signatures.validated.subscribed.each do |signature|
                    expect(signature.get_email_sent_at_for('petition_email')).to eq(petition_timestamp)
                  end
                end
              end
              it "should email out to the validated signees who have opted in when the delayed job runs" do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::context(emails out the petition email)::it#should email out to the validated signees who have opted in when the delayed job runs has a flog score of 44            
perform_enqueued_jobs do
                  do_post
                  expect(deliveries.length).to eq 5
                  expect(deliveries.map(&:to)).to eq([
                    [petition.creator.email],
                    ['laura_0@example.com'],
                    ['laura_1@example.com'],
                    ['laura_2@example.com'],
                    ['petitionscommittee@parliament.uk']
                  ])
                end
              end
            end
          end
          describe 'with invalid params' do
            let(:petition_email_attributes) do
              { subject: "", body: "" }
            end
            it 're-renders the petitions/show template' do
              do_post
              expect(response).to be_success
              expect(response).to render_template('petitions/show')
            end
            it 'leaves the in-memory instance with errors' do
              do_post
              expect(assigns(:email)).to be_present
              expect(assigns(:email).errors).not_to be_empty
            end
            it 'does not stores the email details in the db' do
              do_post
              petition.reload
              expect(petition.emails).to be_empty
            end
          end
        end
        shared_examples_for 'trying to email supporters of a petition in the wrong state' do
          it 'raises a 404 error' do
            expect {
              do_post
            }.to raise_error ActiveRecord::RecordNotFound
          end
          it 'does not stores the supplied email details in the db' do
            suppress(ActiveRecord::RecordNotFound) { do_post }
            petition.reload
            expect(petition.emails).to be_empty
          end
        end
        describe 'for a pending petition' do
          before { petition.update_column(:state, Petition::PENDING_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a validated petition' do
          before { petition.update_column(:state, Petition::VALIDATED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a sponsored petition' do
          before { petition.update_column(:state, Petition::SPONSORED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
      end
      context 'when clicking the Save button' do  -       
def do_post(overrides = {})
          params = {
            petition_id: petition.id,
            petition_email: petition_email_attributes,
            save: "Save"
          }
          post :create, params.merge(overrides)
        end
        describe 'for an open petition' do
          it 'fetches the requested petition' do
            do_post
            expect(assigns(:petition)).to eq petition
          end
          describe 'with valid params' do
            it 'redirects to the petition show page' do
              do_post
              expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
            end
            it 'tells the moderator that their changes were saved' do
              do_post
              expect(flash[:notice]).to eq 'Created other parliamentary business successfully'
            end
            it 'stores the supplied email details in the db' do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Save button)::describe(for an open petition)::describe(with valid params)::it#stores the supplied email details in the db has a flog score of 50            
do_post
              petition.reload
              email = petition.emails.last
              expect(email).to be_present
              expect(email.subject).to eq "Petition email subject"
              expect(email.body).to eq "Petition email body"
              expect(email.sent_by).to eq user.pretty_name
            end
            context "does not email out the petition email" do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Save button)::describe(for an open petition)::describe(with valid params)::context#does not email out the petition email has a flog score of 37            
before do
                3.times do |i|
                  attributes = {
                    name: "Laura #{i}",
                    email: "laura_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Sarah #{i}",
                    email: "sarah_#{i}@example.com",
                    notify_by_email: false,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Brian #{i}",
                    email: "brian_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  FactoryBot.create(:pending_signature, attributes)
                end
                petition.reload
              end
              it "does not queue a job to process the emails" do
                assert_enqueued_jobs 0 do
                  do_post
                end
              end
            end
          end
          describe 'with invalid params' do
            let(:petition_email_attributes) do
              { subject: "", body: "" }
            end
            it 're-renders the petitions/show template' do
              do_post
              expect(response).to be_success
              expect(response).to render_template('petitions/show')
            end
            it 'leaves the in-memory instance with errors' do
              do_post
              expect(assigns(:email)).to be_present
              expect(assigns(:email).errors).not_to be_empty
            end
            it 'does not stores the email details in the db' do
              do_post
              petition.reload
              expect(petition.emails).to be_empty
            end
          end
        end
        shared_examples_for 'trying to email supporters of a petition in the wrong state' do
          it 'raises a 404 error' do
            expect {
              do_post
            }.to raise_error ActiveRecord::RecordNotFound
          end
          it 'does not store the supplied email details in the db' do
            suppress(ActiveRecord::RecordNotFound) { do_post }
            petition.reload
            expect(petition.emails).to be_empty
          end
        end
        describe 'for a pending petition' do
          before { petition.update_column(:state, Petition::PENDING_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a validated petition' do
          before { petition.update_column(:state, Petition::VALIDATED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a sponsored petition' do
          before { petition.update_column(:state, Petition::SPONSORED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
      end
      context 'when clicking the Preview button' do  -       
def do_post(overrides = {})  -                 describe(logged in as moderator user)::describe(POST /)::context#do_post has a flog score of 27            
params = {
            petition_id: petition.id,
            petition_email: petition_email_attributes,
            save_and_preview: "Save and preview"
          }
          post :create, params.merge(overrides)
        end
        describe 'for an open petition' do
          it 'fetches the requested petition' do
            do_post
            expect(assigns(:petition)).to eq petition
          end
          describe 'with valid params' do
            it 'redirects to the petition show page' do
              do_post
              expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
            end
            it 'tells the moderator that their changes were saved' do
              do_post
              expect(flash[:notice]).to eq 'Preview email successfully sent'
            end
            it 'stores the supplied email details in the db' do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Preview button)::describe(for an open petition)::describe(with valid params)::it#stores the supplied email details in the db has a flog score of 50            
do_post
              petition.reload
              email = petition.emails.last
              expect(email).to be_present
              expect(email.subject).to eq "Petition email subject"
              expect(email.body).to eq "Petition email body"
              expect(email.sent_by).to eq user.pretty_name
            end
            context "does not email out the petition email" do  -                 describe(logged in as moderator user)::describe(POST /)::context(when clicking the Preview button)::describe(for an open petition)::describe(with valid params)::context#does not email out the petition email has a flog score of 37            
before do
                3.times do |i|
                  attributes = {
                    name: "Laura #{i}",
                    email: "laura_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Sarah #{i}",
                    email: "sarah_#{i}@example.com",
                    notify_by_email: false,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Brian #{i}",
                    email: "brian_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  FactoryBot.create(:pending_signature, attributes)
                end
                petition.reload
              end
              it "does not queue a job to process the emails" do
                assert_enqueued_jobs 0 do
                  do_post
                end
              end
            end
          end
          describe 'with invalid params' do
            let(:petition_email_attributes) do
              { subject: "", body: "" }
            end
            it 're-renders the petitions/show template' do
              do_post
              expect(response).to be_success
              expect(response).to render_template('petitions/show')
            end
            it 'leaves the in-memory instance with errors' do
              do_post
              expect(assigns(:email)).to be_present
              expect(assigns(:email).errors).not_to be_empty
            end
            it 'does not stores the email details in the db' do
              do_post
              petition.reload
              expect(petition.emails).to be_empty
            end
          end
        end
        shared_examples_for 'trying to email supporters of a petition in the wrong state' do
          it 'raises a 404 error' do
            expect {
              do_post
            }.to raise_error ActiveRecord::RecordNotFound
          end
          it 'does not store the supplied email details in the db' do
            suppress(ActiveRecord::RecordNotFound) { do_post }
            petition.reload
            expect(petition.emails).to be_empty
          end
        end
        describe 'for a pending petition' do
          before { petition.update_column(:state, Petition::PENDING_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a validated petition' do
          before { petition.update_column(:state, Petition::VALIDATED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a sponsored petition' do
          before { petition.update_column(:state, Petition::SPONSORED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
      end
    end
    describe 'GET /:id/edit' do
      let(:email) do
        FactoryBot.create(
          :petition_email,
          petition: petition,
          subject: "Petition email subject",
          body: "Petition email body"
        )
      end
      describe 'for an open petition' do  -       
it 'fetches the requested petition' do
          get :edit, petition_id: petition.id, id: email.id
          expect(assigns(:petition)).to eq petition
        end
        it 'fetches the requested email' do
          get :edit, petition_id: petition.id, id: email.id
          expect(assigns(:email)).to eq email
        end
        it 'responds successfully and renders the petition_emails/edit template' do  -                 describe(logged in as moderator user)::describe(GET /:id/edit)::describe(for an open petition)::it#responds successfully and renders the petition_emails/edit template has a flog score of 27            
get :edit, petition_id: petition.id, id: email.id
          expect(response).to be_success
          expect(response).to render_template('petition_emails/edit')
        end
      end
      shared_examples_for 'trying to view the email petitioners form of a petition in the wrong state' do  -       
it 'raises a 404 error' do
          expect {
            get :new, petition_id: petition.id, id: email.id
          }.to raise_error ActiveRecord::RecordNotFound
        end
      end
      describe 'for a pending petition' do
        before { petition.update_column(:state, Petition::PENDING_STATE) }
        it_behaves_like 'trying to view the email petitioners form of a petition in the wrong state'
      end
      describe 'for a validated petition' do
        before { petition.update_column(:state, Petition::VALIDATED_STATE) }
        it_behaves_like 'trying to view the email petitioners form of a petition in the wrong state'
      end
      describe 'for a sponsored petition' do
        before { petition.update_column(:state, Petition::SPONSORED_STATE) }
        it_behaves_like 'trying to view the email petitioners form of a petition in the wrong state'
      end
    end
    describe 'PATCH /:id' do
      let(:email) do
        FactoryBot.create(
          :petition_email,
          petition: petition,
          subject: "Petition email subject",
          body: "Petition email body"
        )
      end
      let(:petition_email_attributes) do
        {
          subject: "New petition email subject",
          body: "New petition email body"
        }
      end
      context 'when clicking the Email button' do
        def do_patch(overrides = {})  -       
params = {
            petition_id: petition.id,
            id: email.id,
            petition_email: petition_email_attributes,
            save_and_email: "Email"
          }
          patch :update, params.merge(overrides)
        end
        describe 'for an open petition' do
          it 'fetches the requested petition' do
            do_patch
            expect(assigns(:petition)).to eq petition
          end
          it 'fetches the requested email' do
            do_patch
            expect(assigns(:email)).to eq email
          end
          describe 'with valid params' do  -       
it 'redirects to the petition show page' do
              do_patch
              expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
            end
            it 'tells the moderator that their email will be sent overnight' do
              do_patch
              expect(flash[:notice]).to eq 'Email will be sent overnight'
            end
            it 'stores the supplied email details in the db' do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::it#stores the supplied email details in the db has a flog score of 50            
do_patch
              petition.reload
              email = petition.emails.last
              expect(email).to be_present
              expect(email.subject).to eq "New petition email subject"
              expect(email.body).to eq "New petition email body"
              expect(email.sent_by).to eq user.pretty_name
            end
            context "emails out the petition email" do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::context#emails out the petition email has a flog score of 37            
before do
                3.times do |i|
                  attributes = {
                    name: "Laura #{i}",
                    email: "laura_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Sarah #{i}",
                    email: "sarah_#{i}@example.com",
                    notify_by_email: false,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Brian #{i}",
                    email: "brian_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  FactoryBot.create(:pending_signature, attributes)
                end
                petition.reload
              end
              it "queues a job to process the emails" do
                assert_enqueued_jobs 1 do
                  do_patch
                end
              end
              it "stamps the 'petition_email' email sent timestamp on each signature when the job runs" do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::context(emails out the petition email)::it#stamps the 'petition_email' email sent timestamp on each signature when the job runs has a flog score of 44            
perform_enqueued_jobs do
                  do_patch
                  petition.reload
                  petition_timestamp = petition.get_email_requested_at_for('petition_email')
                  expect(petition_timestamp).not_to be_nil
                  petition.signatures.validated.subscribed.each do |signature|
                    expect(signature.get_email_sent_at_for('petition_email')).to eq(petition_timestamp)
                  end
                end
              end
              it "should email out to the validated signees who have opted in when the delayed job runs" do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Email button)::describe(for an open petition)::describe(with valid params)::context(emails out the petition email)::it#should email out to the validated signees who have opted in when the delayed job runs has a flog score of 44            
perform_enqueued_jobs do
                  do_patch
                  expect(deliveries.length).to eq 5
                  expect(deliveries.map(&:to)).to eq([
                    [petition.creator.email],
                    ['laura_0@example.com'],
                    ['laura_1@example.com'],
                    ['laura_2@example.com'],
                    ['petitionscommittee@parliament.uk']
                  ])
                end
              end
            end
          end
          describe 'with invalid params' do  -       
let(:petition_email_attributes) do
              { subject: "", body: "" }
            end
            it 're-renders the petitions/show template' do
              do_patch
              expect(response).to be_success
              expect(response).to render_template('petitions/show')
            end
            it 'leaves the in-memory instance with errors' do
              do_patch
              expect(assigns(:email)).to be_present
              expect(assigns(:email).errors).not_to be_empty
            end
            it 'does not stores the email details in the db' do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Email button)::describe(for an open petition)::describe(with invalid params)::it#does not stores the email details in the db has a flog score of 38            
do_patch
              email.reload
              expect(email).to be_present
              expect(email.subject).to eq "Petition email subject"
              expect(email.body).to eq "Petition email body"
            end
          end
        end
        shared_examples_for 'trying to email supporters of a petition in the wrong state' do  -       
it 'raises a 404 error' do
            expect {
              do_patch
            }.to raise_error ActiveRecord::RecordNotFound
          end
          it 'does not stores the supplied email details in the db' do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Email button)::shared_examples_for(trying to email supporters of a petition in the wrong state)::it#does not stores the supplied email details in the db has a flog score of 38            
suppress(ActiveRecord::RecordNotFound) { do_patch }
            email.reload
            expect(email).to be_present
            expect(email.subject).to eq "Petition email subject"
            expect(email.body).to eq "Petition email body"
          end
        end
        describe 'for a pending petition' do
          before { petition.update_column(:state, Petition::PENDING_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a validated petition' do
          before { petition.update_column(:state, Petition::VALIDATED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a sponsored petition' do
          before { petition.update_column(:state, Petition::SPONSORED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
      end
      context 'when clicking the Save button' do
        def do_patch(overrides = {})  -       
params = {
            petition_id: petition.id,
            id: email.id,
            petition_email: petition_email_attributes,
            save: "Save"
          }
          patch :update, params.merge(overrides)
        end
        describe 'for an open petition' do
          it 'fetches the requested petition' do
            do_patch
            expect(assigns(:petition)).to eq petition
          end
          it 'fetches the requested email' do
            do_patch
            expect(assigns(:email)).to eq email
          end
          describe 'with valid params' do
            it 'redirects to the petition show page' do
              do_patch
              expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
            end
            it 'tells the moderator that their changes were saved' do
              do_patch
              expect(flash[:notice]).to eq 'Updated other parliamentary business successfully'
            end
            it 'stores the supplied email details in the db' do  -       
-                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Save button)::describe(for an open petition)::describe(with valid params)::it#stores the supplied email details in the db has a flog score of 54            
do_patch
              email.reload
              expect(email).to be_present
              expect(email.subject).to eq "New petition email subject"
              expect(email.body).to eq "New petition email body"
              expect(email.sent_by).to eq user.pretty_name
            end
            context "does not email out the petition email" do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Save button)::describe(for an open petition)::describe(with valid params)::context#does not email out the petition email has a flog score of 37            
before do
                3.times do |i|
                  attributes = {
                    name: "Laura #{i}",
                    email: "laura_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Sarah #{i}",
                    email: "sarah_#{i}@example.com",
                    notify_by_email: false,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Brian #{i}",
                    email: "brian_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  FactoryBot.create(:pending_signature, attributes)
                end
                petition.reload
              end
              it "does not queue a job to process the emails" do
                assert_enqueued_jobs 0 do
                  do_patch
                end
              end
            end
          end
          describe 'with invalid params' do  -       
let(:petition_email_attributes) do
              { subject: "", body: "" }
            end
            it 're-renders the petitions/show template' do
              do_patch
              expect(response).to be_success
              expect(response).to render_template('petitions/show')
            end
            it 'leaves the in-memory instance with errors' do
              do_patch
              expect(assigns(:email)).to be_present
              expect(assigns(:email).errors).not_to be_empty
            end
            it 'does not stores the email details in the db' do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Save button)::describe(for an open petition)::describe(with invalid params)::it#does not stores the email details in the db has a flog score of 30            
do_patch
              email.reload
              expect(email.subject).to eq("Petition email subject")
              expect(email.body).to eq("Petition email body")
            end
          end
        end
        shared_examples_for 'trying to email supporters of a petition in the wrong state' do  -       
it 'raises a 404 error' do
            expect {
              do_patch
            }.to raise_error ActiveRecord::RecordNotFound
          end
          it 'does not store the supplied email details in the db' do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Save button)::shared_examples_for(trying to email supporters of a petition in the wrong state)::it#does not store the supplied email details in the db has a flog score of 30            
suppress(ActiveRecord::RecordNotFound) { do_patch }
            email.reload
            expect(email.subject).to eq("Petition email subject")
            expect(email.body).to eq("Petition email body")
          end
        end
        describe 'for a pending petition' do
          before { petition.update_column(:state, Petition::PENDING_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a validated petition' do
          before { petition.update_column(:state, Petition::VALIDATED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a sponsored petition' do
          before { petition.update_column(:state, Petition::SPONSORED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
      end
      context 'when clicking the Preview button' do
        def do_patch(overrides = {})  -       
-                 describe(logged in as moderator user)::describe(PATCH /:id)::context#do_patch has a flog score of 37            
params = {
            petition_id: petition.id,
            id: email.id,
            petition_email: petition_email_attributes,
            save_and_preview: "Save and preview"
          }
          patch :update, params.merge(overrides)
        end
        describe 'for an open petition' do
          it 'fetches the requested petition' do
            do_patch
            expect(assigns(:petition)).to eq petition
          end
          it 'fetches the requested email' do
            do_patch
            expect(assigns(:email)).to eq email
          end
          describe 'with valid params' do
            it 'redirects to the petition show page' do
              do_patch
              expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
            end
            it 'tells the moderator that their changes were saved' do
              do_patch
              expect(flash[:notice]).to eq 'Preview email successfully sent'
            end
            it 'stores the supplied email details in the db' do  -       
-                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Preview button)::describe(for an open petition)::describe(with valid params)::it#stores the supplied email details in the db has a flog score of 54            
do_patch
              email.reload
              expect(email).to be_present
              expect(email.subject).to eq "New petition email subject"
              expect(email.body).to eq "New petition email body"
              expect(email.sent_by).to eq user.pretty_name
            end
            context "does not email out the petition email" do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Preview button)::describe(for an open petition)::describe(with valid params)::context#does not email out the petition email has a flog score of 37            
before do
                3.times do |i|
                  attributes = {
                    name: "Laura #{i}",
                    email: "laura_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Sarah #{i}",
                    email: "sarah_#{i}@example.com",
                    notify_by_email: false,
                    petition: petition
                  }
                  s = FactoryBot.create(:pending_signature, attributes)
                  s.validate!
                end
                2.times do |i|
                  attributes = {
                    name: "Brian #{i}",
                    email: "brian_#{i}@example.com",
                    notify_by_email: true,
                    petition: petition
                  }
                  FactoryBot.create(:pending_signature, attributes)
                end
                petition.reload
              end
              it "does not queue a job to process the emails" do
                assert_enqueued_jobs 0 do
                  do_patch
                end
              end
            end
            it "should email out a preview email" do  -       
-                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Preview button)::describe(for an open petition)::describe(with valid params)::it#should email out a preview email has a flog score of 34            
perform_enqueued_jobs do
                do_patch
                expect(deliveries.length).to eq 1
                expect(deliveries.map(&:to)).to eq([
                  ['petitionscommittee@parliament.uk']
                ])
              end
            end
          end
          describe 'with invalid params' do  -       
let(:petition_email_attributes) do
              { subject: "", body: "" }
            end
            it 're-renders the petitions/show template' do
              do_patch
              expect(response).to be_success
              expect(response).to render_template('petitions/show')
            end
            it 'leaves the in-memory instance with errors' do
              do_patch
              expect(assigns(:email)).to be_present
              expect(assigns(:email).errors).not_to be_empty
            end
            it 'does not stores the email details in the db' do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Preview button)::describe(for an open petition)::describe(with invalid params)::it#does not stores the email details in the db has a flog score of 30            
do_patch
              email.reload
              expect(email.subject).to eq("Petition email subject")
              expect(email.body).to eq("Petition email body")
            end
          end
        end
        shared_examples_for 'trying to email supporters of a petition in the wrong state' do  -       
it 'raises a 404 error' do
            expect {
              do_patch
            }.to raise_error ActiveRecord::RecordNotFound
          end
          it 'does not store the supplied email details in the db' do  -                 describe(logged in as moderator user)::describe(PATCH /:id)::context(when clicking the Preview button)::shared_examples_for(trying to email supporters of a petition in the wrong state)::it#does not store the supplied email details in the db has a flog score of 30            
suppress(ActiveRecord::RecordNotFound) { do_patch }
            email.reload
            expect(email.subject).to eq("Petition email subject")
            expect(email.body).to eq("Petition email body")
          end
        end
        describe 'for a pending petition' do
          before { petition.update_column(:state, Petition::PENDING_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a validated petition' do
          before { petition.update_column(:state, Petition::VALIDATED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
        describe 'for a sponsored petition' do
          before { petition.update_column(:state, Petition::SPONSORED_STATE) }
          it_behaves_like 'trying to email supporters of a petition in the wrong state'
        end
      end
    end
    describe 'DELETE /:id' do
      let(:email) do
        FactoryBot.create(
          :petition_email,
          petition: petition,
          subject: "Petition email subject",
          body: "Petition email body"
        )
      end
      def do_delete(overrides = {})  -       
params = { petition_id: petition.id, id: email.id }
        delete :destroy, params.merge(overrides)
      end
      describe 'for an open petition' do  -                 describe(logged in as moderator user)::describe(DELETE /:id)::describe#for an open petition has a flog score of 65            
let(:moderated) { double(:moderated) }
        let(:emails) { double(:emails) }
        before do
          expect(Petition).to receive(:moderated).and_return(moderated)
          expect(moderated).to receive(:find).with("#{petition.id}").and_return(petition)
          expect(petition).to receive(:emails).and_return(emails)
          expect(emails).to receive(:find).with("#{email.id}").and_return(email)
        end
        it 'fetches the requested petition' do
          do_delete
          expect(assigns(:petition)).to eq petition
        end
        it 'fetches the requested email' do
          do_delete
          expect(assigns(:email)).to eq email
        end
        context "when the delete is successful" do  -       
before do
            expect(email).to receive(:destroy).and_return(true)
          end
          it 'redirects to the petition show page' do
            do_delete
            expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
          end
          it 'tells the moderator that the record was deleted' do
            do_delete
            expect(flash[:notice]).to eq 'Deleted other parliamentary business successfully'
          end
        end
        context "when the delete is unsuccessful" do  -       
before do
            expect(email).to receive(:destroy).and_return(false)
          end
          it 'redirects to the petition show page' do
            do_delete
            expect(response).to redirect_to "https://moderate.petition.parliament.uk/admin/petitions/#{petition.id}"
          end
          it 'tells the moderator to contact support' do
            do_delete
            expect(flash[:notice]).to eq 'Unable to delete other parliamentary business - please contact support'
          end
        end
      end
    end
  end
end