From 13f29eb797d32338fb073bef8cc37b4e1dbde9f1 Mon Sep 17 00:00:00 2001 From: Evgeny Mandrikov <138671+Godin@users.noreply.github.com> Date: Fri, 18 Jan 2019 18:25:51 +0100 Subject: Preserve empty class and sourcefile nodes in XML report (#817) --- .../src/org/jacoco/core/analysis/CoverageBuilderTest.java | 10 +++------- .../jacoco/core/internal/analysis/ClassCoverageImplTest.java | 2 +- 2 files changed, 4 insertions(+), 8 deletions(-) (limited to 'org.jacoco.core.test/src/org') diff --git a/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageBuilderTest.java b/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageBuilderTest.java index a4a38954..40c35e8f 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageBuilderTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageBuilderTest.java @@ -13,7 +13,6 @@ package org.jacoco.core.analysis; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertTrue; import java.util.Arrays; import java.util.Collection; @@ -106,13 +105,10 @@ public class CoverageBuilderTest { } @Test - public void testIgnoreClassesWithoutCode() { - final MethodCoverageImpl method = new MethodCoverageImpl("doit", "()V", - null); - addClass(123L, false, "Sample", null, method); + public void should_not_ignore_empty_classes() { + addClass(123L, false, "Empty", null); - final Collection classes = coverageBuilder.getClasses(); - assertTrue(classes.isEmpty()); + assertEquals(1, coverageBuilder.getClasses().size()); } @Test(expected = IllegalStateException.class) diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/ClassCoverageImplTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/ClassCoverageImplTest.java index b219a878..57d281b3 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/ClassCoverageImplTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/ClassCoverageImplTest.java @@ -93,7 +93,7 @@ public class ClassCoverageImplTest { assertEquals(CounterImpl.COUNTER_0_0, node.getInstructionCounter()); assertEquals(CounterImpl.COUNTER_0_0, node.getBranchCounter()); assertEquals(CounterImpl.COUNTER_0_0, node.getMethodCounter()); - assertEquals(CounterImpl.COUNTER_1_0, node.getClassCounter()); + assertEquals(CounterImpl.COUNTER_0_0, node.getClassCounter()); } @Test -- cgit v1.2.3