rautomation 0.12.0 → 0.13.0

data/ext/WindowsForms/WindowsForms/DataEntryForm.cs

@@ -43,5 +43,10 @@ namespace WindowsForms
 
         }
 
+        private void toggleMultiSelect_Click(object sender, EventArgs e)
+        {
+            personListView.MultiSelect = !personListView.MultiSelect;
+        }
+
     }
 }

data/ext/WindowsForms/WindowsForms/MainFormWindow.Designer.cs

@@ -29,14 +29,14 @@
         private void InitializeComponent()
         {
             this.components = new System.ComponentModel.Container();
-            System.Windows.Forms.TreeNode treeNode6 = new System.Windows.Forms.TreeNode("Child 1");
-            System.Windows.Forms.TreeNode treeNode7 = new System.Windows.Forms.TreeNode("Grandchild 1");
-            System.Windows.Forms.TreeNode treeNode8 = new System.Windows.Forms.TreeNode("Child 2", new System.Windows.Forms.TreeNode[] {
-            treeNode7});
-            System.Windows.Forms.TreeNode treeNode9 = new System.Windows.Forms.TreeNode("Parent One", new System.Windows.Forms.TreeNode[] {
-            treeNode6,
-            treeNode8});
-            System.Windows.Forms.TreeNode treeNode10 = new System.Windows.Forms.TreeNode("Parent Two");
+            System.Windows.Forms.TreeNode treeNode1 = new System.Windows.Forms.TreeNode("Child 1");
+            System.Windows.Forms.TreeNode treeNode2 = new System.Windows.Forms.TreeNode("Grandchild 1");
+            System.Windows.Forms.TreeNode treeNode3 = new System.Windows.Forms.TreeNode("Child 2", new System.Windows.Forms.TreeNode[] {
+            treeNode2});
+            System.Windows.Forms.TreeNode treeNode4 = new System.Windows.Forms.TreeNode("Parent One", new System.Windows.Forms.TreeNode[] {
+            treeNode1,
+            treeNode3});
+            System.Windows.Forms.TreeNode treeNode5 = new System.Windows.Forms.TreeNode("Parent Two");
             this.label1 = new System.Windows.Forms.Label();
             this.aboutButton = new System.Windows.Forms.Button();
             this.button1 = new System.Windows.Forms.Button();
@@ -72,6 +72,7 @@
             this.buttonDataGridView = new System.Windows.Forms.Button();
             this.automatableMonthCalendar1 = new System.Windows.Forms.MonthCalendar();
             this.toolTip1 = new System.Windows.Forms.ToolTip(this.components);
+            this.addSomeFruitsToolStripMenuItem = new System.Windows.Forms.ToolStripMenuItem();
             this.groupBox1.SuspendLayout();
             this.groupBox2.SuspendLayout();
             this.menuStrip1.SuspendLayout();
@@ -356,7 +357,8 @@
             // 
             this.faileToolStripMenuItem.DropDownItems.AddRange(new System.Windows.Forms.ToolStripItem[] {
             this.aboutToolStripMenuItem,
-            this.roundaboutWayToolStripMenuItem});
+            this.roundaboutWayToolStripMenuItem,
+            this.addSomeFruitsToolStripMenuItem});
             this.faileToolStripMenuItem.Name = "faileToolStripMenuItem";
             this.faileToolStripMenuItem.Size = new System.Drawing.Size(37, 20);
             this.faileToolStripMenuItem.Text = "&File";
@@ -395,19 +397,19 @@
             // 
             this.treeView.Location = new System.Drawing.Point(490, 285);
             this.treeView.Name = "treeView";
