The snapshot has 2 lightweight members only and hence does not need another indirection.

1 files changed, 3 insertions, 3 deletions

diff --git a/searchcore/src/tests/proton/persistenceengine/persistence_handler_map/persistence_handler_map_test.cpp b/searchcore/src/tests/proton/persistenceengine/persistence_handler_map/persistence_handler_map_test.cpp
index ef12b694187..941628891fe 100644
--- a/searchcore/src/tests/proton/persistenceengine/persistence_handler_map/persistence_handler_map_test.cpp
+++ b/searchcore/src/tests/proton/persistenceengine/persistence_handler_map/persistence_handler_map_test.cpp
@@ -59,10 +59,10 @@ assertNullHandler(const IPersistenceHandler::SP &handler)
 }
 
 void
-assertSnapshot(const std::vector<IPersistenceHandler::SP> &exp, const HandlerSnapshot::UP &snapshot)
+assertSnapshot(const std::vector<IPersistenceHandler::SP> &exp, HandlerSnapshot snapshot)
 {
-    EXPECT_EQUAL(exp.size(), snapshot->size());
-    auto &sequence = snapshot->handlers();
+    EXPECT_EQUAL(exp.size(), snapshot.size());
+    auto &sequence = snapshot.handlers();
     for (size_t i = 0; i < exp.size() && sequence.valid(); ++i, sequence.next()) {
         EXPECT_EQUAL(exp[i].get(), sequence.get());
     }