TS-41980 Handle nested jars for converter

cqse · Feb 25, 2025 · 84a5108 · 84a5108
1 parent 7e96a77
commit 84a5108
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/agent/src/main/java/com/teamscale/jacoco/agent/convert/Converter.java b/agent/src/main/java/com/teamscale/jacoco/agent/convert/Converter.java
@@ -60,7 +60,7 @@ public void runJaCoCoReportGeneration() throws IOException, AgentOptionParseExce
 		try (Benchmark benchmark = new Benchmark("Generating the XML report")) {
 			generator.convert(new Dump(sessionInfo, executionDataStore), Paths.get(arguments.outputFile).toFile());
 		} catch (EmptyReportException e) {
-			logger.warn("Converted report was emtpy.", e);
+			logger.warn("Converted report was empty.", e);
 		}
 	}
 

diff --git a/report-generator/src/main/kotlin/com/teamscale/report/jacoco/FilteringAnalyzer.kt b/report-generator/src/main/kotlin/com/teamscale/report/jacoco/FilteringAnalyzer.kt
@@ -69,11 +69,10 @@ open class FilteringAnalyzer(
 	 */
 	@Throws(IOException::class)
 	protected open fun analyzeJar(input: InputStream, location: String): Int {
-		ZipInputStream(BashFileSkippingInputStream(input)).use { zip ->
-			return generateSequence { zip.nextEntry(location) }
+		val zip = ZipInputStream(BashFileSkippingInputStream(input))
+		return generateSequence { zip.nextEntry(location) }
 				.map { entry -> analyzeAll(zip, "$location@${entry.name}") }
 				.sum()
-		}
 	}
 
 	/** Copied from [org.jacoco.core.analysis.Analyzer.nextEntry].  */

diff --git a/report-generator/src/main/kotlin/com/teamscale/report/util/BashFileSkippingInputStream.kt b/report-generator/src/main/kotlin/com/teamscale/report/util/BashFileSkippingInputStream.kt
@@ -46,8 +46,6 @@ class BashFileSkippingInputStream(input: InputStream) : FilterInputStream(Buffer
 			`in`.mark(BUFFER_SIZE)
 			bytesRead = `in`.read(buffer, 0, BUFFER_SIZE)
 		}
-
-		throw IOException("ZIP header not found in the input stream.")
 	}
 
 	/**
-Original file line number
+Diff line change
@@ Expand Up @@
     			`in`.mark(BUFFER_SIZE)
     			bytesRead = `in`.read(buffer, 0, BUFFER_SIZE)
     		}
-    		throw IOException("ZIP header not found in the input stream.")
     	}
     	/**
@@ Expand Down @@