quotest: don't fail on past name changes coming in sync

Sync may bring past events in a few batches, it turns out, and quotest
used to choke on name changes from its own past executions.

1 files changed, 2 insertions, 3 deletions

diff --git a/tests/quotest.cpp b/tests/quotest.cpp
index ed5c34b1..afe26f7a 100644
--- a/tests/quotest.cpp
+++ b/tests/quotest.cpp
@@ -499,7 +499,6 @@ TEST_IMPL(changeName)
                      if (localUser != arrivedUser)
                          return false;
 
-                     localUser->rename({});
                      const auto& arrivedNewName = arrivedUser->name(targetRoom);
                      // Old names may diverge e.g. because the original name
                      // hasn't been known to Quotient
@@ -508,8 +507,8 @@ TEST_IMPL(changeName)
 
                      clog << "Names mismatch: found " << newName.toStdString()
                           << " instead of " << arrivedNewName.toStdString()
-                          << endl;
-                     FAIL_TEST();
+                          << "; waiting for the next change" << endl;
+                     return false;
                  });
     return false;
 }