AK: Make Vector::remove_all_matching() return removal success

This matches the behavior of other remove_*_matching() functions.

1 files changed, 23 insertions, 0 deletions

diff --git a/Tests/AK/TestVector.cpp b/Tests/AK/TestVector.cpp
index bf3d807431..2289c24639 100644
--- a/Tests/AK/TestVector.cpp
+++ b/Tests/AK/TestVector.cpp
@@ -259,6 +259,29 @@ TEST_CASE(vector_remove)
     EXPECT_EQ(ints[0], 4);
 }
 
+TEST_CASE(remove_all_matching)
+{
+    Vector<int> ints;
+
+    ints.append(1);
+    ints.append(2);
+    ints.append(3);
+    ints.append(4);
+
+    EXPECT_EQ(ints.size(), 4u);
+
+    EXPECT_EQ(ints.remove_all_matching([&](int value) { return value > 2; }), true);
+    EXPECT_EQ(ints.remove_all_matching([&](int) { return false; }), false);
+
+    EXPECT_EQ(ints.size(), 2u);
+
+    EXPECT_EQ(ints.remove_all_matching([&](int) { return true; }), true);
+
+    EXPECT(ints.is_empty());
+
+    EXPECT_EQ(ints.remove_all_matching([&](int) { return true; }), false);
+}
+
 TEST_CASE(nonnullownptrvector)
 {
     struct Object {