page_object_wrapper 1.1.1 → 1.1.2

data/README.md

@@ -82,6 +82,8 @@ where required attributes are marked with (*)
           textarea(:id => 'f2').set data
         end
 
+        action_alias(:fill_textarea_alias, :some_test_page){ action :fill_textarea }
+
         table(:table_without_header) do
           locator :summary => 'Each row names a Nordic country and specifies its total area and land area, in square kilometers'
         end
@@ -139,46 +141,49 @@ Defined elements can be accessed with their labels.
 *preconditions*  
 **tp** is a :some\_test\_page object opened in the browser
 
-    context "argument = :fresh_food" do
-      it "populates all xxx elements with :fresh_food" do
+    context "argument = :fresh_food":
+      it "populates all xxx elements with :fresh_food":
         tp = PageObjectWrapper.current_page
         tp.feed_test_elements(:fresh_food)
-      end
-    end
 
-    context "argument = nil" do
-      it "populates all xxx elements with :fresh_food" do
+    context "argument = nil":
+      it "populates all xxx elements with :fresh_food":
         tp = PageObjectWrapper.current_page
         tp.feed_test_elements
-      end
-    end
 
-    context "argument = :missing_food" do
-      it "populates all xxx elements with :missing_food" do
+    context "argument = :missing_food":
+      it "populates all xxx elements with :missing_food":
         tp = PageObjectWrapper.open_page(:some_test_page)
         tp.feed_test_elements(:missing_food)
-      end
-    end
+
+    context "xxx is alias":
+      it "executes corresponding action":
+        tp = PageObjectWrapper.open_page(:some_test_page)
+        tp.fire_fill_textarea_alias
+        @b.textarea(:id => 'f2').value.should eq('Default data')
 
 
 #### fire\_xxx
 *preconditions*  
 **tp** is a :some\_test\_page object opened in the browser
 
-    it "executes fire_block in Watir::Browser context" do
+    it "executes fire_block in Watir::Browser context":
       tp = PageObjectWrapper.open_page(:some_test_page)
       tp.fire_fill_textarea
-    end
 
-    it "can be invoked with parameters" do
+    it "can be invoked with parameters":
       tp = PageObjectWrapper.current_page
       tp.fire_fill_textarea('User defined data')
-    end
 
-    it "returns next_page" do
+    it "returns next_page":
       tp = PageObjectWrapper.current_page
       np = tp.fire_press_cool_button
-    end
+
+    context "xxx is alias":
+      it "executes corresponding action":
+        tp = PageObjectWrapper.open_page(:some_test_page)
+        tp.fire_fill_textarea_alias
+        @b.textarea(:id => 'f2').value.should eq('Default data')
 
 #### select\_from\_xxx
 *preconditions*  
@@ -189,36 +194,27 @@ correct arguments are:
 :column1 is a column value from which you want to receive   
 :column2 is a column which is used to get specific row   
 
-    context "where == nil" do
-      it "returns last row value from provided column" do
+    context "where == nil":
+      it "returns last row value from provided column":
         tp.select_from_table_without_header(:column_0).text.should eq 'Sweden'
         tp.select_from_table_without_header(:column_1).text.should eq '449,964'
         tp.select_from_table_without_header(:column_2).text.should eq '410,928'
-      end
-    end
 
-    context "where not nil" do
-      context "found by String" do
-        it "returns found cells" do
+    context "where not nil":
+      context "found by String":
+        it "returns found cells":
           tp.select_from_table_without_header(:column_0, :column_1 => '103,000').text.should eq 'Iceland'
           tp.select_from_table_with_header(:country, :total_area => '337,030').text.should eq 'Finland'
-        end
-        it "returns nil" do
+        it "returns nil":
           tp.select_from_table_without_header(:column_0, :column_1 => '123').should eq nil
           tp.select_from_table_with_header(:country, :total_area => '123').should eq nil
-        end
-      end
-      context "found by Regexp" do
-        it "returns found cells" do
+      context "found by Regexp":
+        it "returns found cells":
           tp.select_from_table_without_header(:column_0, :column_1 => /103/).text.should eq 'Iceland'
           tp.select_from_table_with_header(:country, :total_area => /337/).text.should eq 'Finland'
-        end
-        it "returns nil" do
+        it "returns nil":
           tp.select_from_table_without_header(:column_0, :column_1 => /123/).should eq nil
           tp.select_from_table_with_header(:country, :total_area => /123/).should eq nil
-        end
-      end
-    end
 
 #### each\_xxx
 TODO

data/good_pages/another_test_page.rb

