watirgrid 0.0.3b → 0.0.3

data/EXAMPLES.rdoc

@@ -91,10 +91,10 @@ Watir has been extended with a Grid class that can be used as follows:
   require 'rubygems'
   require 'watirgrid'
 
-  grid = Watir::Grid.new(:ring_server_port => 12358)
-  grid.start(:quantity => 1, :read_all => true, :browser_type => 'ie')
+  browsers = Watir::Grid.new(:ring_server_port => 12358)
+  @browsers = browsers.start(:quantity => 1, :read_all => true, :browser_type => 'ie')
   threads = []
-  grid.browsers.each do |browser|
+  @browsers.each do |browser|
     threads << Thread.new do 
       b = browser[:object].new_browser
       b.goto("http://www.google.com")
@@ -105,13 +105,13 @@ Watir has been extended with a Grid class that can be used as follows:
   threads.each {|thread| thread.join}
 
 Stepping through this example we first instantiate a browsers object, specifying which ring server port to broadcast on when looking for available providers:
-  grid = Watir::Grid.new(:ring_server_port => 12358)
+  browsers = Watir::Grid.new(:ring_server_port => 12358)
 
 You may also need to tell the code which host the ring server is on:
-  grid = Watir::Grid.new(:ring_server_port => 12358, :ring_server_host => 143.238.105.61)
+  browsers = Watir::Grid.new(:ring_server_port => 12358, :ring_server_host => 143.238.105.61)
 
 Next we start up the grid, specifying the number of browsers we wish to use, and the method of accessing the tuple space:
-  grid.start(:quantity => 1, :read_all => true)
+  browsers.start(:quantity => 1, :read_all => true)
   
 There are two methods for accessing the tuple space. 
   :read_all => true

data/README.rdoc

@@ -40,10 +40,10 @@ This should find the recently started Ring Server and register a tuple for the W
 You will now be able to execute commands across remote browsers on your grid network.
 
 e.g.
-  grid = Watir::Grid.new(:ring_server_port => 12358)
-  grid.start(:quantity => 1, :read_all => true)
+  browsers = Watir::Grid.new(:ring_server_port => 12358)
+  @browsers = browsers.start(:quantity => 1, :read_all => true)
   threads = []
-  grid.browsers.each do |browser|
+  @browsers.each do |browser|
     threads << Thread.new do 
       b = browser[:object].new_browser
       b.goto("http://www.google.com")

data/Rakefile

@@ -11,7 +11,7 @@ begin
     gem.homepage = "http://github.com/90kts/watirgrid"
     gem.authors = ["Tim Koopmans"]
     gem.add_development_dependency "rspec", ">= 1.2.9"
-    gem.version = "0.0.3b"
+    gem.version = "0.0.3"
     # gem is a Gem::Specification... see http://www.rubygems.org/read/chapter/20 for additional settings
   end
 rescue LoadError

data/examples/simple.rb

@@ -1,12 +1,12 @@
 require 'rubygems'
 require '../lib/watirgrid'
 
-grid = Watir::Grid.new(:ring_server_port => 12358, 
-:ring_server_host => '192.168.1.101', :loglevel => Logger::DEBUG)
-grid.start(:quantity => 1, :read_all => true)
+browsers = Watir::Grid.new(:ring_server_port => 12358, 
+:ring_server_host => '192.168.1.122', :loglevel => Logger::DEBUG)
+@browsers = browsers.start(:quantity => 2, :read_all => true)
 
 threads = []
-grid.browsers.each do |browser|
+@browsers.each do |browser|
   threads << Thread.new do 
     b = browser[:object].new_browser
     b.goto("http://www.google.com")

data/lib/watirgrid.rb

@@ -23,14 +23,11 @@ module Watir
       @log  = Logger.new(logfile, 'daily')
       @log.level = params[:loglevel] || Logger::ERROR
       @log.datetime_format = "%Y-%m-%d %H:%M:%S "   
-      
-      @browsers = []
-      @tuples = []
     end
 
     ##
     # Start required services
-    def start(params = {})      
+    def start(params = {})
       start_drb_server
       find_ring_server
       get_tuples(params)
