operawatir 0.4.3.pre1-jruby → 0.5-jruby

data/lib/operawatir/quickwidgets/quick_searchfield.rb

@@ -8,9 +8,12 @@ module OperaWatir
     end
 
     ######################################################################
-    # Enters the search text into the search field, and waits for page 
+    # Enters the search text into the search field, and waits for page
     # loading to finish
     #
+    # @example (RSpec)
+    #    browser.quick_searchfield(:name, "tbs_MainSearch").search_with_text("old cars").should include "http://www.google"
+    #
     # @param [String] url   text to search with
     #
     # @return [String] text in the address field after the page is loaded
@@ -19,13 +22,13 @@ module OperaWatir
     def search_with_text(search_text)
       # Must focus field before calling enter_text...
       focus_with_click
-      
+
       # Enters text in a field and then hits enter
       t = enter_text_and_hit_enter(search_text)
 
-      # return text in addressfield (in same window as search field)      
+      # return text in addressfield (in same window as search field)
       driver.findWidgetByName(WIDGET_ENUM_MAP[:addressfield], @window_id, "tba_address_field", "Document Toolbar").getText
     end
-    
+
   end
 end

data/lib/operawatir/quickwidgets/quick_tab.rb

@@ -1,12 +1,12 @@
 module OperaWatir
   class QuickTab < QuickButton
- 
+
     # @private
     # Checks the type of the widget is correct
     def correct_type?
       @element.getType == WIDGET_ENUM_MAP[:tabbutton]
     end
-    
+
     ######################################################################
     # Drag and drop this tab on tab tab_target
     #
@@ -17,10 +17,10 @@ module OperaWatir
     def move_with_drag(tab_target)
       raise(Exceptions::UnknownObjectException) unless tab_target.type == :tabbutton
       drag_and_drop_on(tab_target, :center)
-      
+
       sleep(0.1)
     end
-    
+
     ######################################################################
     # Drag and drop this tab on another tab to add it to its tab group
     #
@@ -32,34 +32,45 @@ module OperaWatir
     #               in which case it represents only 1 tab, itself
     def group_with_drag(tab_target)
       raise(Exceptions::UnknownObjectException) unless tab_target.type == :tabbutton
-      
+
       # Drop on the edge of a tab to make the grouping work
       drag_and_drop_on(tab_target, :edge)
-      
+
       sleep(0.1)
-      
+
       #Get the  number of tabs in the group
       element(true).value
     end
-    
-        
+
+
     ######################################################################
     # Clicks the tab button, and waits for the tab to be shown / switches to the page
     #
     # @param [String] win_name name of the window that will be opened (Pass a blank string for any window)
     #
-    # @return [int] Window ID of the window activated, 
+    # @return [int] Window ID of the window activated,
     #               0 if the window is already the active one,
-    #               or if no window is active 
+    #               or if no window is active
     #
     # @raise [DesktopExceptions::WidgetNotVisibleException] if the tab button
     #            is not visible
     #
     def activate_tab_with_click
       wait_start
-      click 
+      click
       wait_for_window_activated("Document Window")
     end
 
+    ###########################################################################
+    # Double clicks the tab and waits for it to be closed
+    #
+    # @return [int] Window ID of the window that was closed
+    #
+    def close_window_with_doubleclick
+      wait_start
+      click(:left, 2)
+      wait_for_window_close("Document Window")
+    end
+
   end
 end

data/lib/operawatir/quickwidgets/quick_thumbnail.rb

@@ -1,12 +1,12 @@
 module OperaWatir
   class QuickThumbnail < QuickButton
- 
+
     # @private
     # Checks the type of the widget is correct
     def correct_type?
       @element.getType == WIDGET_ENUM_MAP[:thumbnail]
     end
-    
+
     ######################################################################
     # Drag and drop this speeddial onto the speed dial tab_target
     #
@@ -18,9 +18,9 @@ module OperaWatir
     def move_with_drag(tab_target)
       raise(Exceptions::UnknownObjectException) unless tab_target.type == :thumbnail
       drag_and_drop_on(tab_target, :center)
-      
+
       sleep(0.1)
     end
-    
+
   end
 end

data/lib/operawatir/quickwidgets/quick_toolbar.rb