-            treeNode6.Name = "Child 1";
-            treeNode6.Text = "Child 1";
-            treeNode7.Name = "Grandchild 1";
-            treeNode7.Text = "Grandchild 1";
-            treeNode8.Name = "Child 2";
-            treeNode8.Text = "Child 2";
-            treeNode9.Name = "Parent One";
-            treeNode9.Text = "Parent One";
-            treeNode10.Name = "Parent Two";
-            treeNode10.Text = "Parent Two";
+            treeNode1.Name = "Child 1";
+            treeNode1.Text = "Child 1";
+            treeNode2.Name = "Grandchild 1";
+            treeNode2.Text = "Grandchild 1";
+            treeNode3.Name = "Child 2";
+            treeNode3.Text = "Child 2";
+            treeNode4.Name = "Parent One";
+            treeNode4.Text = "Parent One";
+            treeNode5.Name = "Parent Two";
+            treeNode5.Text = "Parent Two";
             this.treeView.Nodes.AddRange(new System.Windows.Forms.TreeNode[] {
-            treeNode9,
-            treeNode10});
+            treeNode4,
+            treeNode5});
             this.treeView.Size = new System.Drawing.Size(159, 177);
             this.treeView.TabIndex = 16;
             // 
@@ -435,6 +437,13 @@
             this.automatableMonthCalendar1.Name = "automatableMonthCalendar1";
             this.automatableMonthCalendar1.TabIndex = 19;
             // 
+            // addSomeFruitsToolStripMenuItem
+            // 
+            this.addSomeFruitsToolStripMenuItem.Name = "addSomeFruitsToolStripMenuItem";
+            this.addSomeFruitsToolStripMenuItem.Size = new System.Drawing.Size(166, 22);
+            this.addSomeFruitsToolStripMenuItem.Text = "Add Some Fruits";
+            this.addSomeFruitsToolStripMenuItem.Click += new System.EventHandler(this.addSomeFruitsToolStripMenuItem_Click);
+            // 
             // MainFormWindow
             // 
             this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 13F);
@@ -511,7 +520,8 @@
         private System.Windows.Forms.TextBox multiLineTextField;
         private System.Windows.Forms.Button buttonDataGridView;
         private System.Windows.Forms.MonthCalendar automatableMonthCalendar1;
-        private System.Windows.Forms.ToolTip toolTip1;
+        private System.Windows.Forms.ToolTip toolTip1;
+        private System.Windows.Forms.ToolStripMenuItem addSomeFruitsToolStripMenuItem;
     }
 }
 

data/ext/WindowsForms/WindowsForms/MainFormWindow.cs

@@ -1,4 +1,6 @@
 using System;
+using System.Collections.Generic;
+using System.Linq;
 using System.Windows.Forms;
 using UIA.Extensions;
 
@@ -74,7 +76,34 @@ namespace WindowsForms
 
         private void FruitListBox_SelectedIndexChanged(object sender, EventArgs e)
         {
-            fruitsLabel.Text = (string) FruitListBox.SelectedItem;
+            var selectedFruits = (FruitListBox.SelectedItems.Cast<object>().Select(item => item.ToString()));
+            fruitsLabel.Text = String.Join(",", selectedFruits);
+        }
+
+        private void addSomeFruitsToolStripMenuItem_Click(object sender, EventArgs e)
+        {
+            10.Times(AddAnotherFruit);
+        }
+
+        private void AddAnotherFruit(int fruitNumber)
+        {
+            FruitListBox.Items.Add("Fruit " + fruitNumber);
+        }
+    }
+
+    static class EnumerableExtensions
+    {
+        public static void ForEach<T>(this IEnumerable<T> items, Action<T> doIt)
+        {
+            foreach (var item in items)
+            {
+                doIt(item);
+            }
+        }
+
+        public static void Times(this int howMany, Action<int> doIt)
+        {
+            Enumerable.Range(0, howMany).ForEach(doIt);
         }
     }
 }

data/lib/rautomation/adapter/ms_uia/select_list.rb

@@ -24,11 +24,11 @@ module RAutomation
             UiaDll::select_list_select_index @select_list.search_information, @index
           end
 
-          alias_method :set, :select
-        end
+          def clear
+            UiaDll::remove_from_selection @select_list.search_information, @index
+          end
 
-        def set(value)
-          UiaDll::select_list_select_value(search_information, value)
+          alias_method :set, :select
         end
 
         def options(options = {})
