page_object_wrapper 1.4.8 → 1.4.9

data/lib/page_object_wrapper/PageObject.rb

@@ -24,6 +24,7 @@ module PageObjectWrapper
     FEED = Regexp.new(/^feed_([\w_]+)$/)
     FIRE_ACTION = Regexp.new(/^fire_([\w_]+)$/)
     SELECT_FROM = Regexp.new(/^select_from_([\w_]+)$/)
+    SELECT_ROW_FROM = Regexp.new(/^select_row_from_([\w_]+)$/)
     PAGINATION_EACH = Regexp.new(/^([\w_]+)_each$/)
     PAGINATION_OPEN = Regexp.new(/^([\w_]+)_open$/)
     VALIDATE = Regexp.new(/^validate_([\w_\?]+)$/)
@@ -96,6 +97,10 @@ module PageObjectWrapper
           # page_object.select_from_some_table(:header_column, {:column => 'value'})
           table = table_for($1)
           select_from(table, *args)
+        when (SELECT_ROW_FROM.match(method_name) and has_table?($1))
+          # page_object.select_row_from_some_table(:number => 1, :column1 => value1, :column2 => value3, ...)
+          table = table_for($1)
+          select_row_from(table, args[0])
         when (PAGINATION_EACH.match(method_name) and has_pagination?($1))
           # page_object.each_pagination
           pagination = pagination_for($1)
@@ -147,6 +152,9 @@ module PageObjectWrapper
         when (SELECT_FROM.match(method_name) and has_table?($1))
           # page_object.select_from_some_table(:header_column, {:column => 'value'})
           true
+        when (SELECT_ROW_FROM.match(method_name) and has_table?($1))
+          # page_object.select_from_some_table(:header_column, {:column => 'value'})
+          true
         when (PAGINATION_EACH.match(method_name) and has_pagination?($1))
           # page_object.each_pagination
           true
@@ -506,6 +514,49 @@ module PageObjectWrapper
       end
     end
 
+    def select_row_from(table, query)
+      conditions = query.clone
+      conditions.delete(:number)
+      raise PageObjectWrapper::BrowserNotFound if @@browser.nil? or not @@browser.exist?
+      t = @@browser.table(table.locator_value)
+      found_row = {}
+      candidate_rows = nil
+      raise ArgumentError, "argument should be a meaningful Hash, got #{query.inspect}" if not query.is_a?(Hash) or query.empty?
+      if query.has_key?(:number) and query[:number].class != Fixnum
+        raise ArgumentError, "arguments key :number should have Integer value, got #{query[:number].class}"
+      end
+      if not conditions.empty? and (conditions.keys.collect(&:class).uniq != [Symbol] or conditions.values.collect(&:class).uniq != [String])
+        raise ArgumentError, "arguments hash should be like :symbol => 'a string' (for all columns except :number), got #{query.inspect}"
+      end
+      
+      if query.has_key? :number
+        candidate_rows = [t[query[:number]]]
+        query.delete(:number)
+      else
+        candidate_rows = t.rows
+      end
+
+      candidate_rows.each{ |r|
+        conditions_met = true
+        unless query.empty?
+          query.each_key{ |column_name|
+            raise ArgumentError, "column #{column_name.inspect} not in table header and not == :number" if not table.header_value.include?(column_name) 
+            column_index = table.header_value.index(column_name)
+            conditions_met = false if r[column_index].text != query[column_name]
+          }
+        end
+        if conditions_met
+          column_index = 0
+          r.cells.each{ |cell| 
+            found_row[table.header_value[column_index]] = cell
+            column_index += 1
+          }
+          return found_row
+        end
+      }
+      return nil
+    end
+
     def run_each_subpage(p, opts=nil, &block)
       raise PageObjectWrapper::BrowserNotFound if @@browser.nil? or not @@browser.exist?
       limit = opts[:limit] if not opts.nil?

data/lib/version.rb

@@ -1,3 +1,3 @@
 module PageObjectWrapper
-  VERSION = "1.4.8" unless const_defined?(:VERSION)
+  VERSION = "1.4.9" unless const_defined?(:VERSION)
 end

