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");