Question

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

每个行为的新测试用例文件
卸载静态类
新JVM

我对这些选择并不着迷，还有什么更好的吗？

Answer 1

如果你无法避免静态初始化器提取它的方法。然后测试方法。您的静态初始化程序看起来像static { myMethod(); }，几乎不会被破坏。

Answer 2

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

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

除了静态初始化之外，听起来你的代码违反了single-responsibility principle，因为类正在从外部源加载自己，并且可能还有一些用途。

所以你需要进行一些重构，例如，如果你的代码看起来像这样（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)
    {
      ...
    }
  }
}

然后你可以将这大部分逻辑提取到一个自定义 reader 类中，如下所示：

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方法现在非常简单，我可能不会打扰。

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

2 个答案: