#6 Fixed overwriting of packageNames

Author: jonashackt

src/main/java/de/codecentric/cxf/BootCxfMojo.java

@@ -280,17 +280,17 @@ protected String generatePackageNameFromTargetNamespaceInWsdl(String targetNames
     }
 
     protected void writeSeiAndWebServiceClientPackageToCxfSpringBootMavenPropterties(String outputDirectory, String packageName) throws MojoExecutionException {
-        writeCxfSpringBootMavenProperties(outputDirectory, packageName, SEI_AND_WEB_SERVICE_CLIENT_PACKAGE_NAME_KEY);
+        writeCxfSpringBootMavenProperties(outputDirectory, SEI_AND_WEB_SERVICE_CLIENT_PACKAGE_NAME_KEY, packageName);
     }
 
     protected void writeSeiImplementationPackageToCxfSpringBootMavenPropterties(String outputDirectory, String packageName) throws MojoExecutionException {
-        writeCxfSpringBootMavenProperties(outputDirectory, packageName, SEI_IMPLEMENTATION_PACKAGE_NAME_KEY);
+        writeCxfSpringBootMavenProperties(outputDirectory, SEI_IMPLEMENTATION_PACKAGE_NAME_KEY, packageName);
     }
 
-    protected void writeCxfSpringBootMavenProperties(String outputDirectory, String packageName, String propertyKey) throws MojoExecutionException {
+    protected void writeCxfSpringBootMavenProperties(String outputDirectory, String propertyKey, String packageName) throws MojoExecutionException {
         try {
             File cxfSpringBootMavenProperties = new File(outputDirectory + "/" + CXF_SPRING_BOOT_MAVEN_PROPERTIES_FILE_NAME);
-            FileUtils.writeStringToFile(cxfSpringBootMavenProperties, propertyKey + "=" + packageName, Charset.defaultCharset());
+            FileUtils.writeStringToFile(cxfSpringBootMavenProperties, propertyKey + "=" + packageName + "\n", Charset.defaultCharset(), true);
 
         } catch (IOException ioExc) {
             throw new MojoExecutionException("Could not inject packageName into " + CXF_SPRING_BOOT_MAVEN_PROPERTIES_FILE_NAME + "." +

src/test/java/de/codecentric/cxf/BootCxfMojoTest.java

@@ -57,7 +57,7 @@ public class BootCxfMojoTest {
     @Test public void
     does_write_cxfSpringBootMavenProperties() throws MojoExecutionException, IOException {
 
-        bootCxfMojo.writeCxfSpringBootMavenProperties(buildDirectory, "foo.bar", "foo.key");
+        bootCxfMojo.writeCxfSpringBootMavenProperties(buildDirectory, "foo.key", "foo.bar");
 
         File cxfSpringBootMavenProperties = findCxfSpringBootMavenPropertiesInClasspath();
         assertThat(cxfSpringBootMavenProperties.getName(), is(equalTo(BootCxfMojo.CXF_SPRING_BOOT_MAVEN_PROPERTIES_FILE_NAME)));
@@ -87,6 +87,16 @@ public class BootCxfMojoTest {
         assertThat(content, containsString(BootCxfMojo.SEI_IMPLEMENTATION_PACKAGE_NAME_KEY + "=" + seiImplementationPackageName));
     }
 
+    @Test public void
+    does_not_overwrite_packages_in_cxfSpringBootMavenProperties_file() throws MojoExecutionException, IOException {
+        bootCxfMojo.writeCxfSpringBootMavenProperties(buildDirectory, "foo.key", "foo.bar");
+        bootCxfMojo.writeCxfSpringBootMavenProperties(buildDirectory, "bar.key", "bar.boar");
+
+        String content = FileUtils.readFileToString(findCxfSpringBootMavenPropertiesInClasspath(), Charset.defaultCharset());
+        assertThat(content, containsString("foo.key" + "=" + "foo.bar"));
+        assertThat(content, containsString("bar.key" + "=" + "bar.boar"));
+    }
+
     private File findCxfSpringBootMavenPropertiesInClasspath() throws IOException {
         return findInClasspath("classpath*:**/" + BootCxfMojo.CXF_SPRING_BOOT_MAVEN_PROPERTIES_FILE_NAME).getFile();
     }