selenium webdriver（c＃）-循环访问选择菜单的最佳方法-然后验证元素样式是否更新？

Question

I would like to iterate through all available options inside a select menu, click on each option in turn, then verify that clicking on each option will update a particular element with a width and height value.

Here is the code for the select menu:

 <span id="SizeSelectContainer" class="dropdownButton" style="min-width: 100px; max-width: 120px;">
            <select id="SizeSelect" title="Zoom" tabindex="23" data-l10n-id="zoom" style="min-width: 126px;">
                <option id="SizeAutoOption" title="" value="auto" selected="selected" data-l10n-id="pagesizeauto">Automatic Size</option>
                <option id="SizeActualOption" title="" value="page-size-actual" data-l10n-id="pagesizeactual">Actual Size</option>
                <option title="" value="0.1">10</option>
                <option title="" value="0.5">50</option>
                <option title="" value="1">100</option>                         
            </select>
        </span>

When I click on each option, the value within another element located elsewhere on the page will change (see below) - There is no "selected" attribute applied to any of the drop down options when selected (apart from the default option) so I can't assert the selections based on a "selected" attribute.

What I'm going to do (which I haven't coded in my sample yet) is that when a particular select option is selected, another page element (shown below) will update to hold new width and height values:

<div id="Container" class="page" style="width: 547px; height: 865px; data-loaded="true">

Here is what I have come up with so far - Note that I know I'm not asserting anything yet

        IWebElement AutoSize = UtilityClass.driver.FindElement(By.Id("SizeAutoOption"));
        IWebElement ActualSize = UtilityClass.driver.FindElement(By.Id("SizeActualOption"));

        IWebElement Ten = UtilityClass.driver.FindElement(By.XPath("//*[@id='SizeSelect']/option[3]"));
        IWebElement Fifty = UtilityClass.driver.FindElement(By.XPath("//*[@id='SizeSelect']/option[4]"));
        IWebElement Hundred = UtilityClass.driver.FindElement(By.XPath("//*[@id='SizeSelect']/option[5]"));  

        Object[] SelectElements = new Object[] { AutoSize, ActualSize, Ten, Fifty, Hundred };

        for (int i = 0; i < SelectElements.Count(); i++)
        {
            if (i == 0)
            {
                AutoSize.Click();
                System.Threading.Thread.Sleep(1000);
            }
            else if (i == 1)
            {
                ActualSize.Click();
                System.Threading.Thread.Sleep(1000);
            }
            else if (i == 2)
            {
                Ten.Click();
                System.Threading.Thread.Sleep(1000);
            }
            else if (i == 3)
            {
                Fifty.Click();
                System.Threading.Thread.Sleep(1000);
            }
            else if (i == 4)
            {
                Hundred.Click();
                System.Threading.Thread.Sleep(1000);
            }
         }

I'm really just looking for some confirmation on whether or not this will be the best way to approach this. The reason I ask is that I have essentially simplified the code sample above and I actually have over 50 select options to verify - not to mention that I have many other select menus with as many options contained inside them to automate.

Answer 1

You've got several problems here.

The first is that a for loop is not what you want to do.

The second is that Selenium wraps all the "find me an option within a select" functionality for you in a SelectElement class so you don't have to do this in the first place:

The third is that your XPath constructs are heavily based on position. Position is bad because it's something that can change. If I add in an option at the very top of that list, your test will break and it should not. I've not broken application functionality, merely just changed the list order, and your test is going to fail.

The fourth problem is you are holding references to your element and you've said the page is going to change when you click the option. This, may , cause a StaleReferenceException dependant on what the click actually does. I say may because it shouldn't but if your application is coded to do something that causes Selenium to think "the reference to this is no longer valid" it will throw exceptions.

The fifth issue is you are looping through an array of object s which is unnecessary. You know their type. It's IWebElement . You are not harnessing the power of the class if you just define it as Object .

The final issue is that you are using Thread.Sleep . You are clearly waiting for something so use the explicit wait within Selenium to actually wait for something instead of "lets wait a few seconds".

https://github.com/SeleniumHQ/selenium/blob/master/dotnet/src/support/UI/SelectElement.cs

(remember Selenium is open source)

SelectElement actualSizeSelect = new SelectElement(UtilityClass.driver.FindElement(By.Id("SizeSelect")));

This then gives you the ability to search by text:

actualSizeSelect.SelectByText("10");

Now your test won't care where exactly in that list "10" is!

This would also give you the ability to loop through all the options within a given select like so:

IList<IWebElement> options = actualSizeSelect.AllOptions;

You now have a list of option s within that select . So let's loop through them:

foreach (IWebElement option in options) 
{
    option.Click(); // just a demo to show you.
}