JUnit5与Mockito测试中条件判断失效问题：无法覆盖目标代码块的解决方案求助

我来帮你分析下问题所在，以及如何解决这个测试覆盖的问题。

首先，你的测试没生效的核心原因很明确：你Mock的mockedFile对象，根本没被实现类里的getData()方法用到。

看实现类的代码：

File[] files = new File(APP_CONST.DIRECTORY).listFiles();

这里是直接new File(...)创建了一个全新的File实例，和你测试类里@Mock的那个mockedFile完全是两个独立的对象。所以你对mockedFile的所有Mock设置（比如listFiles()返回files数组），都不会影响到这个新创建的File实例，它的listFiles()方法还是会去读取实际的文件系统（测试环境下这个目录大概率不存在，所以返回null），自然就进入了返回空Map的分支。

接下来给你两种可行的解决方案，推荐第一种，因为更符合可测试性的代码设计：

方案一：重构实现类，让File对象可Mock #

我们可以把创建File对象的逻辑抽成一个独立的方法，这样测试时就能Mock这个方法，返回我们预设的File实例。

修改后的实现类代码： #

@Component
public class AppConfig {

    // 保持原有的目录常量，或者改成配置注入更灵活
    private final String directoryPath = APP_CONST.DIRECTORY;

    // 抽取出获取目录File的方法，用protected修饰方便测试Mock
    protected File getDirectoryFile() {
        return new File(directoryPath);
    }

    private Map<String, String> getData() {
        // 不再直接new File，而是调用抽出来的方法
        File directory = getDirectoryFile();
        File[] files = directory.listFiles();
        
        if (null == files) {
            return Collections.emptyMap();
        }
        
        List<String> keysList = getKeyList(files);
        return keyMap(directoryPath, keysList);
    }

    // 你的其他方法：getKeyList、keyMap等
}

修改后的测试类代码： #

@ExtendWith(MockitoExtension.class)
class AppConfigTest {
    @Mock
    private File mockedDirectory; // Mock我们需要的目录File对象

    // 使用@Spy来部分Mock AppConfig，这样可以只Mock特定方法，其他方法保留原逻辑
    @Spy
    @InjectMocks
    private AppConfig appConfig;

    @Test
    void getData_shouldReturnKeyMap_whenFilesExist() throws Exception {
        // 准备测试用的文件数组
        File mockFile = new File("test-key-file");
        File[] mockFiles = {mockFile};

        // 告诉Mockito：当调用appConfig的getDirectoryFile()时，返回我们Mock的directory
        doReturn(mockedDirectory).when(appConfig).getDirectoryFile();
        // Mock目录的listFiles()方法，返回我们预设的文件数组
        when(mockedDirectory.listFiles()).thenReturn(mockFiles);

        // 执行测试方法
        Map<String, String> result = appConfig.getData();

        // 验证结果非空，说明已经进入了keyMap的分支
        assertNotNull(result);
        
        // 如果keyMap是public方法，还可以验证它是否被正确调用
        // verify(appConfig).keyMap(eq(APP_CONST.DIRECTORY), anyList());
    }
}

方案二：使用PowerMock Mock File构造函数（不推荐，除非无法重构） #

如果因为某些原因不能修改实现类，可以用PowerMock来拦截new File(...)的构造调用，返回我们Mock的对象。不过这种方式会破坏代码的封装性，而且PowerMock的配置相对复杂，不推荐作为首选。

测试类示例（需要引入PowerMock依赖）： #

@ExtendWith({MockitoExtension.class, PowerMockExtension.class})
@PrepareForTest(AppConfig.class) // 指定要Mock构造函数的类
class AppConfigTest {
    @Mock
    private File mockedDirectory;

    @InjectMocks
    private AppConfig appConfig;

    @Test
    void getData_shouldReturnKeyMap_whenFilesExist() throws Exception {
        File[] mockFiles = {new File("test-file")};
        when(mockedDirectory.listFiles()).thenReturn(mockFiles);

        // 拦截new File(APP_CONST.DIRECTORY)的调用，返回我们的mockedDirectory
        PowerMockito.whenNew(File.class).withArguments(APP_CONST.DIRECTORY).thenReturn(mockedDirectory);

        Map<String, String> result = appConfig.getData();
        assertNotNull(result);
    }
}

另外还要注意：你之前测试类里的@Mock和@TempDir同时用在mockedFile字段上是错误的，这两个注解不能共存——@TempDir是用来创建真实的临时目录，而@Mock是创建Mock对象，两者冲突，所以之前的这个字段设置本身就有问题。

内容的提问来源于stack exchange，提问作者continuousLearner