Add tests to verify that transformed filters are processed correctly

git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1687142 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/test/java/org/apache/maven/shared/artifact/filter/collection/TestFilterArtifacts.java b/src/test/java/org/apache/maven/shared/artifact/filter/collection/TestFilterArtifacts.java
index 6ac222b..8a66b35 100644
--- a/src/test/java/org/apache/maven/shared/artifact/filter/collection/TestFilterArtifacts.java
+++ b/src/test/java/org/apache/maven/shared/artifact/filter/collection/TestFilterArtifacts.java
@@ -58,7 +58,7 @@
         FileUtils.deleteDirectory( outputFolder );
 
         ArtifactStubFactory fact = new ArtifactStubFactory( outputFolder, false );
-        @SuppressWarnings( "unchecked" )
+
         Set<Artifact> artifacts = fact.getReleaseAndSnapshotArtifacts();
         FilterArtifacts fa = new FilterArtifacts();
 
@@ -120,7 +120,7 @@
         File outputFolder = new File( "target/filters/" );
         FileUtils.deleteDirectory( outputFolder );
         ArtifactStubFactory fact = new ArtifactStubFactory( outputFolder, false );
-        @SuppressWarnings( "unchecked" )
+
         Set<Artifact> artifacts = fact.getClassifiedArtifacts();
         FilterArtifacts fa = new FilterArtifacts();
         fa.addFilter( new ClassifierFilter( "", "four" ) );
diff --git a/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/EclipseAetherFilterTransformerTest.java b/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/EclipseAetherFilterTransformerTest.java
index 9ab6635..aac8deb 100644
--- a/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/EclipseAetherFilterTransformerTest.java
+++ b/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/EclipseAetherFilterTransformerTest.java
@@ -19,6 +19,8 @@
  * under the License.
  */
 
+import static org.junit.Assert.*;
+
 import java.util.Arrays;
 import java.util.Collections;
 
@@ -29,6 +31,10 @@
 import org.apache.maven.shared.artifact.filter.resolve.PatternInclusionsFilter;
 import org.apache.maven.shared.artifact.filter.resolve.ScopeFilter;
 import org.apache.maven.shared.artifact.filter.resolve.TransformableFilter;
+import org.eclipse.aether.artifact.DefaultArtifact;
+import org.eclipse.aether.graph.DefaultDependencyNode;
+import org.eclipse.aether.graph.Dependency;
+import org.eclipse.aether.graph.DependencyNode;
 import org.eclipse.aether.util.filter.AndDependencyFilter;
 import org.eclipse.aether.util.filter.ExclusionsDependencyFilter;
 import org.eclipse.aether.util.filter.OrDependencyFilter;
@@ -39,24 +45,36 @@
 
 public class EclipseAetherFilterTransformerTest
 {
-
     private EclipseAetherFilterTransformer transformer = new EclipseAetherFilterTransformer();
 
     @Test
     public void testTransformAndFilter()
     {
-        AndFilter filter = new AndFilter( Arrays.<TransformableFilter>asList( ScopeFilter.including( "compile" ), 
-                                                                              ScopeFilter.including( "test" ) ) );
+        AndFilter filter = new AndFilter(
+              Arrays.<TransformableFilter>asList( ScopeFilter.including( "compile" ),
+                                                   new ExclusionsFilter( Collections.singletonList( "x:a" ) ) ) );
 
         AndDependencyFilter dependencyFilter = (AndDependencyFilter) filter.transform( transformer );
+        
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "test" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "test" ), null ) );
     }
 
     @Test
     public void testTransformExclusionsFilter()
     {
-        ExclusionsFilter filter = new ExclusionsFilter( Collections.singletonList( "runtime" ) );
+        ExclusionsFilter filter = new ExclusionsFilter( Collections.singletonList( "x:a" ) );
 
         ExclusionsDependencyFilter dependencyFilter = (ExclusionsDependencyFilter) filter.transform( transformer );
+
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "compile" ), null ) );
     }
 
     @Test
@@ -67,6 +85,11 @@
 
         OrDependencyFilter dependencyFilter = (OrDependencyFilter) filter.transform( transformer );
 
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "test" ), null ) );
+        
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
     }
 
     @Test
@@ -75,26 +98,45 @@
         ScopeFilter filter = ScopeFilter.including( Collections.singletonList( "runtime" ) );
 
         ScopeDependencyFilter dependencyFilter = (ScopeDependencyFilter) filter.transform( transformer );
+        
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "test" ), null ) );
     }
 
     @Test
     public void testTransformPatternExclusionsFilter()
     {
         PatternExclusionsFilter filter =
-            new PatternExclusionsFilter( Collections.singletonList( "org.apache.maven:*" ) );
+            new PatternExclusionsFilter( Collections.singletonList( "x:*" ) );
 
         PatternExclusionsDependencyFilter dependencyFilter =
             (PatternExclusionsDependencyFilter) filter.transform( transformer );
+
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "runtime" ), null ) );
     }
 
     @Test
     public void testTransformPatternInclusionsFilter()
     {
         PatternInclusionsFilter filter =
-            new PatternInclusionsFilter( Collections.singletonList( "org.apache.maven:*" ) );
+            new PatternInclusionsFilter( Collections.singletonList( "g:*" ) );
 
         PatternInclusionsDependencyFilter dependencyFilter =
             (PatternInclusionsDependencyFilter) filter.transform( transformer );
+
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "runtime" ), null ) );
+    }
+    
+    private DependencyNode newDependencyNode( String string, String scope )
+    {
+        return new DefaultDependencyNode( new Dependency( new DefaultArtifact( string ), scope ) );
     }
 
 }
