Update Test related gemsupdate-test-gems

Upgrade to latest version the following gems:
* capybara
* capybara-screenshot
* selenium-webdriver

Updated specs to fix inconsistencies with capybara 3.x

1 files changed, 7 insertions, 2 deletions

diff --git a/spec/support/shared_examples/features/master_manages_access_requests_shared_example.rb b/spec/support/shared_examples/features/master_manages_access_requests_shared_example.rb
index 75ad948e42c..1b2caef405e 100644
--- a/spec/support/shared_examples/features/master_manages_access_requests_shared_example.rb
+++ b/spec/support/shared_examples/features/master_manages_access_requests_shared_example.rb
@@ -1,10 +1,11 @@
 RSpec.shared_examples 'Maintainer manages access requests' do
-  let(:user) { create(:user) }
   let(:maintainer) { create(:user) }
+  let(:user) { create(:user) }
 
   before do
     entity.request_access(user)
     entity.respond_to?(:add_owner) ? entity.add_owner(maintainer) : entity.add_maintainer(maintainer)
+
     sign_in(maintainer)
   end
 
@@ -19,7 +20,7 @@ RSpec.shared_examples 'Maintainer manages access requests' do
 
     expect_visible_access_request(entity, user)
 
-    accept_confirm { click_on 'Grant access' }
+    click_on 'Grant access'
 
     expect_no_visible_access_request(entity, user)
 
@@ -43,9 +44,13 @@ RSpec.shared_examples 'Maintainer manages access requests' do
     expect(entity.requesters.exists?(user_id: user)).to be_truthy
     expect(page).to have_content "Users requesting access to #{entity.name} 1"
     expect(page).to have_content user.name
+
+    WaitForRequests.wait_for_requests
   end
 
   def expect_no_visible_access_request(entity, user)
+    WaitForRequests.wait_for_requests
+
     expect(entity.requesters.exists?(user_id: user)).to be_falsy
     expect(page).not_to have_content "Users requesting access to #{entity.name}"
   end