Fixed a remove bug not removing the last item.

Author: Bytekeeper

src/main/java/org/bk/ass/query/PositionQueries.java

@@ -122,7 +122,7 @@ public boolean remove(Object o) {
       for (i = 0; i < replacement.length && !Objects.equals(node.values[i], o); i++) {
         replacement[i] = node.values[i];
       }
-      if (i == replacement.length) return false;
+      if (i == node.values.length) return false;
       for (; i < replacement.length; i++) {
         replacement[i] = node.values[i + 1];
       }

src/test/java/org/bk/ass/query/PositionQueriesTest.java

@@ -241,6 +241,19 @@ void shouldRemoveItem() {
     assertThat(removed).isTrue();
   }
 
+  @Test
+  void shouldRemoveLastItem() {
+    // GIVEN
+    PositionQueries<Position> finder =
+        new PositionQueries<>(Collections.singletonList(new Position(0, 0)), Function.identity());
+
+    // WHEN
+    boolean removed = finder.remove(new Position(0, 0));
+
+    // THEN
+    assertThat(removed).isTrue();
+  }
+
   @Test
   void shouldRemoveAllItems() {
     // GIVEN