chore: OpManagerTest fixes

romange · romange · commit 6f2925d22280 · 2025-01-20T09:44:18.000+02:00
diff --git a/src/server/tiering/op_manager_test.cc b/src/server/tiering/op_manager_test.cc
@@ -55,7 +55,8 @@ struct OpManagerTest : PoolTestBase, OpManager {
 
   void NotifyStashed(EntryId id, const io::Result<DiskSegment>& segment) override {
     ASSERT_TRUE(segment);
-    stashed_[id] = *segment;
+    auto [it, inserted] = stashed_.emplace(id, *segment);
+    ASSERT_TRUE(inserted) << "Duplicate stash for " << id;
   }
 
   bool NotifyFetched(EntryId id, std::string_view value, DiskSegment segment,

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,8 @@ struct OpManagerTest : PoolTestBase, OpManager {`
`55`	`55`
`56`	`56`	`void NotifyStashed(EntryId id, const io::Result<DiskSegment>& segment) override {`
`57`	`57`	`ASSERT_TRUE(segment);`
`58`		`- stashed_[id] = *segment;`
	`58`	`+ auto [it, inserted] = stashed_.emplace(id, *segment);`
	`59`	`+ ASSERT_TRUE(inserted) << "Duplicate stash for " << id;`
`59`	`60`	`}`
`60`	`61`
`61`	`62`	`bool NotifyFetched(EntryId id, std::string_view value, DiskSegment segment,`