@@ -50,6 +50,10 @@ module RAutomation
           UiaDll::selection(search_information)
         end
 
+        def values
+          UiaDll::selections(search_information)
+        end
+
         def option(options)
           UiaDll::select_options(search_information).each_with_index do |item, item_no|
             return SelectListOption.new(self, item, item_no) if options[:text] == item
@@ -58,10 +62,6 @@ module RAutomation
           nil
         end
 
-        def select(index)
-          UiaDll::select_list_select_index search_information, index
-        end
-
         def exist?
           super && matches_type?(Constants::UIA_COMBOBOX_CONTROL_TYPE)
         end

data/lib/rautomation/adapter/ms_uia/table.rb

@@ -50,6 +50,18 @@ module RAutomation
           @locators[:index] || 0
         end
 
+        def select
+          UiaDll::table_select search_information, row
+        end
+
+        def selected?
+          UiaDll::table_row_is_selected search_information, row
+        end
+
+        def clear
+          UiaDll::table_remove_from_selection search_information, row
+        end
+
         def value
           UiaDll::table_value_at search_information, @locators[:index]
         end
@@ -96,14 +108,6 @@ module RAutomation
           all_strings
         end
 
-        def select(which_item)
-          UiaDll::table_select search_information, which_item
-        end
-
-        def selected?(which_item)
-          UiaDll::table_row_is_selected search_information, which_item
-        end
-
         def row_count
           UiaDll::table_row_count search_information
         end

data/lib/rautomation/adapter/ms_uia/uia_dll.rb

@@ -185,6 +185,8 @@ module RAutomation
         # Select List methods
         attach_function :SelectList_Selection,
                         [SearchCriteria.by_ref, :pointer, :int], :void
+        attach_function :SelectList_Selections,
+                        [SearchCriteria.by_ref, :pointer], :int
         attach_function :SelectList_Options,
                         [SearchCriteria.by_ref, :pointer], :int
         attach_function :select_list_count, :SelectList_Count,
@@ -197,6 +199,10 @@ module RAutomation
                         [SearchCriteria.by_ref, :int], :bool
         attach_function :select_list_select_value, :SelectList_SelectValue,
                         [SearchCriteria.by_ref, :pointer], :int
+        attach_function :SelectList_RemoveIndex,
+                        [SearchCriteria.by_ref, :int, :pointer, :int], :void
+        attach_function :SelectList_RemoveValue,
+                        [SearchCriteria.by_ref, :string, :pointer, :int], :void
 
         def self.select_options(search_information)
           strings_from(:SelectList_Options, search_information)
@@ -206,6 +212,19 @@ module RAutomation
           string_from(:SelectList_Selection, search_information)
         end
 
+        def self.selections(search_information)
+          strings_from(:SelectList_Selections, search_information)
+        end
+
+        def self.remove_from_selection(search_information, which_item)
+          case which_item
+            when Fixnum
+              can_throw(:SelectList_RemoveIndex, search_information, which_item)
+            when String
+              can_throw(:SelectList_RemoveValue, search_information, which_item)
+          end
+        end
+
         # Spinner methods
         attach_function :Spinner_GetValue,
                         [SearchCriteria.by_ref, :pointer, :int], :double
@@ -290,18 +309,22 @@ module RAutomation
         attach_function :Table_ValueAt,
                         [SearchCriteria.by_ref, :int, :int, :pointer, :int], :void
         attach_function :Table_SelectByIndex,
-                        [SearchCriteria.by_ref, :int], :void
+                        [SearchCriteria.by_ref, :int, :pointer, :int], :void
         attach_function :Table_SelectByValue,
-                        [SearchCriteria.by_ref, :string], :void
+                        [SearchCriteria.by_ref, :string, :pointer, :int], :void
         attach_function :table_row_is_selected, :Table_IsSelectedByIndex,
                         [SearchCriteria.by_ref, :int], :bool