diff --git a/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/SonatypeAetherFilterTransformerTest.java b/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/SonatypeAetherFilterTransformerTest.java
index 7358904..15e833b 100644
--- a/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/SonatypeAetherFilterTransformerTest.java
+++ b/src/test/java/org/apache/maven/shared/artifact/filter/resolve/transform/SonatypeAetherFilterTransformerTest.java
@@ -19,6 +19,9 @@
  * under the License.
  */
 
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
 import java.util.Arrays;
 import java.util.Collections;
 
@@ -29,12 +32,16 @@
 import org.apache.maven.shared.artifact.filter.resolve.PatternInclusionsFilter;
 import org.apache.maven.shared.artifact.filter.resolve.ScopeFilter;
 import org.apache.maven.shared.artifact.filter.resolve.TransformableFilter;
+import org.sonatype.aether.graph.Dependency;
+import org.sonatype.aether.graph.DependencyNode;
+import org.sonatype.aether.util.artifact.DefaultArtifact;
 import org.sonatype.aether.util.filter.AndDependencyFilter;
 import org.sonatype.aether.util.filter.ExclusionsDependencyFilter;
 import org.sonatype.aether.util.filter.OrDependencyFilter;
 import org.sonatype.aether.util.filter.PatternExclusionsDependencyFilter;
 import org.sonatype.aether.util.filter.PatternInclusionsDependencyFilter;
 import org.sonatype.aether.util.filter.ScopeDependencyFilter;
+import org.sonatype.aether.util.graph.DefaultDependencyNode;
 import org.junit.Test;
 
 public class SonatypeAetherFilterTransformerTest
@@ -45,18 +52,31 @@
     @Test
     public void testTransformAndFilter()
     {
-        AndFilter filter = new AndFilter( Arrays.<TransformableFilter>asList( ScopeFilter.including( "compile" ), 
-                                                                              ScopeFilter.including( "test" ) ) );
+        AndFilter filter = new AndFilter(
+                  Arrays.<TransformableFilter>asList( ScopeFilter.including( "compile" ),
+                                                      new ExclusionsFilter( Collections.singletonList( "x:a" ) ) ) );
 
         AndDependencyFilter dependencyFilter = (AndDependencyFilter) filter.transform( transformer );
+        
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "test" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "test" ), null ) );
     }
 
     @Test
     public void testTransformExclusionsFilter()
     {
-        ExclusionsFilter filter = new ExclusionsFilter( Collections.singletonList( "runtime" ) );
+        ExclusionsFilter filter = new ExclusionsFilter( Collections.singletonList( "x:a" ) );
 
         ExclusionsDependencyFilter dependencyFilter = (ExclusionsDependencyFilter) filter.transform( transformer );
+
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "compile" ), null ) );
     }
 
     @Test
@@ -67,6 +87,11 @@
 
         OrDependencyFilter dependencyFilter = (OrDependencyFilter) filter.transform( transformer );
 
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "test" ), null ) );
+        
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
     }
 
     @Test
@@ -75,26 +100,45 @@
         ScopeFilter filter = ScopeFilter.including( Collections.singletonList( "runtime" ) );
 
         ScopeDependencyFilter dependencyFilter = (ScopeDependencyFilter) filter.transform( transformer );
+        
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "compile" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "g:a:v", "test" ), null ) );
     }
 
     @Test
     public void testTransformPatternExclusionsFilter()
     {
         PatternExclusionsFilter filter =
-            new PatternExclusionsFilter( Collections.singletonList( "org.apache.maven:*" ) );
+            new PatternExclusionsFilter( Collections.singletonList( "x:*" ) );
 
         PatternExclusionsDependencyFilter dependencyFilter =
             (PatternExclusionsDependencyFilter) filter.transform( transformer );
+        
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "runtime" ), null ) );
     }
 
     @Test
     public void testTransformPatternInclusionsFilter()
     {
         PatternInclusionsFilter filter =
-            new PatternInclusionsFilter( Collections.singletonList( "org.apache.maven:*" ) );
+            new PatternInclusionsFilter( Collections.singletonList( "g:*" ) );
 
         PatternInclusionsDependencyFilter dependencyFilter =
             (PatternInclusionsDependencyFilter) filter.transform( transformer );
+        
+        assertTrue( dependencyFilter.accept( newDependencyNode( "g:a:v", "runtime" ), null ) );
+
+        assertFalse( dependencyFilter.accept( newDependencyNode( "x:a:v", "runtime" ), null ) );
+    }
+    
+    private DependencyNode newDependencyNode( String string, String scope )
+    {
+        return new DefaultDependencyNode( new Dependency( new DefaultArtifact( string ), scope ) );
     }
 
 }