You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2020/11/26 17:04:30 UTC
[sling-org-apache-sling-feature-extension-apiregions] branch master
updated: Update test to latest analyser api to avoid using deprecated
methods
This is an automated email from the ASF dual-hosted git repository.
cziegeler pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-feature-extension-apiregions.git
The following commit(s) were added to refs/heads/master by this push:
new c06aa88 Update test to latest analyser api to avoid using deprecated methods
c06aa88 is described below
commit c06aa884f2aab1cd3b531674d70171db6bf21e27
Author: Carsten Ziegeler <cz...@apache.org>
AuthorDate: Thu Nov 26 18:04:11 2020 +0100
Update test to latest analyser api to avoid using deprecated methods
---
pom.xml | 2 +-
.../CheckApiRegionsCrossFeatureDupsTest.java | 89 +++++++++++++---------
2 files changed, 56 insertions(+), 35 deletions(-)
diff --git a/pom.xml b/pom.xml
index a980994..25a0259 100644
--- a/pom.xml
+++ b/pom.xml
@@ -62,7 +62,7 @@
<dependency>
<groupId>org.apache.sling</groupId>
<artifactId>org.apache.sling.feature.analyser</artifactId>
- <version>1.3.12</version>
+ <version>1.3.13-SNAPSHOT</version>
<scope>provided</scope>
</dependency>
<dependency>
diff --git a/src/test/java/org/apache/sling/feature/extension/apiregions/analyser/CheckApiRegionsCrossFeatureDupsTest.java b/src/test/java/org/apache/sling/feature/extension/apiregions/analyser/CheckApiRegionsCrossFeatureDupsTest.java
index 3765fe3..1d8652e 100644
--- a/src/test/java/org/apache/sling/feature/extension/apiregions/analyser/CheckApiRegionsCrossFeatureDupsTest.java
+++ b/src/test/java/org/apache/sling/feature/extension/apiregions/analyser/CheckApiRegionsCrossFeatureDupsTest.java
@@ -20,6 +20,7 @@ import org.apache.sling.feature.ArtifactId;
import org.apache.sling.feature.Feature;
import org.apache.sling.feature.analyser.Analyser;
import org.apache.sling.feature.analyser.AnalyserResult;
+import org.apache.sling.feature.analyser.AnalyserResult.ArtifactReport;
import org.apache.sling.feature.analyser.task.AnalyserTask;
import org.apache.sling.feature.builder.ArtifactProvider;
import org.apache.sling.feature.extension.apiregions.scanner.ApiRegionsExtensionScanner;
@@ -55,13 +56,16 @@ public class CheckApiRegionsCrossFeatureDupsTest {
Collections.singletonMap("warningPackages", "x.y.z"));
Analyser a = new Analyser(scanner, configs, at);
AnalyserResult res = a.analyse(f);
- assertEquals(1, res.getErrors().size());
- assertEquals(0, res.getWarnings().size());
-
- String err = res.getErrors().get(0);
- assertTrue(err.contains("a.b.c"));
- assertTrue(err.contains("g:f3:1"));
- assertTrue(err.contains("feature-export2"));
+ assertEquals(1, res.getArtifactErrors().size());
+ assertEquals(0, res.getArtifactWarnings().size());
+ assertEquals(0, res.getGlobalWarnings().size());
+ assertEquals(0, res.getGlobalErrors().size());
+
+ ArtifactReport err = res.getArtifactErrors().get(0);
+ assertEquals(ArtifactId.parse("g:exp2:1"), err.getKey());
+ assertTrue(err.getValue().contains("a.b.c"));
+ assertTrue(err.getValue().contains("g:f3:1"));
+ assertTrue(err.getValue().contains("feature-export2"));
}
@Test
@@ -80,13 +84,16 @@ public class CheckApiRegionsCrossFeatureDupsTest {
Collections.singletonMap("api-regions-crossfeature-dups", cfg);
Analyser a = new Analyser(scanner, configs, at);
AnalyserResult res = a.analyse(f);
- assertEquals(0, res.getErrors().size());
- assertEquals(1, res.getWarnings().size());
-
- String err = res.getWarnings().get(0);
- assertTrue(err.contains("a.b.c"));
- assertTrue(err.contains("g:f3:1"));
- assertTrue(err.contains("feature-export2"));
+ assertEquals(0, res.getArtifactErrors().size());
+ assertEquals(1, res.getArtifactWarnings().size());
+ assertEquals(0, res.getGlobalWarnings().size());
+ assertEquals(0, res.getGlobalErrors().size());
+
+ ArtifactReport err = res.getArtifactWarnings().get(0);
+ assertEquals(ArtifactId.parse("g:exp2:1"), err.getKey());
+ assertTrue(err.getValue().contains("a.b.c"));
+ assertTrue(err.getValue().contains("g:f3:1"));
+ assertTrue(err.getValue().contains("feature-export2"));
}
@Test
@@ -104,13 +111,16 @@ public class CheckApiRegionsCrossFeatureDupsTest {
Collections.singletonMap("api-regions-crossfeature-dups", cfg);
Analyser a = new Analyser(scanner, configs, at);
AnalyserResult res = a.analyse(f);
- assertEquals(0, res.getErrors().size());
- assertEquals(1, res.getWarnings().size());
-
- String err = res.getWarnings().get(0);
- assertTrue(err.contains("a.b.c"));
- assertTrue(err.contains("g:f3:1"));
- assertTrue(err.contains("feature-export2"));
+ assertEquals(0, res.getArtifactErrors().size());
+ assertEquals(1, res.getArtifactWarnings().size());
+ assertEquals(0, res.getGlobalWarnings().size());
+ assertEquals(0, res.getGlobalErrors().size());
+
+ ArtifactReport err = res.getArtifactWarnings().get(0);
+ assertEquals(ArtifactId.parse("g:exp2:1"), err.getKey());
+ assertTrue(err.getValue().contains("a.b.c"));
+ assertTrue(err.getValue().contains("g:f3:1"));
+ assertTrue(err.getValue().contains("feature-export2"));
}
@Test
@@ -128,8 +138,10 @@ public class CheckApiRegionsCrossFeatureDupsTest {
Collections.singletonMap("api-regions-crossfeature-dups", cfg);
Analyser a = new Analyser(scanner, configs, at);
AnalyserResult res = a.analyse(f);
- assertEquals(0, res.getErrors().size());
- assertEquals(0, res.getWarnings().size());
+ assertEquals(0, res.getArtifactErrors().size());
+ assertEquals(0, res.getArtifactWarnings().size());
+ assertEquals(0, res.getGlobalWarnings().size());
+ assertEquals(0, res.getGlobalErrors().size());
}
@Test
@@ -143,12 +155,18 @@ public class CheckApiRegionsCrossFeatureDupsTest {
AnalyserTask at = new CheckApiRegionsCrossFeatureDups();
Analyser a = new Analyser(scanner, at);
AnalyserResult res = a.analyse(f);
- assertEquals(2, res.getErrors().size());
- assertEquals(0, res.getWarnings().size());
-
- String allErrs = res.getErrors().get(0) + res.getErrors().get(1);
- assertTrue(allErrs.contains("a.b.c"));
- assertTrue(allErrs.contains("zzz.zzz"));
+ assertEquals(2, res.getArtifactErrors().size());
+ assertEquals(0, res.getArtifactWarnings().size());
+ assertEquals(0, res.getGlobalWarnings().size());
+ assertEquals(0, res.getGlobalErrors().size());
+
+ ArtifactReport err1 = res.getArtifactErrors().get(0);
+ assertEquals(ArtifactId.parse("g:extra:1"), err1.getKey());
+ assertTrue(err1.getValue().contains("zzz.zzz"));
+
+ ArtifactReport err2 = res.getArtifactErrors().get(1);
+ assertEquals(ArtifactId.parse("g:exp2:1"), err2.getKey());
+ assertTrue(err2.getValue().contains("a.b.c"));
}
@Test
@@ -165,11 +183,14 @@ public class CheckApiRegionsCrossFeatureDupsTest {
Collections.singletonMap("regions", "something,global"));
Analyser a = new Analyser(scanner, configs, at);
AnalyserResult res = a.analyse(f);
- assertEquals(1, res.getErrors().size());
- assertEquals(0, res.getWarnings().size());
-
- String err = res.getErrors().get(0);
- assertTrue(err.contains("zzz.zzz"));
+ assertEquals(1, res.getArtifactErrors().size());
+ assertEquals(0, res.getArtifactWarnings().size());
+ assertEquals(0, res.getGlobalWarnings().size());
+ assertEquals(0, res.getGlobalErrors().size());
+
+ ArtifactReport err1 = res.getArtifactErrors().get(0);
+ assertEquals(ArtifactId.parse("g:extra:1"), err1.getKey());
+ assertTrue(err1.getValue().contains("zzz.zzz"));
}
private Scanner getScanner() throws IOException {