RubyGems - capybara-webkit - Versions diffs - 0.5.0 → 0.6.0 - Mend

capybara-webkit 0.5.0 → 0.6.0

Files changed (39) hide show

data/.gitignore +3 -0
data/CONTRIBUTING.md +38 -0
data/Gemfile +3 -2
data/Gemfile.lock +28 -28
data/README.md +39 -4
data/capybara-webkit.gemspec +2 -2
data/lib/capybara-webkit.rb +4 -0
data/lib/capybara/driver/webkit.rb +23 -3
data/lib/capybara/driver/webkit/browser.rb +44 -6
data/lib/capybara/driver/webkit/node.rb +15 -4
data/lib/capybara/driver/webkit/socket_debugger.rb +43 -0
data/lib/capybara_webkit_builder.rb +9 -2
data/spec/browser_spec.rb +26 -0
data/spec/driver_rendering_spec.rb +80 -0
data/spec/driver_spec.rb +186 -15
data/spec/integration/driver_spec.rb +4 -0
data/spec/integration/session_spec.rb +2 -2
data/src/Body.h +12 -0
data/src/Connection.cpp +5 -2
data/src/Header.cpp +18 -0
data/src/Header.h +11 -0
data/src/NetworkAccessManager.cpp +22 -0
data/src/NetworkAccessManager.h +18 -0
data/src/Render.cpp +19 -0
data/src/Render.h +12 -0
data/src/Reset.cpp +3 -0
data/src/Server.cpp +5 -2
data/src/Server.h +1 -0
data/src/Source.cpp +9 -2
data/src/Source.h +7 -0
data/src/WebPage.cpp +79 -5
data/src/WebPage.h +9 -0
data/src/body.cpp +11 -0
data/src/capybara.js +19 -4
data/src/find_command.h +3 -0
data/src/main.cpp +2 -1
data/src/webkit_server.pro +2 -2
metadata +34 -9
data/spec/support/socket_debugger.rb +0 -42

data/lib/capybara_webkit_builder.rb CHANGED Viewed

@@ -3,23 +3,30 @@ require "fileutils"
 module CapybaraWebkitBuilder
   extend self
+  def make_bin
+    make_binaries = ['gmake', 'make']
+    make_binaries.detect { |make| system("which #{make}") }
+  end
   def makefile
     qmake_binaries = ['qmake', 'qmake-qt4']
     qmake = qmake_binaries.detect { |qmake| system("which #{qmake}") }
     case RUBY_PLATFORM
     when /linux/
       system("#{qmake} -spec linux-g++")
+    when /freebsd/
+      system("#{qmake} -spec freebsd-g++")
     else
       system("#{qmake} -spec macx-g++")
     end
   end
   def qmake
-    system("make qmake")
+    system("#{make_bin} qmake")
   end
   def build
-    system("make") or return false
+    system(make_bin) or return false
     FileUtils.mkdir("bin") unless File.directory?("bin")
     FileUtils.cp("src/webkit_server", "bin", :preserve => true)

data/spec/browser_spec.rb ADDED Viewed

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require 'capybara/driver/webkit/browser'
+describe Capybara::Driver::Webkit::Browser do
+  let(:browser) { Capybara::Driver::Webkit::Browser.new }
+  describe '#server_port' do
+    subject { browser.server_port }
+    it 'returns a valid port number' do
+      should be_a(Integer)
+    end
+    it 'returns a port in the allowed range' do
+      should be_between 0x400, 0xffff
+    end
+  end
+  context 'random port' do
+    it 'chooses a new port number for a new browser instance' do
+      new_browser = Capybara::Driver::Webkit::Browser.new
+      new_browser.server_port.should_not == browser.server_port
+    end
+  end
+end

data/spec/driver_rendering_spec.rb ADDED Viewed