+        attach_function :Table_RemoveRowByIndex,
+                        [SearchCriteria.by_ref, :int, :pointer, :int], :void
+        attach_function :Table_RemoveRowByValue,
+                        [SearchCriteria.by_ref, :string, :pointer, :int], :void
 
         def self.table_select(search_information, which_item)
           case which_item
             when Integer
-              Table_SelectByIndex search_information, which_item
+              can_throw(:Table_SelectByIndex, search_information, which_item)
             when String
-              Table_SelectByValue search_information, which_item
+              can_throw(:Table_SelectByValue, search_information, which_item)
           end
         end
 
@@ -321,6 +344,15 @@ module RAutomation
           strings_from :Table_GetValues, search_information
         end
 
+        def self.table_remove_from_selection(search_information, which_row)
+          case which_row
+            when Fixnum
+              can_throw(:Table_RemoveRowByIndex, search_information, which_row)
+            when String
+              can_throw(:Table_RemoveRowByValue, search_information, which_row)
+          end
+        end
+
         # String methods
         attach_function :clean_up_strings, :String_CleanUp,
                         [:pointer, :int], :void

data/spec/adapter/ms_uia/button_spec.rb

@@ -30,12 +30,12 @@ describe "MsUia::Button", :if => SpecHelper.adapter == :ms_uia do
 
   it "cannot click disabled button" do
     window = RAutomation::Window.new(:title => SpecHelper::DATA[:window1_title])
-    lambda { window.button(:id => "disabledButton").click }.should raise_error
+    expect { window.button(:id => "disabledButton").click }.to raise_error
   end
 
   it "cannot set focus to disabled button" do
     window = RAutomation::Window.new(:title => SpecHelper::DATA[:window1_title])
-    lambda { window.button(:id => "disabledButton").focus }.should raise_error
+    expect { window.button(:id => "disabledButton").focus }.to raise_error
   end
 
 end

data/spec/adapter/ms_uia/checkbox_spec.rb

@@ -46,9 +46,9 @@ describe "MsUia::Checkbox", :if => SpecHelper.adapter == :ms_uia do
   end
 
   it "cannot check a disabled checkbox" do
-    lambda {
+    expect {
       RAutomation::Window.new(:title => "MainFormWindow").checkbox(:id => "checkBoxDisabled").set
-    }.should raise_error
+    }.to raise_error
   end
 
 end

data/spec/adapter/ms_uia/listbox_spec.rb

@@ -1,32 +1,30 @@
 require 'spec_helper'
 
 describe "MsUia::ListBox", :if => SpecHelper.adapter == :ms_uia do
+  let(:window) { RAutomation::Window.new(:title => "MainFormWindow") }
+  let(:label) { window.label(:id => "fruitsLabel") }
+  let(:list_box) { window.list_box(:id => "FruitListBox") }
 
   it "#exists" do
-    RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox").should exist
+    list_box.should exist
   end
 
   it "checks for ListBox class" do
-    RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "textField").should_not exist
-    RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox").should exist
+    window.list_box(:id => "textField").should_not exist
+    list_box.should exist
   end
 
   it "counts items" do
-    list_box = RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox")
     list_box.count.should == 3
   end
 
   it "lists items" do
-    list_box = RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox")
-
     list_box.items[0].value.should == "Apple"
     list_box.items[1].value.should == "Orange"
     list_box.items[2].value.should == "Mango"
   end
 
   it "returns a value" do
-    list_box = RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox")
-
     list_box.value.should == ""
     list_box.select(0)
     list_box.value.should == "Apple"
@@ -37,16 +35,12 @@ describe "MsUia::ListBox", :if => SpecHelper.adapter == :ms_uia do
   end
 
   it "#selected?" do
-    list_box = RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox")
-
     list_box.selected?(2).should == false
     list_box.select(2)
     list_box.selected?(2).should == true
   end
 
   it "#select" do
-    list_box = RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox")
-
     list_box.select(1)
     list_box.selected?(1).should == true
 
@@ -58,18 +52,22 @@ describe "MsUia::ListBox", :if => SpecHelper.adapter == :ms_uia do
   end
 
   it "#strings" do
