You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by vs...@apache.org on 2008/02/21 00:07:02 UTC

svn commit: r629649 - in /maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing: ./ stubs/

Author: vsiveton
Date: Wed Feb 20 15:07:00 2008
New Revision: 629649

URL: http://svn.apache.org/viewvc?rev=629649&view=rev
Log:
o fixed checkstyle
o updated some javadoc

Modified:
    maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/AbstractMojoTestCase.java
    maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/ArtifactStubFactory.java
    maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/ArtifactStub.java
    maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/DefaultArtifactHandlerStub.java
    maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/MavenProjectStub.java
    maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactRepository.java
    maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactResolver.java

Modified: maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/AbstractMojoTestCase.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/AbstractMojoTestCase.java?rev=629649&r1=629648&r2=629649&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/AbstractMojoTestCase.java (original)
+++ maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/AbstractMojoTestCase.java Wed Feb 20 15:07:00 2008
@@ -188,7 +188,8 @@
         if ( pluginConfiguration != null )
         {
             /* requires v10 of plexus container for lookup on expression evaluator
-             ExpressionEvaluator evaluator = (ExpressionEvaluator) getContainer().lookup( ExpressionEvaluator.ROLE, "stub-evaluator" );
+             ExpressionEvaluator evaluator = (ExpressionEvaluator) getContainer().lookup( ExpressionEvaluator.ROLE,
+                                                                                         "stub-evaluator" );
              */
             ExpressionEvaluator evaluator = new ResolverExpressionEvaluatorStub();
 

Modified: maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/ArtifactStubFactory.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/ArtifactStubFactory.java?rev=629649&r1=629648&r2=629649&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/ArtifactStubFactory.java (original)
+++ maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/ArtifactStubFactory.java Wed Feb 20 15:07:00 2008
@@ -51,15 +51,15 @@
  */
 public class ArtifactStubFactory
 {
-    File workingDir;
+    private File workingDir;
 
-    boolean createFiles;
+    private boolean createFiles;
 
-    File srcFile;
+    private File srcFile;
 
-    boolean createUnpackableFile;
+    private boolean createUnpackableFile;
 
-    ArchiverManager archiverManager;
+    private ArchiverManager archiverManager;
 
     /**
      * Default constructor. This should be used only if real files aren't needed...just the artifact objects
@@ -167,7 +167,7 @@
 
         if ( createFiles )
         {
-            setArtifactFile( artifact,this.workingDir,this.srcFile,this.createUnpackableFile );
+            setArtifactFile( artifact, this.workingDir, this.srcFile, this.createUnpackableFile );
         }
         return artifact;
     }
@@ -282,10 +282,10 @@
      * @param artifact
      * @return
      */
-    static public String getUnpackableFileName( Artifact artifact )
+    public static String getUnpackableFileName( Artifact artifact )
     {
-        return "" + artifact.getGroupId() + "-" + artifact.getArtifactId() + "-" + artifact.getVersion() + "-" +
-            artifact.getClassifier() + "-" + artifact.getType() + ".txt";
+        return "" + artifact.getGroupId() + "-" + artifact.getArtifactId() + "-" + artifact.getVersion() + "-"
+            + artifact.getClassifier() + "-" + artifact.getType() + ".txt";
     }
 
     /**
@@ -442,7 +442,8 @@
 
     /**
      * @return a set of <code>DefaultArtifact</code>, i.e.:
-     * <code>one:group-one:jar:a:1.0, three:group-three:jar:a:1.0, four:group-four:jar:a:1.0, two:group-two:jar:a:1.0</code>
+     * <code>one:group-one:jar:a:1.0, three:group-three:jar:a:1.0, four:group-four:jar:a:1.0,
+     * two:group-two:jar:a:1.0</code>
      * @throws IOException if any
      */
     public Set getGroupIdArtifacts()
@@ -577,9 +578,8 @@
                 classifierString = "-" + artifact.getClassifier();
             }
 
-            destFileName =
-                artifact.getArtifactId() + versionString + classifierString + "." +
-                    artifact.getArtifactHandler().getExtension();
+            destFileName = artifact.getArtifactId() + versionString + classifierString + "."
+                + artifact.getArtifactHandler().getExtension();
         }
         return destFileName;
     }