@@ -0,0 +1,80 @@
+require 'spec_helper'
+require 'capybara/driver/webkit'
+require 'mini_magick'
+describe Capybara::Driver::Webkit, "rendering an image" do
+  before(:all) do
+    # Set up the tmp directory and file name
+    tmp_dir    = File.join(PROJECT_ROOT, 'tmp')
+    FileUtils.mkdir_p tmp_dir
+    @file_name = File.join(tmp_dir, 'render-test.png')
+    app = lambda do |env|
+      body = <<-HTML
+        <html>
+          <body>
+            <h1>Hello World</h1>
+          </body>
+        </html>
+      HTML
+      [200,
+        { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+        [body]]
+    end
+    @driver = Capybara::Driver::Webkit.new(app, :browser => $webkit_browser)
+    @driver.visit("/hello/world?success=true")
+  end
+  after(:all) { @driver.reset! }
+  def render(options)
+    FileUtils.rm_f @file_name
+    @driver.render @file_name, options
+    @image = MiniMagick::Image.open @file_name
+  end
+  context "with default options" do
+    before(:all){ render({}) }
+    it "should be a PNG" do
+      @image[:format].should == "PNG"
+    end
+    it "width default to 1000px (with 15px less for the scrollbar)" do
+      @image[:width].should be < 1001
+      @image[:width].should be > 1000-17
+    end
+    it "height should be at least 10px" do
+      @image[:height].should >= 10
+    end
+  end
+  context "with dimensions set larger than necessary" do
+    before(:all){ render(:width => 500, :height => 400) }
+    it "width should match the width given" do
+      @image[:width].should == 500
+    end
+    it "height should match the height given" do
+      @image[:height].should > 10
+    end
+  end
+  context "with dimensions set smaller than the document's default" do
+    before(:all){ render(:width => 50, :height => 10) }
+    it "width should be greater than the width given" do
+      @image[:width].should > 50
+    end
+    it "height should be greater than the height given" do
+      @image[:height].should > 10
+    end
+  end
+end

data/spec/driver_spec.rb CHANGED Viewed

@@ -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>
@@ -56,12 +56,12 @@ describe Capybara::Driver::Webkit do
     it "raises error for missing frame by index" do
       expect { subject.within_frame(1) { } }.
-        to raise_error(Capybara::Driver::Webkit::WebkitError)
+        to raise_error(Capybara::Driver::Webkit::WebkitInvalidResponseError)
     end
     it "raise_error for missing frame by id" do
       expect { subject.within_frame("foo") { } }.
-        to raise_error(Capybara::Driver::Webkit::WebkitError)
+        to raise_error(Capybara::Driver::Webkit::WebkitInvalidResponseError)
     end
     it "returns an attribute's value" do
@@ -119,6 +119,7 @@ describe Capybara::Driver::Webkit do
                 <div id="invisible">Can't see me</div>
               </div>
               <input type="text" disabled="disabled"/>
+              <input id="checktest" type="checkbox" checked="checked"/>
               <script type="text/javascript">
                 document.write("<p id='greeting'>he" + "llo</p>");
               </script>
@@ -142,7 +143,7 @@ describe Capybara::Driver::Webkit do
     it "raises an error for an invalid xpath query" do
       expect { subject.find("totally invalid salad") }.
-        to raise_error(Capybara::Driver::Webkit::WebkitError, /xpath/i)
+        to raise_error(Capybara::Driver::Webkit::WebkitInvalidResponseError, /xpath/i)
     end
     it "returns an attribute's value" do
@@ -176,10 +177,6 @@ describe Capybara::Driver::Webkit do
       subject.source.should =~ %r{<html>.*greeting.*}m
     end
-    it "aliases body as source" do
-      subject.body.should == subject.source
-    end
     it "evaluates Javascript and returns a string" do
       result = subject.evaluate_script(%<document.getElementById('greeting').innerText>)
       result.should == "hello"
@@ -237,7 +234,7 @@ describe Capybara::Driver::Webkit do
     it "raises an error for failing Javascript" do
       expect { subject.execute_script(%<invalid salad>) }.
-        to raise_error(Capybara::Driver::Webkit::WebkitError)
+        to raise_error(Capybara::Driver::Webkit::WebkitInvalidResponseError)
     end
     it "doesn't raise an error for Javascript that doesn't return anything" do
@@ -253,6 +250,10 @@ describe Capybara::Driver::Webkit do
       subject.find("//input").first.should be_disabled
     end
+    it "reads checked property" do
+      subject.find("//input[@id='checktest']").first.should be_checked
+    end
     it "finds visible elements" do
       subject.find("//p").first.should be_visible
       subject.find("//*[@id='invisible']").first.should_not be_visible
@@ -311,6 +312,12 @@ describe Capybara::Driver::Webkit do
       input.value.should == "newvalue"
     end
+    it "sets an input's nil value" do
+      input = subject.find("//input").first
+      input.set(nil)
+      input.value.should == ""
+    end
     it "sets a select's value" do
       select = subject.find("//select").first
       select.set("Monkey")
@@ -365,28 +372,36 @@ describe Capybara::Driver::Webkit do
       checked_box['checked'].should be_true
     end
+    it "knows a checked box is checked using checked?" do
+      checked_box.should be_checked
+    end
     it "knows an unchecked box is unchecked" do
       unchecked_box['checked'].should_not be_true
     end
+    it "knows an unchecked box is unchecked using checked?" do
+      unchecked_box.should_not be_checked
+    end
     it "checks an unchecked box" do
       unchecked_box.set(true)
-      unchecked_box['checked'].should be_true
+      unchecked_box.should be_checked
     end
     it "unchecks a checked box" do
       checked_box.set(false)
-      checked_box['checked'].should_not be_true
+      checked_box.should_not be_checked
     end
     it "leaves a checked box checked" do
       checked_box.set(true)
-      checked_box['checked'].should be_true
+      checked_box.should be_checked
     end
     it "leaves an unchecked box unchecked" do
       unchecked_box.set(false)
-      unchecked_box['checked'].should_not be_true
+      unchecked_box.should_not be_checked
     end
     let(:enabled_input)  { subject.find("//input[@name='foo']").first }
@@ -620,7 +635,7 @@ describe Capybara::Driver::Webkit do
         wait_for_error_to_complete
         subject.find("//body")
       }.
