java - 通过比较器模拟排序列表 (EasyMock)

Question

当排序对象是模拟对象时，如何为排序函数(使用比较器)编写单元测试？

举个小例子:我有一个有电影的图书馆。三个类:Library、Movie、myCmpMovies(实现比较器)。我正在测试类库，将其隔离。对于这个目标，我模拟了 Movie 类，以测试库中的所有方法。

使用 EasyMock。

我正在以这种方式对电影列表进行排序:

class Library{
    ArrayList<Movie> movies;
    public List<Movie> getSortedMoviesByDatesAndNames() {
        List<Movie> movies = this.getMovies();
        Comparator myCmpMovies = new myCmpMovies();
        Collections.sort(movies, myCmpMovies);
        return movies;
    }
}

现在，我正在通过单元测试来测试此方法。模拟对象是 Movie。我只是尝试这样的事情:

    @Test
    public List<Movie> TestGetSortedMoviesByDatesAndNames() {
        movie1= createMock(Movie.class);
        movie2= createMock(Movie.class);
        movie3= createMock(Movie.class);

        ....
    }

我知道有一些关于这个主题的问题，但我认为这些不是同一个问题。

提前致谢。

Answer 1

对您的评论进行一些编辑:如果您确实必须坚持使用“笨拙”的电影类(class)，并且您想进行 mock ，我会这样做:

Movie mockMovieWith(String title, Date releaseDate, ...) {
  Movie mockedMovie = createMock(Movie.class);
  expect(mockedMovie.getTitle()).andStubReturn(title);
  expect(mockedMovie.getReleaseDate())...
  replay(mockedMovie);
  return mockedMovie;
}

换句话说:使用辅助方法创建一个模拟的 Movie 对象，该对象具有您认为应该/将由稍后将使用模拟对象的“真实”代码使用的属性。

但如下所示；你真的应该在这里添加一些“反腐败层”。含义:当“外部”库 Movie 类很难使用时，请包装一些东西 - 这样您的代码就不会被该 Movie 类中的不良设计破坏!

...为了记录，这是最初的答案。

你在很多层面上都搞错了。首先，您使用方法中的局部变量遮蔽您的字段电影。这很少是一个好主意，因为它可能会导致各种微妙的错误。

然后:首先要测试的不是方法；它是比较器本身。含义:您正在依赖 Collections.sort();无需测试该部分。您希望确保您的比较器执行其应执行的操作。

从这个意义上说，您的第一个测试用例应该简单地定义两个 Movie 对象；然后调用比较器的compareTo方法并检查预期结果。

引出您问题的真正主题:您应该不需要需要模拟 Movie 对象。这些东西代表某种数据。从这个意义上说:您不需要实例化模拟数据，您只需创建具有已知内容的真实 Movie 对象。

例如，假设电影的核心属性是它的标题，您可以这样:

Movie movieA = new Movie("Title A");
Movie movieB = new Movie("Title B");

这就是您要输入比较器的内容。或者可能进入您的电影列表(您仍然需要至少进行一项“集成”测试，以确保 getSortedMovies...() 执行其应有的操作)。

如果您认为需要模拟电影对象，那么您的设计就很糟糕。就是这么简单。如果实例化 Movie 对象并不“那么”容易(如上所示)，那么您应该完全启用它。

否则，您将不得不执行以下操作:

Movie mockedMovieA = createMock(Movie.class);

然后提供模拟期望的所有属性，例如:

expect(mockedMovieA.getTitle()).andReturn("Title A");

这将非常麻烦且容易出错。

因此，如前所述，您应该研究的核心事情:创建一个设计，允许您在单元测试中使用“真实”Movie 对象。如果你的 Movie 类包含太多的东西，这是不可能的......那么这清楚地表明，你的 Movie 类太大并且正在做不属于它的事情。

解决此类“太大”问题的第一种方法:您可以分离一个“Movie”接口(interface)，其中包含当前 Movie 类的核心方法，例如 getTitle()、getReleaseDate() 等。然后所有你的代码只适用于该接口(interface)......这很容易被 mock ；或者也由更“愚蠢”的仅测试类实现。