-    list_box = RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox")
-
     list_box.strings.should == ["Apple", "Orange", "Mango"]
   end
 
   it "fires events when the index changes" do
-    list_box = RAutomation::Window.new(:title => "MainFormWindow").list_box(:id => "FruitListBox")
-    label = RAutomation::Window.new(:title => "MainFormWindow").label(:id => "fruitsLabel")
-
     ['Apple', 'Orange', 'Mango'].each_with_index do |value, index|
       list_box.select index
       label.value.should eq(value)
     end
   end
+
+  it 'fires events even when the item is not in the current view' do
+    window.menu(text: 'File').menu(text: 'Add Some Fruits').open
+
+    list_box.strings.each_with_index do |value, index|
+      list_box.select index
+      label.value.should eq(value)
+    end
+  end
 end

data/spec/adapter/ms_uia/radio_spec.rb

@@ -34,6 +34,6 @@ describe "MsUia::RadioButton", :if => SpecHelper.adapter == :ms_uia do
   end
 
   it "cannot set a disabled radio button" do
-    lambda { RAutomation::Window.new(:title => "MainFormWindow").radio(:value => "Option Disabled").set }.should raise_error
+    expect { RAutomation::Window.new(:title => "MainFormWindow").radio(:value => "Option Disabled").set }.to raise_error
   end
 end

data/spec/adapter/ms_uia/select_list_spec.rb

@@ -1,101 +1,103 @@
 require 'spec_helper'
 
-describe "MsUia::SelectList", :if => SpecHelper.adapter == :ms_uia do
-  it "#select_list" do
-    RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox").should exist
+describe 'MsUia::SelectList', :if => SpecHelper.adapter == :ms_uia do
+  let(:window) { RAutomation::Window.new(:title => 'MainFormWindow') }
+  let(:fruits_combo) { window.select_list(:id => 'FruitsComboBox') }
+  let(:fruits_list) { window.select_list(:id => 'FruitListBox') }
+  let(:fruit_label) { window.label(:id => 'fruitsLabel') }
+  let(:disabled_combo) { window.select_list(:id => 'comboBoxDisabled') }
+  let(:about) do
+    window.button(:value => 'About').click { true }
+    about = RAutomation::Window.new(:title => 'About')
+  end
+  let(:multi_fruits) do
+    about.tab_control(:id => 'tabControl').set 'Multi-Select ListBox'
+    about.select_list(:id => 'multiFruitListBox')
+  end
+
+  it '#fruits_combo' do
+    fruits_combo.should exist
 
     RAutomation::Window.wait_timeout = 0.1
-    expect {RAutomation::Window.new(:title => "non-existent-window").
+    expect {RAutomation::Window.new(:title => 'non-existent-window').
             select_list(:class => /COMBOBOX/i)}.
             to raise_exception(RAutomation::UnknownWindowException)
   end
 
-  it "check for select list class" do
-    RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "textField").should_not exist
-    RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox").should exist
+  it 'check for select list class' do
+    window.select_list(:id => 'textField').should_not exist
+    fruits_combo.should exist
   end
 
-  it "#options" do
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox")
-    select_list.options.size.should == 5
-
-    expected_options = ["Apple", "Caimito", "Coconut", "Orange", "Passion Fruit"]
-    select_list.options.map {|option| option.text}.should == expected_options
-  end
+  it '#options' do
+    fruits_combo.options.size.should == 5
 
-  it "#selected? & #select" do
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox")
-    select_list.options(:text => "Apple")[0].should_not be_selected
-    select_list.options(:text => "Apple")[0].select.should be_true
-    select_list.options(:text => "Apple")[0].should be_selected
+    expected_options = ['Apple', 'Caimito', 'Coconut', 'Orange', 'Passion Fruit']
+    fruits_combo.options.map {|option| option.text}.should == expected_options
   end
 
