[maven-studies] 02/02: ignore attached buildinfo files through attach type instead of filename

[hb...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

hboutemy pushed a commit to branch maven-buildinfo-plugin
in repository https://gitbox.apache.org/repos/asf/maven-studies.git

commit 4eb0129ec4fd5c79186e2df26731590c2b9910b7
Author: Hervé Boutemy <hb...@apache.org>
AuthorDate: Sun Jan 5 17:09:28 2020 +0100

    ignore attached buildinfo files through attach type instead of filename
---
 src/it/multi/verify.groovy                                     | 1 +
 src/main/java/org/apache/maven/plugins/buildinfo/SaveMojo.java | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/it/multi/verify.groovy b/src/it/multi/verify.groovy
index b8faa3c..50b2e6b 100644
--- a/src/it/multi/verify.groovy
+++ b/src/it/multi/verify.groovy
@@ -51,6 +51,7 @@ assert buildinfo.contains( "outputs.1.coordinates=org.apache.maven.plugins.it:mu
 assert buildinfo.contains( "outputs.1.0.filename=multi-modA-1.0-SNAPSHOT.jar" )
 assert buildinfo.contains( "outputs.2.coordinates=org.apache.maven.plugins.it:multi-modB" )
 assert buildinfo.contains( "outputs.2.0.filename=multi-modB-1.0-SNAPSHOT.jar" )
+assert !buildinfo.contains( ".buildinfo" )
 
 // check existence of buildinfos in local repository
 File local = new File( basedir, "../../local-repo/org/apache/maven/plugins/it/multi/1.0-SNAPSHOT/multi-1.0-SNAPSHOT.buildinfo")
diff --git a/src/main/java/org/apache/maven/plugins/buildinfo/SaveMojo.java b/src/main/java/org/apache/maven/plugins/buildinfo/SaveMojo.java
index 0c19827..8a157ac 100644
--- a/src/main/java/org/apache/maven/plugins/buildinfo/SaveMojo.java
+++ b/src/main/java/org/apache/maven/plugins/buildinfo/SaveMojo.java
@@ -248,7 +248,7 @@ public class SaveMojo
                 // ignore pgp signatures
                 continue;
             }
-            if ( attached.getFile().getName().equals( "buildinfo" ) )
+            if ( attached.getType().equals( "buildinfo" ) )
             {
                 // ignore buildinfo files (during aggregate)
                 continue;