-        to raise_error(Capybara::Driver::Webkit::WebkitError, %r{/error})
+        to raise_error(Capybara::Driver::Webkit::WebkitInvalidResponseError, %r{/error})
     end
     def wait_for_error_to_complete
@@ -651,7 +666,7 @@ describe Capybara::Driver::Webkit do
     it "raises a webkit error and then continues" do
       subject.find("//input").first.click
-      expect { subject.find("//p") }.to raise_error(Capybara::Driver::Webkit::WebkitError)
+      expect { subject.find("//p") }.to raise_error(Capybara::Driver::Webkit::WebkitInvalidResponseError)
       subject.visit("/")
       subject.find("//p").first.text.should == "hello"
     end
@@ -681,4 +696,160 @@ describe Capybara::Driver::Webkit do
       subject.find("//p").first.text.should == "success"
     end
   end
+  context "custom header" do
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html><body>
+            <p id="user-agent">#{env['HTTP_USER_AGENT']}</p>
+            <p id="x-capybara-webkit-header">#{env['HTTP_X_CAPYBARA_WEBKIT_HEADER']}</p>
+            <p id="accept">#{env['HTTP_ACCEPT']}</p>
+            <a href="/">/</a>
+          </body></html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+    before do
+      subject.header('user-agent', 'capybara-webkit/custom-user-agent')
+      subject.header('x-capybara-webkit-header', 'x-capybara-webkit-header')
+      subject.header('accept', 'text/html')
+      subject.visit('/')
+    end
+    it "can set user_agent" do
+      subject.find('id("user-agent")').first.text.should == 'capybara-webkit/custom-user-agent'
+      subject.evaluate_script('navigator.userAgent').should == 'capybara-webkit/custom-user-agent'
+    end
+    it "keep user_agent in next page" do
+      subject.find("//a").first.click
+      subject.find('id("user-agent")').first.text.should == 'capybara-webkit/custom-user-agent'
+      subject.evaluate_script('navigator.userAgent').should == 'capybara-webkit/custom-user-agent'
+    end
+    it "can set custom header" do
+      subject.find('id("x-capybara-webkit-header")').first.text.should == 'x-capybara-webkit-header'
+    end
+    it "can set Accept header" do
+      subject.find('id("accept")').first.text.should == 'text/html'
+    end
+    it "can reset all custom header" do
+      subject.reset!
+      subject.visit('/')
+      subject.find('id("user-agent")').first.text.should_not == 'capybara-webkit/custom-user-agent'
+      subject.evaluate_script('navigator.userAgent').should_not == 'capybara-webkit/custom-user-agent'
+      subject.find('id("x-capybara-webkit-header")').first.text.should be_empty
+      subject.find('id("accept")').first.text.should_not == 'text/html'
+    end
+  end
+  context "no response app" do
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html><body>
+            <form action="/error"><input type="submit"/></form>
+          </body></html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+    it "raises a webkit error for the requested url" do
+      make_the_server_go_away
+      expect {
+        subject.find("//body")
+      }.
+       to raise_error(Capybara::Driver::Webkit::WebkitNoResponseError, %r{response})
+      make_the_server_come_back
+    end
+    def make_the_server_come_back
+      subject.browser.instance_variable_get(:@socket).unstub!(:gets)
+      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)
+      subject.browser.instance_variable_get(:@socket).stub!(:print)
+    end
+  end
+  context "custom font app" do
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html>
+            <head>
+              <style type="text/css">
+                p { font-family: "Verdana"; }
+              </style>
+            </head>
+            <body>
+              <p id="text">Hello</p>
+            </body>
+          </html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+    it "ignores custom fonts" do
+      font_family = subject.evaluate_script(<<-SCRIPT)
+        var element = document.getElementById("text");
+        element.ownerDocument.defaultView.getComputedStyle(element, null).getPropertyValue("font-family");
+      SCRIPT
+      font_family.should == "Arial"
+    end
+  end
+  context "with socket debugger" do
+    let(:socket_debugger_class){ Capybara::Driver::Webkit::SocketDebugger }
+    let(:browser_with_debugger){
+      Capybara::Driver::Webkit::Browser.new(:socket_class => socket_debugger_class)
+    }
+    let(:driver_with_debugger){ Capybara::Driver::Webkit.new(@app, :browser => browser_with_debugger) }
+    before(:all) do
+      @app = lambda do |env|
+        body = <<-HTML
+          <html><body>
+            <div id="parent">
+              <div class="find">Expected</div>
+            </div>
+            <div class="find">Unexpected</div>
+          </body></html>
+        HTML
+        [200,
+          { 'Content-Type' => 'text/html', 'Content-Length' => body.length.to_s },
+          [body]]
+      end
+    end
+    it "prints out sent content" do
+      socket_debugger_class.any_instance.stub(:received){|content| content }
+      sent_content = ['Find', 1, 17, "//*[@id='parent']"]
+      socket_debugger_class.any_instance.should_receive(:sent).exactly(sent_content.size).times
+      driver_with_debugger.find("//*[@id='parent']")
+    end
+    it "prints out received content" do
+      socket_debugger_class.any_instance.stub(:sent)
+      socket_debugger_class.any_instance.should_receive(:received).at_least(:once).and_return("ok")
+      driver_with_debugger.find("//*[@id='parent']")
+    end
+  end
 end

data/spec/integration/driver_spec.rb CHANGED Viewed

@@ -11,6 +11,10 @@ describe Capybara::Driver::Webkit do
   it_should_behave_like "driver with javascript support"
   it_should_behave_like "driver with cookies support"
+  it "returns the rack server port" do
+    @driver.server_port.should  eq(@driver.instance_variable_get(:@rack_server).port)
+  end
   # Can't support:
   # it_should_behave_like "driver with header support"