diff --git a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/AbstractGeneratorTestCase.java b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/AbstractGeneratorTestCase.java index b3b1d41..a2020f4 100644 --- a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/AbstractGeneratorTestCase.java +++ b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/AbstractGeneratorTestCase.java @@ -23,6 +23,7 @@ import junit.framework.TestCase; import org.apache.maven.plugin.descriptor.MojoDescriptor; import org.apache.maven.plugin.descriptor.Parameter; import org.apache.maven.plugin.descriptor.PluginDescriptor; +import org.apache.maven.tools.plugin.DefaultPluginToolsRequest; import org.codehaus.plexus.component.repository.ComponentDependency; import org.codehaus.plexus.util.FileUtils; @@ -91,7 +92,7 @@ public abstract class AbstractGeneratorTestCase FileUtils.deleteDirectory( destinationDirectory ); destinationDirectory.mkdir(); - generator.execute( destinationDirectory, pluginDescriptor ); + generator.execute( destinationDirectory, new DefaultPluginToolsRequest( null, pluginDescriptor ) ); validate( destinationDirectory ); diff --git a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/PluginHelpGeneratorTest.java b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/PluginHelpGeneratorTest.java index 8c0eca7..78565f2 100644 --- a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/PluginHelpGeneratorTest.java +++ b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/generator/PluginHelpGeneratorTest.java @@ -26,34 +26,5 @@ package org.apache.maven.tools.plugin.generator; public class PluginHelpGeneratorTest extends AbstractGeneratorTestCase { - public void testToText() - throws Exception - { - String javadoc = null; - assertEquals( "", PluginHelpGenerator.toText( javadoc ) ); - javadoc = ""; - assertEquals( "", PluginHelpGenerator.toText( javadoc ) ); - - // line breaks - javadoc = "Line1\nLine2"; - assertEquals( "Line1 Line2", PluginHelpGenerator.toText( javadoc ) ); - javadoc = "Line1\rLine2"; - assertEquals( "Line1 Line2", PluginHelpGenerator.toText( javadoc ) ); - javadoc = "Line1\r\nLine2"; - assertEquals( "Line1 Line2", PluginHelpGenerator.toText( javadoc ) ); - javadoc = "Line1
Line2"; - assertEquals( "Line1\nLine2", PluginHelpGenerator.toText( javadoc ) ); - - // true HTML - javadoc = "Generates something for the project."; - assertEquals( "Generates something for the project.", PluginHelpGenerator.toText( javadoc ) ); - - // wrong HTML - javadoc = "Generates something for the project."; - assertEquals( "Generates something for the project.", PluginHelpGenerator.toText( javadoc ) ); - - // javadoc inline tags - javadoc = "Generates {@code something} for the project."; - assertEquals( "Generates something for the project.", PluginHelpGenerator.toText( javadoc ) ); - } + // inherits tests from base class } diff --git a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/scanner/DefaultMojoScannerTest.java b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/scanner/DefaultMojoScannerTest.java index e5ee239..af192a5 100644 --- a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/scanner/DefaultMojoScannerTest.java +++ b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/scanner/DefaultMojoScannerTest.java @@ -26,6 +26,7 @@ import org.apache.maven.plugin.descriptor.InvalidPluginDescriptorException; import org.apache.maven.plugin.descriptor.MojoDescriptor; import org.apache.maven.plugin.descriptor.PluginDescriptor; import org.apache.maven.project.MavenProject; +import org.apache.maven.tools.plugin.DefaultPluginToolsRequest; import org.apache.maven.tools.plugin.extractor.ExtractionException; import java.io.File; @@ -80,7 +81,7 @@ public class DefaultMojoScannerTest { PluginDescriptor pluginDescriptor = createPluginDescriptor(); - scanner.populatePluginDescriptor( project, pluginDescriptor ); + scanner.populatePluginDescriptor( new DefaultPluginToolsRequest( project, pluginDescriptor ) ); checkResult( pluginDescriptor, extractors.keySet() ); } @@ -97,7 +98,7 @@ public class DefaultMojoScannerTest PluginDescriptor pluginDescriptor = createPluginDescriptor(); scanner.setActiveExtractors( activeExtractors ); - scanner.populatePluginDescriptor( project, pluginDescriptor ); + scanner.populatePluginDescriptor( new DefaultPluginToolsRequest( project, pluginDescriptor ) ); checkResult( pluginDescriptor, Arrays.asList( new String[]{"one", "three"} ) ); } @@ -108,7 +109,7 @@ public class DefaultMojoScannerTest PluginDescriptor pluginDescriptor = createPluginDescriptor(); scanner.setActiveExtractors( null ); - scanner.populatePluginDescriptor( project, pluginDescriptor ); + scanner.populatePluginDescriptor( new DefaultPluginToolsRequest( project, pluginDescriptor ) ); checkResult( pluginDescriptor, extractors.keySet() ); } @@ -121,7 +122,7 @@ public class DefaultMojoScannerTest scanner.setActiveExtractors( Collections.EMPTY_SET ); try { - scanner.populatePluginDescriptor( project, pluginDescriptor ); + scanner.populatePluginDescriptor( new DefaultPluginToolsRequest( project, pluginDescriptor ) ); fail( "Expected exception" ); } catch (InvalidPluginDescriptorException e) @@ -144,7 +145,7 @@ public class DefaultMojoScannerTest try { - scanner.populatePluginDescriptor( project, pluginDescriptor ); + scanner.populatePluginDescriptor( new DefaultPluginToolsRequest( project, pluginDescriptor ) ); fail( "No error for unknown extractor" ); } catch ( ExtractionException e ) diff --git a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/util/PluginUtilsTest.java b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/util/PluginUtilsTest.java index ab6285a..d895439 100644 --- a/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/util/PluginUtilsTest.java +++ b/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/util/PluginUtilsTest.java @@ -205,4 +205,35 @@ public class PluginUtilsTest assertEquals( "label", PluginUtils.decodeJavadocTags( javadoc ) ); } + public void testToText() + throws Exception + { + String javadoc = null; + assertEquals( "", PluginUtils.toText( javadoc ) ); + javadoc = ""; + assertEquals( "", PluginUtils.toText( javadoc ) ); + + // line breaks + javadoc = "Line1\nLine2"; + assertEquals( "Line1 Line2", PluginUtils.toText( javadoc ) ); + javadoc = "Line1\rLine2"; + assertEquals( "Line1 Line2", PluginUtils.toText( javadoc ) ); + javadoc = "Line1\r\nLine2"; + assertEquals( "Line1 Line2", PluginUtils.toText( javadoc ) ); + javadoc = "Line1
Line2"; + assertEquals( "Line1\nLine2", PluginUtils.toText( javadoc ) ); + + // true HTML + javadoc = "Generates something for the project."; + assertEquals( "Generates something for the project.", PluginUtils.toText( javadoc ) ); + + // wrong HTML + javadoc = "Generates something for the project."; + assertEquals( "Generates something for the project.", PluginUtils.toText( javadoc ) ); + + // javadoc inline tags + javadoc = "Generates {@code something} for the project."; + assertEquals( "Generates something for the project.", PluginUtils.toText( javadoc ) ); + } + }