SLING-10477 : Review usage of UserEntryHandler in RecollectorVaultPackageScanner (test documentation)
diff --git a/src/test/java/org/apache/sling/feature/cpconverter/ConverterUserAndPermissionTest.java b/src/test/java/org/apache/sling/feature/cpconverter/ConverterUserAndPermissionTest.java
index 81a86aa..8355e13 100644
--- a/src/test/java/org/apache/sling/feature/cpconverter/ConverterUserAndPermissionTest.java
+++ b/src/test/java/org/apache/sling/feature/cpconverter/ConverterUserAndPermissionTest.java
@@ -135,6 +135,15 @@
}
}
+ /**
+ * "demo-cp3.zip" contains the same content as "demo-cp.zip" but with altered order leading to ACE being read before
+ * the corresponding system-user, whose principal is referenced in the ACE.
+ *
+ * This test would fail if user/group information was not collected during the first pass (i.e. corresponding
+ * handlers listed in {@link org.apache.sling.feature.cpconverter.vltpkg.RecollectorVaultPackageScanner}.
+ *
+ * @throws Exception
+ */
@Test
public void testConvertPackageWithUsersGroupsAndServiceUsersRepPolicyFirst() throws Exception {
URL packageUrl = getClass().getResource("demo-cp3.zip");
@@ -159,7 +168,13 @@
deleteDirTree(outputDirectory);
}
}
-
+
+ /**
+ * "demo-cp2.zip" contains the same content as "demo-cp.zip" except for the application content below /apps.
+ * The content package therefore gets type CONTENT assigned (and not MIXED).
+ *
+ * @throws Exception
+ */
@Test
public void testConvertCONTENTPackageWithUsersGroupsAndServiceUsers() throws Exception {
URL packageUrl = getClass().getResource("demo-cp2.zip");