capybara-webkit 0.7.2 → 0.8.0

data/lib/capybara/driver/webkit/cookie_jar.rb

@@ -0,0 +1,55 @@
+require 'webrick'
+
+# A simple cookie jar implementation.
+# Does not take special cookie attributes
+# into account like expire, max-age, httponly, secure
+class Capybara::Driver::Webkit::CookieJar
+  attr_reader :browser
+
+  def initialize(browser)
+    @browser = browser
+  end
+
+  def [](*args)
+    cookie = find(*args)
+    cookie && cookie.value
+  end
+
+  def find(name, domain = nil, path = "/")
+    # we are sorting by path size because more specific paths take
+    # precendence
+    cookies.sort_by { |c| -c.path.size }.find { |c|
+      c.name.downcase == name.downcase &&
+      (!domain || valid_domain?(c, domain)) &&
+      (!path   || valid_path?(c, path))
+    }
+  end
+
+ protected
+
+  def valid_domain?(cookie, domain)
+    ends_with?(("." + domain).downcase,
+               normalize_domain(cookie.domain).downcase)
+  end
+
+  def normalize_domain(domain)
+    domain = "." + domain unless domain[0,1] == "."
+    domain
+  end
+
+  def valid_path?(cookie, path)
+    starts_with?(path, cookie.path)
+  end
+
+  def ends_with?(str, suffix)
+    str[-suffix.size..-1] == suffix
+  end
+
+  def starts_with?(str, prefix)
+    str[0, prefix.size] == prefix
+  end
+
+  def cookies
+    browser.get_cookies.map { |c| WEBrick::Cookie.parse_set_cookie(c) }
+  end
+end

data/lib/capybara/driver/webkit/node.rb

@@ -74,7 +74,11 @@ class Capybara::Driver::Webkit
     end
 
     def path
-      raise Capybara::NotSupportedByDriverError
+      invoke "path"
+    end
+
+    def submit(opts = {})
+      invoke "submit"
     end
 
     def trigger(event)

data/lib/capybara/driver/webkit/version.rb

@@ -0,0 +1,7 @@
+module Capybara
+  module Driver
+    class Webkit
+      VERSION = '0.8.0'.freeze
+    end
+  end
+end

data/lib/capybara/webkit/matchers.rb

@@ -0,0 +1,39 @@
+module Capybara
+  module Webkit
+    module RspecMatchers
+      extend RSpec::Matchers::DSL
+
+      matcher :have_errors do |expected|
+        match do |actual|
+          actual = resolve(actual)
+          actual.error_messages.any?
+        end
+
+        failure_message_for_should do |actual|
+          "Expected Javascript errors, but there were none."
+        end
+        failure_message_for_should_not do |actual|
+          actual = resolve(actual)
+          "Expected no Javascript errors, got:\n#{error_messages_for(actual)}"
+        end
+
+        def error_messages_for(obj)
+          obj.error_messages.map do |m|
+            "  - #{m[:message]}"
+          end.join("\n")
+        end
+
+        def resolve(actual)
+          if actual.respond_to? :page
+            actual.page.driver
+          elsif actual.respond_to? :driver
+            actual.driver
+          else
+            actual
+          end
+        end
+      end
+
+    end
+  end
+end

data/lib/capybara_webkit_builder.rb

@@ -1,4 +1,5 @@
 require "fileutils"
+require "rbconfig"
 
 module CapybaraWebkitBuilder
   extend self
@@ -11,7 +12,7 @@ module CapybaraWebkitBuilder
   def makefile
     qmake_binaries = ['qmake', 'qmake-qt4']
     qmake = qmake_binaries.detect { |qmake| system("which #{qmake}") }
-    case RUBY_PLATFORM
+    case RbConfig::CONFIG['host_os']
     when /linux/
       system("#{qmake} -spec linux-g++")
     when /freebsd/

data/spec/browser_spec.rb

@@ -1,10 +1,16 @@
 require 'spec_helper'
+require 'self_signed_ssl_cert'
 require 'stringio'
 require 'capybara/driver/webkit/browser'
+require 'socket'
+require 'base64'
 
 describe Capybara::Driver::Webkit::Browser do
 
   let(:browser) { Capybara::Driver::Webkit::Browser.new }
+  let(:browser_ignore_ssl_err) {
+    Capybara::Driver::Webkit::Browser.new(:ignore_ssl_errors => true)
+  }
 
   describe '#server_port' do
     subject { browser.server_port }
