kotovalexarian-likes-github/capybara-webkit
1
0
Fork 0
mirror of https://github.com/thoughtbot/capybara-webkit synced 2023-03-27 23:22:28 -04:00
Code Releases Activity

Ensure pending commands will block until the new window is loaded

Browse source
This commit is contained in:
Matthew Horan 2012-05-19 16:02:20 -04:00
parent efa577c3f0
commit 7149ec7a39
2 changed files with 14 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
spec/driver_spec.rb
View file

@ -1553,15 +1553,18 @@ describe Capybara::Driver::Webkit do
context "javascript new window app" do
before(:all) do
@app = lambda do |env|
if env['PATH_INFO'] == '/new_window'
request = ::Rack::Request.new(env)
if request.path == '/new_window'
body = <<-HTML
<html>
<script type="text/javascript">
window.open('http://#{env['HTTP_HOST']}/test');
window.open('http://#{request.host_with_port}/test?#{request.query_string}');
</script>
</html>
HTML
else
params = request.params
sleep params['sleep'].to_i if params['sleep']
body = "<html><p>finished</p></html>"
end
[200,
@ -1576,5 +1579,12 @@ describe Capybara::Driver::Webkit do
subject.find("//p").first.text.should == "finished"
end
end
it "waits for the new window to load" do
subject.visit("/new_window?sleep=1")
subject.within_window(nil) do
lambda { Timeout::timeout(1) { subject.find("//p") } }.should_not raise_error(Timeout::Error)
end
end
end
end

2
src/Connection.cpp
View file

@ -72,6 +72,8 @@ void Connection::writeResponse(Response *response) {
}
void Connection::changeWindow(WebPage *newPage) {
disconnect(m_page, SIGNAL(pageFinished(bool)), this, SLOT(pendingLoadFinished(bool)));
m_page = newPage;
connect(m_page, SIGNAL(pageFinished(bool)), this, SLOT(pendingLoadFinished(bool)));
}