Skip to content

Commit 69a5707

Browse files
authored
Add const to the condition function (#8644)
1 parent 4a4dbe2 commit 69a5707

File tree

6 files changed

+9
-9
lines changed

6 files changed

+9
-9
lines changed

ydb/core/testlib/actors/block_events.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ namespace NActors {
1414
template<class TEvType>
1515
class TBlockEvents : public std::deque<typename TEvType::TPtr> {
1616
public:
17-
TBlockEvents(TTestActorRuntime& runtime, std::function<bool(typename TEvType::TPtr&)> condition = {})
17+
TBlockEvents(TTestActorRuntime& runtime, std::function<bool(const typename TEvType::TPtr&)> condition = {})
1818
: Runtime(runtime)
1919
, Condition(std::move(condition))
2020
, Holder(Runtime.AddObserver<TEvType>(

ydb/core/testlib/actors/test_runtime_ut.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -794,7 +794,7 @@ Y_UNIT_TEST_SUITE(TActorTest) {
794794
auto source = runtime.Register(new TSourceActor(target), /* nodeIdx */ 1);
795795
runtime.EnableScheduleForActor(source);
796796

797-
TBlockEvents<TEvTrigger> block(runtime, [&](TEvTrigger::TPtr& ev){ return ev->GetRecipientRewrite() == target; });
797+
TBlockEvents<TEvTrigger> block(runtime, [&](const TEvTrigger::TPtr& ev){ return ev->GetRecipientRewrite() == target; });
798798
runtime.WaitFor("blocked 3 events", [&]{ return block.size() >= 3; });
799799
UNIT_ASSERT_VALUES_EQUAL(block.size(), 3u);
800800
UNIT_ASSERT_VALUES_EQUAL(values.size(), 0u);

ydb/core/tx/datashard/datashard_ut_build_index.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ Y_UNIT_TEST_SUITE(TTxDataShardBuildIndexScan) {
167167

168168
CreateShardedTableForIndex(server, sender, "/Root", "table-2", 1, false);
169169

170-
TBlockEvents<TEvDataShard::TEvCompactBorrowed> block(runtime, [&](TEvDataShard::TEvCompactBorrowed::TPtr& event) {
170+
TBlockEvents<TEvDataShard::TEvCompactBorrowed> block(runtime, [&](const TEvDataShard::TEvCompactBorrowed::TPtr& event) {
171171
return runtime.FindActorName(event->Sender) == "FLAT_SCHEMESHARD_ACTOR";
172172
});
173173

ydb/core/tx/datashard/datashard_ut_stats.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -443,7 +443,7 @@ Y_UNIT_TEST_SUITE(DataShardStats) {
443443

444444
UpsertRows(server, sender);
445445

446-
TBlockEvents<NSharedCache::TEvResult> block(runtime, [&](NSharedCache::TEvResult::TPtr& event) {
446+
TBlockEvents<NSharedCache::TEvResult> block(runtime, [&](const NSharedCache::TEvResult::TPtr& event) {
447447
return runtime.FindActorName(event->GetRecipientRewrite()) == "DATASHARD_STATS_BUILDER";
448448
});
449449

ydb/core/tx/mediator/mediator_ut.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -685,7 +685,7 @@ Y_UNIT_TEST_SUITE(MediatorTest) {
685685

686686
// Block plan steps at tablet 1
687687
TBlockEvents<TEvTxProcessing::TEvPlanStep> blockedPlan1(runtime,
688-
[&](TEvTxProcessing::TEvPlanStep::TPtr& ev) {
688+
[&](const TEvTxProcessing::TEvPlanStep::TPtr& ev) {
689689
return ev->Get()->Record.GetTabletID() == tablet1;
690690
});
691691

@@ -943,12 +943,12 @@ Y_UNIT_TEST_SUITE(MediatorTest) {
943943
WatcherState->Updates.clear();
944944

945945
TBlockEvents<TEvTxProcessing::TEvPlanStep> blockedPlan1(runtime,
946-
[&](TEvTxProcessing::TEvPlanStep::TPtr& ev) {
946+
[&](const TEvTxProcessing::TEvPlanStep::TPtr& ev) {
947947
return ev->Get()->Record.GetTabletID() == tablet1;
948948
});
949949

950950
TBlockEvents<TEvTxProcessing::TEvPlanStep> blockedPlan2(runtime,
951-
[&](TEvTxProcessing::TEvPlanStep::TPtr& ev) {
951+
[&](const TEvTxProcessing::TEvPlanStep::TPtr& ev) {
952952
return ev->Get()->Record.GetTabletID() == tablet2;
953953
});
954954

ydb/core/tx/time_cast/time_cast_ut.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -228,9 +228,9 @@ namespace NKikimr::NMediatorTimeCastTest {
228228

229229
TBlockEvents<TEvTxProcessing::TEvPlanStepAck> blockAcks(runtime);
230230
TBlockEvents<TEvTxProcessing::TEvPlanStep> blockPlan1(runtime,
231-
[&](auto& ev) { return ev->Get()->Record.GetTabletID() == tablet1; });
231+
[&](const auto& ev) { return ev->Get()->Record.GetTabletID() == tablet1; });
232232
TBlockEvents<TEvTxProcessing::TEvPlanStep> blockPlan2(runtime,
233-
[&](auto& ev) { return ev->Get()->Record.GetTabletID() == tablet2; });
233+
[&](const auto& ev) { return ev->Get()->Record.GetTabletID() == tablet2; });
234234

235235
ui64 stepTx1 = PlanTx(server, 1, { tablet1, tablet2 });
236236
Cerr << "... tx1 planned at step " << stepTx1 << Endl;

0 commit comments

Comments
 (0)