@@ -23,13 +29,155 @@ describe Capybara::Driver::Webkit::Browser do
       new_browser.server_port.should_not == browser.server_port
     end
   end
-  
+
   it 'forwards stdout to the given IO object' do
     io = StringIO.new
     new_browser = Capybara::Driver::Webkit::Browser.new(:stdout => io)
     new_browser.execute_script('console.log("hello world")')
     sleep(0.5)
-    io.string.should == "hello world\n"
+    io.string.should include "hello world\n"
+  end
+
+  context 'handling of SSL validation errors' do
+    before do
+      # set up minimal HTTPS server
+      @host = "127.0.0.1"
+      @server = TCPServer.new(@host, 0)
+      @port = @server.addr[1]
+
+      # set up SSL layer
+      ssl_serv = OpenSSL::SSL::SSLServer.new(@server, $openssl_self_signed_ctx)
+
+      @server_thread = Thread.new(ssl_serv) do |serv|
+        while conn = serv.accept do
+          # read request
+          request = []
+          until (line = conn.readline.strip).empty?
+            request << line
+          end
+
+          # write response
+          html = "<html><body>D'oh!</body></html>"
+          conn.write "HTTP/1.1 200 OK\r\n"
+          conn.write "Content-Type:text/html\r\n"
+          conn.write "Content-Length: %i\r\n" % html.size
+          conn.write "\r\n"
+          conn.write html
+          conn.close
+        end
+      end
+    end
+
+    after do
+      @server_thread.kill
+      @server.close
+    end
+
+    it "doesn't accept a self-signed certificate by default" do
+      lambda { browser.visit "https://#{@host}:#{@port}/" }.should raise_error
+    end
+
+    it 'accepts a self-signed certificate if configured to do so' do
+      browser_ignore_ssl_err.visit "https://#{@host}:#{@port}/"
+    end
   end
+  describe "forking" do
+    it "only shuts down the server from the main process" do
+      browser.reset!
+      pid = fork {}
+      Process.wait(pid)
+      expect { browser.reset! }.not_to raise_error
+    end
+  end
+
+  describe '#set_proxy' do
+    before do
+      @host = '127.0.0.1'
+      @user = 'user'
+      @pass = 'secret'
+      @url  = "http://example.org/"
+
+      @server = TCPServer.new(@host, 0)
+      @port = @server.addr[1]
+
+      @proxy_requests = []
+      @proxy = Thread.new(@server, @proxy_requests) do |serv, proxy_requests|
+        while conn = serv.accept do
+          # read request
+          request = []
+          until (line = conn.readline.strip).empty?
+            request << line
+          end
 
+          # send response
+          auth_header = request.find { |h| h =~ /Authorization:/i }
+          if auth_header || request[0].split(/\s+/)[1] =~ /^\//
+            html = "<html><body>D'oh!</body></html>"
+            conn.write "HTTP/1.1 200 OK\r\n"
+            conn.write "Content-Type:text/html\r\n"
+            conn.write "Content-Length: %i\r\n" % html.size
+            conn.write "\r\n"
+            conn.write html
+            conn.close
+            proxy_requests << request if auth_header
+          else
+            conn.write "HTTP/1.1 407 Proxy Auth Required\r\n"
+            conn.write "Proxy-Authenticate: Basic realm=\"Proxy\"\r\n"
+            conn.write "\r\n"
+            conn.close
+            proxy_requests << request
+          end
+        end
+      end
+
+      browser.set_proxy(:host => @host,
+                        :port => @port,
+                        :user => @user,
+                        :pass => @pass)
+      browser.visit @url
+      @proxy_requests.size.should == 2
+      @request = @proxy_requests[-1]
+    end
+
+    after do
+      @proxy.kill
+      @server.close
+    end
+
+    it 'uses the HTTP proxy correctly' do
+      @request[0].should match /^GET\s+http:\/\/example.org\/\s+HTTP/i
+      @request.find { |header|
+        header =~ /^Host:\s+example.org$/i }.should_not be nil
+    end
+
+    it 'sends correct proxy authentication' do
+      auth_header = @request.find { |header|
+        header =~ /^Proxy-Authorization:\s+/i }
+      auth_header.should_not be nil
+
+      user, pass = Base64.decode64(auth_header.split(/\s+/)[-1]).split(":")
+      user.should == @user
+      pass.should == @pass
+    end
+
+    it "uses the proxies' response" do
+      browser.body.should include "D'oh!"
+    end
+
+    it 'uses original URL' do
+      browser.url.should == @url
+    end
+
+    it 'uses URLs changed by javascript' do
+      browser.execute_script "window.history.pushState('', '', '/blah')"
+      browser.requested_url.should == 'http://example.org/blah'
+    end
+
+    it 'is possible to disable proxy again' do
+      @proxy_requests.clear
+      browser.clear_proxy
+      browser.visit "http://#{@host}:#{@port}/"
+      @proxy_requests.size.should == 0
+    end
+  end
 end

data/spec/cookie_jar_spec.rb

@@ -0,0 +1,48 @@
+require 'spec_helper'
+require 'capybara/driver/webkit/cookie_jar'
+
+describe Capybara::Driver::Webkit::CookieJar do
+  let(:browser) {
+    browser = double("Browser")
+    browser.stub(:get_cookies) { [
+        "cookie1=1; domain=.example.org; path=/",
+        "cookie1=2; domain=.example.org; path=/dir1/",
+        "cookie1=3; domain=.facebook.com; path=/",
+        "cookie2=4; domain=.sub1.example.org; path=/",
+      ] }
+    browser
+  }
+
+  subject { Capybara::Driver::Webkit::CookieJar.new(browser) }
+
+  describe "#find" do
+    it "returns a cookie object" do
+      subject.find("cookie1", "www.facebook.com").domain.should == ".facebook.com"
+    end
+
+    it "returns the right cookie for every given domain/path" do
+      subject.find("cookie1", "example.org").value.should == "1"
+      subject.find("cookie1", "www.facebook.com").value.should == "3"
+      subject.find("cookie2", "sub1.example.org").value.should == "4"
+    end
+
+    it "does not return a cookie from other domain" do
+      subject.find("cookie2", "www.example.org").should == nil
+    end
+
+    it "respects path precedence rules" do
+      subject.find("cookie1", "www.example.org").value.should == "1"
+      subject.find("cookie1", "www.example.org", "/dir1/123").value.should == "2"
+    end
+  end
+
+  describe "#[]" do
+    it "returns the first matching cookie's value" do
+      subject["cookie1", "example.org"].should == "1"
+    end
+
+    it "returns nil if no cookie is found" do
+      subject["notexisting"].should == nil
+    end
+  end
+end

data/spec/driver_spec.rb

@@ -20,7 +20,7 @@ describe Capybara::Driver::Webkit do
           p_id = "greeting"
           msg  = "hello"
           iframe = "<iframe id=\"f\" src=\"/?iframe=true\"></iframe>"
-        end 
+        end
         body = <<-HTML
           <html>
             <head>
@@ -140,12 +140,16 @@ describe Capybara::Driver::Webkit do
       @app = lambda do |env|
         [200, {"Content-Type" => "text/css", "Content-Length" => body.length.to_s}, [body]]
       end
+      subject.visit("/")
     end
 
     it "renders unsupported content types gracefully" do
-      subject.visit("/")
       subject.body.should =~ /css/
     end
+
+    it "sets the response headers with respect to the unsupported request" do
+      subject.response_headers["Content-Type"].should == "text/css"
+    end
   end
 
   context "hello app" do
@@ -219,9 +223,9 @@ describe Capybara::Driver::Webkit do
       subject.current_url.should == "http://127.0.0.1:#{port}/hello/world?success=true"
     end
 
-    it "escapes URLs" do
-      subject.visit("/hello there")
-      subject.current_url.should =~ /hello%20there/
+    it "does not double-encode URLs" do
+      subject.visit("/hello/world?success=%25true")
+      subject.current_url.should =~ /success=\%25true/
     end
 
     it "visits a page with an anchor" do
@@ -316,6 +320,40 @@ describe Capybara::Driver::Webkit do
     end
   end
 
+  context "console messages app" do
+
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html>
+            <head>
+            </head>
+            <body>
+              <script type="text/javascript">
+                console.log("hello");
+                console.log("hello again");
+                oops
+              </script>
+            </body>
+          </html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+
+    it "collects messages logged to the console" do
+      subject.console_messages.first.should include :source, :message => "hello", :line_number => 6
+      subject.console_messages.length.should eq 3
+    end
+
+    it "logs errors to the console" do
+      subject.error_messages.length.should eq 1
+    end
+
+  end
+
   context "form app" do
     before(:all) do
       @app = lambda do |env|