@@ -75,37 +72,15 @@ module Watir
     ##
     # Get all tuple spaces on ringserver
     def get_tuples(params = {})
-      quantity = calculate_quantity(params[:quantity]) 
-      read_tuples(params[:architecture], params[:browser_type])
-      @log.info("Found #{@tuples.size} tuples.")
-      if @tuples.size > -1 then
-        @tuples[0..quantity].each do |tuple|
-          if params[:hostnames]
-            filter_tuple_by_hostname(tuple, params)
-          else
-            add_tuple_to_browsers(tuple)
-            take_tuple(tuple) if params[:take_all] == true
-          end      
-        end
-      end
-    end
-
-    ##
-    # Sets the quantity (upper limit of array) of tuples to retrieve
-    # This is because some users prefer not to specify a zero based
-    # index when asking for n browsers
-    def calculate_quantity(quantity)
-      if (quantity.nil? or quantity == 0) then
+      if (params[:quantity].nil? or params[:quantity] == 0) then
         quantity = -1 
       else
-        quantity -= 1
+        quantity = params[:quantity] - 1
       end
-    end
-
-    ##
-    # Read all tuples filtered by architecture and browser type
-    # then populate the tuples accessor
-    def read_tuples(architecture, browser_type)
+      architecture = params[:architecture] || nil
+      browser_type = params[:browser_type] || nil
+      
+      @browsers = []
       @tuples = @ring_server.read_all([
         :name,
         nil, # watir provider
@@ -114,30 +89,29 @@ module Watir
         nil, # hostname
         architecture,
         browser_type])
-    end
 
-    ##
-    # Filter tuple by hostnames
-    def filter_tuple_by_hostname(tuple, params={})
-      hostname = tuple[4]
-      if (params[:hostnames][hostname]) then
-        add_tuple_to_browsers(tuple)
-        take_tuple(tuple) if params[:take_all] == true
+      @log.info("Found #{@tuples.size} tuples.")
+      if @tuples.size > -1 then
+         @log.debug("Iterating from 0 to #{quantity}")
+        @tuples[0..quantity].each do |tuple|
+          @log.debug("Iterating through #{@tuples.size} tuples")
+          hostname = tuple[4]
+          if params[:hostnames] then
+            if params[:hostnames][hostname] then
+              @browsers << tuple_to_hash(tuple)
+              @ring_server.take(tuple)if params[:take_all] == true
+            end
+          else
+            @browsers << tuple_to_hash(tuple)
+            @ring_server.take(tuple)if params[:take_all] == true
+          end
+        end
+      else
+        @browsers
       end
+      @browsers
     end
-
-    ##
-    # Add a tuple to the browsers accessor
-    def add_tuple_to_browsers(tuple)
-      @browsers <<  tuple_to_hash(tuple)
-    end
-
-    ##
-    # Take a tuple from the tuple space
-    def take_tuple(tuple)
-      @ring_server.take(tuple)
-    end
-
+    
     ##
     # Convert tuple into a hash for easier handling
     def tuple_to_hash(tuple)
@@ -151,7 +125,6 @@ module Watir
       tuple_hash[:browser_type] = tuple[6]
       tuple_hash
     end
-  
   end
 
 end

data/spec/grid_spec.rb

@@ -14,37 +14,37 @@ describe 'WatirGrid' do
     end
   end
 
-  it 'should return how many grid are available in the tuplespace' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:read_all => true)
-    grid.size.should == 5
+  it 'should return how many browsers are available in the tuplespace' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:read_all => true)
+    browsers.size.should == 5
   end
 
-  it 'should read any 2 grid in the tuplespace' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 2, :read_all => true)
-    grid.size.should == 2
+  it 'should read any 2 browsers in the tuplespace' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 2, :read_all => true)
+    browsers.size.should == 2
   end
 
   it 'should take any 1 browser in the tuplespace' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1, :take_all => true)
-    grid.size.should == 1 
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1, :take_all => true)
+    browsers.size.should == 1 
   end
 
-  it 'should take all grid remaining in tuplespace' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:take_all => true)
-    grid.size.should == 4
+  it 'should take all browsers remaining in tuplespace' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:take_all => true)
+    browsers.size.should == 4
   end
 
