pjfanning commented on code in PR #999:
URL: https://github.com/apache/poi/pull/999#discussion_r2701064312
##########
poi-ooxml/src/test/java/org/apache/poi/xslf/TestXSLFBugs.java:
##########
@@ -236,7 +236,7 @@ void bug61589() throws IOException {
try (XMLSlideShow ppt3 = writeOutAndReadBack(dest)) {
XSLFSlide slide3 = ppt3.getSlides().get(0);
XSLFTextBox shape3 = (XSLFTextBox) slide3.getShapes().get(0);
- XSLFTextParagraph p3 = shape3.getTextParagraphs().get(1);
+ XSLFTextParagraph p3 = shape3.getTextParagraphs().get(2);
Review Comment:
Why is there is an extra paragraph here? the test doesn't seem to manipulate
the text.
If we are going to modify the behaviour, we should check the 0 and 1
paragraphs to see that we don't duplicate the paragraphs by accident.
I'm not convinced that the other test changes are valid either. It seems
like we are hacking the tests instead of improving the tests.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
