Only show verify/reject buttons for new schools

To prevent verified schools from being rejected and vice versa.

Author: chrisroos

app/views/admin/schools/show.html.erb

@@ -37,7 +37,7 @@ as well as a link to its edit page.
       method: :post,
       style: "background-color: green; margin: 5px;",
       data: { confirm: t("administrate.actions.confirm_verify_school") }
-    ) unless page.resource.verified? %>
+    ) unless page.resource.verified? || page.resource.rejected? %>
 
     <%= link_to(
       t("administrate.actions.reject_school"),
@@ -46,7 +46,7 @@ as well as a link to its edit page.
       method: :patch,
       style: "background-color: hsla(0, 88%, 33%, 1); margin: 5px 5px 5px 0;",
       data: { confirm: t("administrate.actions.confirm_reject_school") }
-    ) unless page.resource.rejected? %>
+    ) unless page.resource.verified? || page.resource.rejected? %>
   </div>
 </header>
 

spec/features/admin/schools_spec.rb

@@ -46,16 +46,16 @@
         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'))
+      it 'does not include a link to reject school' do
+        expect(response.body).not_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'))
+      it 'does not include a link to verify school' do
+        expect(response.body).not_to include(I18n.t('administrate.actions.verify_school'))
       end
 
       it 'does not include a link to reject school' do