Comments

• Sumitra Members

  October 2011

  Hi KeithM,
  
  Here is a better way which you could use:
  

  <browser>
  function listElements() {
     var $items= new Array();
     $items = _list(0).children;
     var $textElements = new Array();
     for(var $i=0; $i<$items.length; $i++){
         $textElements[$i] = _getText($items[$i]);
     }
  return $textElements;
  }
  </browser>
  var $expected = new Array("Item 1","Item 2", "Item 3", "Item 4");
  _set($elements, listElements());
  _assertEqual($expected, $elements);
  

  
  Regards
  Sumitra
• KeithM Members

  October 2011

  Thanks Sumitra. Your solution would work except that with the way these pages/menus on our website are coded, Sahi sees the array as having 35 items (all possible menu items) instead of the 4 visible ones in my example. I tried using _setStrictVisibilityCheck around your code, but that didn't help.
  
  Is there any logical reason that
  
  _assertNotTrue(_isVisible(_listItem("Item 1", _under(_listItem("/.*/")))));
  
  doesn't work for the first item, but seems to work for the last item?
• Sumitra Members

  October 2011

  Hi KeithM,
  In your case, _assertNotTrue() just checks if Item1 is under ANY listitem and if item1 is visible it will return true. Instead you should just verify if the element above item1 is not visible.
  
  I have modified my earlier code:
  The HTML code for the list items is:

  <ul>
  <li style="display:none">Item 0</li>
  <li>Item 1</li>
  <li>Item 2</li>
  <li style="display:none">Item 2.5</li>
  <li>Item 3</li>
  <li>Item 4</li>
  <li style="display:none">Item 5</li>
  </ul>
  

  
  The script for this would be:
  

  <browser>
  function listElements() {
     var $items= new Array();
     $items = _list(0).children;
     var $textElements = new Array();
     for(var $i=0; $i<$items.length; $i++){
    if(_isVisible($items[$i]))
         $textElements.push(_getText($items[$i]));
     }
  return $textElements;
  }
  </browser>
  
  

  
  Regards
  Sumitra
• KeithM Members

  November 2011

  Unfortunately, modifying the HTML with a 'ghost' Item 0 to make some automation script work isn't an option in my environment. Thanks again for the help, though. Maybe an _over API could be considered in the future?