Add tests for Admin::SchoolsController

I want to make sure that verifying and rejecting schools continues to
work as I make changes to the functionality.

Unfortunately we now see the following deprecation warning in our tests:

> DEPRECATION WARNING: The option :class_name is deprecated.
> Administrate should detect it automatically.

This is caused by our use of `Field::BelongsTo` to associate the school
and user in the `SchoolDashboard` configuration[1]. The `School#creator`
method isn't actually a `belongs_to` association so we have to pass the
`class_name` to let Administrate know that the method returns a user. I
can't see a way of avoiding this deprecation warning at the moment.

[1]: https://github.com/RaspberryPiFoundation/editor-api/blob/6a8f7903112de73b018d4b50f7701f0e0bc40924/app/dashboards/school_dashboard.rb#L14

Author: chrisroos

spec/features/admin/schools_spec.rb

@@ -0,0 +1,158 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'Schools', type: :request do
+  let(:admin_user) { create(:admin_user) }
+
+  before do
+    sign_in_as(admin_user)
+  end
+
+  describe 'GET #index' do
+    it 'responds 200' do
+      get admin_schools_path
+      expect(response).to have_http_status(:success)
+    end
+  end
+
+  describe 'GET #show' do
+    let(:creator) { create(:user) }
+    let(:verified_at) { nil }
+    let(:rejected_at) { nil }
+    let(:school) { create(:school, creator_id: creator.id, verified_at:, rejected_at:) }
+
+    before do
+      stub_user_info_api_for(creator)
+      get admin_school_path(school)
+    end
+
+    it 'responds 200' do
+      expect(response).to have_http_status(:success)
+    end
+
+    it 'includes link to verify school' do
+      expect(response.body).to include(I18n.t('administrate.actions.verify_school'))
+    end
+
+    it 'includes link to reject school' do
+      expect(response.body).to include(I18n.t('administrate.actions.reject_school'))
+    end
+
+    describe 'when the school is verified' do
+      let(:verified_at) { Time.zone.now }
+
+      it 'does not include a link to verify school' do
+        expect(response.body).not_to include(I18n.t('administrate.actions.verify_school'))
+      end
+
+      it 'includes link to reject school' do
+        expect(response.body).to include(I18n.t('administrate.actions.reject_school'))
+      end
+    end
+
+    describe 'when the school is rejected' do
+      let(:rejected_at) { Time.zone.now }
+
+      it 'includes a link to verify school' do
+        expect(response.body).to include(I18n.t('administrate.actions.verify_school'))
+      end
+
+      it 'does not include a link to reject school' do
+        expect(response.body).not_to include(I18n.t('administrate.actions.reject_school'))
+      end
+    end
+  end
+
+  describe 'POST #verify_school' do
+    let(:creator) { create(:user) }
+    let(:verified_at) { nil }
+    let(:school) { create(:school, creator_id: creator.id, verified_at:) }
+    let(:verification_result) { nil }
+    let(:verification_service) { instance_double(SchoolVerificationService, verify: verification_result) }
+
+    before do
+      stub_user_info_api_for(creator)
+      allow(SchoolVerificationService).to receive(:new).with(school.id).and_return(verification_service)
+
+      post admin_school_verify_school_path(school)
+    end
+
+    it 'redirects to school path' do
+      expect(response).to redirect_to(admin_school_path(school))
+    end
+
+    describe 'when verification was successful' do
+      let(:verification_result) { true }
+
+      before do
+        follow_redirect!
+      end
+
+      it 'displays success message' do
+        expect(response.body).to include(I18n.t('administrate.controller.verify_school.success'))
+      end
+    end
+
+    describe 'when verification was unsuccessful' do
+      let(:verification_result) { false }
+
+      before do
+        follow_redirect!
+      end
+
+      it 'displays failure message' do
+        expect(response.body).to include(I18n.t('administrate.controller.verify_school.error'))
+      end
+    end
+  end
+
+  describe 'PUT #reject_school' do
+    let(:creator) { create(:user) }
+    let(:school) { create(:school, creator_id: creator.id) }
+    let(:rejection_result) { nil }
+    let(:verification_service) { instance_double(SchoolVerificationService, reject: rejection_result) }
+
+    before do
+      stub_user_info_api_for(creator)
+      allow(SchoolVerificationService).to receive(:new).with(school.id).and_return(verification_service)
+
+      patch admin_school_reject_school_path(school)
+    end
+
+    it 'redirects to school path' do
+      expect(response).to redirect_to(admin_school_path(school))
+    end
+
+    describe 'when rejection was successful' do
+      let(:rejection_result) { true }
+
+      before do
+        follow_redirect!
+      end
+
+      it 'displays success message' do
+        expect(response.body).to include(I18n.t('administrate.controller.reject_school.success'))
+      end
+    end
+
+    describe 'when rejection was unsuccessful' do
+      let(:rejection_result) { false }
+
+      before do
+        follow_redirect!
+      end
+
+      it 'displays failure message' do
+        expect(response.body).to include(I18n.t('administrate.controller.reject_school.error'))
+      end
+    end
+  end
+
+  private
+
+  def sign_in_as(user)
+    allow(User).to receive(:from_omniauth).and_return(user)
+    get '/auth/callback'
+  end
+end