3

我经常使用Selenium Webdriver，并且我编写了很多“实用”方法来使其更易于使用。我将这些类放在一个WebDriverUtil类中，现在该文件的长度超过 1,200 行。每种方法都WebDriverUtil试图将我与使用区分开来，WebDriver因为这是我经常使用的东西，不会干枯继续写作。

例如，这是我要放入的方法WebDriverUtil。

public void waitUntilVisible(final WebElement webElement) {
    new WebDriverWait(webDriver, 10).until(new Predicate<WebDriver>() {
        @Override
        public boolean apply(WebDriver webDriver) {
            return webElement.isDisplayed();
        }
    });
}

如果我有 1,200 行充满这样的方法的代码，我有上帝对象吗？如果是这样，我该如何解决？

我应该把我的行为分成这样的装饰器类吗？

public class WebElementDecorator implements WebElement {
    private WebElement webElement;
    private final WebDriver webDriver;

    public WebElementDecorator(WebElement webElement, WebDriver webDriver) {
        this.webElement = webElement;
        this.webDriver = webDriver;
    }

    public void waitUntilVisible() {
        new WebDriverWait(webDriver, 10).until(new Predicate<WebDriver>() {
            @Override
            public boolean apply(WebDriver webDriver) {
                return webElement.isDisplayed();
            }
        });
    }

    public void click() {
        webElement.click();
    }

    //... other WebElement methods
}
4

1 回答 1

1

如果我有 1200 行充满这样的方法的代码，我有上帝对象吗？

单独的代码行数并不能充分表明一个类是否像上帝一样。由于糟糕的编码风格、过度工程、过度专业化方法的不同变体、冗长的语言、内联注释等，一个类可能会被代码臃肿。

神级是一个充满责任感的班级。这里有两个试金石，可以确定你的 util 类是否已经进化为神类：

  1. 更改测试时对 util 类的影响。如果对测试子集的更改导致您经常更改和重新编译您的 util 类，那么这可能表明您的 util 类服务于太多的 master。理想的情况是，对测试子集的更改只会影响那些与测试直接相关的 util 方法（如果需要）。

  2. 更改 util 类时对测试类的影响。如果更改 util 类的一部分会导致许多不相关的测试出现意外失败，那么您的 util 类可能已经将它的触角扩展到了您的测试中。

如果是这样，我该如何解决？我应该把我的行为分成这样的装饰器类吗？

最好以小的增量步骤开始重构。使用您的代码示例，我将首先将所有代码简单地提取到一个名为或其他Wait..Until..Predicate名称的单独类中，并使用 ,等方法。示例用法如下所示：WaitUntilEvent()isVisible()isEnabled()isSelected()

WaitUntilEvent waitUntil = new WaitUntilEvent(webElement, webDriver);
waitUntil.isVisible();
webElement.click();
// etc..

如果我需要更改我的测试要求Wait..Until..Predicate（例如超时间隔），我知道只有一个类需要编辑。然后可以将其进一步重构为until(PredicateIsTrue).then(PerformAction)等until(FunctionIsTrue).then(PerformAction)。我更喜欢这种方法，而不是包罗万象的上帝，class WebElementDecorator它可能最终会以许多装饰方法捕获许多不同的行为。

于 2015-08-14T05:23:09.743 回答