@@ -6,6 +6,6 @@ module OperaWatir
     def correct_type?
       @element.getType == WIDGET_ENUM_MAP[:toolbar]
     end
-    
+
   end
 end

data/lib/operawatir/quickwidgets/quick_treeitem.rb

@@ -10,7 +10,7 @@ module OperaWatir
     ######################################################################
     # Set focus to the tree item by clicking on it
     #
-    # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeview 
+    # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeview
     #            the treeitem is in is not visible
     # @raise [DesktopExceptions::WidgetDisabledException] if the treeitem
     #            is disabled
@@ -20,24 +20,24 @@ module OperaWatir
       scroll_item_into_view unless visible?
       super
     end
-        
+
     ######################################################################
     # Expands a tree item when it is clicked
     #
-    # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeitem 
+    # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeitem
     #            is not visible
     #
     def expand_with_click
       # For now there is no difference to focusing
       focus_with_click
     end
-    
+
     alias_method :collapse_with_click, :expand_with_click
 
     ######################################################################
     # Expands a tree item when it is double clicked
     #
-    # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeitem 
+    # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeitem
     #            is not visible
     #
     def expand_with_double_click
@@ -46,9 +46,9 @@ module OperaWatir
       # No event yet so just cheat and sleep
       sleep(0.1);
     end
-    
+
     alias_method :collapse_with_double_click, :expand_with_double_click
-    
+
     ######################################################################
     # Presses a key including modifiers
     #
@@ -70,7 +70,7 @@ module OperaWatir
       key_press_direct(key, *opts)
       sleep(0.1)
     end
-        
+
     ######################################################################
     # Checks if the treeitem is selected
     #
@@ -81,24 +81,24 @@ module OperaWatir
     def selected?
       element.isSelected
     end
-     
-    
+
+
     ######################################################################
-    # Switches to the tree view tab by clicking on it (e.g. on the 
-    # Advanced page of the preferences dialog) 
+    # Switches to the tree view tab by clicking on it (e.g. on the
+    # Advanced page of the preferences dialog)
     #
     # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeitem
     #            is not visible
     #
     def activate_tab_with_click
       click
-      
+
       # No event yet so just cheat and sleep
       sleep(0.1);
     end
-    
+
     ######################################################################
-    # Double clicks the tree item, and waits for the window with 
+    # Double clicks the tree item, and waits for the window with
     # window name win_name to be shown
     #
     # @param [String] win_name name of the window that will be opened (Pass a blank string for any window)
@@ -109,15 +109,31 @@ module OperaWatir
     #            is not visible
     #
     def open_window_with_double_click(win_name)
-      wait_start
-      click(:left, 2)
-      wait_for_window_shown(win_name)
+      open_window_with_click_internal(win_name, 2)
     end
-    
+
     alias_method :open_dialog_with_double_click, :open_window_with_double_click
 
+
+    ######################################################################
+    # Clicks the tree item, and waits for the window with
+    # window name win_name to be shown
+    #
+    # @param [String] win_name name of the window that will be opened (Pass a blank string for any window)
+    #
+    # @return [int] Window ID of the window shown or 0 if no window is shown
+    #
+    # @raise [DesktopExceptions::WidgetNotVisibleException] if the treeitem
+    #            is not visible
+    #
+    def open_window_with_click(win_name)
+      open_window_with_click_internal(win_name, 1)
+    end
+
+    alias_method :open_dialog_with_click, :open_window_with_click
+
     ######################################################################
-    # Double clicks the tree item, and waits for the window with 
+    # Double clicks the tree item, and waits for the window with
     # window name win_name to be loaded with the url of the treeitem
     #
     # @param [String] win_name name of the window that will be loaded (Pass a blank string for any window)
@@ -132,21 +148,21 @@ module OperaWatir
       click(:left, 2)
       wait_for_window_loaded(win_name)
     end
-    
+
 
 private
     # @private
     # Scrolls the item into view if required
     def scroll_item_into_view
-      
+
       # Make sure we have a window id
       win_id = window_id >= 0 ? window_id : driver.getActiveQuickWindowID()
-      
+
       # Filter only treeitems in parent_treeview
       treeitems = driver.getQuickWidgetList(win_id).select do |wdg|
         wdg.getType == QuickWidget::WIDGET_ENUM_MAP[:treeitem] && wdg.getParentName == parent_name
       end