-  it 'should find no more grid in the tuplespace' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:read_all => true)
-    grid.size.should == 0
+  it 'should find no more browsers in the tuplespace' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:read_all => true)
+    browsers.size.should == 0
   end
 
-  it 'should register 4 new grid in the tuplespace' do
+  it 'should register 4 new browsers in the tuplespace' do
     1.upto(4) do 
       provider = Provider.new(:ring_server_port => 12351, 
         :loglevel => Logger::ERROR, :browser_type => 'safari')
@@ -53,63 +53,64 @@ describe 'WatirGrid' do
   end
 
   it 'should take any 1 browser based on browser type' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1,
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1,
       :take_all => true, :browser_type => 'safari')
-    grid.size.should == 1
+    browsers.size.should == 1
   end
 
-  it 'should fail to find any grid based on a specific browser type' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1,
+  it 'should fail to find any browsers based on a specific browser type' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1,
       :take_all => true, :browser_type => 'firefox')
-    grid.size.should == 0
+    browsers.size.should == 0
   end
 
-  it 'should fail to find any grid based on a unknown browser type' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1,
+  it 'should fail to find any browsers based on a unknown browser type' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1,
       :take_all => true, :browser_type => 'penguin')
-    grid.size.should == 0
+    browsers.size.should == 0
   end
 
   it 'should take any 1 browser based on specific architecture type' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1, 
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1, 
       :take_all => true, :architecture => 'universal-darwin10.0')
-    grid.size.should == 1
+    browsers.size.should == 1
   end
 
-  it 'should fail to find any grid based on unknown architecture type' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1, 
+  it 'should fail to find any browsers based on unknown architecture type' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1, 
       :take_all => true, :architecture => 'geos-1992')
-    grid.size.should == 0
+    browsers.size.should == 0
   end
 
   it 'should take any 1 browser based on specific hostname' do
     hostname = `hostname`.strip
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1,
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1,
       :take_all => true, 
       :hostnames => { hostname => "127.0.0.1"}
       )
-    grid.size.should == 1
+    browsers.size.should == 1
   end
 
-  it 'should fail to find any grid based on unknown hostname' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1,
+  it 'should fail to find any browsers based on unknown hostname' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:quantity => 1,
       :take_all => true, :hostnames => { 
         "tokyo" => "127.0.0.1"})
-    grid.size.should == 0
+    browsers.size.should == 0
   end
 
   it 'should take the last browser and execute some watir commands' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:quantity => 1, :take_all => true)
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    @browsers = browsers.start(:quantity => 1,
+      :take_all => true)
     threads = []
-    grid.browsers.each do |browser|
+    @browsers.each do |browser|
       threads << Thread.new do 
         browser[:hostname].should == `hostname`.strip
         browser[:architecture].should == Config::CONFIG['arch']
@@ -121,18 +122,18 @@ describe 'WatirGrid' do
       end
     end
     threads.each {|thread| thread.join}
-    grid.size.should == 1
+    browsers.size.should == 1
   end
 
-  it 'should find no more grid in the tuplespace' do
-    grid = Watir::Grid.new(:ring_server_port => 12351)
-    grid.start(:read_all => true)
-    grid.size.should == 0
+  it 'should find no more browsers in the tuplespace' do
+    browsers = Watir::Grid.new(:ring_server_port => 12351)
+    browsers.start(:read_all => true)
+    browsers.size.should == 0
   end
 
   it 'should register a new browser on a remote provider' do
     pending('provision of remote registration') do
-      grid.size.should == 0
+      browsers.size.should == 0
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: watirgrid
 version: !ruby/object:Gem::Version 
-  version: 0.0.3b
+  version: 0.0.3
 platform: ruby
 authors: 
 - Tim Koopmans
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-12-05 00:00:00 +11:00
+date: 2009-12-01 00:00:00 +11:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -69,9 +69,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
   version: 
 required_rubygems_version: !ruby/object:Gem::Requirement 
   requirements: 
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version 
-      version: 1.3.1
+      version: "0"
   version: 
 requirements: []