capybara-webkit 0.6.1 → 0.7.0

data/Appraisals

@@ -0,0 +1,7 @@
+appraise "1.0" do
+  gem "capybara", "~> 1.0.0"
+end
+
+appraise "1.1" do
+  gem "capybara", "~> 1.1.0"
+end

data/Gemfile

@@ -1,6 +1,8 @@
 source "http://rubygems.org"
-gem "rspec", '~> 2.6.0', :require => false
-gem "capybara"
-gem "sinatra", :require => false
-gem "mini_magick", :require => false
-gem 'rake'
+gemspec
+# gem "appraisal"
+# gem "rspec", '~> 2.6.0', :require => false
+# gem "capybara"
+# gem "sinatra", :require => false
+# gem "mini_magick", :require => false
+# gem 'rake'

data/Gemfile.lock

@@ -1,24 +1,33 @@
+PATH
+  remote: .
+  specs:
+    capybara-webkit (0.7.0)
+      capybara (< 1.2, >= 1.0.0)
+
 GEM
   remote: http://rubygems.org/
   specs:
-    capybara (1.0.0)
+    appraisal (0.3.8)
+      bundler
+      rake
+    capybara (1.1.1)
       mime-types (>= 1.16)
       nokogiri (>= 1.3.3)
       rack (>= 1.0.0)
       rack-test (>= 0.5.4)
-      selenium-webdriver (~> 0.2.0)
+      selenium-webdriver (~> 2.0)
       xpath (~> 0.1.4)
-    childprocess (0.1.9)
+    childprocess (0.2.2)
       ffi (~> 1.0.6)
     diff-lcs (1.1.2)
     ffi (1.0.9)
-    json_pure (1.5.3)
+    json_pure (1.6.1)
     mime-types (1.16)
     mini_magick (3.2.1)
       subexec (~> 0.0.4)
     nokogiri (1.5.0)
-    rack (1.3.0)
-    rack-test (0.6.0)
+    rack (1.3.2)
+    rack-test (0.6.1)
       rack (>= 1.0)
     rake (0.9.2)
     rspec (2.6.0)
@@ -30,8 +39,8 @@ GEM
       diff-lcs (~> 1.1.2)
     rspec-mocks (2.6.0)
     rubyzip (0.9.4)
-    selenium-webdriver (0.2.2)
-      childprocess (>= 0.1.9)
+    selenium-webdriver (2.7.0)
+      childprocess (>= 0.2.1)
       ffi (>= 1.0.7)
       json_pure
       rubyzip
@@ -47,7 +56,8 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  capybara
+  appraisal
+  capybara-webkit!
   mini_magick
   rake
   rspec (~> 2.6.0)

data/README.md

@@ -3,36 +3,13 @@ capybara-webkit
 
 A [capybara](https://github.com/jnicklas/capybara) driver that uses [WebKit](http://webkit.org) via [QtWebKit](http://doc.qt.nokia.com/4.7/qtwebkit.html).
 
-Dependent on Qt
----------------
+Qt Dependency
+-------------
 
-capybara-webkit depends on a WebKit implementation from Qt, a cross-platform development toolkit. You'll need to download the Qt libraries to build and install the gem.
-
-OS X Lion 10.7:
-
-Install Qt via [homebrew](http://mxcl.github.com/homebrew/)(can take more than an hour) using:
-
-    brew install qt --build-from-source
-
-OS X < 10.7:
-
-[Download the non-debug Cocoa package](http://qt.nokia.com/downloads/qt-for-open-source-cpp-development-on-mac-os-x).
-
-Ubuntu:
-
-    apt-get install libqt4-dev
-
-Fedora:
-
-    yum install qt-webkit-devel
-
-Gentoo Linux:
-
-    emerge x11-libs/qt-webkit
-
-Other Linux distributions:
-
-[Download this package](http://qt.nokia.com/downloads/linux-x11-cpp).
+capybara-webkit depends on a WebKit implementation from Qt, a cross-platform
+development toolkit. You'll need to download the Qt libraries to build and
+install the gem. You can find instructions for downloading and installing QT on
+the [capybara-webkit wiki](https://github.com/thoughtbot/capybara-webkit/wiki/Installing-QT)
 
 CI
 --
@@ -73,11 +50,9 @@ The names and logos for thoughtbot are trademarks of thoughtbot, inc.
 Notes
 -----
 
-This capybara WebKit driver will listen on port 8200, this may conflict
-with other services.
+capybara-webkit will listen on port 8200. This may conflict with other services.
 
 License
 -------
 
-capybara-webkit is Copyright © 2011 thoughtbot, inc. It is free software, and may be redistributed under the terms specified in the LICENSE file.
-
+capybara-webkit is Copyright (c) 2011 thoughtbot, inc. It is free software, and may be redistributed under the terms specified in the LICENSE file.

data/Rakefile

@@ -3,6 +3,7 @@ require 'bundler/setup'
 require 'rspec/core/rake_task'
 require 'rake/gempackagetask'
 require 'capybara_webkit_builder'
+require 'appraisal'
 
 desc "Generate a Makefile using qmake"
 file 'Makefile' do

data/capybara-webkit.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = "capybara-webkit"
-  s.version = "0.6.1"
+  s.version = "0.7.0"
   s.authors = ["thoughtbot", "Joe Ferris", "Jason Morrison", "Tristan Dunn",
                "Joshua Clayton", "Yuichi Tateno", "Aaron Gibralter",
                "Vasily Reys", "petrushka", "John Bintz", "Chad Pytel",
@@ -13,7 +13,12 @@ Gem::Specification.new do |s|
   s.require_path = "lib"
   s.rubygems_version = "1.3.5"
   s.summary = "Headless Webkit driver for Capybara"
-  s.add_runtime_dependency "capybara", "~> 1.0.0"
+  s.add_runtime_dependency "capybara", [">= 1.0.0", "< 1.2"]
+  s.add_development_dependency "rspec", "~> 2.6.0"
+  s.add_development_dependency "sinatra"
+  s.add_development_dependency "mini_magick"
+  s.add_development_dependency "rake"
+  s.add_development_dependency "appraisal"
   s.extensions = "extconf.rb"
 end
 

data/gemfiles/1.0.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "capybara", "~> 1.0.0"
+
+gemspec :path=>"../"

data/gemfiles/1.0.gemfile.lock

@@ -0,0 +1,65 @@
+PATH
+  remote: /Users/jferris/Source/capybara-webkit
+  specs:
+    capybara-webkit (0.6.1)
+      capybara (< 1.2, >= 1.0.0)
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    appraisal (0.3.8)
+      bundler
+      rake
+    capybara (1.0.1)
+      mime-types (>= 1.16)
+      nokogiri (>= 1.3.3)
+      rack (>= 1.0.0)
+      rack-test (>= 0.5.4)
+      selenium-webdriver (~> 2.0)
+      xpath (~> 0.1.4)
+    childprocess (0.2.2)
+      ffi (~> 1.0.6)
+    diff-lcs (1.1.3)
+    ffi (1.0.9)
+    json_pure (1.6.1)
+    mime-types (1.16)
+    mini_magick (3.3)
+      subexec (~> 0.1.0)
+    nokogiri (1.5.0)
+    rack (1.3.3)
+    rack-test (0.6.1)
+      rack (>= 1.0)
+    rake (0.9.2)
+    rspec (2.6.0)
+      rspec-core (~> 2.6.0)
+      rspec-expectations (~> 2.6.0)
+      rspec-mocks (~> 2.6.0)
+    rspec-core (2.6.4)
+    rspec-expectations (2.6.0)
+      diff-lcs (~> 1.1.2)
+    rspec-mocks (2.6.0)
+    rubyzip (0.9.4)
+    selenium-webdriver (2.7.0)
+      childprocess (>= 0.2.1)
+      ffi (>= 1.0.7)
+      json_pure
+      rubyzip
+    sinatra (1.2.6)
+      rack (~> 1.1)
+      tilt (< 2.0, >= 1.2.2)
+    subexec (0.1.0)
+    tilt (1.3.3)
+    xpath (0.1.4)
+      nokogiri (~> 1.3)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  appraisal
+  capybara (~> 1.0.0)
+  capybara-webkit!
+  mini_magick
+  rake
+  rspec (~> 2.6.0)
+  sinatra

data/gemfiles/1.1.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "capybara", "~> 1.1.0"
+
+gemspec :path=>"../"

data/gemfiles/1.1.gemfile.lock

@@ -0,0 +1,65 @@
+PATH
+  remote: /Users/jferris/Source/capybara-webkit
+  specs:
+    capybara-webkit (0.6.1)
+      capybara (< 1.2, >= 1.0.0)
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    appraisal (0.3.8)
+      bundler
+      rake
+    capybara (1.1.1)
+      mime-types (>= 1.16)
+      nokogiri (>= 1.3.3)
+      rack (>= 1.0.0)
+      rack-test (>= 0.5.4)
+      selenium-webdriver (~> 2.0)
+      xpath (~> 0.1.4)
+    childprocess (0.2.2)
+      ffi (~> 1.0.6)
+    diff-lcs (1.1.3)
+    ffi (1.0.9)
+    json_pure (1.6.1)
+    mime-types (1.16)
+    mini_magick (3.3)
+      subexec (~> 0.1.0)
+    nokogiri (1.5.0)
+    rack (1.3.3)
+    rack-test (0.6.1)
+      rack (>= 1.0)
+    rake (0.9.2)
+    rspec (2.6.0)
+      rspec-core (~> 2.6.0)
+      rspec-expectations (~> 2.6.0)
+      rspec-mocks (~> 2.6.0)
+    rspec-core (2.6.4)
+    rspec-expectations (2.6.0)
+      diff-lcs (~> 1.1.2)
+    rspec-mocks (2.6.0)
+    rubyzip (0.9.4)
+    selenium-webdriver (2.7.0)
+      childprocess (>= 0.2.1)
+      ffi (>= 1.0.7)
+      json_pure
+      rubyzip
+    sinatra (1.2.6)
+      rack (~> 1.1)
+      tilt (< 2.0, >= 1.2.2)
+    subexec (0.1.0)
+    tilt (1.3.3)
+    xpath (0.1.4)
+      nokogiri (~> 1.3)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  appraisal
+  capybara (~> 1.1.0)
+  capybara-webkit!
+  mini_magick
+  rake
+  rspec (~> 2.6.0)
+  sinatra

data/lib/capybara-webkit.rb

@@ -1,11 +1 @@
-require "capybara"
-require "capybara/driver/webkit"
-
-Capybara.register_driver :webkit do |app|
-  Capybara::Driver::Webkit.new(app)
-end
-
-Capybara.register_driver :webkit_debug do |app|
-  browser = Capybara::Driver::Webkit::Browser.new(:socket_class => Capybara::Driver::Webkit::SocketDebugger)
-  Capybara::Driver::Webkit.new(app, :browser => browser)
-end
+require "capybara/webkit"

data/lib/capybara/driver/webkit.rb

@@ -10,6 +10,9 @@ class Capybara::Driver::Webkit
   class WebkitNoResponseError < StandardError
   end
 
+  class NodeNotAttachedError < Capybara::ElementNotFound
+  end
+
   attr_reader :browser
 
   def initialize(app, options={})

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

@@ -86,6 +86,18 @@ class Capybara::Driver::Webkit
       command "Render", path, width, height
     end
 
+    def set_cookie(cookie)
+      command "SetCookie", cookie
+    end
+
+    def clear_cookies
+      command "ClearCookies"
+    end
+
+    def get_cookies
+      command("GetCookies").lines.map{ |line| line.strip }.select{ |line| !line.empty? }
+    end
+
     private
 
     def start_server

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

@@ -1,7 +1,10 @@
 class Capybara::Driver::Webkit
   class Node < Capybara::Driver::Node
+    NBSP = "\xC2\xA0"
+    NBSP.force_encoding("UTF-8") if NBSP.respond_to?(:force_encoding)
+
     def text
-      invoke "text"
+      invoke("text").gsub(NBSP, ' ').gsub(/\s+/u, ' ').strip
     end
 
     def [](name)
@@ -85,7 +88,23 @@ class Capybara::Driver::Webkit
     end
 
     def invoke(name, *args)
-      browser.command "Node", name, native, *args
+      if allow_unattached_nodes? || attached?
+        browser.command "Node", name, native, *args
+      else
+        raise Capybara::Driver::Webkit::NodeNotAttachedError
+      end
+    end
+
+    def allow_unattached_nodes?
+      !automatic_reload?
+    end
+
+    def automatic_reload?
+      Capybara.respond_to?(:automatic_reload) && Capybara.automatic_reload
+    end
+
+    def attached?
+      browser.command("Node", "isAttached", native) == "true"
     end
 
     def browser

data/lib/capybara/webkit.rb

@@ -0,0 +1,11 @@
+require "capybara"
+require "capybara/driver/webkit"
+
+Capybara.register_driver :webkit do |app|
+  Capybara::Driver::Webkit.new(app)
+end
+
+Capybara.register_driver :webkit_debug do |app|
+  browser = Capybara::Driver::Webkit::Browser.new(:socket_class => Capybara::Driver::Webkit::SocketDebugger)
+  Capybara::Driver::Webkit.new(app, :browser => browser)
+end

data/spec/driver_spec.rb

@@ -104,6 +104,50 @@ describe Capybara::Driver::Webkit do
     end
   end
 
+  context "redirect app" do
+    before(:all) do
+      @app = lambda do |env|
+        if env['PATH_INFO'] == '/target'
+          content_type = "<p>#{env['CONTENT_TYPE']}</p>"
+          [200, {"Content-Type" => "text/html", "Content-Length" => content_type.length.to_s}, [content_type]]
+        elsif env['PATH_INFO'] == '/form'
+          body = <<-HTML
+            <html>
+              <body>
+                <form action="/redirect" method="POST" enctype="multipart/form-data">
+                  <input name="submit" type="submit" />
+                </form>
+              </body>
+            </html>
+          HTML
+          [200, {"Content-Type" => "text/html", "Content-Length" => body.length.to_s}, [body]]
+        else
+          [301, {"Location" => "/target"}, [""]]
+        end
+      end
+    end
+
+    it "should redirect without content type" do
+      subject.visit("/form")
+      subject.find("//input").first.click
+      subject.find("//p").first.text.should == ""
+    end
+  end
+
+  context "css app" do
+    before(:all) do
+      body = "css"
+      @app = lambda do |env|
+        [200, {"Content-Type" => "text/css", "Content-Length" => body.length.to_s}, [body]]
+      end
+    end
+
+    it "renders unsupported content types gracefully" do
+      subject.visit("/")
+      subject.body.should =~ /css/
+    end
+  end
+
   context "hello app" do
     before(:all) do
       @app = lambda do |env|
@@ -115,6 +159,7 @@ describe Capybara::Driver::Webkit do
               </style>
             </head>
             <body>
+              <div class='normalize'>Spaces&nbsp;not&nbsp;normalized&nbsp;</div>
               <div id="display_none">
                 <div id="invisible">Can't see me</div>
               </div>
@@ -132,6 +177,13 @@ describe Capybara::Driver::Webkit do
       end
     end
 
+    it "handles anchor tags" do
+      subject.visit("#test")
+      subject.find("//*[contains(., 'hello')]").should_not be_empty
+      subject.visit("#test")
+      subject.find("//*[contains(., 'hello')]").should_not be_empty
+    end
+
     it "finds content after loading a URL" do
       subject.find("//*[contains(., 'hello')]").should_not be_empty
     end
@@ -158,6 +210,10 @@ describe Capybara::Driver::Webkit do
       subject.find("//p").first.text.should == "hello"
     end
 
+    it "normalizes a node's text" do
+      subject.find("//div[contains(@class, 'normalize')]").first.text.should == "Spaces not normalized"
+    end
+
     it "returns the current URL" do
       port = subject.instance_variable_get("@rack_server").port
       subject.current_url.should == "http://127.0.0.1:#{port}/hello/world?success=true"
@@ -835,6 +891,60 @@ describe Capybara::Driver::Webkit do
     end
   end
 
+  context "cookie-based app" do
+    before(:all) do
+      @cookie = 'cookie=abc; domain=127.0.0.1; path=/'
+      @app = lambda do |env|
+        request = ::Rack::Request.new(env)
+
+        body = <<-HTML
+          <html><body>
+            <p id="cookie">#{request.cookies["cookie"] || ""}</p>
+          </body></html>
+        HTML
+        [200,
+          { 'Content-Type'   => 'text/html; charset=UTF-8',
+            'Content-Length' => body.length.to_s,
+            'Set-Cookie'     => @cookie,
+          },
+          [body]]
+      end
+    end
+
+    def echoed_cookie
+      subject.find('id("cookie")').first.text
+    end
+
+    it "remembers the cookie on second visit" do
+      echoed_cookie.should == ""
+      subject.visit "/"
+      echoed_cookie.should == "abc"
+    end
+
+    it "uses a custom cookie" do
+      subject.browser.set_cookie @cookie
+      subject.visit "/"
+      echoed_cookie.should == "abc"
+    end
+
+    it "clears cookies" do
+      subject.browser.clear_cookies
+      subject.visit "/"
+      echoed_cookie.should == ""
+    end
+
+    it "allows enumeration of cookies" do
+      cookies = subject.browser.get_cookies
+
+      cookies.size.should == 1
+
+      cookie = Hash[cookies[0].split(/\s*;\s*/).map { |x| x.split("=", 2) }]
+      cookie["cookie"].should == "abc"
+      cookie["domain"].should include "127.0.0.1"
+      cookie["path"].should == "/"
+    end
+  end
+
   context "with socket debugger" do
     let(:socket_debugger_class){ Capybara::Driver::Webkit::SocketDebugger }
     let(:browser_with_debugger){
@@ -871,4 +981,36 @@ describe Capybara::Driver::Webkit do
       driver_with_debugger.find("//*[@id='parent']")
     end
   end
+
+  context "remove node app" do
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html>
+            <div id="parent">
+              <p id="removeMe">Hello</p>
+            </div>
+          </html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+
+    before { set_automatic_reload false }
+    after { set_automatic_reload true }
+
+    def set_automatic_reload(value)
+      if Capybara.respond_to?(:automatic_reload)
+        Capybara.automatic_reload = value
+      end
+    end
+
+    it "allows removed nodes when reloading is disabled" do
+      node = subject.find("//p[@id='removeMe']").first
+      subject.evaluate_script("document.getElementById('parent').innerHTML = 'Magic'")
+      node.text.should == 'Hello'
+    end
+  end
 end