Modified: maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/ArtifactStub.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/ArtifactStub.java?rev=629649&r1=629648&r2=629649&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/ArtifactStub.java (original)
+++ maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/ArtifactStub.java Wed Feb 20 15:07:00 2008
@@ -341,8 +341,8 @@
      */
     public boolean isSnapshot()
     {
-        return Artifact.VERSION_FILE_PATTERN.matcher( version ).matches() ||
-        version.endsWith( Artifact.SNAPSHOT_VERSION );
+        return Artifact.VERSION_FILE_PATTERN.matcher( version ).matches()
+            || version.endsWith( Artifact.SNAPSHOT_VERSION );
     }
 
     /**

Modified: maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/DefaultArtifactHandlerStub.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/DefaultArtifactHandlerStub.java?rev=629649&r1=629648&r2=629649&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/DefaultArtifactHandlerStub.java (original)
+++ maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/DefaultArtifactHandlerStub.java Wed Feb 20 15:07:00 2008
@@ -62,7 +62,7 @@
     }
 
     /**
-     * @param t the artifact handler type
+     * @param type the artifact handler type
      */
     public DefaultArtifactHandlerStub( String type )
     {
@@ -137,8 +137,7 @@
     }
 
     /**
-     * @param theAddedToClasspath
-     *            The addedToClasspath to set.
+     * @param theAddedToClasspath The addedToClasspath to set.
      */
     public void setAddedToClasspath( boolean theAddedToClasspath )
     {
@@ -146,8 +145,7 @@
     }
 
     /**
-     * @param theClassifier
-     *            The classifier to set.
+     * @param theClassifier The classifier to set.
      */
     public void setClassifier( String theClassifier )
     {
@@ -155,8 +153,7 @@
     }
 
     /**
-     * @param theDirectory
-     *            The directory to set.
+     * @param theDirectory The directory to set.
      */
     public void setDirectory( String theDirectory )
     {
@@ -164,8 +161,7 @@
     }
 
     /**
-     * @param theExtension
-     *            The extension to set.
+     * @param theExtension The extension to set.
      */
     public void setExtension( String theExtension )
     {
@@ -173,8 +169,7 @@
     }
 
     /**
-     * @param theIncludesDependencies
-     *            The includesDependencies to set.
+     * @param theIncludesDependencies The includesDependencies to set.
      */
     public void setIncludesDependencies( boolean theIncludesDependencies )
     {
@@ -182,8 +177,7 @@
     }
 
     /**
-     * @param theLanguage
-     *            The language to set.
+     * @param theLanguage The language to set.
      */
     public void setLanguage( String theLanguage )
     {
@@ -191,8 +185,7 @@
     }
 
     /**
-     * @param thePackaging
-     *            The packaging to set.
+     * @param thePackaging The packaging to set.
      */
     public void setPackaging( String thePackaging )
     {
@@ -200,8 +193,7 @@
     }
 
     /**
-     * @param theType
-     *            The type to set.
+     * @param theType The type to set.
      */
     public void setType( String theType )
     {

Modified: maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/MavenProjectStub.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/MavenProjectStub.java?rev=629649&r1=629648&r2=629649&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/MavenProjectStub.java (original)
+++ maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/MavenProjectStub.java Wed Feb 20 15:07:00 2008
@@ -161,21 +161,25 @@
      */
     public MavenProjectStub()
     {
-        this( (Model) new Model() );
+        this( new Model() );
     }
 
-    // kinda dangerous...
+    /**
+     * @param model the given model
+     */
     public MavenProjectStub( Model model )
     {
-        //  super(model);
         super( (Model) null );
         this.model = model;
     }
 
-    // kinda dangerous...
+    /**
+     * No project model is associated
+     *
+     * @param project the given project
+     */
     public MavenProjectStub( MavenProject project )
     {
-        //super(project);
         super( (Model) null );
     }
 
@@ -190,41 +194,31 @@
         return "";
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getArtifact()
-     */
+    /** {@inheritDoc} */
     public Artifact getArtifact()
     {
         return artifact;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setArtifact(org.apache.maven.artifact.Artifact)
-     */
+    /** {@inheritDoc} */
     public void setArtifact( Artifact artifact )
     {
         this.artifact = artifact;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getModel()
-     */
+    /** {@inheritDoc} */
     public Model getModel()
     {
         return model;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getParent()
-     */
+    /** {@inheritDoc} */
     public MavenProject getParent()
     {
         return parent;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setParent(org.apache.maven.project.MavenProject)
-     */
+    /** {@inheritDoc} */
     public void setParent( MavenProject mavenProject )
     {
         this.parent = mavenProject;
@@ -250,9 +244,7 @@
         return Collections.EMPTY_LIST;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#hasParent()
-     */
+    /** {@inheritDoc} */
     public boolean hasParent()
     {
         if ( parent != null )
@@ -263,25 +255,19 @@
         return false;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getFile()
-     */
+    /** {@inheritDoc} */
     public File getFile()
     {
         return file;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setFile(java.io.File)
-     */
+    /** {@inheritDoc} */
     public void setFile( File file )
     {
         this.file = file;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getBasedir()
-     */
+    /** {@inheritDoc} */
     public File getBasedir()
     {
         return new File( PlexusTestCase.getBasedir() );
@@ -317,9 +303,7 @@
         return null;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#addCompileSourceRoot(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void addCompileSourceRoot( String string )
     {
         if ( compileSourceRoots == null )
@@ -332,9 +316,7 @@
         }
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#addScriptSourceRoot(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void addScriptSourceRoot( String string )
     {
         if ( scriptSourceRoots == null )
@@ -347,9 +329,7 @@
         }
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#addTestCompileSourceRoot(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void addTestCompileSourceRoot( String string )
     {
         if ( testCompileSourceRoots == null )
@@ -362,33 +342,25 @@
         }
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getCompileSourceRoots()
-     */
+    /** {@inheritDoc} */
     public List getCompileSourceRoots()
     {
         return compileSourceRoots;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getScriptSourceRoots()
-     */
+    /** {@inheritDoc} */
     public List getScriptSourceRoots()
     {
         return scriptSourceRoots;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getTestCompileSourceRoots()
-     */
+    /** {@inheritDoc} */
     public List getTestCompileSourceRoots()
     {
         return testCompileSourceRoots;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getCompileClasspathElements()
-     */
+    /** {@inheritDoc} */
     public List getCompileClasspathElements()
         throws DependencyResolutionRequiredException
     {
@@ -403,84 +375,64 @@
         this.compileArtifacts = compileArtifacts;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getCompileArtifacts()
-     */
+    /** {@inheritDoc} */
     public List getCompileArtifacts()
     {
         return compileArtifacts;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getCompileDependencies()
-     */
+    /** {@inheritDoc} */
     public List getCompileDependencies()
     {
         return compileDependencies;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getTestClasspathElements()
-     */
+    /** {@inheritDoc} */
     public List getTestClasspathElements()
         throws DependencyResolutionRequiredException
     {
         return testClasspathElements;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getTestArtifacts()
-     */
+    /** {@inheritDoc} */
     public List getTestArtifacts()
     {
         return testArtifacts;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getTestDependencies()
-     */
+    /** {@inheritDoc} */
     public List getTestDependencies()
     {
         return testDependencies;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getRuntimeClasspathElements()
-     */
+    /** {@inheritDoc} */
     public List getRuntimeClasspathElements()
         throws DependencyResolutionRequiredException
     {
         return runtimeClasspathElements;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getRuntimeArtifacts()
-     */
+    /** {@inheritDoc} */
     public List getRuntimeArtifacts()
     {
         return runtimeArtifacts;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getRuntimeDependencies()
-     */
+    /** {@inheritDoc} */
     public List getRuntimeDependencies()
     {
         return runtimeDependencies;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getSystemClasspathElements()
-     */
+    /** {@inheritDoc} */
     public List getSystemClasspathElements()
         throws DependencyResolutionRequiredException
     {
         return systemClasspathElements;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getSystemArtifacts()
-     */
+    /** {@inheritDoc} */
     public List getSystemArtifacts()
     {
         return systemArtifacts;
@@ -654,22 +606,19 @@
         this.model = model;
     }
 
+    /** {@inheritDoc} */
     public List getSystemDependencies()
     {
         return systemDependencies;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setModelVersion(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setModelVersion( String string )
     {
         this.modelVersion = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getModelVersion()
-     */
+    /** {@inheritDoc} */
     public String getModelVersion()
     {
         return modelVersion;
@@ -685,113 +634,85 @@
         return "";
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setGroupId(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setGroupId( String string )
     {
         this.groupId = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getGroupId()
-     */
+    /** {@inheritDoc} */
     public String getGroupId()
     {
         return groupId;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setArtifactId(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setArtifactId( String string )
     {
         this.artifactId = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getArtifactId()
-     */
+    /** {@inheritDoc} */
     public String getArtifactId()
     {
         return artifactId;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setName(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setName( String string )
     {
         this.name = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getName()
-     */
+    /** {@inheritDoc} */
     public String getName()
     {
         return name;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setVersion(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setVersion( String string )
     {
         this.version = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getVersion()
-     */
+    /** {@inheritDoc} */
     public String getVersion()
     {
         return version;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getPackaging()
-     */
+    /** {@inheritDoc} */
     public String getPackaging()
     {
         return packaging;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setPackaging(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setPackaging( String string )
     {
         this.packaging = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setInceptionYear(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setInceptionYear( String string )
     {
         this.inceptionYear = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getInceptionYear()
-     */
+    /** {@inheritDoc} */
     public String getInceptionYear()
     {
         return inceptionYear;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setUrl(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setUrl( String string )
     {
         this.url = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getUrl()
-     */
+    /** {@inheritDoc} */
     public String getUrl()
     {
         return url;
@@ -867,17 +788,13 @@
         return null;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setDescription(java.lang.String)
-     */
+    /** {@inheritDoc} */
     public void setDescription( String string )
     {
         this.description = string;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getDescription()
-     */
+    /** {@inheritDoc} */
     public String getDescription()
     {
         return description;
@@ -1013,11 +930,13 @@
         // nop
     }
 
+    /** {@inheritDoc} */
     public void setBuild( Build build )
     {
         this.build = build;
     }
 
+    /** {@inheritDoc} */
     public Build getBuild()
     {
         return build;
@@ -1083,17 +1002,13 @@
         return null;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setLicenses(java.util.List)
-     */
+    /** {@inheritDoc} */
     public void setLicenses( List licenses )
     {
         this.licenses = licenses;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getLicenses()
-     */
+    /** {@inheritDoc} */
     public List getLicenses()
     {
         return licenses;
@@ -1319,33 +1234,25 @@
         // nop
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getCollectedProjects()
-     */
+    /** {@inheritDoc} */
     public List getCollectedProjects()
     {
         return collectedProjects;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setCollectedProjects(java.util.List)
-     */
+    /** {@inheritDoc} */
     public void setCollectedProjects( List list )
     {
         this.collectedProjects = list;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setPluginArtifactRepositories(java.util.List)
-     */
+    /** {@inheritDoc} */
     public void setPluginArtifactRepositories( List list )
     {
         this.pluginArtifactRepositories = list;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getPluginArtifactRepositories()
-     */
+    /** {@inheritDoc} */
     public List getPluginArtifactRepositories()
     {
         return pluginArtifactRepositories;
@@ -1371,25 +1278,19 @@
         return Collections.EMPTY_LIST;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setActiveProfiles(java.util.List)
-     */
+    /** {@inheritDoc} */
     public void setActiveProfiles( List list )
     {
         activeProfiles = list;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getActiveProfiles()
-     */
+    /** {@inheritDoc} */
     public List getActiveProfiles()
     {
         return activeProfiles;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#addAttachedArtifact(org.apache.maven.artifact.Artifact)
-     */
+    /** {@inheritDoc} */
     public void addAttachedArtifact( Artifact artifact )
     {
         if ( attachedArtifacts == null )
@@ -1402,9 +1303,7 @@
         }
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getAttachedArtifacts()
-     */
+    /** {@inheritDoc} */
     public List getAttachedArtifacts()
     {
         return attachedArtifacts;
@@ -1472,49 +1371,37 @@
         // nop
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getDependencyArtifacts()
-     */
+    /** {@inheritDoc} */
     public Set getDependencyArtifacts()
     {
         return dependencyArtifacts;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setDependencyArtifacts(java.util.Set)
-     */
+    /** {@inheritDoc} */
     public void setDependencyArtifacts( Set set )
     {
         this.dependencyArtifacts = set;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setReleaseArtifactRepository(org.apache.maven.artifact.repository.ArtifactRepository)
-     */
+    /** {@inheritDoc} */
     public void setReleaseArtifactRepository( ArtifactRepository artifactRepository )
     {
         this.releaseArtifactRepository = artifactRepository;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setSnapshotArtifactRepository(org.apache.maven.artifact.repository.ArtifactRepository)
-     */
+    /** {@inheritDoc} */
     public void setSnapshotArtifactRepository( ArtifactRepository artifactRepository )
     {
         this.snapshotArtifactRepository = artifactRepository;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setOriginalModel(org.apache.maven.model.Model)
-     */
+    /** {@inheritDoc} */
     public void setOriginalModel( Model model )
     {
         this.originalModel = model;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getOriginalModel()
-     */
+    /** {@inheritDoc} */
     public Model getOriginalModel()
     {
         return originalModel;
@@ -1591,25 +1478,19 @@
         return Collections.EMPTY_MAP;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#isExecutionRoot()
-     */
+    /** {@inheritDoc} */
     public boolean isExecutionRoot()
     {
         return executionRoot;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#setExecutionRoot(boolean)
-     */
+    /** {@inheritDoc} */
     public void setExecutionRoot( boolean b )
     {
         this.executionRoot = b;
     }
 
-    /**
-     * @see org.apache.maven.project.MavenProject#getDefaultGoal()
-     */
+    /** {@inheritDoc} */
     public String getDefaultGoal()
     {
         return defaultGoal;

Modified: maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactRepository.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactRepository.java?rev=629649&r1=629648&r2=629649&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactRepository.java (original)
+++ maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactRepository.java Wed Feb 20 15:07:00 2008
@@ -32,7 +32,7 @@
 public class StubArtifactRepository
     implements ArtifactRepository
 {
-    String baseDir = null;
+    private String baseDir = null;
 
     /**
      * Default constructor

Modified: maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactResolver.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactResolver.java?rev=629649&r1=629648&r2=629649&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactResolver.java (original)
+++ maven/plugin-tools/trunk/maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/stubs/StubArtifactResolver.java Wed Feb 20 15:07:00 2008
@@ -43,12 +43,11 @@
 public class StubArtifactResolver
     implements ArtifactResolver
 {
+    private boolean throwArtifactResolutionException;
 
-    boolean throwArtifactResolutionException;
+    private boolean throwArtifactNotFoundException;
 
-    boolean throwArtifactNotFoundException;
-
-    ArtifactStubFactory factory;
+    private ArtifactStubFactory factory;
 
     /**
      * Default constructor