SFT: avoid testing aggregator features

[odlparent.git] / features4-test / src / main / java / org / opendaylight / odlparent / featuretest / PerRepoTestRunner.java

diff --git a/features4-test/src/main/java/org/opendaylight/odlparent/featuretest/PerRepoTestRunner.java b/features4-test/src/main/java/org/opendaylight/odlparent/featuretest/PerRepoTestRunner.java
index b44ef832484aa0e361af1048064c48791f4044c4..ab4b7aa41b915279deab734f01533e495b43be59 100644 (file)
--- a/features4-test/src/main/java/org/opendaylight/odlparent/featuretest/PerRepoTestRunner.java
+++ b/features4-test/src/main/java/org/opendaylight/odlparent/featuretest/PerRepoTestRunner.java
@@ -118,7 +118,14 @@ public class PerRepoTestRunner extends ParentRunner<PerFeatureRunner> {
         final List<PerFeatureRunner> runners = new ArrayList<>();
         final List<Feature> featureList = features.getFeature();
         for (final Feature f : featureList) {
-            runners.add(new PerFeatureRunner(repoUrl, f.getName(), f.getVersion(), testClass));
+            // If the features have more than one feature, ignore any feature with the same name as the
+            // repository — these are the aggregator features generated by the Karaf Maven plugin, and
+            // which are expensive to test
+            if (featureList.size() == 1 || !f.getName().equals(features.getName())) {
+                runners.add(new PerFeatureRunner(repoUrl, f.getName(), f.getVersion(), testClass));
+            } else {
+                LOG.warn("Skipping {}, it's an aggregator feature", f.getName());
+            }
         }
         return runners;
     }