@@ -497,8 +535,13 @@ describe Capybara::Driver::Webkit do
         body = <<-HTML
           <html><body>
             <form action="/" method="GET">
-              <input class="watch" type="text"/>
+              <input class="watch" type="email"/>
+              <input class="watch" type="number"/>
               <input class="watch" type="password"/>
+              <input class="watch" type="search"/>
+              <input class="watch" type="tel"/>
+              <input class="watch" type="text"/>
+              <input class="watch" type="url"/>
               <textarea class="watch"></textarea>
               <input class="watch" type="checkbox"/>
               <input class="watch" type="radio"/>
@@ -540,9 +583,11 @@ describe Capybara::Driver::Webkit do
        %w{change blur}).flatten
     end
 
-    it "triggers text input events" do
-      subject.find("//input[@type='text']").first.set(newtext)
-      subject.find("//li").map(&:text).should == keyevents
+    %w(email number password search tel text url).each do | field_type |
+      it "triggers text input events on inputs of type #{field_type}" do
+        subject.find("//input[@type='#{field_type}']").first.set(newtext)
+        subject.find("//li").map(&:text).should == keyevents
+      end
     end
 
     it "triggers textarea input events" do
@@ -550,11 +595,6 @@ describe Capybara::Driver::Webkit do
       subject.find("//li").map(&:text).should == keyevents
     end
 
-    it "triggers password input events" do
-      subject.find("//input[@type='password']").first.set(newtext)
-      subject.find("//li").map(&:text).should == keyevents
-    end
-
     it "triggers radio input events" do
       subject.find("//input[@type='radio']").first.set(true)
       subject.find("//li").map(&:text).should == %w(click change)
@@ -838,7 +878,7 @@ describe Capybara::Driver::Webkit do
           [body]]
       end
     end
