本周狗哥的QBVmAgent算是搭建的七七八八, 我和強sir拉下代碼各自體驗. 妹想到我這邊12個單元測試竟然有兩個個死活跑不過去. 而強Sir和狗哥那邊都是可以正常跑過的. 黑人問號??? 想到狗哥和強Sir是MacOS, 而我的是Win10, 所以應(yīng)該是由于系統(tǒng)導(dǎo)致的不一致性問題.

IDEA下的測試截圖

這三處測試的行為都是在同一目錄下生成配置文件my.conf, 如果配置文件已存在, 則需要先刪除文件. 創(chuàng)建新配置文件的代碼如下:

private static void writeConfig(LinkedHashMap<String, String> map) {
        URL str = MyConfFileUtils.class.getClassLoader().getResource(appYmlFile);
        assert str != null;
        //去頭尾
        String temp = StringUtils.remove(str.toString(), "file:");
        temp = StringUtils.remove(temp, appYmlFile);
        File conf = new File(String.format("%s%s", temp, MY_CONF));
        assert !conf.exists() || conf.delete();
...省略

Debug進去后發(fā)現(xiàn)是conf.delete()返回false, 而delete調(diào)用的是本地方法, 由操作系統(tǒng)進行刪除.

聯(lián)想到之前在Linux下開啟兩個SSH連接對同一個文件一邊用Vim打開, 另一邊用rm刪除的時候, 系統(tǒng)并沒有提示任何異常. 猜想應(yīng)該是類Unix系統(tǒng)打開文件的同時依然可以刪除文件, 而Windows在一個文件打開的同時顯然是不可以刪除的. 所以無法刪除的原因應(yīng)該是別的進/線程占用了此文件.

由此結(jié)論進一步猜測, 是否是因為測試是并行的, 因此產(chǎn)生文件占用?折騰了一陣, 包括查詢Junit串行跑測試的一些方法, 應(yīng)用后發(fā)現(xiàn)問題依然存在. 而且由測試的日志時間確認, 測試本身就是串行執(zhí)行的.

測試的日志時間

線索到這里就死掉了, 正當我蛋疼的時候, 突然發(fā)現(xiàn)modifyConfigTest, 這個測試對my.conf開啟了一個FileInputStream, 并且沒有關(guān)閉:

FileInputStream inputStream = FileUtils.openInputStream(conf);
        Long filelength = conf.length();
        byte[] filecontent = new byte[filelength.intValue()];
        inputStream.read(filecontent);
        //缺少了inputStream.close()

原來如此, 看來問題是出在對該文件操作的文件流沒有正確關(guān)閉, 導(dǎo)致Windows平臺下無法刪除該文件. 添加關(guān)閉操作之后, 后面的測試全部通過.

StackOverflow傳送門 在StackOverflow上面也找到了一樣的問題, 看了看回答, 跟我的猜想是一致的.

StackOverflow回答

根據(jù)該回答, 在Windows中, 如果當前程序打開了該文件, 并嘗試刪除的時候, 你需要先關(guān)閉該文件然后刪除. 如果是別的程序打開了該文件, 那么你需要想辦法搞清楚誰打開了此文件. 而在Linux中, 基本上沒有什么可以阻止你刪除文件.

最后, 根據(jù)該回答下一些熱心群眾的建議, File.delete()是一個過時的方法, 在刪除失敗時只有一個boolean值返回. 推薦使用Files.delete(Path path)方法, 在文件被占用時操作會拋出FileSystemException異常. 而且, 上面的代碼中還存在著文件路徑過度依賴字符串進行處理的問題, 推薦使用Path進行路徑的拼接.
根據(jù)以上兩點, 對最上文處的代碼修改后, 如下所示:

private static void writeConfig(LinkedHashMap<String, String> map) {
        URL url = MyConfFileUtils.class.getClassLoader().getResource(appYmlFile);
        assert url != null;
        try {
            Path dirPath = Paths.get(url.toURI()).getParent(); // 改為由Path處理路徑
            Path currentPath = Paths.get(dirPath.toString(), MY_CONF);
            Files.deleteIfExists(currentPath); // 之前為 assert !conf.exists() || conf.delete();
...省略
        } catch (IOException | URISyntaxException e) {
            e.printStackTrace();
        }
    }