java - Selenium [Java] PageFactory 设计 : Where do I write my Assertions following Page Object Model

Question

我正在遵循页面对象模型来自动化一个应用程序中的流程。在其中一个模块中，我必须断言页面标题和更多消息。截至目前，我将断言代码放入 PageFactory 本身中，如下所示:

public class EditPost {

    WebDriver driver;

    public EditPost(WebDriver editPostDriver)
    {
        this.driver=editPostDriver;
    }

    @FindBy(how=How.XPATH,using="//*[@id='message']/p")
    WebElement post_published;

    public void assert_message()
    {
        String actual_message_title=post_published.getText();
        Assert.assertEquals(actual_message_title, "Post published. View post");
        System.out.println("Message: Post published, Successfully Verified");
    }
}

我从实现 TestNG 的主文件中调用断言方法，如下所示:

@Test (priority=5)
public void assert_message()
{
    //Created Page Object using Page Factory
    EditPost edit_post = PageFactory.initElements(driver, EditPost.class);
    edit_post.assert_message();

}

目前，我正在通过 3 个包运行执行。浏览器工厂的“Helper”包、PageFactories 的“Pages”包和测试用例的“Testcase”包。

我的目标是向前迈进，我想重用为所有不同实用程序编写的代码。

我的问题是:

1. 根据 PageFactory 和页面对象模型的概念，我的方法正确吗？或者我需要将断言移至“Helper”包中吗？或者我应该为断言创建一个单独的库/包？ (在接下来的日子里，我可能需要在单个页面上执行多个断言)

2. 在下一个冲刺中，我可能需要执行一些其他 Activity ，例如拍摄所有/失败的测试用例的屏幕截图。那么我如何保持设计的结构化和组织性，以便我可以重用代码/库/以最佳方式利用它们？

Answer 1

根据我见过的大多数网站，最佳实践是将断言保留在页面对象之外。下面是 Selenium 文档中的一个示例。

http://www.seleniumhq.org/docs/06_test_design_considerations.jsp#page-object-design-pattern

There is a lot of flexibility in how the page objects may be designed, but there are a few basic rules for getting the desired maintainability of your test code.

Page objects themselves should never make verifications or assertions. This is part of your test and should always be within the test’s code, never in an page object. The page object will contain the representation of the page, and the services the page provides via methods but no code related to what is being tested should be within the page object.

There is one, single, verification which can, and should, be within the page object and that is to verify that the page, and possibly critical elements on the page, were loaded correctly. This verification should be done while instantiating the page object. In the examples above, both the SignInPage and HomePage constructors check that the expected page is available and ready for requests from the test.

页面对象应返回产品名称、产品价格、当前选择的数量等信息。然后测试代码将断言返回的字符串与预期匹配。

assert_message() 将变为 getMessage() 并以 String 形式返回消息。见下文。

public String getMessage()
{
    return driver.findElement(messageLocator).getText();
}

(注意:请继续阅读，了解我为何将 PageFactory 元素更改为此处的定位器。)

然后在您的测试代码中，您将拥有

Assert.assertEquals(editPost.getMessage(), "Post published. View post");

现在您已将断言代码保留在测试脚本中并保留在页面对象之外。

查看您的代码，我会提出一些进一步的建议。

1. 我建议您阅读一些 Java 命名约定。有很多网站都有推荐，我认为它们之间有很多相似之处，但这里是 oracle recommendations首先。你的方法名称应该是

   verbs, in mixed case with the first letter lowercase, with the first letter of each internal word capitalized.

   因此 assert_message() 会变成 assertMessage() 等等。 _s 让它看起来更像 python。

2. 定位器的优先顺序:ID、CSS 选择器，以及在极少数情况下的 XPath。 ID 应该始终是您的首选，因为它(根据 W3C 定义)在页面上应该是唯一的。 CSS 选择器应该是下一个，因为它是最快的(在我的测试中比 ID 更快)，具有最好的浏览器支持，并且跨浏览器实现最一致。 XPath 应该只保留用于 CSS 选择器无法完成的事情，例如通过包含的文本查找元素。与 CSS 选择器相比，XPath 定位器的性能较差，并且没有与 CSS 选择器相同级别的支持。例如，您的 XPath 定位器可以轻松转换为 CSS 选择器“#message > p”。

   这里有一些 CSS 选择器引用，可帮助您入门。

   CSS Selectors reference

   CSS Selector tips

3. 删除PageFactory。是的，它似乎使事情变得更容易，但我认为在很多情况下它会导致更多问题，例如陈旧元素异常等。更喜欢根据需要抓取页面。在类的顶部声明所有定位器，并在需要时在方法中使用它们。

   public class EditPost {
   
       WebDriver driver;
   
       By messageLocator = By.cssSelector("#message > p")
   
       public EditPost(WebDriver editPostDriver)
       {
           this.driver = editPostDriver;
       }
   
       public String getMessage()
       {
           return driver.findElement(messageLocator).getText();
       }
   }
   

我知道这比您要求的要多，但希望它有帮助。