@@ -1,4 +1,3 @@
-require 'page_object_wrapper'
 PageObjectWrapper.define_page(:another_test_page) do
   locator 'http://www.cs.tut.fi/~jkorpela/www/testel.html'
   uniq_h1 :text => 'Testing display of HTML elements'

data/good_pages/google_page.rb

@@ -0,0 +1,4 @@
+PageObjectWrapper.define_page(:google_page) do
+  locator 'google.com'
+  uniq_input :id => 'gbqfq'
+end

data/good_pages/google_wron_uniq_page.rb

@@ -0,0 +1,4 @@
+PageObjectWrapper.define_page(:wrong_google_page) do
+  locator 'google.com'
+  uniq_element :id => 'foobar'
+end

data/good_pages/goole_as_page.rb

@@ -0,0 +1,4 @@
+PageObjectWrapper.define_page(:google_as_page) do
+  locator '/advanced_search'
+  uniq_text_field :name => 'as_q' 
+end

data/good_pages/some_test_page.rb

@@ -1,4 +1,3 @@
-require 'page_object_wrapper'
 PageObjectWrapper.define_page(:some_test_page) do
   locator 'http://www.cs.tut.fi/~jkorpela/www/testel.html'
   uniq_h1 :text => 'Testing display of HTML elements'

data/good_pages/test_table_page.rb

@@ -1,4 +1,3 @@
-require 'page_object_wrapper'
 PageObjectWrapper.define_page(:test_page_with_table) do
   locator 'http://www.cs.tut.fi/cgi-bin/run/~jkorpela/echo.cgi?hidden+field=42&foo=bar&text=Default+text.&textarea=Default+text.%0D%0A&radio=2&checkbox2=on&select1=two+%28default%29&select2=two+%28default%29'
   uniq_h1 :text => 'Echoing submitted form data'

data/lib/page_object_wrapper/PageObject.rb

@@ -16,6 +16,7 @@ class PageObject < DslElementWithLocator
   @@pages = []
   @@current_page = nil
 
+  UNIQ_ELEMENT_WAIT_PERIOD = 10
   FOOD_TYPES = [:missing_food, :fresh_food]
   FEED_ALL = Regexp.new(/^feed_all$/)
   FEED_SET = Regexp.new(/^feed_([\w_]+)$/)
@@ -26,7 +27,7 @@ class PageObject < DslElementWithLocator
     
   def initialize(label)
     super label
-    @uniq_element_type = 'element'
+    @uniq_element_type = nil
     @uniq_element_hash = {}
     @esets = []
     @elements = []
@@ -136,8 +137,14 @@ class PageObject < DslElementWithLocator
     raise PageObjectWrapper::BrowserNotFound if @@browser.nil?
     raise PageObjectWrapper::UnknownPageObject, label if not @@pages.collect(&:label_value).include?(label)
     page_object = PageObject.find_page_object(label)
-    watir_uniq_element = @@browser.send page_object.uniq_element_type, page_object.uniq_element_hash
-    raise PageObjectWrapper::UnmappedPageObject, "#{label} <=> #{@@browser.url}" if not watir_uniq_element.present?
+    if not page_object.uniq_element_type.nil?
+      watir_uniq_element = @@browser.send page_object.uniq_element_type, page_object.uniq_element_hash
+      begin
+        watir_uniq_element.wait_until_present UNIQ_ELEMENT_WAIT_PERIOD
+      rescue Watir::Wait::TimeoutError => e
+        raise PageObjectWrapper::UnmappedPageObject, "#{label} <=> #{@@browser.url} (#{e.message})" if not watir_uniq_element.present?
+      end
+    end
     @@current_page = page_object
   end
 
@@ -270,8 +277,7 @@ private
   end
 
   def feed_elements(elements, food_type=nil)
