测试类静态初始化程序的最佳方法是什么？

Question

我有一个具有相当复杂的静态初始化的类.我正在从目录中读取文件,然后解析这些json文件,映射到对象,并填写列表.你可以想象,可能会出现一些异常,我需要覆盖并测试这些代码分支.问题是这个静态初始化只运行一次/ Testcase文件.解决方案我喜欢:

• 每个行为的新testcase文件
• 卸载静态类
• 新的JVM

我对这些选项并不着迷,有什么更好的吗？

Answer 1

如果你无法避免静态初始化器提取它的身体方法.然后测试方法.静态初始化程序看起来static { myMethod(); }几乎不可能被打破.

Answer 2

单元测试的一个重要因素是构建代码以使其适合测试。

不幸的是，正如您所发现的，执行复杂 IO 操作的具有过多静态初始化的代码并不是一个易于测试的结构。

除了静态初始化之外，听起来您的代码违反了单一职责原则，因为类从外部源加载自身并且可能还有其他用途。

因此，您需要进行一些重构，例如，如果您的代码看起来像这样（JSON 解析，为了清晰起见，用 CSV 解析替换）：

public MyClass
{
  private static List<MyObject> myObjects = new ArrayList<>();

  static
  {
    try
    {
      try (BufferedReader reader = new BufferedReader(new FileReader(myfile.csv))
      {
        String line;

        while ((line = reader.readLine()) != null) 
        {
          String[] tokens = line.split(",");
          myObjects.add(new MyObject(tokens[0], tokens[1], tokens[2]));
        }
      }
    }
    catch (IndexOutOfBoundsException e)
    {
      ...
    }
    catch (IOException e)
    {
      ...
    }
  }
}

然后，您可以将大部分逻辑提取到自定义阅读器类中，如下所示：

public class MyObjectReader implements Closeable
{
  private BufferredReader reader;

  public MyObjectReader(Reader reader)
  {
    this.reader = new BufferredReader(reader);
  }

  public MyObject read() throws IOException
  {
    String line = reader.readLine();

    if (line != null)
    {
      String[] tokens = line.split(",");

      if (tokens.length < 3)
      {
        throw new IOException("Invalid line encountered: " + line);
      }

      return new MyObject(tokens[0], tokens[1], tokens[2]);
    }
    else
    {
      return null;
    }
  }

  public void close() throws IOException
  {
    this.reader.close();
  }
}

该类MyObjectReader是完全可测试的，重要的是不依赖于存在的文件或其他资源，因此您可以像这样测试它：

public MyObjectReaderTest
{
  @Test
  public void testRead() throws IOException
  {
    String input = "value1.1,value1.2,value1.3\n" +
      "value2.1,value2.2,value2.3\n" +
      "value3.1,value3.2,value3.3";

    try (MyObjectReader reader = new MyObjectReader(new StringReader(input)))
    {
      assertEquals(new MyObject("value1.1", "value1.2", "value1.3"), reader.read());
      assertEquals(new MyObject("value2.1", "value2.2", "value2.3"), reader.read());
      assertEquals(new MyObject("value3.1", "value3.2", "value3.3"), reader.read());
      assertNull(reader.read());
    }
  }

  @Test(expected=IOException.class)
  public void testReadWithInvalidLine() throws IOException
  {
    String input = "value1.1,value1.2";

    try (MyObjectReader reader = new MyObjectReader(new StringReader(input)))
    {
      reader.read();
    }
  }
}

如果没有看到你的代码或不知道文件格式，就很难对此进行扩展，但希望你能明白要点。

最后，您的静态初始化将很简单：

public MyClass
{
  private static List<MyObject> myObjects = new ArrayList<>();

  static
  {
    loadMyObjects(new FileReader("myfile.csv"));
  }

  /* package */ static void loadMyObjects(Reader reader)
  {
    try
    {
      try (MyObjectReader reader = new new MyObjectReader(reader))
      {
        MyObject myObject;

        while ((myObject = reader.read()) != null) 
        {
          myObjects.add(myObject);
        }
      }
    }
    catch (IOException e)
    {
      ...
    }
  }
}    

也许值得在这里测试快乐路径，但就我个人而言，该loadMyObjects方法现在非常简单，我可能不会打扰。