As an example, if i send a file with this function:
public class CobrancaConverter {
public CobrancaEntity convert(NFeCobrancaEntity cobranca, NfePK nfeKey) {
if (cobranca == null) {
return null;
} else {
DuplicatasConverter duplicatasConverter = new DuplicatasConverter();
CobrancaEntity entity = new CobrancaEntity();
entity.setY03Nfat(cobranca.getNfat());
entity.setY04Vorig(cobranca.getvOrig());
entity.setY05Vdesc(cobranca.getvDesc());
entity.setY06Vliq(cobranca.getvLiq());
entity.setKey(nfeKey);
List<CobrancaDuplicataEntity> duplicatas = duplicatasConverter.convert(cobranca.getDuplicatas(), nfeKey);
entity.setDuplicatas(duplicatas);
return entity;
}
}
}
Jacoco doesn't recognize the "if" as a branch and doesn't count it towards the covered branchs percentage, but if i create a public function calling a private function, like this, it works.
public class CobrancaConverter {
public CobrancaEntity converter(NFeCobrancaEntity cobranca, NfePK nfeKey){
return convert(cobranca, nfeKey);
}
private CobrancaEntity convert(NFeCobrancaEntity cobranca, NfePK nfeKey) {
if (cobranca == null) {
return null;
} else {
DuplicatasConverter duplicatasConverter = new DuplicatasConverter();
CobrancaEntity entity = new CobrancaEntity();
entity.setY03Nfat(cobranca.getNfat());
entity.setY04Vorig(cobranca.getvOrig());
entity.setY05Vdesc(cobranca.getvDesc());
entity.setY06Vliq(cobranca.getvLiq());
entity.setKey(nfeKey);
List<CobrancaDuplicataEntity> duplicatas = duplicatasConverter.convert(cobranca.getDuplicatas(), nfeKey);
entity.setDuplicatas(duplicatas);
return entity;
}
}
}
Shouldn't Jacoco recognize both cases? Am i doing something wrong, is this expected or this is a bug i should report?
Am i doing something wrong, is this expected or this is a bug i should report?