-      
+
       # Get the first visible item
       lowest = treeitems.find { | item| item.visible? } #This will always be a parent item, not a child
       # Assume list is ordered? => lower row means scroll up, higher row means scroll down
@@ -156,10 +172,10 @@ private
       qw = QuickTreeItem.new(self,lowest)
       qw.focus_with_click
       key_press_direct(key)
-      
+
       #First scroll
       key_press_direct(key)
-      
+
       visible_treeitems = driver.getQuickWidgetList(win_id).select do |wdg|
         wdg.getType == QuickWidget::WIDGET_ENUM_MAP[:treeitem] && wdg.getParentName == parent_name && wdg.visible?
       end
@@ -167,12 +183,12 @@ private
       # Stop scrolling if we have run through the whole list, the item is then a child and won't get visible
       max_times = treeitems.length / visible_treeitems.length + 1
       until element(true).visible? || max_times < 0
-          key_press_direct(key) 
+          key_press_direct(key)
           max_times-=1
       end
-      
+
     end
   end
-  
+
 end
 

data/lib/operawatir/quickwidgets/quick_treeview.rb

@@ -18,11 +18,11 @@ module OperaWatir
       end.select { |item| item != nil }
       treeitems.select {|item| item.parent_name == name }
     end
-    
+
     def num_treeitems
       treeitems.select { |item| item.position[1] == 0 }.length
     end
-    
+
   end
 end
 

data/lib/operawatir/quickwidgets/quick_widget.rb

@@ -3,13 +3,16 @@ module OperaWatir
     include DesktopCommon
     include DesktopContainer
     include Deprecated
-    
+
+    ConditionTimeout = 10.0
+
     # @private
     # window_id is set if constructor is called on a (parent) window
     # location is set is this is called on a (parent) widget
     def initialize(container, method, selector=nil, location=nil, window_id=-1, type=nil)
+
       @container = container
-                            
+
       if method.is_a? Java::ComOperaCoreSystems::QuickWidget
         @elm = method
         @type = WIDGET_ENUM_MAP.invert[method.getType]
@@ -21,9 +24,9 @@ module OperaWatir
         @window_id  = window_id
       end
     end
-    
+
     #######################################################################
-    # 
+    #
     # Hovers widget and waits for window to be shown
     #
     #
@@ -32,7 +35,25 @@ module OperaWatir
       element.hover
       wait_for_window_shown(win_name)
     end
-    
+
+    ######################################################################
+    # Rightclicks the widget, and waits for the menu with name
+    # menu_name to be shown
+    #
+    # @param [String] menu name (from standard_menu.ini)
+    #
+    # @raise [DesktopExceptions::WidgetNotVisibleException] if the button
+    #            is not visible
+    #
+    # @return [String] name of menu opened if it matches parameter menu_name
+    #              otherwise empty string
+    #
+    def open_menu_with_rightclick(menu_name)
+      wait_start
+      click(:right)
+      wait_for_menu_shown(menu_name)
+    end
+
     ######################################################################
     # Checks whether a widget exists or not
     #
@@ -44,26 +65,26 @@ module OperaWatir
         false
     end
     alias_method :exists?, :exist?
-    
+
     ######################################################################
     # Checks if a widget is enabled or not
     #
     # @return [Boolean] true if enabled otherwise false
     #
     # @raise [Exceptions::UnknownObjectException] if the widget could not be found
-    #           using the specified method  
+    #           using the specified method
     #
     def enabled?
       element.isEnabled
     end
-      
+
     ######################################################################
     # Checks if a widget is visible or not
     #
     # @return [Boolean] true if visible otherwise false
     #
     # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
-    #           using the specified method  
+    #           using the specified method
     #
     def visible?
       element.isVisible
@@ -78,24 +99,24 @@ module OperaWatir
     # @return [String] text of the widget
     #
     # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
-    #           using the specified method  
+    #           using the specified method
     #
     def text
       element.getText
     end
-    
+
     ######################################################################
     # Gets the type of a widget
     #
     # @return [Symbol] type of the widget (e.g. :dropdown, :button)
     #
-    # @raise [Exceptions::UnknownObjectException] if the widget cannot be found 
+    # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
     #               using the specified method