data/spec/select_raw_from_xxx_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+require 'shared_examples'
+
+describe "page_object.select_from_xxx" do
+  context "browser is closed" do
+    it "raises PageObjectWrapper::BrowserNotFound" do
+      tp = PageObjectWrapper.receive_page(:some_test_page)
+      expect{ tp.select_row_from_table_without_header(:column_2 => '') }.to raise_error(PageObjectWrapper::BrowserNotFound)
+    end
+  end
+  context "browser is opened" do
+    before(:all){
+      @b = Watir::Browser.new
+      PageObjectWrapper.use_browser @b
+    }
+    after(:all){ PageObjectWrapper.browser.quit }
+    let!(:tp){ PageObjectWrapper.open_page(:some_test_page)}
+
+    context "wrong arguments" do
+      context "arguments is not a meaningful Hash" do
+        it "raises ArgumentError" do
+          expect{ tp.select_row_from_table_without_header('') }.to raise_error(ArgumentError)
+          expect{ tp.select_row_from_table_with_header({}) }.to raise_error(ArgumentError)
+        end
+      end
+
+      context "values inside arguments Hash are not strings or :number != Integer" do
+        it "raises ArgumentError" do
+          expect{ tp.select_row_from_table_with_header(:country => []) }.to raise_error(ArgumentError)
+          expect{ tp.select_row_from_table_with_header(:number => 'string') }.to raise_error(ArgumentError)
+        end
+      end
+
+      context "specified column not found inside table" do
+        it "raises ArgumentError" do
+          expect{ tp.select_row_from_table_without_header(:unknown_column => 'some value') }.to raise_error(ArgumentError)
+          expect{ tp.select_row_from_table_with_header(:column_1 => 'some value') }.to raise_error(ArgumentError)
+          expect{ tp.select_row_from_table_with_header(:unknown_column => 'some value') }.to raise_error(ArgumentError)
+        end
+      end
+    end
+
+    context "table with header" do
+      it "returns first found row excluding table header" do
+        t_row = tp.select_row_from_table_with_header(:number => 1)
+        t_row.should be_a Hash
+        t_row[:country].text.should eq 'Denmark'
+        t_row = tp.select_row_from_table_with_header(:country => 'Denmark')
+        t_row.should be_a Hash
+        t_row[:country].text.should eq 'Denmark'
+        t_row = tp.select_row_from_table_with_header(:number => 1, :country => 'Denmark')
+        t_row.should be_a Hash
+        t_row[:country].text.should eq 'Denmark'
+      end
+    end
+
+    context "table without header" do
+      it "returns first found row including table header" do
+        t_row = tp.select_row_from_table_without_header(:number => 1)
+        t_row.should be_a Hash
+        t_row[:column_0].text.should eq 'Denmark'
+        t_row = tp.select_row_from_table_without_header(:column_0 => 'Denmark')
+        t_row.should be_a Hash
+        t_row[:column_0].text.should eq 'Denmark'
+        t_row = tp.select_row_from_table_without_header(:number => 1, :column_0 => 'Denmark')
+        t_row.should be_a Hash
+        t_row[:column_0].text.should eq 'Denmark'
+      end
+    end
+
+    context "several rows found" do
+      it "returns first found row" do
+        table_page = tp.fire_press_cool_button
+        t_row = table_page.select_row_from_test_table(:column_1 => 'Default text.')
+        t_row.should be_a Hash
+        t_row[:column_0].text.should eq 'text'
+      end
+    end
+
+    context "no rows found" do
+      it "returns nil" do
+        t_row = tp.select_row_from_table_with_header(:number => 1, :country => 'Russia')
+        t_row.should eq nil
+        t_row = tp.select_row_from_table_with_header(:country => 'Russia', :land_area => '100')
+        t_row.should eq nil
+        t_row = tp.select_row_from_table_without_header(:number => 1, :column_1 => 'Russia')
+        t_row.should eq nil
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: page_object_wrapper
 version: !ruby/object:Gem::Version
-  version: 1.4.8
+  version: 1.4.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-01 00:00:00.000000000 Z
+date: 2013-04-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: watir-webdriver
-  requirement: &10964880 !ruby/object:Gem::Requirement
+  requirement: &13065260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *10964880
+  version_requirements: *13065260
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &10980640 !ruby/object:Gem::Requirement
+  requirement: &13100340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *10980640
+  version_requirements: *13100340
 - !ruby/object:Gem::Dependency
   name: babosa
-  requirement: &10980020 !ruby/object:Gem::Requirement
+  requirement: &13099320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *10980020
+  version_requirements: *13099320
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &10979200 !ruby/object:Gem::Requirement
+  requirement: &13094780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: 2.0.0
   type: :development
   prerelease: false
-  version_requirements: *10979200
+  version_requirements: *13094780
 - !ruby/object:Gem::Dependency
   name: debugger
-  requirement: &10978560 !ruby/object:Gem::Requirement
+  requirement: &13116000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,7 +65,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10978560
+  version_requirements: *13116000
 description: Wraps watir-webdriver with convenient testing interface, based on PageObjects
   automation testing pattern. Simplifies resulting automated test understanding.
 email:
@@ -116,6 +116,7 @@ files:
 - spec/press_xxx_spec.rb
 - spec/required_elements_spec.rb
 - spec/select_from_xxx_spec.rb
+- spec/select_raw_from_xxx_spec.rb
 - spec/shared_examples.rb
 - spec/spec_helper.rb
 - spec/xxx_each.rb
@@ -156,6 +157,7 @@ test_files:
 - spec/press_xxx_spec.rb
 - spec/required_elements_spec.rb
 - spec/select_from_xxx_spec.rb
+- spec/select_raw_from_xxx_spec.rb
 - spec/shared_examples.rb
 - spec/spec_helper.rb
 - spec/xxx_each.rb