Document sparseness trick in test

Author: FlorentinD

proc/community/src/test/java/org/neo4j/gds/CommunityProcCompanionTest.java

@@ -164,6 +164,7 @@ void minComponentSizeWithSparseProperties() {
         inputBuilder.set(3, 42);
         var input = inputBuilder.build();
 
+        // we mimic the sparseness here through size > valueStored
         LongNodePropertyValues sparseProperties = new TestSparseNodePropertyValues(4, 3, input::get);
 
         var config = ConfigWithComponentSize.of(CypherMapWrapper.empty().withNumber("minComponentSize", 2L));
@@ -175,10 +176,8 @@ void minComponentSizeWithSparseProperties() {
             () -> { throw new UnsupportedOperationException("Not implemented"); }
         );
 
-        // null from the beginning
         assertThat(filteredProperties.value(0)).isNull();
         assertThat(filteredProperties.value(1)).isEqualTo(Values.longValue(42));
-        // filtered out
         assertThat(filteredProperties.value(2)).isNull();
         assertThat(filteredProperties.value(3)).isEqualTo(Values.longValue(42));
     }
@@ -191,6 +190,7 @@ void consecutiveIdsWithSparseProperties() {
         inputBuilder.set(3, 42);
         var input = inputBuilder.build();
 
+        // we mimic the sparseness here through size > valueStored
         LongNodePropertyValues sparseProperties = new TestSparseNodePropertyValues(4, 3, input::get);
 
         var config = ConfigWithComponentSize.of(CypherMapWrapper.create(Map.of("consecutiveIds", true)));
@@ -202,12 +202,10 @@ void consecutiveIdsWithSparseProperties() {
             () -> { throw new UnsupportedOperationException("Not implemented"); }
         );
 
-        // null from the beginning
-        assertThat(filteredProperties.value(0)).isNull();
-        assertThat(filteredProperties.value(1)).isEqualTo(Values.longValue(0));
-        // filtered out
-        assertThat(filteredProperties.value(2)).isEqualTo(Values.longValue(1));
-        assertThat(filteredProperties.value(3)).isEqualTo(Values.longValue(0));
+        assertThat(filteredProperties.value(0)).isEqualTo(Values.longValue(0));
+        assertThat(filteredProperties.value(1)).isEqualTo(Values.longValue(1));
+        assertThat(filteredProperties.value(2)).isEqualTo(Values.longValue(2));
+        assertThat(filteredProperties.value(3)).isEqualTo(Values.longValue(1));
     }
 
     private static final class TestNodePropertyValues implements LongNodePropertyValues {