You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by cs...@apache.org on 2022/06/10 06:30:00 UTC
[maven-assembly-plugin] branch master updated: [MASSEMBLY-959] Update to Java8 and drop m-shared-utils (#67)
This is an automated email from the ASF dual-hosted git repository.
cstamas pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-assembly-plugin.git
The following commit(s) were added to refs/heads/master by this push:
new 36b9e527 [MASSEMBLY-959] Update to Java8 and drop m-shared-utils (#67)
36b9e527 is described below
commit 36b9e5272ff9c77bae474dff129d39d86716d22c
Author: Tamas Cservenak <ta...@cservenak.net>
AuthorDate: Fri Jun 10 08:29:56 2022 +0200
[MASSEMBLY-959] Update to Java8 and drop m-shared-utils (#67)
Update to Java8 and drop m-shared-utils
---
pom.xml | 19 +++++---------
src/it/projects/reproducible/verify.groovy | 2 +-
.../filter/ComponentsXmlArchiverFileFilter.java | 10 ++++----
.../plugins/assembly/io/DefaultAssemblyReader.java | 8 +++---
.../maven/plugins/assembly/io/URLLocation.java | 4 +--
.../assembly/mojos/AbstractAssemblyMojo.java | 18 ++++++++++---
.../repository/DefaultRepositoryAssembler.java | 2 +-
.../assembly/utils/AssemblyFormatUtils.java | 2 +-
.../phase/DependencySetAssemblyPhaseTest.java | 19 +++++++-------
.../archive/phase/ModuleSetAssemblyPhaseTest.java | 30 +++++++++++-----------
.../archive/phase/RepositoryAssemblyPhaseTest.java | 5 ++--
.../assembly/archive/task/AddArtifactTaskTest.java | 7 +++--
.../archive/task/AddDependencySetsTaskTest.java | 6 ++---
13 files changed, 66 insertions(+), 66 deletions(-)
diff --git a/pom.xml b/pom.xml
index 24ca9411..d2ab0c03 100644
--- a/pom.xml
+++ b/pom.xml
@@ -64,7 +64,7 @@ under the License.
</distributionManagement>
<properties>
- <javaVersion>7</javaVersion>
+ <javaVersion>8</javaVersion>
<mdoVersion>2.1.1</mdoVersion>
<mavenVersion>3.2.5</mavenVersion>
<slf4jVersion>1.7.5</slf4jVersion>
@@ -140,11 +140,6 @@ under the License.
<scope>provided</scope>
</dependency>
- <dependency>
- <groupId>org.apache.maven.shared</groupId>
- <artifactId>maven-shared-utils</artifactId>
- <version>3.3.4</version>
- </dependency>
<dependency>
<groupId>org.apache.maven.shared</groupId>
<artifactId>maven-common-artifact-filters</artifactId>
@@ -176,12 +171,12 @@ under the License.
<dependency>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-archiver</artifactId>
- <version>4.2.1</version>
+ <version>4.2.7</version>
</dependency>
<dependency>
<groupId>commons-io</groupId>
<artifactId>commons-io</artifactId>
- <version>2.6</version>
+ <version>2.11.0</version>
</dependency>
<dependency>
<groupId>org.apache.maven.shared</groupId>
@@ -191,17 +186,17 @@ under the License.
<dependency>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-io</artifactId>
- <version>3.2.0</version>
+ <version>3.3.1</version>
</dependency>
<dependency>
<groupId>org.apache.maven</groupId>
<artifactId>maven-archiver</artifactId>
- <version>3.5.1</version>
+ <version>3.5.2</version>
</dependency>
<dependency>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-utils</artifactId>
- <version>3.3.0</version>
+ <version>3.4.2</version>
</dependency>
<!-- Test -->
@@ -246,7 +241,7 @@ under the License.
<dependency>
<groupId>org.mockito</groupId>
<artifactId>mockito-core</artifactId>
- <version>2.28.2</version>
+ <version>4.5.1</version>
<scope>test</scope>
</dependency>
<dependency>
diff --git a/src/it/projects/reproducible/verify.groovy b/src/it/projects/reproducible/verify.groovy
index 2693e3a4..14b429c7 100644
--- a/src/it/projects/reproducible/verify.groovy
+++ b/src/it/projects/reproducible/verify.groovy
@@ -27,4 +27,4 @@ assert deployDir.exists()
assert new File( deployDir, 'reproducible-1.0-src.zip.sha1' ).text == '5ce34fc133d47cbc9c81195877dbe10b9ec7d864'
assert new File( deployDir, 'reproducible-1.0-src.tar.sha1' ).text == '0b9dc1da069705a93b4954a198c18bd248822bf8'
-assert new File( deployDir, 'reproducible-1.0-src.jar.sha1' ).text == '9b41cf2a80bc91c1f56b769bbef61c5dfd57974c'
+assert new File( deployDir, 'reproducible-1.0-src.jar.sha1' ).text == '2e84c54834ad80e58b4564471a02356904c7b1e3'
diff --git a/src/main/java/org/apache/maven/plugins/assembly/filter/ComponentsXmlArchiverFileFilter.java b/src/main/java/org/apache/maven/plugins/assembly/filter/ComponentsXmlArchiverFileFilter.java
index be7ccb8c..215d51e0 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/filter/ComponentsXmlArchiverFileFilter.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/filter/ComponentsXmlArchiverFileFilter.java
@@ -19,8 +19,8 @@ package org.apache.maven.plugins.assembly.filter;
* under the License.
*/
-import org.apache.maven.shared.utils.ReaderFactory;
-import org.apache.maven.shared.utils.WriterFactory;
+import org.apache.commons.io.input.XmlStreamReader;
+import org.apache.commons.io.output.XmlStreamWriter;
import org.codehaus.plexus.archiver.Archiver;
import org.codehaus.plexus.archiver.ArchiverException;
import org.codehaus.plexus.archiver.ResourceIterator;
@@ -35,10 +35,10 @@ import javax.inject.Named;
import java.io.BufferedReader;
import java.io.File;
-import java.io.FileOutputStream;
import java.io.IOException;
import java.io.Reader;
import java.io.Writer;
+import java.nio.file.Files;
import java.util.Collections;
import java.util.LinkedHashMap;
import java.util.List;
@@ -106,7 +106,7 @@ public class ComponentsXmlArchiverFileFilter
f.deleteOnExit();
- try ( Writer fileWriter = WriterFactory.newXmlWriter( new FileOutputStream( f ) ) )
+ try ( Writer fileWriter = new XmlStreamWriter( Files.newOutputStream( f.toPath() ) ) )
{
final Xpp3Dom dom = new Xpp3Dom( "component-set" );
final Xpp3Dom componentDom = new Xpp3Dom( "components" );
@@ -182,7 +182,7 @@ public class ComponentsXmlArchiverFileFilter
if ( ComponentsXmlArchiverFileFilter.COMPONENTS_XML_PATH.equals( entry ) )
{
- try ( Reader reader = new BufferedReader( ReaderFactory.newXmlReader( fileInfo.getContents() ) ) )
+ try ( Reader reader = new BufferedReader( new XmlStreamReader( fileInfo.getContents() ) ) )
{
addComponentsXml( reader );
}
diff --git a/src/main/java/org/apache/maven/plugins/assembly/io/DefaultAssemblyReader.java b/src/main/java/org/apache/maven/plugins/assembly/io/DefaultAssemblyReader.java
index b82fff56..b08d5063 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/io/DefaultAssemblyReader.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/io/DefaultAssemblyReader.java
@@ -22,6 +22,7 @@ package org.apache.maven.plugins.assembly.io;
import javax.inject.Named;
import javax.inject.Singleton;
+import org.apache.commons.io.input.XmlStreamReader;
import org.apache.maven.plugins.assembly.AssemblerConfigurationSource;
import org.apache.maven.plugins.assembly.InvalidAssemblerConfigurationException;
import org.apache.maven.plugins.assembly.interpolation.AssemblyExpressionEvaluator;
@@ -40,7 +41,6 @@ import org.apache.maven.plugins.assembly.model.io.xpp3.ComponentXpp3Reader;
import org.apache.maven.plugins.assembly.resolved.AssemblyId;
import org.apache.maven.plugins.assembly.utils.InterpolationConstants;
import org.apache.maven.project.MavenProject;
-import org.apache.maven.shared.utils.ReaderFactory;
import org.codehaus.plexus.interpolation.PrefixAwareRecursionInterceptor;
import org.codehaus.plexus.interpolation.RecursionInterceptor;
import org.codehaus.plexus.interpolation.fixed.FixedStringSearchInterpolator;
@@ -206,7 +206,7 @@ public class DefaultAssemblyReader implements AssemblyReader
}
}
- try ( Reader reader = ReaderFactory.newXmlReader( resourceAsStream ) )
+ try ( Reader reader = new XmlStreamReader( resourceAsStream ) )
{
final Assembly assembly = readAssembly( reader, ref, null, configSource );
assemblies.add( assembly );
@@ -237,7 +237,7 @@ public class DefaultAssemblyReader implements AssemblyReader
}
}
- try ( Reader r = ReaderFactory.newXmlReader( descriptor ) )
+ try ( Reader r = new XmlStreamReader( descriptor ) )
{
final Assembly assembly =
readAssembly( r, descriptor.getAbsolutePath(), descriptor.getParentFile(), configSource );
@@ -275,7 +275,7 @@ public class DefaultAssemblyReader implements AssemblyReader
}
- try ( Reader r = ReaderFactory.newXmlReader( location.getInputStream() ) )
+ try ( Reader r = new XmlStreamReader( location.getInputStream() ) )
{
File dir = null;
if ( location.getFile() != null )
diff --git a/src/main/java/org/apache/maven/plugins/assembly/io/URLLocation.java b/src/main/java/org/apache/maven/plugins/assembly/io/URLLocation.java
index faaac78b..bd7008f3 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/io/URLLocation.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/io/URLLocation.java
@@ -23,7 +23,7 @@ import java.io.File;
import java.io.IOException;
import java.net.URL;
-import org.apache.maven.shared.utils.io.FileUtils;
+import org.apache.commons.io.IOUtils;
/**
* The URL Location.
@@ -72,7 +72,7 @@ class URLLocation
tempFile.deleteOnExit();
}
- FileUtils.copyURLToFile( url, tempFile );
+ IOUtils.copy( url, tempFile );
setFile( tempFile );
}
diff --git a/src/main/java/org/apache/maven/plugins/assembly/mojos/AbstractAssemblyMojo.java b/src/main/java/org/apache/maven/plugins/assembly/mojos/AbstractAssemblyMojo.java
index 2611b36f..c3aaa9fb 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/mojos/AbstractAssemblyMojo.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/mojos/AbstractAssemblyMojo.java
@@ -20,6 +20,8 @@ package org.apache.maven.plugins.assembly.mojos;
*/
import java.io.File;
+import java.io.IOException;
+import java.io.UncheckedIOException;
import java.util.Collections;
import java.util.Date;
import java.util.List;
@@ -48,11 +50,11 @@ import org.apache.maven.plugins.assembly.utils.InterpolationConstants;
import org.apache.maven.project.MavenProject;
import org.apache.maven.project.MavenProjectHelper;
import org.apache.maven.shared.filtering.MavenReaderFilter;
-import org.apache.maven.shared.utils.cli.CommandLineUtils;
import org.codehaus.plexus.configuration.PlexusConfiguration;
import org.codehaus.plexus.interpolation.fixed.FixedStringSearchInterpolator;
import org.codehaus.plexus.interpolation.fixed.PrefixedPropertiesValueSource;
import org.codehaus.plexus.interpolation.fixed.PropertiesBasedValueSource;
+import org.codehaus.plexus.util.cli.CommandLineUtils;
/**
* @author <a href="mailto:brett@apache.org">Brett Porter</a>
@@ -622,9 +624,17 @@ public abstract class AbstractAssemblyMojo extends AbstractMojo implements Assem
private FixedStringSearchInterpolator createEnvInterpolator()
{
- PrefixedPropertiesValueSource envProps = new PrefixedPropertiesValueSource( Collections.singletonList( "env." ),
- CommandLineUtils.getSystemEnvVars( false ), true );
- return FixedStringSearchInterpolator.create( envProps );
+ try
+ {
+ PrefixedPropertiesValueSource envProps =
+ new PrefixedPropertiesValueSource( Collections.singletonList( "env." ),
+ CommandLineUtils.getSystemEnvVars( false ), true );
+ return FixedStringSearchInterpolator.create( envProps );
+ }
+ catch ( IOException e )
+ {
+ throw new UncheckedIOException( e );
+ }
}
/**
diff --git a/src/main/java/org/apache/maven/plugins/assembly/repository/DefaultRepositoryAssembler.java b/src/main/java/org/apache/maven/plugins/assembly/repository/DefaultRepositoryAssembler.java
index a1240fb4..a9bbd03d 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/repository/DefaultRepositoryAssembler.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/repository/DefaultRepositoryAssembler.java
@@ -32,6 +32,7 @@ import java.util.List;
import java.util.Map;
import java.util.Set;
+import org.apache.commons.io.FileUtils;
import org.apache.maven.artifact.Artifact;
import org.apache.maven.artifact.resolver.filter.AndArtifactFilter;
import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
@@ -50,7 +51,6 @@ import org.apache.maven.shared.transfer.artifact.resolve.ArtifactResult;
import org.apache.maven.shared.transfer.dependencies.resolve.DependencyResolver;
import org.apache.maven.shared.transfer.dependencies.resolve.DependencyResolverException;
import org.apache.maven.shared.transfer.repository.RepositoryManager;
-import org.apache.maven.shared.utils.io.FileUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
diff --git a/src/main/java/org/apache/maven/plugins/assembly/utils/AssemblyFormatUtils.java b/src/main/java/org/apache/maven/plugins/assembly/utils/AssemblyFormatUtils.java
index cf591ff2..a7276afe 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/utils/AssemblyFormatUtils.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/utils/AssemblyFormatUtils.java
@@ -32,12 +32,12 @@ import org.apache.maven.plugins.assembly.AssemblerConfigurationSource;
import org.apache.maven.plugins.assembly.format.AssemblyFormattingException;
import org.apache.maven.plugins.assembly.model.Assembly;
import org.apache.maven.project.MavenProject;
-import org.apache.maven.shared.utils.Os;
import org.codehaus.plexus.interpolation.fixed.FixedStringSearchInterpolator;
import org.codehaus.plexus.interpolation.fixed.MapBasedValueSource;
import org.codehaus.plexus.interpolation.fixed.PrefixedObjectValueSource;
import org.codehaus.plexus.interpolation.fixed.PrefixedPropertiesValueSource;
import org.codehaus.plexus.interpolation.fixed.PropertiesBasedValueSource;
+import org.codehaus.plexus.util.Os;
import org.codehaus.plexus.util.StringUtils;
import org.slf4j.Logger;
diff --git a/src/test/java/org/apache/maven/plugins/assembly/archive/phase/DependencySetAssemblyPhaseTest.java b/src/test/java/org/apache/maven/plugins/assembly/archive/phase/DependencySetAssemblyPhaseTest.java
index 71798aaf..60661299 100644
--- a/src/test/java/org/apache/maven/plugins/assembly/archive/phase/DependencySetAssemblyPhaseTest.java
+++ b/src/test/java/org/apache/maven/plugins/assembly/archive/phase/DependencySetAssemblyPhaseTest.java
@@ -19,7 +19,7 @@ package org.apache.maven.plugins.assembly.archive.phase;
* under the License.
*/
-import static org.mockito.ArgumentMatchers.anyListOf;
+import static org.mockito.ArgumentMatchers.anyList;
import static org.mockito.ArgumentMatchers.eq;
import static org.mockito.ArgumentMatchers.isNull;
import static org.mockito.Mockito.mock;
@@ -33,7 +33,6 @@ import java.util.Set;
import org.apache.maven.artifact.Artifact;
import org.apache.maven.model.Model;
-import org.apache.maven.plugins.assembly.AssemblerConfigurationSource;
import org.apache.maven.plugins.assembly.InvalidAssemblerConfigurationException;
import org.apache.maven.plugins.assembly.archive.ArchiveCreationException;
import org.apache.maven.plugins.assembly.artifact.DependencyResolutionException;
@@ -92,15 +91,15 @@ public class DependencySetAssemblyPhaseTest
project.setArtifacts( Collections.singleton( artifact ) );
when( dependencyResolver.resolveDependencySets( eq( assembly ),
- isNull(AssemblerConfigurationSource.class),
- anyListOf( DependencySet.class ) ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
+ isNull(),
+ anyList() ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
this.phase.execute( assembly, null, null );
// result of easymock migration, should be assert of expected result instead of verifying methodcalls
verify( dependencyResolver ).resolveDependencySets( eq( assembly ),
- isNull(AssemblerConfigurationSource.class),
- anyListOf( DependencySet.class ) );
+ isNull(),
+ anyList() );
}
@Test
@@ -112,15 +111,15 @@ public class DependencySetAssemblyPhaseTest
assembly.setIncludeBaseDirectory( false );
when( dependencyResolver.resolveDependencySets( eq( assembly ),
- isNull( AssemblerConfigurationSource.class ),
- anyListOf( DependencySet.class ) ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
+ isNull(),
+ anyList() ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
this.phase.execute( assembly, null, null );
// result of easymock migration, should be assert of expected result instead of verifying methodcalls
verify( dependencyResolver ).resolveDependencySets( eq( assembly ),
- isNull( AssemblerConfigurationSource.class ),
- anyListOf( DependencySet.class ) );
+ isNull(),
+ anyList() );
}
private MavenProject newMavenProject( final String groupId, final String artifactId, final String version )
diff --git a/src/test/java/org/apache/maven/plugins/assembly/archive/phase/ModuleSetAssemblyPhaseTest.java b/src/test/java/org/apache/maven/plugins/assembly/archive/phase/ModuleSetAssemblyPhaseTest.java
index c408edee..4952a251 100644
--- a/src/test/java/org/apache/maven/plugins/assembly/archive/phase/ModuleSetAssemblyPhaseTest.java
+++ b/src/test/java/org/apache/maven/plugins/assembly/archive/phase/ModuleSetAssemblyPhaseTest.java
@@ -25,7 +25,7 @@ import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
import static org.mockito.Mockito.any;
-import static org.mockito.Mockito.anyListOf;
+import static org.mockito.Mockito.anyList;
import static org.mockito.Mockito.atLeastOnce;
import static org.mockito.Mockito.eq;
import static org.mockito.Mockito.isNull;
@@ -312,7 +312,7 @@ public class ModuleSetAssemblyPhaseTest
when( dependencyResolver.resolveDependencySets( eq( assembly ),
eq( ms ),
eq( configSource ),
- anyListOf( DependencySet.class ) ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
+ anyList() ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
DefaultAssemblyArchiverTest.setupInterpolators( configSource, module );
this.phase.execute( assembly, archiver, configSource );
@@ -332,7 +332,7 @@ public class ModuleSetAssemblyPhaseTest
verify( dependencyResolver ).resolveDependencySets( eq( assembly ),
eq( ms ),
eq( configSource ),
- anyListOf( DependencySet.class ) );
+ anyList() );
}
@Test
@@ -396,10 +396,10 @@ public class ModuleSetAssemblyPhaseTest
final Set<MavenProject> projects = singleton( project );
- when( dependencyResolver.resolveDependencySets( isNull( Assembly.class ),
- isNull( ModuleSet.class ),
+ when( dependencyResolver.resolveDependencySets( isNull(),
+ isNull(),
eq( configSource ),
- anyListOf( DependencySet.class ) ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
+ anyList() ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
DefaultAssemblyArchiverTest.setupInterpolators( configSource, project );
this.phase.addModuleBinaries( null, null, binaries, projects, archiver, configSource );
@@ -416,10 +416,10 @@ public class ModuleSetAssemblyPhaseTest
verify( archiver ).setFileMode( 511 );
verify( archiver ).setFileMode( 146 );
- verify( dependencyResolver ).resolveDependencySets( isNull( Assembly.class ),
- isNull( ModuleSet.class ),
+ verify( dependencyResolver ).resolveDependencySets( isNull(),
+ isNull(),
eq( configSource ),
- anyListOf( DependencySet.class ) );
+ anyList() );
}
@Test
@@ -486,10 +486,10 @@ public class ModuleSetAssemblyPhaseTest
final Set<MavenProject> projects = singleton( project );
- when( dependencyResolver.resolveDependencySets( isNull( Assembly.class ),
- isNull( ModuleSet.class ),
+ when( dependencyResolver.resolveDependencySets( isNull(),
+ isNull(),
any( AssemblerConfigurationSource.class ),
- anyListOf( DependencySet.class ) ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
+ anyList() ) ).thenReturn( new LinkedHashMap<DependencySet, Set<Artifact>>() );
DefaultAssemblyArchiverTest.setupInterpolators( configSource, project );
this.phase.addModuleBinaries( null, null, binaries, projects, archiver, configSource );
@@ -499,10 +499,10 @@ public class ModuleSetAssemblyPhaseTest
verify( configSource, atLeastOnce() ).getMavenSession();
verify( configSource, atLeastOnce() ).getProject();
- verify( dependencyResolver ).resolveDependencySets( isNull( Assembly.class ),
- isNull( ModuleSet.class ),
+ verify( dependencyResolver ).resolveDependencySets( isNull(),
+ isNull(),
any( AssemblerConfigurationSource.class ),
- anyListOf( DependencySet.class ) );
+ anyList() );
verify( archiver ).addFile( artifactFile, "out/artifact", 511 );
verify( archiver, atLeastOnce() ).getDestFile();
diff --git a/src/test/java/org/apache/maven/plugins/assembly/archive/phase/RepositoryAssemblyPhaseTest.java b/src/test/java/org/apache/maven/plugins/assembly/archive/phase/RepositoryAssemblyPhaseTest.java
index bd492a47..d3f5f6e0 100644
--- a/src/test/java/org/apache/maven/plugins/assembly/archive/phase/RepositoryAssemblyPhaseTest.java
+++ b/src/test/java/org/apache/maven/plugins/assembly/archive/phase/RepositoryAssemblyPhaseTest.java
@@ -23,7 +23,7 @@ import static org.mockito.Mockito.any;
import static org.mockito.Mockito.atLeastOnce;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
import static org.mockito.Mockito.when;
import java.io.File;
@@ -82,8 +82,7 @@ public class RepositoryAssemblyPhaseTest
// result of easymock migration, should be assert of expected result instead of verifying methodcalls
verify( configSource, atLeastOnce() ).getTemporaryRootDirectory();
-
- verifyZeroInteractions( repositoryAssembler );
+ verifyNoInteractions( repositoryAssembler );
}
@Test
diff --git a/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddArtifactTaskTest.java b/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddArtifactTaskTest.java
index c20979b3..aa959f46 100644
--- a/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddArtifactTaskTest.java
+++ b/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddArtifactTaskTest.java
@@ -28,7 +28,6 @@ import static org.mockito.Mockito.when;
import java.io.File;
import java.io.IOException;
-import java.nio.charset.Charset;
import java.util.Arrays;
import org.apache.maven.artifact.Artifact;
@@ -199,7 +198,7 @@ public class AddArtifactTaskTest
task.execute( archiver, configSource );
// result of easymock migration, should be assert of expected result instead of verifying methodcalls
- verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull( Charset.class ) );
+ verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull() );
verify( archiver, atLeastOnce() ).getDestFile();
verify( archiver ).getOverrideDirectoryMode();
verify( archiver ).getOverrideFileMode();
@@ -232,7 +231,7 @@ public class AddArtifactTaskTest
task.execute( archiver, configSource );
// result of easymock migration, should be assert of expected result instead of verifying methodcalls
- verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull( Charset.class ) );
+ verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull() );
verify( archiver, atLeastOnce() ).getDestFile();
verify( archiver ).getOverrideDirectoryMode();
verify( archiver ).getOverrideFileMode();
@@ -270,7 +269,7 @@ public class AddArtifactTaskTest
task.execute( archiver, configSource );
// result of easymock migration, should be assert of expected result instead of verifying methodcalls
- verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull( Charset.class ) );
+ verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull() );
verify( archiver, atLeastOnce() ).getDestFile();
verify( archiver ).getOverrideDirectoryMode();
verify( archiver ).getOverrideFileMode();
diff --git a/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddDependencySetsTaskTest.java b/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddDependencySetsTaskTest.java
index 99ad00d1..21cdaa6a 100644
--- a/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddDependencySetsTaskTest.java
+++ b/src/test/java/org/apache/maven/plugins/assembly/archive/task/AddDependencySetsTaskTest.java
@@ -25,7 +25,6 @@ import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertSame;
import static org.mockito.ArgumentMatchers.any;
-import static org.mockito.ArgumentMatchers.eq;
import static org.mockito.ArgumentMatchers.isNull;
import static org.mockito.Mockito.atLeastOnce;
import static org.mockito.Mockito.mock;
@@ -35,7 +34,6 @@ import static org.mockito.Mockito.when;
import java.io.File;
import java.io.IOException;
-import java.nio.charset.Charset;
import java.util.Arrays;
import java.util.Collections;
import java.util.HashSet;
@@ -343,7 +341,7 @@ public class AddDependencySetsTaskTest
if ( unpack )
{
- verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull( Charset.class ) );
+ verify( archiver ).addArchivedFileSet( any( ArchivedFileSet.class ), isNull() );
}
else
{
@@ -495,7 +493,7 @@ public class AddDependencySetsTaskTest
task.addDependencySet( dependencySet, archiver, configSource );
ArgumentCaptor<ArchivedFileSet> archivedFileSet = ArgumentCaptor.forClass( ArchivedFileSet.class );
- verify( archiver ).addArchivedFileSet( archivedFileSet.capture(), isNull( Charset.class ) );
+ verify( archiver ).addArchivedFileSet( archivedFileSet.capture(), isNull() );
assertThat( archivedFileSet.getValue().isUsingDefaultExcludes(), is( false ) );
ArgumentCaptor<FileSet> fileSet = ArgumentCaptor.forClass( FileSet.class );