-  it "#set" do
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox")
-    select_list.options(:text => "Apple")[0].should_not be_selected
-    select_list.set("Apple")
-    select_list.options(:text => "Apple")[0].should be_selected
+  it '#selected? & #select' do
+    fruits_combo.options(:text => 'Apple')[0].should_not be_selected
+    fruits_combo.options(:text => 'Apple')[0].select.should be_true
+    fruits_combo.options(:text => 'Apple')[0].should be_selected
   end
 
-  it "#value" do
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox")
+  it '#value' do
 
     #default empty state
-    select_list.value.should == ""
+    fruits_combo.value.should == ''
 
-    select_list.options(:text => "Apple")[0].select
-    select_list.value.should == "Apple"
+    fruits_combo.options(:text => 'Apple')[0].select
+    fruits_combo.value.should == 'Apple'
 
-    select_list.options(:text => "Caimito")[0].select
-    select_list.value.should == "Caimito"
+    fruits_combo.options(:text => 'Caimito')[0].select
+    fruits_combo.value.should == 'Caimito'
   end
 
-  it "enabled/disabled" do
-    RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox").should be_enabled
-    RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox").should_not be_disabled
+  it 'enabled/disabled' do
+    fruits_combo.should be_enabled
+    fruits_combo.should_not be_disabled
 
-    RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "comboBoxDisabled").should_not be_enabled
-    RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "comboBoxDisabled").should be_disabled
+    disabled_combo.should_not be_enabled
+    disabled_combo.should be_disabled
   end
 
-  it "#option" do
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox")
+  it '#values' do
+    multi_fruits.values.should eq([]) # => empty state
 
-    select_list.option(:text => "Apple").should_not be_selected
-    select_list.option(:text => "Apple").set
-    select_list.option(:text => "Apple").should be_selected
+    ['Apple', 'Mango'].each do |value|
+      multi_fruits.option(:text => value).select
+    end
+    multi_fruits.values.should eq(['Apple', 'Mango'])
   end
 
-  it "cannot select anything on a disabled select list" do
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "comboBoxDisabled")
-
-    lambda { select_list.option(:text => "Apple").set }.should raise_error
+  it '#option' do
+    fruits_combo.option(:text => 'Apple').should_not be_selected
+    fruits_combo.option(:text => 'Apple').set
+    fruits_combo.option(:text => 'Apple').should be_selected
   end
 
-  it "fires change event when the value is set" do
-
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox")
-
-    select_list.option(:text => "Apple").should_not be_selected
-    select_list.set("Apple")
-    select_list.option(:text => "Apple").should be_selected
-
-    label = RAutomation::Window.new(:title => "MainFormWindow").label(:id => "fruitsLabel")
-    RAutomation::WaitHelper.wait_until { label.exist? }
-    label.value.should == "Apple"
+  it 'cannot select anything on a disabled select list' do
+    expect { disabled_combo.option(:text => 'Apple').set }.to raise_error
   end
 
-  it "fires change event when the index changes" do
+  it 'fires change event when the value is set' do
+    apple_option = fruits_combo.option(:text => 'Apple')
+    apple_option.should_not be_selected
+    apple_option.select
+    apple_option.should be_selected
 
-    select_list = RAutomation::Window.new(:title => "MainFormWindow").select_list(:id => "FruitsComboBox")
-    label = RAutomation::Window.new(:title => "MainFormWindow").label(:id => "fruitsLabel")
+    RAutomation::WaitHelper.wait_until { fruit_label.exist? }
+    fruit_label.value.should == 'Apple'
+  end
 
-    select_list.options[4].select
-    select_list.option(:text => "Passion Fruit").should be_selected
-    label.value.should == "Passion Fruit"
+  it 'fires change event when the index changes' do
+    fruits_combo.options[4].select
+    fruits_combo.option(:text => 'Passion Fruit').should be_selected
+    fruit_label.value.should == 'Passion Fruit'
 
-    select_list.select 3
-    select_list.option(:text => "Orange").should be_selected
-    label.value.should == "Orange"
+    fruits_combo.options[3].select
+    fruits_combo.option(:text => 'Orange').should be_selected
+    fruit_label.value.should == 'Orange'
   end
 end