Make sure mailboxes in test actorsystems are in a locked state (#11837)

ydb-platform · Nov 21, 2024 · ea7a15c · ea7a15c
1 parent 923d372
commit ea7a15c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/ydb/core/util/actorsys_test/testactorsys.h b/ydb/core/util/actorsys_test/testactorsys.h
@@ -492,9 +492,12 @@ class TTestActorSystem {
         }
 
         // register actor in mailbox
-        const auto& it = Mailboxes.try_emplace(TMailboxId(nodeId, poolId, mboxId)).first;
+        auto [it, mboxInserted] = Mailboxes.try_emplace(TMailboxId(nodeId, poolId, mboxId));
         TMailboxInfo& mbox = it->second;
         mbox.Hint = mboxId;
+        if (mboxInserted) {
+            mbox.LockFromFree();
+        }
         mbox.AttachActor(ActorLocalId, actor);
 
         // generate actor id

diff --git a/ydb/library/actors/testlib/test_runtime.cpp b/ydb/library/actors/testlib/test_runtime.cpp
@@ -928,7 +928,7 @@ namespace NActors {
         RegistrationObserver(*this, parentId ? parentId : CurrentRecipient, actorId);
         DoActorInit(node->ActorSystem.Get(), actor, actorId, parentId ? parentId : CurrentRecipient);
 
-        mailbox->Unlock(node->ExecutorPools[0], GetCycleCountFast(), revolvingCounter);
+        // Note: test actorsystem mailboxes stay permanently locked
 
         return actorId;
     }