Polish "Allow adding <extensions>true</extensions> to plugin config"

See gh-921
This commit is contained in:
Andy Wilkinson
2019-06-10 14:12:54 +01:00
parent 3e86a446a8
commit 50160d4931
4 changed files with 6 additions and 6 deletions

View File

@@ -271,8 +271,8 @@ public class MavenBuildWriter {
writeSingleElement(writer, "groupId", plugin.getGroupId()); writeSingleElement(writer, "groupId", plugin.getGroupId());
writeSingleElement(writer, "artifactId", plugin.getArtifactId()); writeSingleElement(writer, "artifactId", plugin.getArtifactId());
writeSingleElement(writer, "version", plugin.getVersion()); writeSingleElement(writer, "version", plugin.getVersion());
if (plugin.shouldLoadExtensions()) { if (plugin.isExtensions()) {
writeSingleElement(writer, "extensions", Boolean.toString(true)); writeSingleElement(writer, "extensions", "true");
} }
writePluginConfiguration(writer, plugin.getConfiguration()); writePluginConfiguration(writer, plugin.getConfiguration());
if (!plugin.getExecutions().isEmpty()) { if (!plugin.getExecutions().isEmpty()) {

View File

@@ -102,7 +102,7 @@ public class MavenPlugin {
: this.configurationCustomization.build(); : this.configurationCustomization.build();
} }
public boolean shouldLoadExtensions() { public boolean isExtensions() {
return this.extensions; return this.extensions;
} }

View File

@@ -102,7 +102,7 @@ class MavenBuildTests {
MavenPlugin testPlugin = build.getPlugins().get(0); MavenPlugin testPlugin = build.getPlugins().get(0);
assertThat(testPlugin.shouldLoadExtensions()).isFalse(); assertThat(testPlugin.isExtensions()).isFalse();
} }
@Test @Test
@@ -112,7 +112,7 @@ class MavenBuildTests {
MavenPlugin testPlugin = build.getPlugins().get(0); MavenPlugin testPlugin = build.getPlugins().get(0);
assertThat(testPlugin.shouldLoadExtensions()).isTrue(); assertThat(testPlugin.isExtensions()).isTrue();
} }
} }

View File

@@ -473,7 +473,7 @@ class MavenBuildWriterTests {
NodeAssert plugin = pom.nodeAtPath("/project/build/plugins/plugin"); NodeAssert plugin = pom.nodeAtPath("/project/build/plugins/plugin");
assertThat(plugin).textAtPath("groupId").isEqualTo("com.example.demo"); assertThat(plugin).textAtPath("groupId").isEqualTo("com.example.demo");
assertThat(plugin).textAtPath("artifactId").isEqualTo("demo-plugin"); assertThat(plugin).textAtPath("artifactId").isEqualTo("demo-plugin");
assertThat(plugin).textAtPath("extensions").isEqualTo(Boolean.toString(true)); assertThat(plugin).textAtPath("extensions").isEqualTo("true");
}); });
} }