I'm using jacoco coverage report on my java maven project, but it doesn't seem 
to recognize branchs on public functions

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?

-- 
You received this message because you are subscribed to the Google Groups 
"JaCoCo and EclEmma Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jacoco/61fc8039-b781-49cb-8151-9ec8aa3e2b36%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to