Question

我已经为以下函数编写了Junit测试用例。当检查JACOCO测试覆盖率时。它显示只有try块被测试用例覆盖。我是编写测试用例的新手。如何在测试用例中涵盖异常和catch块

这是一个方法

  public static List<Student> readCsvFile(String fileName)
    {

       BufferedReader fileReader = null;

//logic to read file    
}
catch (Exception e)
{
   System.out.println("Error in CsvFileReader !!!");
   e.printStackTrace();
        } finally
        {
            try
            {
                fileReader.close();
            } catch (IOException e)
            {
                System.out.println("Error while closing fileReader !!!");
                e.printStackTrace();
            }
        }
        return students;
    }

和TestMethod

@Test
    public void ReadCsvFileTest()
    {
        String fileName = "test.csv";
        List<Student> result = new ArrayList<Student>();
        result = CsvFileReader.readCsvFile(fileName);

        Student student1 = null;

        Iterator<Student> it = result.iterator();
        while (it.hasNext())
        {
            Student s = it.next();
            if ("471908US".equals(s.getId()))
            {
                student1 = s;
                break;
            }
        }

        assertTrue(student1 != null);
    }

Answer 1

在这种情况下，您可能经常考虑在课堂上引入其他依赖项。这是我粗略的例子。为读者创建工厂：

interface BufferedReaderFactory
{
    public BufferedReader createBufferedReader(String fileName) throws IOException;
}

然后你将有一个简单的实现，几乎不需要任何测试，例如类似的东西：

class BufferedReaderFactoryImpl implements BufferedReaderFactory
{
    @Override
    public BufferedReader createBufferedReader(String fileName) throws IOException
    {
        return new BufferedReader(new FileReader(fileName));
    }
}

然后你必须找到一种方法将这种依赖注入你的类。我通常在日常工作中使用Guice但你可以尝试使用构造函数注入和使方法非静态的简单方法。这是一个例子：

class CsvFileReader
{
    private final BufferedReaderFactory factory;

    public CsvFileReader(BufferedReaderFactory factory)
    {
        this.factory = factory;
    }

    public List<Student> readCsvFile(String fileName)
    {

        BufferedReader fileReader = null;
        try
        {
            fileReader = factory.createBufferedReader(fileName);
            ...
        }
        catch(IOException e)
        {
            ...
        }
        finally
        {
            ...
        }
        return new LinkedList<>();
    }
}

使用像Mockito这样的模拟框架，在IOException - s的情况下，此类的行为现在更容易测试（请注意，您也可以返回从工厂抛出异常的模拟）。这是一个示例：

@RunWith(MockitoJUnitRunner.class)
public class MyTest
{
    @Mock
    private BufferedReaderFactory mockFactroy;

    @Test
    public void testIOException() throws IOException
    {
        String ivalidFileName = "invalid.txt";
        //throw exception in case that invalid file name is passed to the factory
        Mockito.when(mockFactroy.createBufferedReader(ivalidFileName)).thenThrow(new IOException("Hello!"));

        CsvFileReader csvFileReader = new CsvFileReader(mockFactroy);
        //invoke with a factory that throws exceptions
        csvFileReader.readCsvFile(ivalidFileName);
        //...
        //and make a sensible test here, e.g. check that empty list is returned, or proper message is logged, etc.
    }
}

当然，你可以在没有Mockito的情况下通过实施测试工厂来做到这一点。但这更麻烦，尤其是在更复杂的用例中。一旦IOException被抛出，您将获得JaCoCo的适当报道报告。

还要注意JaCoCo提到here的限制，在部分的源代码行中，例外显示没有覆盖。为什么？

Answer 2

鉴于您测试的方法的当前签名，获得完全覆盖并不容易：只有在try块中抛出异常时才会执行catch块。

解决此问题的一种方法：不传入文件名，而是传递阅读器对象本身。喜欢：

public static List<Student> readCsvFile(String fileName) {
  return readCsvFile(new BufferedReader(fileName));
}

static List<Student> readCsvFile(BufferedReader reader) { 
  try {
    ...
  } catch( ...

现在，您可以为第二种方法编写几个特定的单元测试。你保持你的测试只是＆＃34;纠正＆＃34;读;但是你添加一个你传入一个模拟读者对象的地方......它只是在某个时候抛出一个异常。请注意，我将这种新方法打包保护 - 您可能不想使用该方法＆＃34; public＆＃34 ;;并将其设为私有将阻止它进行单元测试。

这应该可以帮助您实现全面覆盖。当然，你还需要至少一次测试来覆盖＆＃34;字符串获取方法也是如此。

一些注意事项：

小心重新发明轮子。现有许多CSV解析器。请放心：编写一个正确的 CSV解析器，它能够处理所有＆＃34;纠正＆＃34;输入CSV比听起来要困难得多。如果这不是为了学习目的＆＃34;我强烈建议不编写您自己的CSV解析器。
小心做这些事情静态。如上所述，真正的CSV解析器是一件复杂的事情，值得完全拥有自己的类。所以没有静态帮助器方法 - 你实例化的普通类然后调用它的方法（也可以使用依赖注入，这将有助于解决你所问的问题... ...试试块）
您正在代码示例中捕获异常。不要这样做 - 尝试准确捕捉您的代码实际可能产生的异常（在您的情况下可能是IOException）。

编写Junit测试以涵盖异常和捕获块

2 个答案: