perry2of5 commented on a change in pull request #491:
URL: https://github.com/apache/logging-log4j2/pull/491#discussion_r628334915
##########
File path:
log4j-layout-template-json/src/test/java/org/apache/logging/log4j/layout/template/json/JsonTemplateLayoutNullEventDelimiterTest.java
##########
@@ -71,13 +71,24 @@ public void test() throws Exception {
// Verify the received log events.
final byte[] actualBytes = server.getReceivedBytes();
Assertions.assertThat(actualBytes).startsWith(expectedBytes);
-
- } catch (BindException bindException) {
- System.err.println("Required port " + PORT + " not available,
skipping " + getClass().getSimpleName());
+ } finally {
+ tempConfig.delete();
}
}
+ public void makeTempConfig(final int serverPort, final File
tempConfigFile) throws IOException {
+ try (FileOutputStream fos = new FileOutputStream(tempConfigFile);
+ OutputStreamWriter osw = new OutputStreamWriter(fos,
StandardCharsets.UTF_8);
+ BufferedWriter bw = new BufferedWriter(osw);) {
+ for (String line : Files.readAllLines(
Review comment:
Same as above; I thought I already made it as commit
d70da9e7214d7baa3643363b849e5ec124b7b748. I see that commit in my branch, did
it not show up here? I'm still figuring out git so any advice on how I should
have made this work.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
