From 4a5d51d7bc2b1b23a96c347b6127ec64d7c298b2 Mon Sep 17 00:00:00 2001
From: Kitsune Ral <Kitsune-Ral@users.sf.net>
Date: Tue, 4 Aug 2020 13:50:09 +0200
Subject: 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.
---
 tests/quotest.cpp | 5 ++---
 1 file 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;
 }
-- 
cgit v1.2.3