naruto-lgtm commented on code in PR #1524:
URL: https://github.com/apache/poi/pull/1524#discussion_r4465358237
##########
poi-scratchpad/src/test/java/org/apache/poi/hslf/usermodel/TestAddingSlides.java:
##########
@@ -248,6 +251,51 @@ void testRemoving() throws IOException {
}
}
+ /**
+ * The records of a removed slide and its notes are not written out again
+ */
+ @Test
+ void removedSlideIsNotWrittenOut() throws IOException {
+ ss_empty.createSlide().createTextBox().setText("Text on the removed
slide");
+ ss_empty.createSlide().createTextBox().setText("Text on the kept
slide");
+ try (HSLFSlideShow ppt =
HSLFTestDataSamples.writeOutAndReadBack(ss_empty)) {
+ ppt.removeSlide(0);
+ String text = getRawText(ppt);
+ assertTrue(text.contains("Text on the kept slide"));
Review Comment:
Switched to assertThat with containsString and not(containsString).
--
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]