-    #    
+    #
     def type
       WIDGET_ENUM_MAP.invert[element.getType]
     end
-    
+
     ######################################################################
     # Gets the name of the widget (as it appears in dialog.ini or code)
     #
@@ -107,7 +128,7 @@ module OperaWatir
     def name
       element.getName
     end
-    
+
     ######################################################################
     # Gets a string representation of the widget
     #
@@ -127,7 +148,7 @@ module OperaWatir
     #
     # @param [String] string_id String ID to use to load the string from the current
     #                 language file (e.g. "D_NEW_PREFERENCES_GENERAL")
-    # 
+    #
     # @return [Boolean] true if the text matches, otherwise false
     #
     # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
@@ -140,19 +161,38 @@ module OperaWatir
         res = /#{text}/ =~ element.getText()
         res == nil ? false: true
       else
-        element.verifyText(string_id) 
+        element.verifyText(string_id)
       end
     end
-  
+
     alias_method :has_ui_string?, :verify_text
-    
+
+    ######################################################################
+    # Checks that the text in the widget matches the text as loaded
+    # from the current language file in Opera using the string_id
+    # (Strips &'s from the string before comparing)
+    #
+    # @param [String] string_id String ID to use to load the string from the current
+    #                 language file (e.g. "D_NEW_PREFERENCES_GENERAL")
+    #
+    # @return [Boolean] true if the text matches, otherwise false
+    #
+    # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
+    #               using the specified method
+    #
+    def verify_text(string_id, *params)
+      text = driver.getString(string_id, true)
+      verify_realtext(text, *params)
+    end
+
+
     ######################################################################
     # Checks that the text in the widget includes the text as loaded
     # from the current language file in Opera using the string_id
     #
     # @param [String] string_id String ID to use to load the string from the current
     #                 language file (e.g. "D_NEW_PREFERENCES_GENERAL")
-    # 
+    #
     # @return [Boolean] true if the text is included, otherwise false
     #
     # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
@@ -161,7 +201,7 @@ module OperaWatir
     def verify_includes_text(string_id)
       element.verifyContainsText(string_id)
     end
-    
+
     alias_method :includes_text?, :verify_includes_text
     deprecated :verify_includes_text
     deprecated :includes_text?
@@ -197,10 +237,15 @@ module OperaWatir
     # @return position for elements that have a position, else false
     #
     def position
-      return [row, col] if type == :treeitem
-      return col if type == :tabbutton
-      return col if type == :button
-      false
+      case type
+        when :treeitem then [row, col]
+        when :tabbutton, :button then col
+        else false
+      end
+      #return [row, col] if type == :treeitem
+      #return col if type == :tabbutton
+      #return col if type == :button
+      #false
     end
 
     ########################################################################
@@ -228,13 +273,13 @@ module OperaWatir
     def y
       element.getRect().y
     end
-    
+
     ######################################################################
     # Prints out all of the internal information about the widget. Used
     # to discover the names of widgets and windows to use in the tests
     #
     # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
-    #           using the specified method 
+    #           using the specified method
     #@private
     def print_widget_info
       puts widget_info_string
@@ -245,10 +290,10 @@ module OperaWatir
     # to discover the names of widgets and windows to use in the tests
     #
     # @raise [Exceptions::UnknownObjectException] if the widget cannot be found
-    #           using the specified method 
+    #           using the specified method
     #
     def widget_info_string
-      "   Name: " + name + 
+      "   Name: " + name +
       "\n   Text: " + text +
       "\n   Type: " + type.to_s +
       "\n Parent: " + element.getParentName() +
@@ -259,31 +304,132 @@ module OperaWatir
       "\n    Ref: row=" + element.getRow().to_s + ", col=" + element.getColumn().to_s +
       "\nselected: " + element.isSelected().to_s + "\n"
     end
-          
+
     # @private
     def driver
       @container.driver
     end
-    
+
     # Gets parent widget name
     def parent_name
       element.getParentName()
     end
-    
+
     #################################################################
     # Focus a widget with a click
     #
     def focus_with_click
       click
       # No event yet so just cheat and sleep
-      sleep(0.1);
+      sleep(0.1)
     end
