From 56df7b3916f669edd315808f92e63e7553367f1d Mon Sep 17 00:00:00 2001
From: Maksim Denisov <denisov.maksim@huawei.com>
Date: Fri, 14 Feb 2025 13:04:28 +0100
Subject: Store HashFunction by value

Although references give an additional information about ownership, they introduce additional design difficulties.
---
 .../execution_engine/executor/executor.test.cpp        | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

(limited to 'test/buildtool/execution_engine/executor/executor.test.cpp')

diff --git a/test/buildtool/execution_engine/executor/executor.test.cpp b/test/buildtool/execution_engine/executor/executor.test.cpp
index c317f80e..891c8198 100644
--- a/test/buildtool/execution_engine/executor/executor.test.cpp
+++ b/test/buildtool/execution_engine/executor/executor.test.cpp
@@ -337,7 +337,7 @@ TEST_CASE("Executor: Process artifact", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -355,7 +355,7 @@ TEST_CASE("Executor: Process artifact", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -373,7 +373,7 @@ TEST_CASE("Executor: Process artifact", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -420,7 +420,7 @@ TEST_CASE("Executor: Process action", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -441,7 +441,7 @@ TEST_CASE("Executor: Process action", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -462,7 +462,7 @@ TEST_CASE("Executor: Process action", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -486,7 +486,7 @@ TEST_CASE("Executor: Process action", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -507,7 +507,7 @@ TEST_CASE("Executor: Process action", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
@@ -531,7 +531,7 @@ TEST_CASE("Executor: Process action", "[executor]") {
         auto api = std::make_shared<TestApi>(config);
         Statistics stats{};
         Progress progress{};
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
         ExecutionContext const exec_context{.repo_config = &repo_config,
                                             .apis = &apis,
                                             .remote_context = &remote_context,
-- 
cgit v1.2.3