0

[PDF Ink Signatures] Re-enable PdfInkUndoRedoModelTest.Stress

Fix the test by deleting a stray StartErase() call.

Bug: 335521182
Change-Id: Icbeb40280562aeee74233a47e0a6618b5d2cdcbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5871263
Reviewed-by: Andy Phan <andyphan@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1357736}
This commit is contained in:
Lei Zhang
2024-09-19 18:20:20 +00:00
committed by Chromium LUCI CQ
parent a1675ad205
commit 4821f48f5a

@ -329,8 +329,7 @@ TEST(PdfInkUndoRedoModelTest, DrawDrawUndoEraseUndo) {
ElementsAreArray({5}));
}
// TODO(crbug.com/335521182): Figure out why this times out on bots and enable.
TEST(PdfInkUndoRedoModelTest, DISABLED_Stress) {
TEST(PdfInkUndoRedoModelTest, Stress) {
constexpr size_t kCycles = 10000;
PdfInkUndoRedoModel undo_redo;
size_t id = 0;
@ -340,7 +339,6 @@ TEST(PdfInkUndoRedoModelTest, DISABLED_Stress) {
}
ASSERT_EQ(2 * kCycles, id);
ASSERT_TRUE(undo_redo.StartErase());
for (size_t i = 0; i < kCycles; ++i) {
std::optional<DiscardedDrawCommands> discards = undo_redo.StartErase();
ASSERT_THAT(discards, Optional(DiscardedDrawCommands()));