Raise server errors on visit and synchronize

This should hopefully make them appear earlier and be more helpful.

lib/capybara/node/base.rb

@@ -80,6 +80,7 @@ def synchronize(seconds=Capybara.default_wait_time)
           begin
             yield
           rescue => e
+            session.raise_server_error!
             raise e unless driver.wait?
             raise e unless catch_error?(e)
             raise e if (Time.now - start_time) >= seconds

lib/capybara/session.rb

@@ -81,12 +81,20 @@ def reset!
         assert_no_selector :xpath, "/html/body/*"
         @touched = false
       end
+      raise_server_error!
+    end
+    alias_method :cleanup!, :reset!
+    alias_method :reset_session!, :reset!
+
+    ##
+    #
+    # Raise errors encountered in the server
+    #
+    def raise_server_error!
       raise @server.error if Capybara.raise_server_errors and @server and @server.error
     ensure
       @server.reset_error! if @server
     end
-    alias_method :cleanup!, :reset!
-    alias_method :reset_session!, :reset!
 
     ##
     #
@@ -179,6 +187,8 @@ def title
     # @param [String] url     The URL to navigate to
     #
     def visit(url)
+      raise_server_error!
+
       @touched = true
 
       if url !~ /^http/ and Capybara.app_host

lib/capybara/spec/session/click_link_spec.rb

@@ -14,6 +14,13 @@
     @session.should have_content('Another World')
   end
 
+  it "raises any errors caught inside the server", :requires => [:server] do
+    quietly { @session.visit("/error") }
+    expect do
+      @session.click_link('foo')
+    end.to raise_error(TestApp::TestAppError)
+  end
+
   context "with id given" do
     it "should take user to the linked page" do
       @session.click_link('foo')

lib/capybara/spec/session/visit_spec.rb

@@ -26,6 +26,13 @@
     @session.should have_content('Hello world!')
   end
 
+  it "raises any errors caught inside the server", :requires => [:server] do
+    quietly { @session.visit("/error") }
+    expect do
+      @session.visit("/")
+    end.to raise_error(TestApp::TestAppError)
+  end
+
   context "when Capybara.always_include_port is true" do
 
     let(:root_uri) do