-    
-    #@private
+
+    #################################################################
+    # Focus a widget by moving the mouse over it
+    #
+    def focus_with_hover
+      element.hover
+      #No event yet so just cheat and sleep
+      sleep(0.1)
+    end
+
+    ###############################################################
+    #
+    # double_click_with_condition { block } → res
+    #
+    # Doubleclicks widget and waits until block evaluates to true or timeout is hit
+    #
+    # @return value of block, or false if no block provided
+    #
+    def double_click_with_condition(&condition)
+      click_with_condition_internal(:left, 2, &condition)
+    end
+
+    ###############################################################
+    #
+    # right_click_with_condition { block } → res
+    #
+    # Rightclicks widget and waits until block evaluates to true or timeout is hit
+    #
+    # @return value of block, or false if no block provided
+    #
+    def right_click_with_condition(&condition)
+      click_with_condition_internal(:right, 1, &condition)
+    end
+
+    ##############################################################
+    #
+    # middle_click_with_condition { block } → res
+    #
+    # Middleclicks widget and waits until block evaluates to true or timeout is hit
+    #
+    # @return value of block, or false if no block provided
+    #
+    def middle_click_with_condition(&condition)
+      click_with_condition_internal(:middle, 1, &condition)
+    end
+
+    ###############################################################
+    #
+    # click_with_condition { block } → res
+    #
+    # @example
+    #      browser.quick_button(:name, "[buttonname]").click_with_condition {
+    #            browser.quick_treeview(:name, "[name]").quick_treeitems.length == 4 }.should be_true
+    #
+    # Clicks widget and waits until block evaluates to true or timeout is hit
+    #
+    # @return value of block, or false if no block provided
+    #
+    def click_with_condition(&condition)
+      click_with_condition_internal(:left, 1, &condition)
+    end
+
+
+    #######################################################
+    #
+    #
+    #
     def value
       return element.getValue
     end
-    
+
+    #########################################################
+    #
+    # @return all widgets inside this widget
+    #
+    # @example
+    #    browser.quick_treeview(:name, "Mail View").quick_widgets
+    #
+    # @note
+    #   you can also specify the widgets by type to retrieve only
+    #   a specific type of widgets
+    #
+    # @example
+    #      browser.quick_treeview(:name, "Mail View").quick_treeitems
+    #
+    #
+    def quick_widgets
+      widgets(window_id).select { | w | w.parent_name == name }
+    end
+
+    def quick_tabs
+      quick_tabbuttons
+    end
+
+    WIDGET_ENUM_MAP.keys.each do |widget_type|
+      my_type = "quick_" << widget_type.to_s
+      type = my_type
+      if my_type == "quick_search" || my_type == "quick_checkbox"
+        my_type << "es"
+      else
+        my_type << "s"
+      end
+      define_method(my_type.to_sym) do #|win|
+        quick_widgets.select { |w| w.type == widget_type and w.parent_name == name }
+      end
+    end
+
 protected
     #@private
     # Return the element
@@ -291,14 +437,83 @@ protected
       if (@elm == nil || refresh == true)
         @elm = find
       end
-      
-      raise(Exceptions::UnknownObjectException, "Element #{@selector} not found using #{@method}") unless @elm 
+
+      raise(Exceptions::UnknownObjectException, "Element #{@selector} not found using #{@method}") unless @elm
       @elm
     end
 
-  
+
 private
