From 17224befe7b551ee06ee33fd4f9f4e7961e49d16 Mon Sep 17 00:00:00 2001
From: Scott Cytacki <scott@concord.org>
Date: Fri, 18 Feb 2011 16:29:51 -0500
Subject: [PATCH] Fix document#text

It failed if the default selector was css, and failed with selenium with xpath.  Calling find('.') or find('/') in selenium returns a element that causes errors.  It now calls find(:xpath, '/html') which seems to work everywhere.
---
 lib/capybara/node/document.rb          |  2 +-
 lib/capybara/spec/driver.rb            |  5 +++++
 lib/capybara/spec/session.rb           |  1 +
 lib/capybara/spec/session/text_spec.rb | 19 +++++++++++++++++++
 spec/rspec/matchers_spec.rb            | 10 ++++++++++
 5 files changed, 36 insertions(+), 1 deletion(-)
 create mode 100644 lib/capybara/spec/session/text_spec.rb

diff --git a/lib/capybara/node/document.rb b/lib/capybara/node/document.rb
index deba1113..f72e43ab 100644
--- a/lib/capybara/node/document.rb
+++ b/lib/capybara/node/document.rb
@@ -18,7 +18,7 @@ module Capybara
       # @return [String]    The text of the document
       #
       def text
-        find('.').text
+        find(:xpath, '/html').text
       end
     end
   end
diff --git a/lib/capybara/spec/driver.rb b/lib/capybara/spec/driver.rb
index df0baa6f..b8933d16 100644
--- a/lib/capybara/spec/driver.rb
+++ b/lib/capybara/spec/driver.rb
@@ -99,6 +99,11 @@ shared_examples_for 'driver' do
         @driver.find('//option[@value="sv"]')[0].should_not be_selected
         @driver.find('//h1')[0].should_not be_selected
       end
+
+      it "should return document text on /html selector" do
+        @driver.visit('/with_simple_html')
+        @driver.find('/html')[0].text.should == 'Bar'
+      end
     end
   end
 end
diff --git a/lib/capybara/spec/session.rb b/lib/capybara/spec/session.rb
index dfb087cc..e2843cc4 100644
--- a/lib/capybara/spec/session.rb
+++ b/lib/capybara/spec/session.rb
@@ -66,6 +66,7 @@ shared_examples_for "session" do
   it_should_behave_like "has_select"
   it_should_behave_like "has_table"
   it_should_behave_like "select"
+  it_should_behave_like "text"
   it_should_behave_like "uncheck"
   it_should_behave_like "unselect"
   it_should_behave_like "within"
diff --git a/lib/capybara/spec/session/text_spec.rb b/lib/capybara/spec/session/text_spec.rb
new file mode 100644
index 00000000..146c8468
--- /dev/null
+++ b/lib/capybara/spec/session/text_spec.rb
@@ -0,0 +1,19 @@
+shared_examples_for "text" do
+  describe '#text' do
+    before do
+      @session.visit('/with_simple_html')
+    end
+
+    it "should print the text of the page" do
+      @session.text.should == 'Bar'
+    end
+
+    context "with css as default selector" do
+      before { Capybara.default_selector = :css }
+      it "should print the text of the page" do
+        @session.text.should == 'Bar'
+      end
+      after { Capybara.default_selector = :xpath }
+    end
+  end
+end
diff --git a/spec/rspec/matchers_spec.rb b/spec/rspec/matchers_spec.rb
index 837b4cfc..a36ddeba 100644
--- a/spec/rspec/matchers_spec.rb
+++ b/spec/rspec/matchers_spec.rb
@@ -250,6 +250,16 @@ describe Capybara::RSpecMatchers do
             page.should have_content('No such Text')
           end.to raise_error(/expected there to be content "No such Text" in "(.*)This is a test(.*)"/)
         end
+
+        context "with default selector CSS" do
+          before { Capybara.default_selector = :css }
+          it "fails if has_css? returns false" do
+            expect do
+              page.should have_content('No such Text')
+            end.to raise_error(/expected there to be content "No such Text" in "(.*)This is a test(.*)"/)
+          end
+          after { Capybara.default_selector = :xpath }
+        end
       end
 
       context "with should_not" do