-    raise PageObjectWrapper::BrowserNotFound if @@browser.nil? 
-    raise PageObjectWrapper::BrowserNotFound if not @@browser.exist? 
+    raise PageObjectWrapper::BrowserNotFound if @@browser.nil? or not @@browser.exist?
     food_type ||= :fresh_food
     raise PageObjectWrapper::UnknownFoodType, food_type.inspect if not FOOD_TYPES.include?(food_type)
     elements.each{|e|
@@ -305,7 +311,7 @@ private
   end
 
   def fire_action(a, *args)
-    raise PageObjectWrapper::BrowserNotFound if @@browser.nil?
+    raise PageObjectWrapper::BrowserNotFound if @@browser.nil? or not @@browser.exist?
     block = (a.is_a? Action)? a.fire_block_value : action_for(a.action_value).fire_block_value
     @@browser.instance_exec args, &block
     self.class.map_current_page a.next_page_value
@@ -313,7 +319,7 @@ private
   end
 
   def select_from_table(table, header, where=nil)
-    raise PageObjectWrapper::BrowserNotFound if @@browser.nil?
+    raise PageObjectWrapper::BrowserNotFound if @@browser.nil? or not @@browser.exist?
     t = @@browser.table(table.locator_value)
     raise ArgumentError, "#{header.inspect} not a Symbol" if not header.is_a? Symbol
     raise ArgumentError, "#{header.inspect} not in table header" if not table.header_value.include? header
@@ -344,11 +350,11 @@ private
   end
 
   def each_pagination
-    raise PageObjectWrapper::BrowserNotFound if @@browser.nil?
+    raise PageObjectWrapper::BrowserNotFound if @@browser.nil? or not @@browser.exist?
   end
 
   def open_padination n
-    raise PageObjectWrapper::BrowserNotFound if @@browser.nil?
+    raise PageObjectWrapper::BrowserNotFound if @@browser.nil? or not @@browser.exist?
   end
 
 

data/lib/page_object_wrapper/version.rb

@@ -1,3 +1,3 @@
 module PageObjectWrapper
-  VERSION = "1.1.1"
+  VERSION = "1.1.2"
 end

data/spec/define_page_object_spec.rb

@@ -3,10 +3,6 @@ require 'shared_examples'
 
 describe "define_page_object" do
   let!(:page_object){
-    begin
-      PageObjectWrapper.load('./good_pages')
-    rescue
-    end
     PageObjectWrapper.receive_page(:some_test_page)
   }
 

data/spec/defined_elements_spec.rb

@@ -7,12 +7,8 @@ describe "page_object.xxx" do
     before(:all){
       @b = Watir::Browser.new
       PageObjectWrapper.use_browser @b
-      begin
-        PageObjectWrapper.load('./good_pages')
-      rescue
-      end
     }
-    after(:all){ PageObjectWrapper.browser.close }
+    after(:all){ PageObjectWrapper.browser.quit }
 
     subject{ PageObjectWrapper.open_page(:some_test_page) }
 

data/spec/feed_elements_spec.rb

@@ -3,14 +3,6 @@ require 'shared_examples'
 
 describe "page_object.feed_xxx" do
   context "browser is closed" do
-    before(:all){
-      begin
-        PageObjectWrapper.browser.quit if not PageObjectWrapper.browser.nil?
-        PageObjectWrapper.load('./good_pages')
-      rescue
-      end
-    }
-
     it "raises PageObjectWrapper::BrowserNotFound" do
       tp = PageObjectWrapper.receive_page(:some_test_page)
       expect{ tp.feed_test_elements }.to raise_error(PageObjectWrapper::BrowserNotFound)
@@ -21,12 +13,8 @@ describe "page_object.feed_xxx" do
     before(:all){
       @b = Watir::Browser.new
       PageObjectWrapper.use_browser @b
-      begin
-        PageObjectWrapper.load('./good_pages')
-      rescue
-      end
     }
-    after(:all){PageObjectWrapper.browser.close}
+    after(:all){ PageObjectWrapper.browser.quit }
 
     it "returns current page object" do
       tp = PageObjectWrapper.open_page(:some_test_page)

data/spec/fire_event_spec.rb

@@ -4,14 +4,6 @@ require 'shared_examples'
 describe "page_object.fire_xxx" do
   context "browser is closed" do
     it "raises PageObjectWrapper::BrowserNotFound" do
-      begin
-        PageObjectWrapper.browser.quit if not PageObjectWrapper.browser.nil?
-        begin
-          PageObjectWrapper.load('./good_pages')
-        rescue
-        end
-      rescue
-      end
       tp = PageObjectWrapper.receive_page(:some_test_page)
       expect{ tp.fire_press_cool_button }.to raise_error(PageObjectWrapper::BrowserNotFound)
     end
@@ -20,12 +12,8 @@ describe "page_object.fire_xxx" do
     before(:all){
       @b = Watir::Browser.new
       PageObjectWrapper.use_browser @b
-      begin
-        PageObjectWrapper.load('./good_pages')
-      rescue
-      end
     }
-    after(:all){ PageObjectWrapper.browser.close }
+    after(:all){ PageObjectWrapper.browser.quit }
 
     it "executes fire_block in Watir::Browser context" do
       tp = PageObjectWrapper.open_page(:some_test_page)

data/spec/open_spec.rb

@@ -1,30 +1,9 @@
 require 'spec_helper'
 
 describe "PageObjectWrapper.open_page" do
-  let(:define_page_object){
-    PageObjectWrapper.define_page(:google_page) do
-      locator 'google.com'
-      uniq_input :id => 'gbqfq'
-    end
-  }
-  let(:define_wrong_page_object){
-    PageObjectWrapper.define_page(:wrong_google_page) do
-      locator 'google.com'
-      uniq_element :id => 'foobar'
-    end
-  }
-  let(:define_page_object_with_local_path){
-    PageObjectWrapper.define_page(:google_as_page) do
-      locator '/advanced_search'
-      uniq_text_field :name => 'as_q' 
-    end
-  }
-
   context "browser is closed" do
-    it "raises PageObjectWrapper::BrowserNotFound" do
-      define_page_object
-      PageObject.browser = nil
-      expect{ PageObjectWrapper.open_page(:google_page) }.to raise_error(PageObjectWrapper::BrowserNotFound)
+    it "raises Errno::ECONNREFUSED (I don't understand why here it behaves like this)" do
+      expect{ PageObjectWrapper.open_page(:google_page) }.to raise_error(Errno::ECONNREFUSED)
     end
   end
 
@@ -33,12 +12,9 @@ describe "PageObjectWrapper.open_page" do
     before(:all){ 
       @b = Watir::Browser.new
       PageObjectWrapper.use_browser @b 
-      define_page_object
-      define_wrong_page_object
-      define_page_object_with_local_path
     }
 
-    after(:all){ PageObjectWrapper.browser.close }
+    after(:all){ PageObjectWrapper.browser.quit }
 
     it "raises errors" do        
       expect{ PageObjectWrapper.open_page(:first_arg, :second_arg) }.to raise_error(ArgumentError)

data/spec/select_from_spec.rb

@@ -4,11 +4,6 @@ require 'shared_examples'
 describe "page_object.select_from_xxx" do
   context "browser is closed" do
     it "raises PageObjectWrapper::BrowserNotFound" do
-      begin
-        PageObjectWrapper.load('./good_pages')
-      rescue
-      end
-      PageObject.browser = nil
       tp = PageObjectWrapper.receive_page(:some_test_page)
       expect{ tp.select_from_table_without_header(:column_1, {:column_2 => ''}) }.to raise_error(PageObjectWrapper::BrowserNotFound)
     end
@@ -17,12 +12,8 @@ describe "page_object.select_from_xxx" do
     before(:all){
       @b = Watir::Browser.new
       PageObjectWrapper.use_browser @b
-      begin
-        PageObjectWrapper.load('./good_pages')
-      rescue
-      end
     }
-    after(:all){ PageObjectWrapper.browser.close }
+    after(:all){ PageObjectWrapper.browser.quit }
     let!(:tp){ PageObjectWrapper.open_page(:some_test_page)}
 
     context "wrong arguments" do

data/spec/spec_helper.rb

@@ -1,2 +1,8 @@
 require "page_object_wrapper"
-Selenium::WebDriver::Firefox.path='/opt/firefox/firefox'
+
+RSpec.configure do |config|
+  config.before(:suite){
+    Selenium::WebDriver::Firefox.path='/opt/firefox/firefox'
+    PageObjectWrapper.load('./good_pages')
+  }
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: page_object_wrapper
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-19 00:00:00.000000000Z
+date: 2012-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: watir-webdriver
-  requirement: &20716880 !ruby/object:Gem::Requirement
+  requirement: &22275160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *20716880
+  version_requirements: *22275160
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &20740240 !ruby/object:Gem::Requirement
+  requirement: &22274620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,7 +32,7 @@ dependencies:
         version: 2.0.0
   type: :development
   prerelease: false
-  version_requirements: *20740240
+  version_requirements: *22274620
 description: Wraps watir-webdriver with convenient testing interface, based on PageObjects
   automation testing pattern. Simplifies resulting automated test understanding.
 email:
@@ -48,6 +48,9 @@ files:
 - Rakefile
 - bad_pages/bad_page.rb
 - good_pages/another_test_page.rb
+- good_pages/google_page.rb
+- good_pages/google_wron_uniq_page.rb
+- good_pages/goole_as_page.rb
 - good_pages/some_test_page.rb
 - good_pages/test_table_page.rb
 - img/scheme.png
@@ -66,6 +69,7 @@ files:
 - page_object_wrapper.gemspec
 - spec/define_page_object_spec.rb
 - spec/defined_elements_spec.rb
+- spec/each_page.rb
 - spec/feed_elements_spec.rb
 - spec/fire_event_spec.rb
 - spec/generate_spec.rb
@@ -102,6 +106,7 @@ summary: Wraps watir-webdriver with convenient testing interface.
 test_files:
 - spec/define_page_object_spec.rb
 - spec/defined_elements_spec.rb
+- spec/each_page.rb
 - spec/feed_elements_spec.rb
 - spec/fire_event_spec.rb
 - spec/generate_spec.rb