diff options
author | Marc R. Hoffmann <hoffmann@mountainminds.com> | 2019-10-05 00:46:40 +0200 |
---|---|---|
committer | Evgeny Mandrikov <138671+Godin@users.noreply.github.com> | 2019-10-05 00:46:40 +0200 |
commit | 674055ec1659d3f9dbbc5c7286dd057a5777b42f (patch) | |
tree | e461cbf1978ef5774af8b05d66cfc006fe409c32 /org.jacoco.core.test/src/org/jacoco/core | |
parent | fba553424f9d49ecb1ac324548d50a3a19c1af7d (diff) | |
download | jacoco-674055ec1659d3f9dbbc5c7286dd057a5777b42f.tar.gz |
Format code base consistently (#954)
All *.java files now comply with the Eclipse 2019-09 formatter.
Diffstat (limited to 'org.jacoco.core.test/src/org/jacoco/core')
40 files changed, 342 insertions, 296 deletions
diff --git a/org.jacoco.core.test/src/org/jacoco/core/analysis/AnalyzerTest.java b/org.jacoco.core.test/src/org/jacoco/core/analysis/AnalyzerTest.java index f2a79515..5162daff 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/analysis/AnalyzerTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/analysis/AnalyzerTest.java @@ -200,8 +200,7 @@ public class AnalyzerTest { } /** - * Triggers exception in - * {@link Analyzer#analyzeClass(InputStream, String)}. + * Triggers exception in {@link Analyzer#analyzeClass(InputStream, String)}. */ @Test public void testAnalyzeClass_BrokenStream() throws IOException { @@ -243,8 +242,8 @@ public class AnalyzerTest { public void testAnalyzeAll_Zip() throws IOException { final ByteArrayOutputStream buffer = new ByteArrayOutputStream(); final ZipOutputStream zip = new ZipOutputStream(buffer); - zip.putNextEntry(new ZipEntry( - "org/jacoco/core/analysis/AnalyzerTest.class")); + zip.putNextEntry( + new ZipEntry("org/jacoco/core/analysis/AnalyzerTest.class")); zip.write(TargetLoader.getClassDataAsBytes(AnalyzerTest.class)); zip.finish(); final int count = analyzer.analyzeAll( @@ -297,20 +296,21 @@ public class AnalyzerTest { public void testAnalyzeAll_Pack200() throws IOException { final ByteArrayOutputStream zipbuffer = new ByteArrayOutputStream(); final ZipOutputStream zip = new ZipOutputStream(zipbuffer); - zip.putNextEntry(new ZipEntry( - "org/jacoco/core/analysis/AnalyzerTest.class")); + zip.putNextEntry( + new ZipEntry("org/jacoco/core/analysis/AnalyzerTest.class")); zip.write(TargetLoader.getClassDataAsBytes(AnalyzerTest.class)); zip.finish(); final ByteArrayOutputStream pack200buffer = new ByteArrayOutputStream(); GZIPOutputStream gzipOutput = new GZIPOutputStream(pack200buffer); - Pack200.newPacker().pack( - new JarInputStream(new ByteArrayInputStream( - zipbuffer.toByteArray())), gzipOutput); + Pack200.newPacker() + .pack(new JarInputStream( + new ByteArrayInputStream(zipbuffer.toByteArray())), + gzipOutput); gzipOutput.finish(); - final int count = analyzer.analyzeAll(new ByteArrayInputStream( - pack200buffer.toByteArray()), "Test"); + final int count = analyzer.analyzeAll( + new ByteArrayInputStream(pack200buffer.toByteArray()), "Test"); assertEquals(1, count); assertClasses("org/jacoco/core/analysis/AnalyzerTest"); } @@ -334,8 +334,8 @@ public class AnalyzerTest { @Test public void testAnalyzeAll_Empty() throws IOException { - final int count = analyzer.analyzeAll(new ByteArrayInputStream( - new byte[0]), "Test"); + final int count = analyzer + .analyzeAll(new ByteArrayInputStream(new byte[0]), "Test"); assertEquals(0, count); assertEquals(Collections.emptyMap(), classes); } @@ -381,12 +381,12 @@ public class AnalyzerTest { /** * With JDK 5 triggers exception in - * {@link Analyzer#nextEntry(ZipInputStream, String)}, - * i.e. message will contain only "broken.zip". + * {@link Analyzer#nextEntry(ZipInputStream, String)}, i.e. message will + * contain only "broken.zip". * * With JDK > 5 triggers exception in - * {@link Analyzer#analyzeAll(java.io.InputStream, String)}, - * i.e. message will contain only "broken.zip@brokenentry.txt". + * {@link Analyzer#analyzeAll(java.io.InputStream, String)}, i.e. message + * will contain only "broken.zip@brokenentry.txt". */ @Test public void testAnalyzeAll_BrokenZipEntry() throws IOException { @@ -413,8 +413,8 @@ public class AnalyzerTest { public void testAnalyzeAll_BrokenClassFileInZip() throws IOException { final ByteArrayOutputStream buffer = new ByteArrayOutputStream(); final ZipOutputStream zip = new ZipOutputStream(buffer); - zip.putNextEntry(new ZipEntry( - "org/jacoco/core/analysis/AnalyzerTest.class")); + zip.putNextEntry( + new ZipEntry("org/jacoco/core/analysis/AnalyzerTest.class")); final byte[] brokenclass = TargetLoader .getClassDataAsBytes(AnalyzerTest.class); brokenclass[10] = 0x23; diff --git a/org.jacoco.core.test/src/org/jacoco/core/analysis/CounterComparatorTest.java b/org.jacoco.core.test/src/org/jacoco/core/analysis/CounterComparatorTest.java index cc5e54da..1c3e22e2 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/analysis/CounterComparatorTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/analysis/CounterComparatorTest.java @@ -111,12 +111,14 @@ public class CounterComparatorTest { private void assertCmpLess(Comparator<ICounter> cmp, int missed1, int covered1, int missed2, int covered2) { - assertTrue(cmp.compare(ctr(missed1, covered1), ctr(missed2, covered2)) < 0); + assertTrue(cmp.compare(ctr(missed1, covered1), + ctr(missed2, covered2)) < 0); } private void assertCmpGreater(Comparator<ICounter> cmp, int missed1, int covered1, int missed2, int covered2) { - assertTrue(cmp.compare(ctr(missed1, covered1), ctr(missed2, covered2)) > 0); + assertTrue(cmp.compare(ctr(missed1, covered1), + ctr(missed2, covered2)) > 0); } private CounterImpl ctr(int missed, int covered) { 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 fb9e2ecf..d9a9fab6 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 @@ -142,13 +142,13 @@ public class CoverageBuilderTest { @Test public void testCreateSourceFile() { - final MethodCoverageImpl method1 = new MethodCoverageImpl("doit", - "()V", null); + 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); + final MethodCoverageImpl method2 = new MethodCoverageImpl("doit", "()V", + null); method2.increment(CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, 6); addClass(234L, false, "Second", "Sample.java", method2); @@ -163,13 +163,13 @@ public class CoverageBuilderTest { @Test public void testCreateSourceFileDuplicateClassNameIdentical() { - final MethodCoverageImpl method1 = new MethodCoverageImpl("doit", - "()V", null); + 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); + 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); @@ -184,18 +184,18 @@ public class CoverageBuilderTest { @Test public void testGetBundle() { - final MethodCoverageImpl method1 = new MethodCoverageImpl("doit", - "()V", null); + final MethodCoverageImpl method1 = new MethodCoverageImpl("doit", "()V", + null); method1.increment(CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, 3); addClass(1, false, "org/jacoco/examples/Sample1", null, method1); - final MethodCoverageImpl method2 = new MethodCoverageImpl("doit", - "()V", null); + final MethodCoverageImpl method2 = new MethodCoverageImpl("doit", "()V", + null); method2.increment(CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, 6); addClass(2, false, "org/jacoco/examples/Sample2", null, method2); - final MethodCoverageImpl method3 = new MethodCoverageImpl("doit", - "()V", null); + final MethodCoverageImpl method3 = new MethodCoverageImpl("doit", "()V", + null); method3.increment(CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, 1); addClass(3, false, "Sample3", null, method3); @@ -212,8 +212,7 @@ public class CoverageBuilderTest { IPackageCoverage p1 = packagesByName.get("org/jacoco/examples"); assertNotNull(p1); assertEquals( - new HashSet<String>(Arrays.asList( - "org/jacoco/examples/Sample1", + new HashSet<String>(Arrays.asList("org/jacoco/examples/Sample1", "org/jacoco/examples/Sample2")), getNames(p1.getClasses())); @@ -237,9 +236,10 @@ public class CoverageBuilderTest { m.increment(CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, 3); addClass(3, false, "Sample3", null, m); - final Set<String> actual = getNames(coverageBuilder.getNoMatchClasses()); - final Set<String> expected = new HashSet<String>(Arrays.asList( - "Sample1", "Sample2")); + final Set<String> actual = getNames( + coverageBuilder.getNoMatchClasses()); + final Set<String> expected = new HashSet<String>( + Arrays.asList("Sample1", "Sample2")); assertEquals(expected, actual); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageNodeImplTest.java b/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageNodeImplTest.java index a6a780ff..3bd40221 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageNodeImplTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/analysis/CoverageNodeImplTest.java @@ -53,7 +53,8 @@ public class CoverageNodeImplTest { public void testIncrement() { CoverageNodeImpl parent = new CoverageNodeImpl(ElementType.GROUP, "sample"); - ICoverageNode child = new CoverageNodeImpl(ElementType.GROUP, "sample") { + ICoverageNode child = new CoverageNodeImpl(ElementType.GROUP, + "sample") { { instructionCounter = CounterImpl.getInstance(1, 41); branchCounter = CounterImpl.getInstance(10, 15); @@ -68,8 +69,10 @@ public class CoverageNodeImplTest { parent.getCounter(INSTRUCTION)); assertEquals(CounterImpl.getInstance(1, 41), parent.getInstructionCounter()); - assertEquals(CounterImpl.getInstance(10, 15), parent.getCounter(BRANCH)); - assertEquals(CounterImpl.getInstance(10, 15), parent.getBranchCounter()); + assertEquals(CounterImpl.getInstance(10, 15), + parent.getCounter(BRANCH)); + assertEquals(CounterImpl.getInstance(10, 15), + parent.getBranchCounter()); assertEquals(CounterImpl.getInstance(5, 3), parent.getCounter(LINE)); assertEquals(CounterImpl.getInstance(5, 3), parent.getLineCounter()); assertEquals(CounterImpl.getInstance(4, 2), @@ -86,12 +89,14 @@ public class CoverageNodeImplTest { public void testIncrementCollection() { CoverageNodeImpl parent = new CoverageNodeImpl(ElementType.GROUP, "sample"); - ICoverageNode child1 = new CoverageNodeImpl(ElementType.GROUP, "sample") { + ICoverageNode child1 = new CoverageNodeImpl(ElementType.GROUP, + "sample") { { branchCounter = CounterImpl.getInstance(5, 2); } }; - ICoverageNode child2 = new CoverageNodeImpl(ElementType.GROUP, "sample") { + ICoverageNode child2 = new CoverageNodeImpl(ElementType.GROUP, + "sample") { { branchCounter = CounterImpl.getInstance(3, 3); } @@ -121,7 +126,8 @@ public class CoverageNodeImplTest { assertEquals(CounterImpl.getInstance(4, 4), copy.getInstructionCounter()); assertEquals(CounterImpl.getInstance(5, 5), copy.getLineCounter()); - assertEquals(CounterImpl.getInstance(6, 6), copy.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(6, 6), + copy.getComplexityCounter()); } @Test diff --git a/org.jacoco.core.test/src/org/jacoco/core/analysis/NodeComparatorTest.java b/org.jacoco.core.test/src/org/jacoco/core/analysis/NodeComparatorTest.java index 200b9bca..5187ad27 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/analysis/NodeComparatorTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/analysis/NodeComparatorTest.java @@ -35,9 +35,9 @@ public class NodeComparatorTest { ICoverageNode d3 = new MockNode(30); ICoverageNode d4 = new MockNode(60); ICoverageNode d5 = new MockNode(99); - final List<ICoverageNode> result = CounterComparator.TOTALITEMS.on( - CounterEntity.INSTRUCTION).sort( - Arrays.asList(d3, d5, d1, d4, d2)); + final List<ICoverageNode> result = CounterComparator.TOTALITEMS + .on(CounterEntity.INSTRUCTION) + .sort(Arrays.asList(d3, d5, d1, d4, d2)); assertEquals(Arrays.asList(d1, d2, d3, d4, d5), result); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataReaderWriterTest.java b/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataReaderWriterTest.java index 58f6ad48..9b7ac48c 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataReaderWriterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataReaderWriterTest.java @@ -152,8 +152,8 @@ public class ExecutionDataReaderWriterTest { @Test(expected = IOException.class) public void testMissingHeader() throws IOException { buffer.reset(); - writer.visitClassExecution(new ExecutionData(Long.MIN_VALUE, "Sample", - createData(8))); + writer.visitClassExecution( + new ExecutionData(Long.MIN_VALUE, "Sample", createData(8))); createReaderWithVisitors().read(); } @@ -165,8 +165,8 @@ public class ExecutionDataReaderWriterTest { @Test(expected = EOFException.class) public void testTruncatedFile() throws IOException { - writer.visitClassExecution(new ExecutionData(Long.MIN_VALUE, "Sample", - createData(8))); + writer.visitClassExecution( + new ExecutionData(Long.MIN_VALUE, "Sample", createData(8))); final byte[] content = buffer.toByteArray(); buffer.reset(); buffer.write(content, 0, content.length - 1); @@ -217,16 +217,16 @@ public class ExecutionDataReaderWriterTest { @Test(expected = IOException.class) public void testNoExecutionDataVisitor() throws IOException { - writer.visitClassExecution(new ExecutionData(Long.MIN_VALUE, "Sample", - createData(8))); + writer.visitClassExecution( + new ExecutionData(Long.MIN_VALUE, "Sample", createData(8))); createReader().read(); } @Test public void testMinClassId() throws IOException { final boolean[] data = createData(8); - writer.visitClassExecution(new ExecutionData(Long.MIN_VALUE, "Sample", - data)); + writer.visitClassExecution( + new ExecutionData(Long.MIN_VALUE, "Sample", data)); assertFalse(createReaderWithVisitors().read()); assertArrayEquals(data, store.get(Long.MIN_VALUE).getProbes()); } @@ -234,8 +234,8 @@ public class ExecutionDataReaderWriterTest { @Test public void testMaxClassId() throws IOException { final boolean[] data = createData(8); - writer.visitClassExecution(new ExecutionData(Long.MAX_VALUE, "Sample", - data)); + writer.visitClassExecution( + new ExecutionData(Long.MAX_VALUE, "Sample", data)); assertFalse(createReaderWithVisitors().read()); assertArrayEquals(data, store.get(Long.MAX_VALUE).getProbes()); } @@ -295,7 +295,8 @@ public class ExecutionDataReaderWriterTest { } }); broken[0] = true; - writer.visitClassExecution(new ExecutionData(3, "Sample", createData(1))); + writer.visitClassExecution( + new ExecutionData(3, "Sample", createData(1))); } private ExecutionDataReader createReaderWithVisitors() throws IOException { @@ -328,8 +329,8 @@ public class ExecutionDataReaderWriterTest { } protected ExecutionDataReader createReader() throws IOException { - return new ExecutionDataReader(new ByteArrayInputStream( - buffer.toByteArray())); + return new ExecutionDataReader( + new ByteArrayInputStream(buffer.toByteArray())); } } diff --git a/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataStoreTest.java b/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataStoreTest.java index 93f19645..924c93bb 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataStoreTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataStoreTest.java @@ -206,8 +206,8 @@ public class ExecutionDataStoreTest implements IExecutionDataVisitor { } @Test - public void testReset() throws InstantiationException, - IllegalAccessException { + public void testReset() + throws InstantiationException, IllegalAccessException { final boolean[] data1 = new boolean[] { true, true, false }; store.put(new ExecutionData(1000, "Sample", data1)); store.reset(); diff --git a/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataTest.java b/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataTest.java index 9b777ca5..dedbfddd 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/data/ExecutionDataTest.java @@ -46,8 +46,8 @@ public class ExecutionDataTest { @Test public void testReset() { - final ExecutionData e = new ExecutionData(5, "Example", new boolean[] { - true, false, true }); + final ExecutionData e = new ExecutionData(5, "Example", + new boolean[] { true, false, true }); e.reset(); assertFalse(e.getProbes()[0]); assertFalse(e.getProbes()[1]); @@ -72,10 +72,10 @@ public class ExecutionDataTest { @Test public void testMerge() { - final ExecutionData a = new ExecutionData(5, "Example", new boolean[] { - false, true, false, true }); - final ExecutionData b = new ExecutionData(5, "Example", new boolean[] { - false, false, true, true }); + final ExecutionData a = new ExecutionData(5, "Example", + new boolean[] { false, true, false, true }); + final ExecutionData b = new ExecutionData(5, "Example", + new boolean[] { false, false, true, true }); a.merge(b); // b is merged into a: @@ -93,10 +93,10 @@ public class ExecutionDataTest { @Test public void testMergeSubtract() { - final ExecutionData a = new ExecutionData(5, "Example", new boolean[] { - false, true, false, true }); - final ExecutionData b = new ExecutionData(5, "Example", new boolean[] { - false, false, true, true }); + final ExecutionData a = new ExecutionData(5, "Example", + new boolean[] { false, true, false, true }); + final ExecutionData b = new ExecutionData(5, "Example", + new boolean[] { false, false, true, true }); a.merge(b, false); // b is subtracted from a: diff --git a/org.jacoco.core.test/src/org/jacoco/core/data/SessionInfoTest.java b/org.jacoco.core.test/src/org/jacoco/core/data/SessionInfoTest.java index ba6310d5..15ef14e2 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/data/SessionInfoTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/data/SessionInfoTest.java @@ -36,15 +36,12 @@ public class SessionInfoTest { @Test public void testCompare() { - assertEquals(0, - new SessionInfo("id", 1000, 2000).compareTo(new SessionInfo( - "id", 1234, 2000))); - assertEquals(-1, - new SessionInfo("id", 3333, 1999).compareTo(new SessionInfo( - "id", 1234, 2000))); - assertEquals(+1, - new SessionInfo("id", 1234, 2001).compareTo(new SessionInfo( - "id", 2222, 2000))); + assertEquals(0, new SessionInfo("id", 1000, 2000) + .compareTo(new SessionInfo("id", 1234, 2000))); + assertEquals(-1, new SessionInfo("id", 3333, 1999) + .compareTo(new SessionInfo("id", 1234, 2000))); + assertEquals(+1, new SessionInfo("id", 1234, 2001) + .compareTo(new SessionInfo("id", 2222, 2000))); } @Test diff --git a/org.jacoco.core.test/src/org/jacoco/core/instr/InstrumenterTest.java b/org.jacoco.core.test/src/org/jacoco/core/instr/InstrumenterTest.java index 0fa42ecd..51959ea6 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/instr/InstrumenterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/instr/InstrumenterTest.java @@ -214,8 +214,8 @@ public class InstrumenterTest { new ObjectOutputStream(buffer).writeObject(obj1); // Deserialize with original class definition: - Object obj2 = new ObjectInputStream(new ByteArrayInputStream( - buffer.toByteArray())).readObject(); + Object obj2 = new ObjectInputStream( + new ByteArrayInputStream(buffer.toByteArray())).readObject(); assertEquals("Hello42", obj2.toString()); } @@ -260,8 +260,8 @@ public class InstrumenterTest { new ByteArrayInputStream(buffer.toByteArray()), out, "Test"); assertEquals(1, count); - ZipInputStream zipin = new ZipInputStream(new ByteArrayInputStream( - out.toByteArray())); + ZipInputStream zipin = new ZipInputStream( + new ByteArrayInputStream(out.toByteArray())); assertEquals("Test.class", zipin.getNextEntry().getName()); assertNull(zipin.getNextEntry()); } @@ -406,23 +406,26 @@ public class InstrumenterTest { ByteArrayOutputStream pack200buffer = new ByteArrayOutputStream(); GZIPOutputStream gzipOutput = new GZIPOutputStream(pack200buffer); - Pack200.newPacker().pack( - new JarInputStream(new ByteArrayInputStream( - jarbuffer.toByteArray())), gzipOutput); + Pack200.newPacker() + .pack(new JarInputStream( + new ByteArrayInputStream(jarbuffer.toByteArray())), + gzipOutput); gzipOutput.finish(); ByteArrayOutputStream out = new ByteArrayOutputStream(); - int count = instrumenter.instrumentAll(new ByteArrayInputStream( - pack200buffer.toByteArray()), out, "Test"); + int count = instrumenter.instrumentAll( + new ByteArrayInputStream(pack200buffer.toByteArray()), out, + "Test"); jarbuffer.reset(); Pack200.newUnpacker() - .unpack(new GZIPInputStream(new ByteArrayInputStream( - out.toByteArray())), new JarOutputStream(jarbuffer)); + .unpack(new GZIPInputStream( + new ByteArrayInputStream(out.toByteArray())), + new JarOutputStream(jarbuffer)); assertEquals(1, count); - ZipInputStream zipin = new ZipInputStream(new ByteArrayInputStream( - jarbuffer.toByteArray())); + ZipInputStream zipin = new ZipInputStream( + new ByteArrayInputStream(jarbuffer.toByteArray())); assertEquals("Test.class", zipin.getNextEntry().getName()); assertNull(zipin.getNextEntry()); } @@ -469,8 +472,8 @@ public class InstrumenterTest { new ByteArrayInputStream(buffer.toByteArray()), out, "Test"); assertEquals(0, count); - ZipInputStream zipin = new ZipInputStream(new ByteArrayInputStream( - out.toByteArray())); + ZipInputStream zipin = new ZipInputStream( + new ByteArrayInputStream(out.toByteArray())); assertEquals("META-INF/MANIFEST.MF", zipin.getNextEntry().getName()); assertNull(zipin.getNextEntry()); } @@ -488,8 +491,8 @@ public class InstrumenterTest { new ByteArrayInputStream(buffer.toByteArray()), out, "Test"); assertEquals(0, count); - ZipInputStream zipin = new ZipInputStream(new ByteArrayInputStream( - out.toByteArray())); + ZipInputStream zipin = new ZipInputStream( + new ByteArrayInputStream(out.toByteArray())); assertEquals("META-INF/ALIAS.SF", zipin.getNextEntry().getName()); assertNull(zipin.getNextEntry()); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/ContentTypeDetectorTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/ContentTypeDetectorTest.java index a35c0e65..61577235 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/ContentTypeDetectorTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/ContentTypeDetectorTest.java @@ -160,7 +160,8 @@ public class ContentTypeDetectorTest { } @Test - public void should_detect_java_13_with_preview_features() throws IOException { + public void should_detect_java_13_with_preview_features() + throws IOException { initData(0xCA, 0xFE, 0xBA, 0xBE, 0xFF, 0xFF, 0x00, 0x39); assertEquals(ContentTypeDetector.CLASSFILE, detector.getType()); assertContent(); @@ -174,7 +175,8 @@ public class ContentTypeDetectorTest { } @Test - public void should_detect_java_14_with_preview_features() throws IOException { + public void should_detect_java_14_with_preview_features() + throws IOException { initData(0xCA, 0xFE, 0xBA, 0xBE, 0xFF, 0xFF, 0x00, 0x3A); assertEquals(ContentTypeDetector.CLASSFILE, detector.getType()); assertContent(); @@ -223,9 +225,10 @@ public class ContentTypeDetectorTest { zip.close(); final ByteArrayOutputStream pack200buffer = new ByteArrayOutputStream(); - Pack200.newPacker().pack( - new JarInputStream(new ByteArrayInputStream( - zipbuffer.toByteArray())), pack200buffer); + Pack200.newPacker() + .pack(new JarInputStream( + new ByteArrayInputStream(zipbuffer.toByteArray())), + pack200buffer); initData(pack200buffer.toByteArray()); assertEquals(ContentTypeDetector.PACK200FILE, detector.getType()); assertContent(); diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/Pack200StreamsTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/Pack200StreamsTest.java index d3c5704b..77cf097e 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/Pack200StreamsTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/Pack200StreamsTest.java @@ -47,16 +47,16 @@ public class Pack200StreamsTest { zipout.finish(); ByteArrayOutputStream pack200buffer = new ByteArrayOutputStream(); - Pack200Streams.pack(jarbuffer.toByteArray(), new NoCloseOutputStream( - pack200buffer)); + Pack200Streams.pack(jarbuffer.toByteArray(), + new NoCloseOutputStream(pack200buffer)); jarbuffer.reset(); Pack200.newUnpacker().unpack( new ByteArrayInputStream(pack200buffer.toByteArray()), new JarOutputStream(jarbuffer)); - ZipInputStream zipin = new ZipInputStream(new ByteArrayInputStream( - jarbuffer.toByteArray())); + ZipInputStream zipin = new ZipInputStream( + new ByteArrayInputStream(jarbuffer.toByteArray())); assertEquals("Test.class", zipin.getNextEntry().getName()); assertNull(zipin.getNextEntry()); } @@ -70,9 +70,10 @@ public class Pack200StreamsTest { zipout.finish(); ByteArrayOutputStream pack200buffer = new ByteArrayOutputStream(); - Pack200.newPacker().pack( - new JarInputStream(new ByteArrayInputStream( - jarbuffer.toByteArray())), pack200buffer); + Pack200.newPacker() + .pack(new JarInputStream( + new ByteArrayInputStream(jarbuffer.toByteArray())), + pack200buffer); InputStream result = Pack200Streams.unpack(new NoCloseInputStream( new ByteArrayInputStream(pack200buffer.toByteArray()))); 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 7bdd7f38..3141a44e 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 @@ -116,8 +116,7 @@ public class ClassCoverageImplTest { private MethodCoverageImpl createMethod(boolean covered) { final MethodCoverageImpl m = new MethodCoverageImpl("sample", "()V", null); - m.increment( - covered ? CounterImpl.COUNTER_0_1 : CounterImpl.COUNTER_1_0, + m.increment(covered ? CounterImpl.COUNTER_0_1 : CounterImpl.COUNTER_1_0, CounterImpl.COUNTER_0_0, ISourceNode.UNKNOWN_LINE); m.incrementMethodCounter(); return m; diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/MethodCoverageImplTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/MethodCoverageImplTest.java index eec47b22..647f08d7 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/MethodCoverageImplTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/MethodCoverageImplTest.java @@ -60,8 +60,8 @@ public class MethodCoverageImplTest { @Test public void testIncrementCoveredInstructions() { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); - node.increment(CounterImpl.getInstance(12, 13), - CounterImpl.COUNTER_0_0, 3); + node.increment(CounterImpl.getInstance(12, 13), CounterImpl.COUNTER_0_0, + 3); node.incrementMethodCounter(); assertEquals(CounterImpl.COUNTER_0_1, node.getMethodCounter()); assertEquals(CounterImpl.COUNTER_0_1, node.getComplexityCounter()); @@ -79,7 +79,8 @@ public class MethodCoverageImplTest { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); node.increment(CounterImpl.COUNTER_0_0, CounterImpl.getInstance(2, 0), 3); - assertEquals(CounterImpl.getInstance(1, 0), node.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(1, 0), + node.getComplexityCounter()); } @Test @@ -87,7 +88,8 @@ public class MethodCoverageImplTest { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); node.increment(CounterImpl.COUNTER_0_0, CounterImpl.getInstance(1, 1), 3); - assertEquals(CounterImpl.getInstance(1, 0), node.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(1, 0), + node.getComplexityCounter()); } @Test @@ -95,7 +97,8 @@ public class MethodCoverageImplTest { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); node.increment(CounterImpl.COUNTER_0_0, CounterImpl.getInstance(0, 2), 3); - assertEquals(CounterImpl.getInstance(0, 1), node.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(0, 1), + node.getComplexityCounter()); } @Test @@ -103,7 +106,8 @@ public class MethodCoverageImplTest { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); node.increment(CounterImpl.COUNTER_0_0, CounterImpl.getInstance(3, 0), 3); - assertEquals(CounterImpl.getInstance(2, 0), node.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(2, 0), + node.getComplexityCounter()); } @Test @@ -111,7 +115,8 @@ public class MethodCoverageImplTest { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); node.increment(CounterImpl.COUNTER_0_0, CounterImpl.getInstance(2, 1), 3); - assertEquals(CounterImpl.getInstance(2, 0), node.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(2, 0), + node.getComplexityCounter()); } @Test @@ -119,7 +124,8 @@ public class MethodCoverageImplTest { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); node.increment(CounterImpl.COUNTER_0_0, CounterImpl.getInstance(1, 2), 3); - assertEquals(CounterImpl.getInstance(1, 1), node.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(1, 1), + node.getComplexityCounter()); } @Test @@ -127,6 +133,7 @@ public class MethodCoverageImplTest { MethodCoverageImpl node = new MethodCoverageImpl("sample", "()V", null); node.increment(CounterImpl.COUNTER_0_0, CounterImpl.getInstance(0, 3), 3); - assertEquals(CounterImpl.getInstance(0, 2), node.getComplexityCounter()); + assertEquals(CounterImpl.getInstance(0, 2), + node.getComplexityCounter()); } } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/PackageCoverageTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/PackageCoverageTest.java index 0b7a6fce..abaadb1b 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/PackageCoverageTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/PackageCoverageTest.java @@ -29,12 +29,12 @@ public class PackageCoverageTest { @Test public void testProperties() { - Collection<IClassCoverage> classes = Collections - .singleton((IClassCoverage) new ClassCoverageImpl( - "org/jacoco/test/Sample", 0, false)); - Collection<ISourceFileCoverage> sourceFiles = Collections - .singleton((ISourceFileCoverage) new SourceFileCoverageImpl( - "Sample.java", "org/jacoco/test/Sample")); + Collection<IClassCoverage> classes = Collections.singleton( + (IClassCoverage) new ClassCoverageImpl("org/jacoco/test/Sample", + 0, false)); + Collection<ISourceFileCoverage> sourceFiles = Collections.singleton( + (ISourceFileCoverage) new SourceFileCoverageImpl("Sample.java", + "org/jacoco/test/Sample")); PackageCoverageImpl data = new PackageCoverageImpl("org/jacoco/test", classes, sourceFiles); assertEquals(ICoverageNode.ElementType.PACKAGE, data.getElementType()); diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/SourceNodeImplTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/SourceNodeImplTest.java index 3bc50e7d..50e07d8b 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/SourceNodeImplTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/SourceNodeImplTest.java @@ -25,7 +25,8 @@ public class SourceNodeImplTest { @Test public void testInit() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); assertEquals(ElementType.CLASS, node.getElementType()); assertEquals("Foo", node.getName()); assertEquals(ISourceNode.UNKNOWN_LINE, node.getFirstLine()); @@ -35,7 +36,8 @@ public class SourceNodeImplTest { @Test public void testGetLine() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); node.ensureCapacity(10, 20); assertEquals(LineImpl.EMPTY, node.getLine(5)); assertEquals(LineImpl.EMPTY, node.getLine(15)); @@ -44,21 +46,24 @@ public class SourceNodeImplTest { @Test public void testEnsureCapacityUnknown1() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); node.ensureCapacity(10, ISourceNode.UNKNOWN_LINE); assertEquals(LineImpl.EMPTY, node.getLine(10)); } @Test public void testEnsureCapacityUnknown2() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); node.ensureCapacity(ISourceNode.UNKNOWN_LINE, 10); assertEquals(LineImpl.EMPTY, node.getLine(10)); } @Test public void testIncrementLineUnknown() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); node.increment(CounterImpl.getInstance(1, 2), CounterImpl.getInstance(3, 4), ISourceNode.UNKNOWN_LINE); assertEquals(CounterImpl.getInstance(1, 2), @@ -69,18 +74,19 @@ public class SourceNodeImplTest { @Test public void testIncrementLines() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); node.increment(CounterImpl.getInstance(1, 1), CounterImpl.COUNTER_0_0, 10); node.increment(CounterImpl.getInstance(2, 2), CounterImpl.COUNTER_0_0, 12); - assertEquals(CounterImpl.getInstance(1, 1), node.getLine(10) - .getInstructionCounter()); - assertEquals(CounterImpl.COUNTER_0_0, node.getLine(11) - .getInstructionCounter()); - assertEquals(CounterImpl.getInstance(2, 2), node.getLine(12) - .getInstructionCounter()); + assertEquals(CounterImpl.getInstance(1, 1), + node.getLine(10).getInstructionCounter()); + assertEquals(CounterImpl.COUNTER_0_0, + node.getLine(11).getInstructionCounter()); + assertEquals(CounterImpl.getInstance(2, 2), + node.getLine(12).getInstructionCounter()); } @Test @@ -165,20 +171,22 @@ public class SourceNodeImplTest { private void testIncrementLine(int mi1, int ci1, int mi2, int ci2, int expectedMissedLines, int expectedCoveredLines) { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); node.increment(CounterImpl.getInstance(mi1, ci1), CounterImpl.COUNTER_0_0, 33); node.increment(CounterImpl.getInstance(mi2, ci2), CounterImpl.COUNTER_0_0, 33); assertEquals(CounterImpl.getInstance(expectedMissedLines, expectedCoveredLines), node.getLineCounter()); - assertEquals(CounterImpl.getInstance(mi1 + mi2, ci1 + ci2), node - .getLine(33).getInstructionCounter()); + assertEquals(CounterImpl.getInstance(mi1 + mi2, ci1 + ci2), + node.getLine(33).getInstructionCounter()); } @Test public void testIncrementChildNoLines() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); final SourceNodeImpl child = new SourceNodeImpl(ElementType.CLASS, "Foo") { { @@ -198,7 +206,8 @@ public class SourceNodeImplTest { @Test public void testIncrementChildWithLines() { - final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, "Foo"); + final SourceNodeImpl node = new SourceNodeImpl(ElementType.CLASS, + "Foo"); final SourceNodeImpl child = new SourceNodeImpl(ElementType.CLASS, "Foo"); diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/EnumEmptyConstructorFilterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/EnumEmptyConstructorFilterTest.java index dd37b466..2c430010 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/EnumEmptyConstructorFilterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/EnumEmptyConstructorFilterTest.java @@ -39,7 +39,8 @@ public class EnumEmptyConstructorFilterTest extends FilterTestBase { filter.filter(m, context, output); - assertIgnored(new Range(m.instructions.getFirst(), m.instructions.getLast())); + assertIgnored( + new Range(m.instructions.getFirst(), m.instructions.getLast())); } /** diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/SynchronizedFilterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/SynchronizedFilterTest.java index b89b876e..c25eb4df 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/SynchronizedFilterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/SynchronizedFilterTest.java @@ -153,7 +153,7 @@ public class SynchronizedFilterTest extends FilterTestBase { filter.filter(m, context, output); assertIgnored(new Range((LabelNode) handler.info, - ((LabelNode) exit.info).getPrevious())); + ((LabelNode) exit.info).getPrevious())); } } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/TryWithResourcesJavac11FilterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/TryWithResourcesJavac11FilterTest.java index 94e5a009..1852c5cd 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/TryWithResourcesJavac11FilterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/analysis/filter/TryWithResourcesJavac11FilterTest.java @@ -50,8 +50,8 @@ public class TryWithResourcesJavac11FilterTest extends FilterTestBase { m.visitVarInsn(Opcodes.ALOAD, 0); range1.fromInclusive = m.instructions.getLast(); - m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", - "()V", false); + m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", "()V", + false); m.visitJumpInsn(Opcodes.GOTO, e); range1.toInclusive = m.instructions.getLast(); @@ -59,15 +59,15 @@ public class TryWithResourcesJavac11FilterTest extends FilterTestBase { range2.fromInclusive = m.instructions.getLast(); m.visitVarInsn(Opcodes.ASTORE, 1); m.visitVarInsn(Opcodes.ALOAD, 0); - m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", - "()V", false); + m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", "()V", + false); m.visitJumpInsn(Opcodes.GOTO, t); m.visitVarInsn(Opcodes.ASTORE, 2); m.visitVarInsn(Opcodes.ALOAD, 1); m.visitVarInsn(Opcodes.ALOAD, 2); m.visitMethodInsn(Opcodes.INVOKEVIRTUAL, "java/lang/Throwable", - "addSuppressed", "(Ljava/lang/Throwable;)V", false); + "addSuppressed", "(Ljava/lang/Throwable;)V", false); m.visitLabel(t); m.visitVarInsn(Opcodes.ALOAD, 1); m.visitInsn(Opcodes.ATHROW); @@ -104,8 +104,8 @@ public class TryWithResourcesJavac11FilterTest extends FilterTestBase { range1.fromInclusive = m.instructions.getLast(); m.visitJumpInsn(Opcodes.IFNULL, e); m.visitVarInsn(Opcodes.ALOAD, 0); - m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", - "()V", false); + m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", "()V", + false); m.visitJumpInsn(Opcodes.GOTO, e); range1.toInclusive = m.instructions.getLast(); @@ -115,8 +115,8 @@ public class TryWithResourcesJavac11FilterTest extends FilterTestBase { m.visitVarInsn(Opcodes.ALOAD, 0); m.visitJumpInsn(Opcodes.IFNULL, t); m.visitVarInsn(Opcodes.ALOAD, 0); - m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", - "()V", false); + m.visitMethodInsn(Opcodes.INVOKEINTERFACE, "Resource", "close", "()V", + false); m.visitJumpInsn(Opcodes.GOTO, t); m.visitVarInsn(Opcodes.ASTORE, 2); diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/data/CRC64Test.java b/org.jacoco.core.test/src/org/jacoco/core/internal/data/CRC64Test.java index ad0051b1..d554a06b 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/data/CRC64Test.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/data/CRC64Test.java @@ -77,9 +77,9 @@ public class CRC64Test { */ @Test public void test2() { - final long sum = CRC64.classId(new byte[] { (byte) 0xff, (byte) 0xff, - (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff, - (byte) 0xff, (byte) 0xff }); + final long sum = CRC64.classId( + new byte[] { (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff, + (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff }); assertEquals(0x5300000000000000L, sum); } @@ -90,8 +90,8 @@ public class CRC64Test { */ @Test public void test3() throws UnsupportedEncodingException { - final long sum = CRC64.classId("JACOCO_JACOCO_JACOCO_JACOCO" - .getBytes("ASCII")); + final long sum = CRC64 + .classId("JACOCO_JACOCO_JACOCO_JACOCO".getBytes("ASCII")); assertEquals(0xD8016B38AAD48308L, sum); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/flow/ClassProbesAdapterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/flow/ClassProbesAdapterTest.java index 739c91d2..2b1e2a93 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/flow/ClassProbesAdapterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/flow/ClassProbesAdapterTest.java @@ -32,14 +32,14 @@ public class ClassProbesAdapterTest { boolean frame = false; @Override - public void visitJumpInsnWithProbe(int opcode, Label label, - int probeId, IFrame frame) { + public void visitJumpInsnWithProbe(int opcode, Label label, int probeId, + IFrame frame) { frame.accept(this); } @Override - public void visitTableSwitchInsnWithProbes(int min, int max, - Label dflt, Label[] labels, IFrame frame) { + public void visitTableSwitchInsnWithProbes(int min, int max, Label dflt, + Label[] labels, IFrame frame) { frame.accept(this); } @@ -50,8 +50,8 @@ public class ClassProbesAdapterTest { } @Override - public void visitFrame(int type, int nLocal, Object[] local, - int nStack, Object[] stack) { + public void visitFrame(int type, int nLocal, Object[] local, int nStack, + Object[] stack) { frame = true; } } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/flow/LabelFlowAnalyzerTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/flow/LabelFlowAnalyzerTest.java index 0d279b7d..16e1af04 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/flow/LabelFlowAnalyzerTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/flow/LabelFlowAnalyzerTest.java @@ -128,8 +128,8 @@ public class LabelFlowAnalyzerTest { public void testFlowScenario11() { // Even if the same label is referenced multiple times but from the same // source instruction this is only counted as one target. - analyzer.visitLookupSwitchInsn(label, new int[] { 0, 1 }, new Label[] { - label, label }); + analyzer.visitLookupSwitchInsn(label, new int[] { 0, 1 }, + new Label[] { label, label }); assertFalse(LabelInfo.isMultiTarget(label)); assertFalse(LabelInfo.isSuccessor(label)); } @@ -138,7 +138,8 @@ public class LabelFlowAnalyzerTest { public void testFlowScenario12() { // Even if the same label is referenced multiple times but from the same // source instruction this is only counted as one target. - analyzer.visitTableSwitchInsn(0, 1, label, new Label[] { label, label }); + analyzer.visitTableSwitchInsn(0, 1, label, + new Label[] { label, label }); assertFalse(LabelInfo.isMultiTarget(label)); assertFalse(LabelInfo.isSuccessor(label)); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/flow/MethodProbesAdapterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/flow/MethodProbesAdapterTest.java index 288cb762..3a841105 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/flow/MethodProbesAdapterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/flow/MethodProbesAdapterTest.java @@ -64,16 +64,16 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { } @Override - public void visitJumpInsnWithProbe(int opcode, Label label, - int probeId, IFrame frame) { + public void visitJumpInsnWithProbe(int opcode, Label label, int probeId, + IFrame frame) { rec("visitJumpInsnWithProbe", Integer.valueOf(opcode), label, Integer.valueOf(probeId)); frame.accept(this); } @Override - public void visitTableSwitchInsnWithProbes(int min, int max, - Label dflt, Label[] labels, IFrame frame) { + public void visitTableSwitchInsnWithProbes(int min, int max, Label dflt, + Label[] labels, IFrame frame) { rec("visitTableSwitchInsnWithProbes", Integer.valueOf(min), Integer.valueOf(max), dflt, labels); frame.accept(this); @@ -171,10 +171,10 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { adapter.visitJumpInsn(Opcodes.GOTO, label); - expectedVisitor - .visitJumpInsnWithProbe(Opcodes.GOTO, label, 1000, frame); - expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, - 0, null); + expectedVisitor.visitJumpInsnWithProbe(Opcodes.GOTO, label, 1000, + frame); + expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, 0, + null); } @Test @@ -186,10 +186,10 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { adapter.visitJumpInsn(Opcodes.IFLT, label); expectedVisitor.visitInsn(Opcodes.ICONST_0); - expectedVisitor - .visitJumpInsnWithProbe(Opcodes.IFLT, label, 1000, frame); - expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, - 0, null); + expectedVisitor.visitJumpInsnWithProbe(Opcodes.IFLT, label, 1000, + frame); + expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, 0, + null); } @Test @@ -214,8 +214,8 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { expectedVisitor.visitInsn(Opcodes.ICONST_0); expectedVisitor.visitJumpInsnWithProbe(Opcodes.IF_ICMPEQ, label, 1000, frame); - expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, - 0, null); + expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, 0, + null); } @Test @@ -231,8 +231,8 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { expectedVisitor.visitInsn(Opcodes.ICONST_0); expectedVisitor.visitLookupSwitchInsnWithProbes(label, keys, labels, frame); - expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, - 0, null); + expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, 0, + null); assertEquals(1000, LabelInfo.getProbeId(label)); } @@ -250,8 +250,8 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { expectedVisitor.visitInsn(Opcodes.ICONST_0); expectedVisitor.visitLookupSwitchInsnWithProbes(label, keys, labels, frame); - expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, - 0, null); + expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, 0, + null); assertEquals(LabelInfo.NO_PROBE, LabelInfo.getProbeId(label)); assertEquals(1000, LabelInfo.getProbeId(label2)); } @@ -279,8 +279,8 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { expectedVisitor.visitInsn(Opcodes.ICONST_0); expectedVisitor.visitTableSwitchInsnWithProbes(0, 1, label, labels, frame); - expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, - 0, null); + expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, 0, + null); assertEquals(1000, LabelInfo.getProbeId(label)); } @@ -297,8 +297,8 @@ public class MethodProbesAdapterTest implements IProbeIdGenerator { expectedVisitor.visitInsn(Opcodes.ICONST_0); expectedVisitor.visitTableSwitchInsnWithProbes(0, 1, label, labels, frame); - expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, - 0, null); + expectedVisitor.visitFrame(Opcodes.F_FULL, 1, new Object[] { "Foo" }, 0, + null); assertEquals(LabelInfo.NO_PROBE, LabelInfo.getProbeId(label)); assertEquals(1000, LabelInfo.getProbeId(label2)); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ClassInstrumenterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ClassInstrumenterTest.java index 88c8d537..af6d01ba 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ClassInstrumenterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ClassInstrumenterTest.java @@ -28,9 +28,9 @@ public class ClassInstrumenterTest implements IProbeArrayStrategy { @Before public void setup() { - instrumenter = new ClassInstrumenter(this, new ClassVisitor( - InstrSupport.ASM_API_VERSION) { - }); + instrumenter = new ClassInstrumenter(this, + new ClassVisitor(InstrSupport.ASM_API_VERSION) { + }); } @Test(expected = IllegalStateException.class) @@ -49,14 +49,15 @@ public class ClassInstrumenterTest implements IProbeArrayStrategy { @Test public void testNoMethodVisitor() { - instrumenter = new ClassInstrumenter(this, new ClassVisitor( - InstrSupport.ASM_API_VERSION) { - @Override - public MethodVisitor visitMethod(int access, String name, - String desc, String signature, String[] exceptions) { - return null; - } - }); + instrumenter = new ClassInstrumenter(this, + new ClassVisitor(InstrSupport.ASM_API_VERSION) { + @Override + public MethodVisitor visitMethod(int access, String name, + String desc, String signature, + String[] exceptions) { + return null; + } + }); assertNull(instrumenter.visitMethod(0, "foo", "()V", null, null)); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/DuplicateFrameEliminatorTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/DuplicateFrameEliminatorTest.java index ffcd236c..5c998dbc 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/DuplicateFrameEliminatorTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/DuplicateFrameEliminatorTest.java @@ -83,14 +83,14 @@ public class DuplicateFrameEliminatorTest { @Test public void testTypeInsn() { - testInstructionBetweenFrames(new TypeInsnNode(Opcodes.NEW, - "java/lang/Object")); + testInstructionBetweenFrames( + new TypeInsnNode(Opcodes.NEW, "java/lang/Object")); } @Test public void testFieldInsn() { - testInstructionBetweenFrames(new FieldInsnNode(Opcodes.GETFIELD, "Foo", - "f", "I")); + testInstructionBetweenFrames( + new FieldInsnNode(Opcodes.GETFIELD, "Foo", "f", "I")); } @Test @@ -107,8 +107,8 @@ public class DuplicateFrameEliminatorTest { @Test public void testJumpInsn() { - testInstructionBetweenFrames(new JumpInsnNode(Opcodes.GOTO, - new LabelNode())); + testInstructionBetweenFrames( + new JumpInsnNode(Opcodes.GOTO, new LabelNode())); } @Test @@ -135,8 +135,8 @@ public class DuplicateFrameEliminatorTest { @Test public void testMultiANewArrayInsn() { - testInstructionBetweenFrames(new MultiANewArrayInsnNode( - "java/lang/String", 4)); + testInstructionBetweenFrames( + new MultiANewArrayInsnNode("java/lang/String", 4)); } private void testInstructionBetweenFrames(AbstractInsnNode node) { diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/MethodInstrumenterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/MethodInstrumenterTest.java index cd7a3580..5dfd9536 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/MethodInstrumenterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/MethodInstrumenterTest.java @@ -196,8 +196,8 @@ public class MethodInstrumenterTest { final Label L2 = new Label(); LabelInfo.setProbeId(L0, 0); LabelInfo.setProbeId(L1, 1); - instrumenter.visitTableSwitchInsnWithProbes(3, 5, L0, new Label[] { L1, - L1, L2 }, frame); + instrumenter.visitTableSwitchInsnWithProbes(3, 5, L0, + new Label[] { L1, L1, L2 }, frame); expectedVisitor.visitTableSwitchInsn(3, 4, L0, new Label[] { L1, L1, L2 }); diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ProbeInserterTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ProbeInserterTest.java index 4b07a1a3..acd54ef0 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ProbeInserterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/ProbeInserterTest.java @@ -42,7 +42,8 @@ public class ProbeInserterTest { expected = new MethodRecorder(); expectedVisitor = expected.getVisitor(); arrayStrategy = new IProbeArrayStrategy() { - public int storeInstance(MethodVisitor mv, boolean clinit, int variable) { + public int storeInstance(MethodVisitor mv, boolean clinit, + int variable) { mv.visitLdcInsn(clinit ? "clinit" : "init"); return 5; } @@ -234,11 +235,13 @@ public class ProbeInserterTest { ProbeInserter pi = new ProbeInserter(0, "m", "(J)V", actualVisitor, arrayStrategy); - pi.visitFrame(Opcodes.F_NEW, 3, new Object[] { "Foo", Opcodes.LONG, - "java/lang/String" }, 0, new Object[0]); + pi.visitFrame(Opcodes.F_NEW, 3, + new Object[] { "Foo", Opcodes.LONG, "java/lang/String" }, 0, + new Object[0]); - expectedVisitor.visitFrame(Opcodes.F_NEW, 4, new Object[] { "Foo", - Opcodes.LONG, "[Z", "java/lang/String" }, 0, new Object[0]); + expectedVisitor.visitFrame(Opcodes.F_NEW, 4, + new Object[] { "Foo", Opcodes.LONG, "[Z", "java/lang/String" }, + 0, new Object[0]); } @Test @@ -260,8 +263,8 @@ public class ProbeInserterTest { pi.visitFrame(Opcodes.F_NEW, 2, new Object[] { Opcodes.DOUBLE, "Foo" }, 0, new Object[0]); - expectedVisitor.visitFrame(Opcodes.F_NEW, 3, new Object[] { "[Z", - Opcodes.DOUBLE, "Foo" }, 0, new Object[0]); + expectedVisitor.visitFrame(Opcodes.F_NEW, 3, + new Object[] { "[Z", Opcodes.DOUBLE, "Foo" }, 0, new Object[0]); } @Test @@ -272,8 +275,8 @@ public class ProbeInserterTest { pi.visitFrame(Opcodes.F_NEW, 0, new Object[] {}, 0, new Object[] {}); // The locals in this frame are filled with TOP up to the probe variable - expectedVisitor.visitFrame(Opcodes.F_NEW, 2, new Object[] { - Opcodes.TOP, "[Z", }, 0, new Object[] {}); + expectedVisitor.visitFrame(Opcodes.F_NEW, 2, + new Object[] { Opcodes.TOP, "[Z", }, 0, new Object[] {}); } @Test @@ -284,8 +287,9 @@ public class ProbeInserterTest { pi.visitFrame(Opcodes.F_NEW, 0, new Object[] {}, 0, new Object[] {}); // The locals in this frame are filled with TOP up to the probe variable - expectedVisitor.visitFrame(Opcodes.F_NEW, 3, new Object[] { - Opcodes.TOP, Opcodes.TOP, "[Z", }, 0, new Object[] {}); + expectedVisitor.visitFrame(Opcodes.F_NEW, 3, + new Object[] { Opcodes.TOP, Opcodes.TOP, "[Z", }, 0, + new Object[] {}); } @Test @@ -297,9 +301,11 @@ public class ProbeInserterTest { new Object[] {}); // The locals in this frame are filled with TOP up to the probe variable - expectedVisitor.visitFrame(Opcodes.F_NEW, 5, new Object[] { - Opcodes.DOUBLE, Opcodes.TOP, Opcodes.TOP, Opcodes.TOP, "[Z", }, - 0, new Object[] {}); + expectedVisitor + .visitFrame( + Opcodes.F_NEW, 5, new Object[] { Opcodes.DOUBLE, + Opcodes.TOP, Opcodes.TOP, Opcodes.TOP, "[Z", }, + 0, new Object[] {}); } @Test(expected = IllegalArgumentException.class) diff --git a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/SignatureRemoverTest.java b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/SignatureRemoverTest.java index dbfa1160..9c63d210 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/internal/instr/SignatureRemoverTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/internal/instr/SignatureRemoverTest.java @@ -112,14 +112,12 @@ public class SignatureRemoverTest { + "Name: org/jacoco/example/B.class\r\n" // + "OtherInfo: keep this\r\n" // + "\r\n"; - assertEquals( - createManifest(expected.getBytes("ISO-8859-1")), + assertEquals(createManifest(expected.getBytes("ISO-8859-1")), createManifest(out.toByteArray())); } private static Manifest createManifest(final byte[] bytes) throws IOException { - return new Manifest( - new ByteArrayInputStream(bytes)); + return new Manifest(new ByteArrayInputStream(bytes)); } } diff --git a/org.jacoco.core.test/src/org/jacoco/core/runtime/AgentOptionsTest.java b/org.jacoco.core.test/src/org/jacoco/core/runtime/AgentOptionsTest.java index e220daed..3c01313b 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/runtime/AgentOptionsTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/runtime/AgentOptionsTest.java @@ -93,7 +93,8 @@ public class AgentOptionsTest { assertFalse(options.getAppend()); assertEquals("org.*:com.*", options.getIncludes()); assertEquals("*Test", options.getExcludes()); - assertEquals("org.jacoco.test.TestLoader", options.getExclClassloader()); + assertEquals("org.jacoco.test.TestLoader", + options.getExclClassloader()); assertTrue(options.getInclBootstrapClasses()); assertTrue(options.getInclNoLocationClasses()); assertEquals("testsession", options.getSessionId()); @@ -157,14 +158,16 @@ public class AgentOptionsTest { public void testGetExclClassloader() { AgentOptions options = new AgentOptions( "exclclassloader=org.jacoco.test.TestLoader"); - assertEquals("org.jacoco.test.TestLoader", options.getExclClassloader()); + assertEquals("org.jacoco.test.TestLoader", + options.getExclClassloader()); } @Test public void testSetExclClassloader() { AgentOptions options = new AgentOptions(); options.setExclClassloader("org.jacoco.test.TestLoader"); - assertEquals("org.jacoco.test.TestLoader", options.getExclClassloader()); + assertEquals("org.jacoco.test.TestLoader", + options.getExclClassloader()); assertEquals("exclclassloader=org.jacoco.test.TestLoader", options.toString()); } @@ -415,9 +418,8 @@ public class AgentOptionsTest { String vmArgument = options.getVMArgument(defaultAgentJarFile); - assertEquals( - String.format("-javaagent:%s=append=true", - defaultAgentJarFile.toString()), vmArgument); + assertEquals(String.format("-javaagent:%s=append=true", + defaultAgentJarFile.toString()), vmArgument); } @Test @@ -450,22 +452,21 @@ public class AgentOptionsTest { String vmArgument = options.prependVMArguments("a b c", defaultAgentJarFile); - assertEquals( - String.format("-javaagent:%s= a b c", - defaultAgentJarFile.toString()), vmArgument); + assertEquals(String.format("-javaagent:%s= a b c", + defaultAgentJarFile.toString()), vmArgument); } @Test public void testPrependVMArgumentsReplace() { AgentOptions options = new AgentOptions(); - String vmArgument = options.prependVMArguments(String.format( - "a b -javaagent:%s=append=false c", defaultAgentJarFile), + String vmArgument = options.prependVMArguments( + String.format("a b -javaagent:%s=append=false c", + defaultAgentJarFile), defaultAgentJarFile); - assertEquals( - String.format("-javaagent:%s= a b c", - defaultAgentJarFile.toString()), vmArgument); + assertEquals(String.format("-javaagent:%s= a b c", + defaultAgentJarFile.toString()), vmArgument); } @Test diff --git a/org.jacoco.core.test/src/org/jacoco/core/runtime/ModifiedSystemClassRuntimeTest.java b/org.jacoco.core.test/src/org/jacoco/core/runtime/ModifiedSystemClassRuntimeTest.java index 990eef3e..62586cf0 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/runtime/ModifiedSystemClassRuntimeTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/runtime/ModifiedSystemClassRuntimeTest.java @@ -57,8 +57,9 @@ public class ModifiedSystemClassRuntimeTest extends RuntimeTestBase { * "java.lang.reflect.Module" introduced in JDK 9. */ private Instrumentation newInstrumentationMock() { - return (Instrumentation) Proxy.newProxyInstance(getClass() - .getClassLoader(), new Class[] { Instrumentation.class }, + return (Instrumentation) Proxy.newProxyInstance( + getClass().getClassLoader(), + new Class[] { Instrumentation.class }, new MyInvocationHandler()); } @@ -75,8 +76,8 @@ public class ModifiedSystemClassRuntimeTest extends RuntimeTestBase { added = true; try { // Our class should get instrumented: - final byte[] data = TargetLoader - .getClassDataAsBytes(ModifiedSystemClassRuntimeTest.class); + final byte[] data = TargetLoader.getClassDataAsBytes( + ModifiedSystemClassRuntimeTest.class); verifyInstrumentedClass(TARGET_CLASS_NAME, transformer.transform((ClassLoader) null, TARGET_CLASS_NAME, null, null, data)); diff --git a/org.jacoco.core.test/src/org/jacoco/core/runtime/OfflineInstrumentationAccessGeneratorTest.java b/org.jacoco.core.test/src/org/jacoco/core/runtime/OfflineInstrumentationAccessGeneratorTest.java index 5c658fda..7c38c332 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/runtime/OfflineInstrumentationAccessGeneratorTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/runtime/OfflineInstrumentationAccessGeneratorTest.java @@ -100,13 +100,14 @@ public class OfflineInstrumentationAccessGeneratorTest { null); // Constructor - GeneratorAdapter gen = new GeneratorAdapter(writer.visitMethod( - Opcodes.ACC_PUBLIC, "<init>", "()V", null, new String[0]), + GeneratorAdapter gen = new GeneratorAdapter( + writer.visitMethod(Opcodes.ACC_PUBLIC, "<init>", "()V", null, + new String[0]), Opcodes.ACC_PUBLIC, "<init>", "()V"); gen.visitCode(); gen.loadThis(); - gen.invokeConstructor(Type.getType(Object.class), new Method("<init>", - "()V")); + gen.invokeConstructor(Type.getType(Object.class), + new Method("<init>", "()V")); gen.loadThis(); final int size = generator.generateDataAccessor(classid, className, 2, gen); @@ -117,8 +118,8 @@ public class OfflineInstrumentationAccessGeneratorTest { gen.visitEnd(); // get() - gen = new GeneratorAdapter(writer.visitMethod(Opcodes.ACC_PUBLIC, - "get", "()[Z", null, new String[0]), Opcodes.ACC_PUBLIC, "get", + gen = new GeneratorAdapter(writer.visitMethod(Opcodes.ACC_PUBLIC, "get", + "()[Z", null, new String[0]), Opcodes.ACC_PUBLIC, "get", "()[Z"); gen.visitCode(); gen.getStatic(classType, InstrSupport.DATAFIELD_NAME, @@ -130,8 +131,9 @@ public class OfflineInstrumentationAccessGeneratorTest { writer.visitEnd(); final TargetLoader loader = new TargetLoader(); - return (ITarget) loader.add(className.replace('/', '.'), - writer.toByteArray()).newInstance(); + return (ITarget) loader + .add(className.replace('/', '.'), writer.toByteArray()) + .newInstance(); } /** diff --git a/org.jacoco.core.test/src/org/jacoco/core/runtime/RemoteControlReaderWriterTest.java b/org.jacoco.core.test/src/org/jacoco/core/runtime/RemoteControlReaderWriterTest.java index ea993087..26eaf18e 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/runtime/RemoteControlReaderWriterTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/runtime/RemoteControlReaderWriterTest.java @@ -31,8 +31,8 @@ import org.junit.Test; * The tests don't care about the written binary format, they just verify * symmetry. */ -public class RemoteControlReaderWriterTest extends - ExecutionDataReaderWriterTest { +public class RemoteControlReaderWriterTest + extends ExecutionDataReaderWriterTest { private RemoteControlWriter writer; @@ -94,8 +94,8 @@ public class RemoteControlReaderWriterTest extends @Override protected RemoteControlReader createReader() throws IOException { - return new RemoteControlReader(new ByteArrayInputStream( - buffer.toByteArray())); + return new RemoteControlReader( + new ByteArrayInputStream(buffer.toByteArray())); } @Override diff --git a/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeDataTest.java b/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeDataTest.java index 0700167a..6b64a033 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeDataTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeDataTest.java @@ -139,8 +139,8 @@ public class RuntimeDataTest { writer.visitEnd(); final TargetLoader loader = new TargetLoader(); - Callable<?> callable = (Callable<?>) loader.add("Sample", - writer.toByteArray()).newInstance(); + Callable<?> callable = (Callable<?>) loader + .add("Sample", writer.toByteArray()).newInstance(); final Object[] args = (Object[]) callable.call(); assertEquals(3, args.length, 0.0); assertEquals(Long.valueOf(1000), args[0]); @@ -150,8 +150,8 @@ public class RuntimeDataTest { @Test public void testGenerateAccessCall() throws Exception { - final boolean[] probes = data.getExecutionData(Long.valueOf(1234), - "Sample", 5).getProbes(); + final boolean[] probes = data + .getExecutionData(Long.valueOf(1234), "Sample", 5).getProbes(); final ClassWriter writer = new ClassWriter(0); writer.visit(Opcodes.V1_5, Opcodes.ACC_PUBLIC, "Sample", null, diff --git a/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeTestBase.java b/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeTestBase.java index 9d177a4d..c6b259eb 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeTestBase.java +++ b/org.jacoco.core.test/src/org/jacoco/core/runtime/RuntimeTestBase.java @@ -55,8 +55,8 @@ public abstract class RuntimeTestBase { } @Test - public void testDataAccessor() throws InstantiationException, - IllegalAccessException { + public void testDataAccessor() + throws InstantiationException, IllegalAccessException { ITarget t = generateAndInstantiateClass(1234); data.collect(storage, storage, false); storage.assertData(1234, t.get()); @@ -65,18 +65,18 @@ public abstract class RuntimeTestBase { @Test public void testNoLocalVariablesInDataAccessor() throws InstantiationException, IllegalAccessException { - runtime.generateDataAccessor(1001, "Target", 5, new MethodVisitor( - InstrSupport.ASM_API_VERSION) { - @Override - public void visitVarInsn(int opcode, int var) { - fail("No usage of local variables allowed."); - } - }); + runtime.generateDataAccessor(1001, "Target", 5, + new MethodVisitor(InstrSupport.ASM_API_VERSION) { + @Override + public void visitVarInsn(int opcode, int var) { + fail("No usage of local variables allowed."); + } + }); } @Test - public void testExecutionRecording() throws InstantiationException, - IllegalAccessException { + public void testExecutionRecording() + throws InstantiationException, IllegalAccessException { generateAndInstantiateClass(1001).a(); data.collect(storage, storage, false); storage.assertSize(1); @@ -86,8 +86,8 @@ public abstract class RuntimeTestBase { } @Test - public void testLoadSameClassTwice() throws InstantiationException, - IllegalAccessException { + public void testLoadSameClassTwice() + throws InstantiationException, IllegalAccessException { generateAndInstantiateClass(1001).a(); generateAndInstantiateClass(1001).b(); data.collect(storage, storage, false); @@ -119,13 +119,14 @@ public abstract class RuntimeTestBase { null); // Constructor - GeneratorAdapter gen = new GeneratorAdapter(writer.visitMethod( - Opcodes.ACC_PUBLIC, "<init>", "()V", null, new String[0]), + GeneratorAdapter gen = new GeneratorAdapter( + writer.visitMethod(Opcodes.ACC_PUBLIC, "<init>", "()V", null, + new String[0]), Opcodes.ACC_PUBLIC, "<init>", "()V"); gen.visitCode(); gen.loadThis(); - gen.invokeConstructor(Type.getType(Object.class), new Method("<init>", - "()V")); + gen.invokeConstructor(Type.getType(Object.class), + new Method("<init>", "()V")); gen.loadThis(); final int size = runtime.generateDataAccessor(classid, className, 2, gen); @@ -136,8 +137,8 @@ public abstract class RuntimeTestBase { gen.visitEnd(); // get() - gen = new GeneratorAdapter(writer.visitMethod(Opcodes.ACC_PUBLIC, - "get", "()[Z", null, new String[0]), Opcodes.ACC_PUBLIC, "get", + gen = new GeneratorAdapter(writer.visitMethod(Opcodes.ACC_PUBLIC, "get", + "()[Z", null, new String[0]), Opcodes.ACC_PUBLIC, "get", "()[Z"); gen.visitCode(); gen.getStatic(classType, InstrSupport.DATAFIELD_NAME, @@ -175,8 +176,9 @@ public abstract class RuntimeTestBase { writer.visitEnd(); final TargetLoader loader = new TargetLoader(); - return (ITarget) loader.add(className.replace('/', '.'), - writer.toByteArray()).newInstance(); + return (ITarget) loader + .add(className.replace('/', '.'), writer.toByteArray()) + .newInstance(); } /** diff --git a/org.jacoco.core.test/src/org/jacoco/core/test/TargetLoader.java b/org.jacoco.core.test/src/org/jacoco/core/test/TargetLoader.java index abe546cc..1b482781 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/test/TargetLoader.java +++ b/org.jacoco.core.test/src/org/jacoco/core/test/TargetLoader.java @@ -62,7 +62,8 @@ public class TargetLoader extends ClassLoader { return readBytes(getClassData(loader, name)); } - public static byte[] getClassDataAsBytes(Class<?> clazz) throws IOException { + public static byte[] getClassDataAsBytes(Class<?> clazz) + throws IOException { return readBytes(getClassData(clazz)); } diff --git a/org.jacoco.core.test/src/org/jacoco/core/test/perf/InstrumentationTimeScenario.java b/org.jacoco.core.test/src/org/jacoco/core/test/perf/InstrumentationTimeScenario.java index 64bbba03..1956fe8e 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/test/perf/InstrumentationTimeScenario.java +++ b/org.jacoco.core.test/src/org/jacoco/core/test/perf/InstrumentationTimeScenario.java @@ -28,7 +28,8 @@ public class InstrumentationTimeScenario extends TimedScenario { private final int count; protected InstrumentationTimeScenario(Class<?> target, int count) { - super(String.format("instrumenting %s classes", Integer.valueOf(count))); + super(String.format("instrumenting %s classes", + Integer.valueOf(count))); this.target = target; this.count = count; } diff --git a/org.jacoco.core.test/src/org/jacoco/core/test/perf/PerfOutputWriter.java b/org.jacoco.core.test/src/org/jacoco/core/test/perf/PerfOutputWriter.java index c815c5b6..09b3a973 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/test/perf/PerfOutputWriter.java +++ b/org.jacoco.core.test/src/org/jacoco/core/test/perf/PerfOutputWriter.java @@ -37,8 +37,10 @@ public class PerfOutputWriter implements IPerfOutput { System.getProperty("java.vm.vendor")); writer.printf("JVM Version: %s%n%n", System.getProperty("java.vm.version")); - writer.println("scenario instr ref overhead"); - writer.println("----------------------------------------------------------"); + writer.println( + "scenario instr ref overhead"); + writer.println( + "----------------------------------------------------------"); } public void writeTimeResult(final String description, final long duration, diff --git a/org.jacoco.core.test/src/org/jacoco/core/test/validation/StatementExecutorTest.java b/org.jacoco.core.test/src/org/jacoco/core/test/validation/StatementExecutorTest.java index 3454a12f..7acea650 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/test/validation/StatementExecutorTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/test/validation/StatementExecutorTest.java @@ -42,8 +42,8 @@ public class StatementExecutorTest { @Test
public void should_prefix_arguments() {
- StatementExecutor executor = new StatementExecutor(this,
- "Hello", "world");
+ StatementExecutor executor = new StatementExecutor(this, "Hello",
+ "world");
executor.visitInvocation("ctx", "target1", "!");
diff --git a/org.jacoco.core.test/src/org/jacoco/core/test/validation/ValidationTestBase.java b/org.jacoco.core.test/src/org/jacoco/core/test/validation/ValidationTestBase.java index 0da9d6ac..ca40d87b 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/test/validation/ValidationTestBase.java +++ b/org.jacoco.core.test/src/org/jacoco/core/test/validation/ValidationTestBase.java @@ -132,7 +132,8 @@ public abstract class ValidationTestBase { } assertEquals( "sum of missed instructions of all lines should be equal to missed instructions of file", - source.getCoverage().getInstructionCounter().getMissedCount(), c.getMissedCount()); + source.getCoverage().getInstructionCounter().getMissedCount(), + c.getMissedCount()); } @Test diff --git a/org.jacoco.core.test/src/org/jacoco/core/tools/ExecFileLoaderTest.java b/org.jacoco.core.test/src/org/jacoco/core/tools/ExecFileLoaderTest.java index ddc5310c..9233c13f 100644 --- a/org.jacoco.core.test/src/org/jacoco/core/tools/ExecFileLoaderTest.java +++ b/org.jacoco.core.test/src/org/jacoco/core/tools/ExecFileLoaderTest.java @@ -119,8 +119,8 @@ public class ExecFileLoaderTest { final FileOutputStream out = new FileOutputStream(file); final ExecutionDataWriter writer = new ExecutionDataWriter(out); final int value = id.length(); - writer.visitClassExecution(new ExecutionData(value, id, - new boolean[] { true })); + writer.visitClassExecution( + new ExecutionData(value, id, new boolean[] { true })); writer.visitSessionInfo(new SessionInfo(id, value, value)); out.close(); return file; |