-    
+
+  def open_window_with_click_internal(win_name, num_clicks)
+    wait_start
+    click(:left, num_clicks)
+    wait_for_window_shown(win_name)
+  end
+
+  def click_with_condition_internal(button = :left, times = 1, &blk)
+      return false unless block_given?
+
+      click(button, times)
+
+      start = Time.now
+      until res = yield rescue false do
+        if Time.now - start > ConditionTimeout
+          return false
+        end
+        sleep 0.1
+
+      end
+      res
+    end
+
+# Right click a widget
+  def right_click
+    click(:right, 1)
+  end
+
+  # Click widget
+  def click(button = :left, times = 1, *opts)
+
+    #raise DesktopExceptions::WidgetDisabledException, "Element #{@selector} is disabled" unless enabled?
+
+    #Some buttons etc. aren't visible until hovering them
+    if (visible? == false and type != :dialogtab)
+      element.hover
+      element(true)
+    end
+
+    # Dialog tabs are always visible even if the page they are connected to isn't
+    if visible? == true or type == :dialogtab
+      button = DesktopEnums::MOUSEBUTTON_ENUM_MAP[button]
+      list = Java::JavaUtil::ArrayList.new
+      opts.each { |mod| list << DesktopEnums::KEYMODIFIER_ENUM_MAP[mod] }
+      element.click(button, times, list)
+    else
+      raise(DesktopExceptions::WidgetNotVisibleException, "Widget #{name.length > 0 ? name : text} not visible")
+    end
+  end
+
+  def verify_realtext(text, *params)
+    if text.include? "%1"
+      result = text.scan(/%\d/)
+      if (params.length == result.length)
+        result.length.times do |time|
+          number = time+1
+          p = params[time]
+          text.gsub!("%#{number}", params[time])
+        end
+      end
+      text == element.getText()
+    else
+      params.each do |param|
+        text.gsub!(/%[csduoxefg0-9]/, param)
+      end
+     text == element.getText()
+    end
+  end
+
+
    def drag_and_drop_on(other, drop_pos)
      element.dragAndDropOn(other.element, DROPPOSITION_ENUM_MAP[drop_pos])
    end
@@ -308,7 +523,7 @@ private
       if @selector == nil && @elm != nil
          set_selector
       end
-      
+
       #FIXME: Shouldn't this always be name if present, then text if present, else pos?
       case @method
       when :name
@@ -323,7 +538,7 @@ private
       end
     end
 
-    # Get row    
+    # Get row
     def row
       element.getRow()
     end
@@ -332,46 +547,21 @@ private
     def col
       element.getColumn()
     end
-    
+
     # Gets the window id to use for the search
     def window_id
       # Need to pass on the current setting of @window_id to make
       # nesting of quick widgets work
       @window_id
     end
-    
-    # Click widget
-    def click(button = :left, times = 1, *opts)
-      raise DesktopExceptions::WidgetDisabledException, "Element #{@selector} is disabled" unless enabled?
-  
-      #Some buttons etc. aren't visible until hovering them
-      if (visible? == false and type != :dialogtab)
-        element.hover
-        element(true)
-      end
-  
-      # Dialog tabs are always visible even if the page they are connected to isn't
-      if visible? == true or type == :dialogtab
-        button = DesktopEnums::MOUSEBUTTON_ENUM_MAP[button]
-        list = Java::JavaUtil::ArrayList.new
-        opts.each { |mod| list << DesktopEnums::KEYMODIFIER_ENUM_MAP[mod] }
-        element.click(button, times, list)
-      else
-        raise(DesktopExceptions::WidgetNotVisibleException, "Widget #{name.length > 0 ? name : text} not visible")
-      end
-    end
 
-    # Right click a widget
-    def right_click
-      click(:right, 1)
-    end
-  
+
     # double click widget
     def double_click
-      click(:left, 2) 
+      click(:left, 2)
     end
 
-    
+
     def set_selector
       if @elm.name.length > 0
         @method = :name
@@ -382,18 +572,18 @@ private
       elsif @elm.type == :treeitem
         @method = :pos
         @selector = [@elm.row, @elm.col]
-=begin          
+=begin
       # tabbuttons now specified by generated name
       elsif @elm.type == :tabbutton
         @method = :pos
         @selector = @elm.col
-=end        
+=end
       end
       @location = element.getParentName()
       @window_id = -1
     end
-    
-    # Finds the element on the page.  
+
+    # Finds the element on the page.
     def find
       #If @method set and we do new find because of refresh, we need to get @selector first
       #Have the java object because the construct was done on it
@@ -426,11 +616,11 @@ private
         else
           @element = driver.findWidgetByPosition(WIDGET_ENUM_MAP[@type], @window_id, @selector[0], @selector[1])
         end
-     end
+      end
       if @window_id < 0 && @element != nil
          @window_id = @element.getParentWindowId
       end
-      raise(Exceptions::UnknownObjectException, "Element #{@selector} not found using #{@method}") unless @element 
+      raise(Exceptions::UnknownObjectException, "Element #{@selector} not found using #{@method}") unless @element
       raise(Exceptions::UnknownObjectException, "Element #{@selector} has wrong type #{@element.getType}") unless correct_type?
       @element
     end