Copilot commented on code in PR #395:
URL:
https://github.com/apache/maven-shared-utils/pull/395#discussion_r3644885403
##########
src/test/java/org/apache/maven/shared/utils/xml/pull/Xpp3DomTest.java:
##########
@@ -42,6 +42,22 @@ private Xpp3Dom createElement(String element, String value) {
return t1s1;
}
+ @Test
+ public void removeChildByIndexUpdatesChildMap() {
+ Xpp3Dom parent = new Xpp3Dom("parent");
+ Xpp3Dom first = new Xpp3Dom("child");
+ first.setValue("first");
+ Xpp3Dom second = new Xpp3Dom("child");
+ second.setValue("second");
+ parent.addChild(first);
+ parent.addChild(second);
+
+ parent.removeChild(1);
+
+ assertEquals(1, parent.getChildCount());
+ assertEquals("first", parent.getChild("child").getValue());
+ }
+
Review Comment:
Lines contain trailing whitespace (including an all-whitespace blank line).
This can create noisy diffs and may violate formatting/lint rules in some build
setups; please trim it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
