Skip to content

Commit ef391f6

Browse files
author
Vincent Potucek
committed
Fix unused stream in DefaultPluginXmlFactory#write
1 parent 324e8fb commit ef391f6

File tree

2 files changed

+48
-1
lines changed

2 files changed

+48
-1
lines changed

impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultPluginXmlFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public void write(XmlWriterRequest<PluginDescriptor> request) throws XmlWriterEx
9393
new PluginDescriptorStaxWriter().write(outputStream, content);
9494
} else {
9595
try (OutputStream os = Files.newOutputStream(path)) {
96-
new PluginDescriptorStaxWriter().write(outputStream, content);
96+
new PluginDescriptorStaxWriter().write(os, content);
9797
}
9898
}
9999
} catch (Exception e) {
Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
/*
2+
* Licensed to the Apache Software Foundation (ASF) under one
3+
* or more contributor license agreements. See the NOTICE file
4+
* distributed with this work for additional information
5+
* regarding copyright ownership. The ASF licenses this file
6+
* to you under the Apache License, Version 2.0 (the
7+
* "License"); you may not use this file except in compliance
8+
* with the License. You may obtain a copy of the License at
9+
*
10+
* http://www.apache.org/licenses/LICENSE-2.0
11+
*
12+
* Unless required by applicable law or agreed to in writing,
13+
* software distributed under the License is distributed on an
14+
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15+
* KIND, either express or implied. See the License for the
16+
* specific language governing permissions and limitations
17+
* under the License.
18+
*/
19+
package org.apache.maven.impl;
20+
21+
import java.nio.file.Files;
22+
import java.nio.file.Path;
23+
24+
import org.apache.maven.api.plugin.descriptor.PluginDescriptor;
25+
import org.apache.maven.api.services.xml.XmlWriterRequest;
26+
import org.junit.jupiter.api.Test;
27+
import org.junit.jupiter.api.io.TempDir;
28+
29+
import static org.assertj.core.api.Assertions.assertThat;
30+
31+
class DefaultPluginXmlFactoryReadWriteTest {
32+
33+
private final DefaultPluginXmlFactory defaultPluginXmlFactory = new DefaultPluginXmlFactory();
34+
35+
@TempDir
36+
Path tempDir;
37+
38+
@Test
39+
void writeToPathGeneratesValidXmlFile() throws Exception {
40+
Path xmlFile = tempDir.resolve("output-plugin.xml");
41+
defaultPluginXmlFactory.write(XmlWriterRequest.<PluginDescriptor>builder()
42+
.path(xmlFile)
43+
.content(PluginDescriptor.newBuilder().name("Sample Plugin").build())
44+
.build());
45+
assertThat(Files.readString(xmlFile)).contains("<name>Sample Plugin</name>");
46+
}
47+
}

0 commit comments

Comments
 (0)