-    
+
     it "raises a webkit error for the requested url" do
       make_the_server_go_away
       expect {
@@ -853,7 +893,7 @@ describe Capybara::Driver::Webkit do
       subject.browser.instance_variable_get(:@socket).unstub!(:puts)
       subject.browser.instance_variable_get(:@socket).unstub!(:print)
     end
- 
+
     def make_the_server_go_away
       subject.browser.instance_variable_get(:@socket).stub!(:gets).and_return(nil)
       subject.browser.instance_variable_get(:@socket).stub!(:puts)
@@ -943,6 +983,10 @@ describe Capybara::Driver::Webkit do
       cookie["domain"].should include "127.0.0.1"
       cookie["path"].should == "/"
     end
+
+    it "allows reading access to cookies using a nice syntax" do
+      subject.cookies["cookie"].should == "abc"
+    end
   end
 
   context "with socket debugger" do
@@ -1014,6 +1058,98 @@ describe Capybara::Driver::Webkit do
     end
   end
 
+  context "app with a lot of HTML tags" do
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html>
+            <head>
+              <title>My eBook</title>
+              <meta class="charset" name="charset" value="utf-8" />
+              <meta class="author" name="author" value="Firstname Lastname" />
+            </head>
+            <body>
+              <div id="toc">
+                <table>
+                  <thead id="head">
+                    <tr><td class="td1">Chapter</td><td>Page</td></tr>
+                  </thead>
+                  <tbody>
+                    <tr><td>Intro</td><td>1</td></tr>
+                    <tr><td>Chapter 1</td><td class="td2">1</td></tr>
+                    <tr><td>Chapter 2</td><td>1</td></tr>
+                  </tbody>
+                </table>
+              </div>
+
+              <h1 class="h1">My first book</h1>
+              <p class="p1">Written by me</p>
+              <div id="intro" class="intro">
+                <p>Let's try out XPath</p>
+                <p class="p2">in capybara-webkit</p>
+              </div>
+
+              <h2 class="chapter1">Chapter 1</h2>
+              <p>This paragraph is fascinating.</p>
+              <p class="p3">But not as much as this one.</p>
+
+              <h2 class="chapter2">Chapter 2</h2>
+              <p>Let's try if we can select this</p>
+            </body>
+          </html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+
+    it "builds up node paths correctly" do
+      cases = {
+        "//*[contains(@class, 'author')]"    => "/html/head/meta[2]",
+        "//*[contains(@class, 'td1')]"       => "/html/body/div[@id='toc']/table/thead[@id='head']/tr/td[1]",
+        "//*[contains(@class, 'td2')]"       => "/html/body/div[@id='toc']/table/tbody/tr[2]/td[2]",
+        "//h1"                               => "/html/body/h1",
+        "//*[contains(@class, 'chapter2')]"  => "/html/body/h2[2]",
+        "//*[contains(@class, 'p1')]"        => "/html/body/p[1]",
+        "//*[contains(@class, 'p2')]"        => "/html/body/div[@id='intro']/p[2]",
+        "//*[contains(@class, 'p3')]"        => "/html/body/p[3]",
+      }
+
+      cases.each do |xpath, path|
+        nodes = subject.find(xpath)
+        nodes.size.should == 1
+        nodes[0].path.should == path
+      end
+    end
+  end
+
+  context "css overflow app" do
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html>
+            <head>
+              <style type="text/css">
+                #overflow { overflow: hidden }
+              </style>
+            </head>
+            <body>
+              <div id="overflow">Overflow</div>
+            </body>
+          </html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+
+    it "handles overflow hidden" do
+      subject.find("//div[@id='overflow']").first.text.should == "Overflow"
+    end
+  end
+
   context "javascript redirect app" do
     before(:all) do
       @app = lambda do |env|
@@ -1041,4 +1177,100 @@ describe Capybara::Driver::Webkit do
       end
     end
   end
+
+  context "app with a lot of HTML tags" do
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html>
+            <head>
+              <title>My eBook</title>
+              <meta class="charset" name="charset" value="utf-8" />
+              <meta class="author" name="author" value="Firstname Lastname" />
+            </head>
+            <body>
+              <div id="toc">
+                <table>
+                  <thead id="head">
+                    <tr><td class="td1">Chapter</td><td>Page</td></tr>
+                  </thead>
+                  <tbody>
+                    <tr><td>Intro</td><td>1</td></tr>
+                    <tr><td>Chapter 1</td><td class="td2">1</td></tr>
+                    <tr><td>Chapter 2</td><td>1</td></tr>
+                  </tbody>
+                </table>
+              </div>
+
+              <h1 class="h1">My first book</h1>
+              <p class="p1">Written by me</p>
+              <div id="intro" class="intro">
+                <p>Let's try out XPath</p>
+                <p class="p2">in capybara-webkit</p>
+              </div>
+
+              <h2 class="chapter1">Chapter 1</h2>
+              <p>This paragraph is fascinating.</p>
+              <p class="p3">But not as much as this one.</p>
+
+              <h2 class="chapter2">Chapter 2</h2>
+              <p>Let's try if we can select this</p>
+            </body>
+          </html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+
+    it "builds up node paths correctly" do
+      cases = {
+        "//*[contains(@class, 'author')]"    => "/html/head/meta[2]",
+        "//*[contains(@class, 'td1')]"       => "/html/body/div[@id='toc']/table/thead[@id='head']/tr/td[1]",
+        "//*[contains(@class, 'td2')]"       => "/html/body/div[@id='toc']/table/tbody/tr[2]/td[2]",
+        "//h1"                               => "/html/body/h1",
+        "//*[contains(@class, 'chapter2')]"  => "/html/body/h2[2]",
+        "//*[contains(@class, 'p1')]"        => "/html/body/p[1]",
+        "//*[contains(@class, 'p2')]"        => "/html/body/div[@id='intro']/p[2]",
+        "//*[contains(@class, 'p3')]"        => "/html/body/p[3]",
+      }
+
+      cases.each do |xpath, path|
+        nodes = subject.find(xpath)
+        nodes.size.should == 1
+        nodes[0].path.should == path
+      end
+    end
+  end
+
+  context "form app with server-side handler" do
+    before(:all) do
+      @app = lambda do |env|
+        if env["REQUEST_METHOD"] == "POST"
+          body = "<html><body><p>Congrats!</p></body></html>"
+        else
+          body = <<-HTML
+            <html>
+              <head><title>Form</title>
+              <body>
+                <form action="/" method="POST">
+                  <input type="hidden" name="abc" value="123" />
+                  <input type="submit" value="Submit" />
+                </form>
+              </body>
+            </html>
+          HTML
+        end
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+
+    it "submits a form without clicking" do
+      subject.find("//form")[0].submit
+      subject.body.should include "Congrats"
+    end
+  end
 end