Skip to content

Instantly share code, notes, and snippets.

@metaskills
Last active October 20, 2023 13:08
Show Gist options
  • Save metaskills/1172519 to your computer and use it in GitHub Desktop.
Save metaskills/1172519 to your computer and use it in GitHub Desktop.
Never sleep() using Capybara!
# WAIT! Do consider that `wait` may not be needed. This article describes
# that reasoning. Please read it and make informed decisions.
# https://www.varvet.com/blog/why-wait_until-was-removed-from-capybara/
# Have you ever had to sleep() in Capybara-WebKit to wait for AJAX and/or CSS animations?
describe 'Modal' do
should 'display login errors' do
visit root_path
click_link 'My HomeMarks'
within '#login_area' do
fill_in 'email', with: 'will@not.work'
fill_in 'password', with: 'test'
click_button 'Login'
end
# DO NOT sleep(1) HERE!
assert_modal_visible
page.find(modal_wrapper_id).text.must_match %r{login failed.*use the forgot password}i
end
end
# Avoid it by using Capybara's #wait_until method. My modal visible/hidden helpers
# do just that. The #wait_until uses the default timeout value.
def modal_wrapper_id
'#hmarks_modal_sheet_wrap'
end
def assert_modal_visible
wait_until { page.find(modal_wrapper_id).visible? }
rescue Capybara::TimeoutError
flunk 'Expected modal to be visible.'
end
def assert_modal_hidden
wait_until { !page.find(modal_wrapper_id).visible? }
rescue Capybara::TimeoutError
flunk 'Expected modal to be hidden.'
end
# Examples of waiting for a page loading to show and hide in jQuery Mobile.
def wait_for_loading
wait_until { page.find('html')[:class].include?('ui-loading') }
rescue Capybara::TimeoutError
flunk "Failed at waiting for loading to appear."
end
def wait_for_loaded
wait_until { !page.find('html')[:class].include?('ui-loading') }
rescue Capybara::TimeoutError
flunk "Failed at waiting for loading to complete."
end
def wait_for_page_load
wait_for_loading && wait_for_loaded
end
@metaskills
Copy link
Author

Well that is fine for AJAX requests but I have found that waiting for AJAX requests is a small part of writing good async tests. I think the focus should be on the DOM, especially in regards to animations and transitions. Thankfully, capybara already does this in many ways automatically and in some cases the rspec matches it provides even goes one step further. I have seen way to many capybara-webkit bugs/failures that are in fractions of a second while the DOM is in transition. Less so due to AJAX requests.

@polarblau
Copy link

A variation:

it 'lets the user login', :js => true do
  wait_until_scope_exists '#login-form' do
    fill_in 'Email', :with => 'foo@bar.com' 
    fill_in 'Password', :with => 'secret'
    click_on 'Sign in'
  end
  page.should have_content('Welcome!')
end

# ...

def wait_until_scope_exists(scope, &block)
  wait_until { page.has_css?(scope) }
  within scope, &block
rescue Capybara::TimeoutError
  flunk "Expected '#{scope}' to be present."
end

@megabga
Copy link

megabga commented Sep 20, 2012

Thanks, its what a looking for, works like a charm!

@nathanbain
Copy link

Be aware that #wait_until has been removed from Capybara 2.0.
Read the notes here: https://groups.google.com/forum/?fromgroups=#!topic/ruby-capybara/qQYWpQb9FzY

@NewAlexandria
Copy link

Yes, has_content and has_css is the contemporary practice — letting capybara configs decide, abstract, and dry-up the configs

@Brantron
Copy link

Just found this while researching modal issues LOL

@dpulliam
Copy link

Lol, brandon, i just stumbled on this too! Great stuff ken :)

@Kufert
Copy link

Kufert commented Jun 29, 2017

I have an issue where a plugin I use adds the click event after 200ms, so w/o the sleep the click dose nothing. I can't wait for html/css, as all of that is already on the screen. Is there a way to wait for the click event to be added or am I doomed to use sleep?

@kevinhq
Copy link

kevinhq commented Nov 1, 2018

wait_until is removed from capybara now?

@KamilBehnke
Copy link

@kevinhq yes, it is removed from capybara

@heridev
Copy link

heridev commented May 11, 2019

For that scenario when maybe the loading spinner is blocking the ui and you need to force sleep in a smarter way, what about?

      while page.has_css?('.pageLoader')
        puts "waiting"
        sleep 0.1
      end

@artur79
Copy link

artur79 commented Sep 9, 2021

anyone, some ajax snippet for rails_ujs which check ajax request done ?

@equivalent
Copy link

equivalent commented Oct 20, 2023

there is "wait" option built in capybara.

find("#element_that_is_added_by_turbo", wait: 1)

above will wait up to 1s to element to appear

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment