最近測(cè)試的同學(xué)反饋一個(gè)問題续担，他在測(cè)試完項(xiàng)目以后查看覆蓋率的結(jié)果，發(fā)現(xiàn)有個(gè)邏輯明明已經(jīng)覆蓋了哩盲，但是覆蓋率的報(bào)告里面顯示的卻是沒有覆蓋艰管。
問題分析
以下就是源碼的內(nèi)容甫窟，并且現(xiàn)場(chǎng)演示了接口返回庫(kù)存不足的邏輯。
看到這個(gè)問題就比較捉急了蛙婴。因?yàn)殛P(guān)于jacoco目前比較難的就是這樣子的現(xiàn)象粗井，很大程度是要去了解jacoco的邏輯才行。
因?yàn)楦采w率的情況基本都是在jacoco.exec這個(gè)文件中, 所以我們第一步要去了解的是exec文件是否確定已經(jīng)有覆蓋了這個(gè)代碼邏輯街图。 這里我們可以借助jacococli.jar來(lái)輔助分析浇衬。
java -jar jacococli.jar execinfo [<execfiles> ...]
所以我們直接拉取了被測(cè)主服務(wù)的exec文件進(jìn)行查看。得到了這樣子的結(jié)果餐济。
這個(gè)信息至少能夠得到一個(gè)信息就是這個(gè)代碼被覆蓋了至少有1/3耘擂。就已經(jīng)跟實(shí)際覆蓋率的報(bào)告不符合了。
我們?cè)倏聪聦?duì)應(yīng)的TradeOrderServiceImpl 類中probes 探針覆蓋的情況絮姆。
這個(gè)時(shí)候測(cè)試同學(xué)告訴了一個(gè)很有意義的信息醉冤。這次的測(cè)試計(jì)劃是有兩個(gè)服務(wù)都用到了同一個(gè)倉(cāng)庫(kù)，測(cè)試同學(xué)問了對(duì)應(yīng)的開發(fā)篙悯，開發(fā)給的意見是有沒有可能是同一份代碼在兩個(gè)倉(cāng)庫(kù)里面都用到了蚁阳，所以導(dǎo)致沖突一直沒有被覆蓋到。
所以又拉取了另外一個(gè)服務(wù)的exec文件進(jìn)行查看鸽照。
確實(shí)在另外一份exec的數(shù)據(jù)里面也出現(xiàn)了這個(gè)類螺捐，但是它的覆蓋率情況就比較糟糕了。所以從這里就大概能夠知道覆蓋率的報(bào)告用的是這份的數(shù)據(jù)了矮燎。
原因
那為什么會(huì)出現(xiàn)這樣子的情況呢? 理論上這兩份數(shù)據(jù)應(yīng)該會(huì)被合并成最后一個(gè)才對(duì)吧定血？帶著這個(gè)疑問我們?nèi)タ戳讼逻壿嫛?/p>
public void put(final ExecutionData data) throws IllegalStateException {
final Long id = Long.valueOf(data.getId());
final ExecutionData entry = entries.get(id);
if (entry == null) {
entries.put(id, data);
names.add(data.getName());
} else {
entry.merge(data);
}
}
從這里看出來(lái)，其實(shí)是否會(huì)合并覆蓋率數(shù)據(jù)是根據(jù)data中的id來(lái)進(jìn)行判斷的诞外，如果是相同的情況下澜沟，數(shù)據(jù)才會(huì)被合并。這個(gè)id其實(shí)是classId.難道說(shuō)這兩個(gè)classId是不相同的嗎峡谊？
我們對(duì)比前兩個(gè)截圖其實(shí)就會(huì)發(fā)現(xiàn)了茫虽，其實(shí)這兩個(gè)classId是不一樣的铝条。所以這就是導(dǎo)致了為什么覆蓋率數(shù)據(jù)不正確的原因了，不過(guò)也不能說(shuō)不正確席噩，只能說(shuō)這個(gè)可能更多是一個(gè)概率性的問題，如果說(shuō)最后分析的是主服務(wù)中的exec文件贤壁，其實(shí)覆蓋的數(shù)據(jù)就正確了悼枢。
所以針對(duì)上述這種情況我們只能夠暫時(shí)忽略掉classId這種區(qū)別，只是根據(jù)className來(lái)做判斷處理脾拆。
public void put(final ExecutionData data) throws IllegalStateException {
final Long id = Long.valueOf(data.getId());
final String className= data.getName();
// 這里可能存在有相同的類名的 最好是做合并的處理
ExecutionData entry = entries.get(id);
if (entry == null) {
entry = classEntries.get(className);
// 如果probe不相同 說(shuō)明是兩個(gè)同名的class但是內(nèi)容不相同
if (entry != null && entry.getProbes().length != data.getProbes().length) {
entry = null;
}
}
// final ExecutionData classEntry = classEntries.get(className);
if (entry == null) {
entries.put(id, data);
names.add(data.getName());
classEntries.put(className, data);
} else {
entry.merge(data);
classEntries.put(className, entry);
}
// if (classEntry == null) {
// classEntries.put(className, data);
// names.add(data.getName());
// }else {
// classEntries.put(className, )
//// classEntry.merge(data);
// }
}
所以就是通過(guò)上述的方式就可以解決了馒索，不過(guò)這個(gè)肯定有相應(yīng)的弊端，因?yàn)楸旧韏acoco的設(shè)計(jì)是通過(guò)classId來(lái)進(jìn)行區(qū)分不同的類的名船，而我們這里是通過(guò)className就行數(shù)據(jù)合并 還是有一定問題的绰上。
