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

capybara-webkit 0.5.0 → 0.6.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

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"