Merge pull request #6262 from samvera/coll-ability

skip ActiveFedora specific setup and tests for CollectionAbility

spec/abilities/collection_ability_spec.rb

@@ -10,26 +10,26 @@
   let(:collection_type) { FactoryBot.create(:collection_type) }
   let(:collection_type_gid) { collection_type.to_global_id }
 
-  # rubocop:disable RSpec/InstanceVariable
-  before :all do
-    @current_collection_model = Hyrax.config.collection_model
-
-    # abilities always test Hyrax::PcdmCollection, but only test Collection
-    # if it is configured as the collection_model
-    Hyrax.config.collection_model = 'Collection'
+  # abilities always test Hyrax::PcdmCollection, but only test Collection
+  # if it is configured as the collection_model
+  around(:each) do |example|
+    current_collection_model = Hyrax.config.collection_model
+    Hyrax.config.collection_model = 'Collection' unless Hyrax.config.disable_wings
+
+    example.run
+    Hyrax.config.collection_model = current_collection_model
   end
 
   after :all do
     Hyrax.config.collection_model = @current_collection_model
   end
-  # rubocop:enable RSpec/InstanceVariable
 
   # rubocop:disable RSpec/ExampleLength
   context 'when admin user' do
     let(:current_user) { admin }
     let(:admin) { FactoryBot.create(:admin, email: 'admin@example.com') }
 
-    context 'and collection is an ActiveFedora::Base' do
+    context 'and collection is an ActiveFedora::Base', :active_fedora do
       let!(:collection) do
         FactoryBot.build(:collection_lw, id: 'col_au',
                                          user: user,
@@ -91,7 +91,7 @@
     let(:current_user) { manager }
     let(:manager) { FactoryBot.create(:user, email: 'manager@example.com') }
 
-    context 'and collection is an ActiveFedora::Base' do
+    context 'and collection is an ActiveFedora::Base', :active_fedora do
       let!(:collection) do
         FactoryBot.build(:collection_lw, id: 'col_mu',
                                          user: user,
@@ -177,7 +177,7 @@
     let(:current_user) { depositor }
     let(:depositor) { FactoryBot.create(:user, email: 'depositor@example.com') }
 
-    context 'and collection is an ActiveFedora::Base' do
+    context 'and collection is an ActiveFedora::Base', :active_fedora do
       let!(:collection) do
         FactoryBot.build(:collection_lw, id: 'col_du',
                                          user: user,
@@ -265,7 +265,7 @@
     let(:current_user) { viewer }
     let(:viewer) { FactoryBot.create(:user, email: 'viewer@example.com') }
 
-    context 'and collection is an ActiveFedora::Base' do
+    context 'and collection is an ActiveFedora::Base', :active_fedora do
       let!(:collection) do
         FactoryBot.build(:collection_lw, id: 'col_vu',
                                          user: user,
@@ -351,7 +351,7 @@
     let(:current_user) { other_user }
     let(:other_user) { FactoryBot.create(:user, email: 'other_user@example.com') }
 
-    context 'and collection is an ActiveFedora::Base' do
+    context 'and collection is an ActiveFedora::Base', :active_fedora do
       let!(:collection) do
         FactoryBot.create(:collection_lw, id: 'as',
                                           user: user,