Copilot commented on code in PR #3400:
URL: https://github.com/apache/maven-surefire/pull/3400#discussion_r3646202469
##########
surefire-booter/src/test/java/org/apache/maven/surefire/booter/spi/CommandChannelDecoderTest.java:
##########
@@ -63,12 +61,13 @@
public class CommandChannelDecoderTest {
private static final Random RND = new Random();
- @TempDir
- Path tempFolder;
-
@BeforeEach
public void initTmpFile() {
- File reportsDir = tempFolder.toFile();
+ // Not a JUnit @TempDir: the decoder writes .dumpstream files into
this directory,
Review Comment:
Minor naming: initTmpFile() no longer initializes a temp file/temp dir (it
now configures DumpErrorSingleton to write into target/decoder-dumps). Renaming
the method would better reflect its purpose and avoid confusion for future
maintainers.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
