diff options
author | Brett Kail <brett.kail@oracle.com> | 2015-11-23 10:24:00 -0600 |
---|---|---|
committer | Brett Kail <brett.kail@oracle.com> | 2015-11-23 10:24:00 -0600 |
commit | f2aef08a63e3867a7db781040950d8a54078fcf7 (patch) | |
tree | a64c6fd5ce736edfecf50d79137f2e9999c82b18 /org.jacoco.core.test/src/org/jacoco | |
parent | ec3266ac08f1a02d9e11789b187af634c09f35e6 (diff) | |
download | jacoco-f2aef08a63e3867a7db781040950d8a54078fcf7.tar.gz |
Don't count class stats twice for a package
Fixes #360.
Diffstat (limited to 'org.jacoco.core.test/src/org/jacoco')
-rw-r--r-- | org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageBuilderTest.java | 21 |
1 files changed, 21 insertions, 0 deletions
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 8eeabd3d..f2d5b39c 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 @@ -165,6 +165,27 @@ public class CoverageBuilderTest { } @Test + public void testCreateSourceFileDuplicateClassNameIdentical() { + final MethodCoverageImpl method1 = new MethodCoverageImpl("doit", + "()V", null); + method1.increment(CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, 3); + addClass(123L, false, "Sample", "Sample.java", method1); + + final MethodCoverageImpl method2 = new MethodCoverageImpl("doit", + "()V", null); + method2.increment(CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, 3); + addClass(123L, false, "Sample", "Sample.java", method2); + + final Collection<ISourceFileCoverage> sourcefiles = coverageBuilder + .getSourceFiles(); + assertEquals(1, sourcefiles.size()); + ISourceFileCoverage s = sourcefiles.iterator().next(); + + assertEquals(1, s.getClassCounter().getTotalCount()); + assertEquals(0, s.getClassCounter().getCoveredCount()); + } + + @Test public void testGetBundle() { final MethodCoverageImpl method1 = new MethodCoverageImpl("doit", "()V", null); |