本周狗哥的QBVmAgent算是搭建的七七八八, 我和強sir拉下代碼各自體驗. 妹想到我這邊12個單元測試竟然有兩個個死活跑不過去. 而強Sir和狗哥那邊都是可以正常跑過的. 黑人問號??? 想到狗哥和強Sir是MacOS, 而我的是Win10, 所以應(yīng)該是由于系統(tǒng)導(dǎo)致的不一致性問題.
這三處測試的行為都是在同一目錄下生成配置文件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進(jìn)去后發(fā)現(xiàn)是conf.delete()返回false, 而delete調(diào)用的是本地方法, 由操作系統(tǒng)進(jìn)行刪除.
聯(lián)想到之前在Linux下開啟兩個SSH連接對同一個文件一邊用Vim打開, 另一邊用rm刪除的時候, 系統(tǒng)并沒有提示任何異常. 猜想應(yīng)該是類Unix系統(tǒng)打開文件的同時依然可以刪除文件, 而Windows在一個文件打開的同時顯然是不可以刪除的. 所以無法刪除的原因應(yīng)該是別的進(jìn)/線程占用了此文件.
由此結(jié)論進(jìn)一步猜測, 是否是因為測試是并行的, 因此產(chǎn)生文件占用?折騰了一陣, 包括查詢Junit串行跑測試的一些方法, 應(yīng)用后發(fā)現(xiàn)問題依然存在. 而且由測試的日志時間確認(rèn), 測試本身就是串行執(zhí)行的.
線索到這里就死掉了, 正當(dāng)我蛋疼的時候, 突然發(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上面也找到了一樣的問題, 看了看回答, 跟我的猜想是一致的.
根據(jù)該回答, 在Windows中, 如果當(dāng)前程序打開了該文件, 并嘗試刪除的時候, 你需要先關(guān)閉該文件然后刪除. 如果是別的程序打開了該文件, 那么你需要想辦法搞清楚誰打開了此文件. 而在Linux中, 基本上沒有什么可以阻止你刪除文件.
最后, 根據(jù)該回答下一些熱心群眾的建議, File.delete()是一個過時的方法, 在刪除失敗時只有一個boolean值返回. 推薦使用Files.delete(Path path)方法, 在文件被占用時操作會拋出FileSystemException異常. 而且, 上面的代碼中還存在著文件路徑過度依賴字符串進(jìn)行處理的問題, 推薦使用Path進(jìn)行路徑的拼接.
根據(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();
}
}
