Store HashFunction by value

Although references give an additional information about ownership, they introduce additional design difficulties.

4 files changed, 52 insertions, 53 deletions

diff --git a/test/buildtool/execution_engine/executor/TARGETS b/test/buildtool/execution_engine/executor/TARGETS
index f9faa576..74d0329b 100644
--- a/test/buildtool/execution_engine/executor/TARGETS
+++ b/test/buildtool/execution_engine/executor/TARGETS
@@ -95,7 +95,6 @@
   , "private-deps":
     [ "executor_api_tests"
     , ["@", "catch2", "", "catch2"]
-    , ["@", "gsl", "", "gsl"]
     , ["@", "src", "src/buildtool/auth", "auth"]
     , ["@", "src", "src/buildtool/common", "common"]
     , ["@", "src", "src/buildtool/common", "config"]
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,
diff --git a/test/buildtool/execution_engine/executor/executor_api.test.hpp b/test/buildtool/execution_engine/executor/executor_api.test.hpp
index 4ba486c6..c5a2126a 100644
--- a/test/buildtool/execution_engine/executor/executor_api.test.hpp
+++ b/test/buildtool/execution_engine/executor/executor_api.test.hpp
@@ -157,7 +157,7 @@ static inline void RunHelloWorldCompilation(
     HashFunction const hash_function{TestHashType::ReadFromEnvironment()};
 
     auto api = factory();
-    auto const apis = CreateTestApiBundle(&hash_function, api);
+    auto const apis = CreateTestApiBundle(hash_function, api);
 
     ExecutionContext const exec_context{.repo_config = repo_config,
                                         .apis = &apis,
@@ -290,7 +290,7 @@ static inline void RunGreeterCompilation(
     HashFunction const hash_function{TestHashType::ReadFromEnvironment()};
 
     auto api = factory();
-    auto const apis = CreateTestApiBundle(&hash_function, api);
+    auto const apis = CreateTestApiBundle(hash_function, api);
 
     ExecutionContext const exec_context{.repo_config = repo_config,
                                         .apis = &apis,
@@ -458,7 +458,7 @@ static inline void TestUploadAndDownloadTrees(
                                        .retry_config = &retry_config,
                                        .exec_config = &*remote_config};
 
-    auto const apis = CreateTestApiBundle(&hash_function, api);
+    auto const apis = CreateTestApiBundle(hash_function, api);
 
     ExecutionContext const exec_context{.repo_config = repo_config,
                                         .apis = &apis,
@@ -632,7 +632,7 @@ static inline void TestRetrieveOutputDirectories(
 
         // run action
         auto api = factory();
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
 
         ExecutionContext const exec_context{.repo_config = repo_config,
                                             .apis = &apis,
@@ -686,7 +686,7 @@ static inline void TestRetrieveOutputDirectories(
 
         // run action
         auto api = factory();
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
 
         ExecutionContext const exec_context{.repo_config = repo_config,
                                             .apis = &apis,
@@ -757,7 +757,7 @@ static inline void TestRetrieveOutputDirectories(
 
         // run action
         auto api = factory();
-        auto const apis = CreateTestApiBundle(&hash_function, api);
+        auto const apis = CreateTestApiBundle(hash_function, api);
 
         ExecutionContext const exec_context{.repo_config = repo_config,
                                             .apis = &apis,
@@ -830,7 +830,7 @@ static inline void TestRetrieveOutputDirectories(
 
             // run action
             auto api = factory();
-            auto const apis = CreateTestApiBundle(&hash_function, api);
+            auto const apis = CreateTestApiBundle(hash_function, api);
 
             ExecutionContext const exec_context{
                 .repo_config = repo_config,
@@ -857,7 +857,7 @@ static inline void TestRetrieveOutputDirectories(
 
             // run action
             auto api = factory();
-            auto const apis = CreateTestApiBundle(&hash_function, api);
+            auto const apis = CreateTestApiBundle(hash_function, api);
 
             ExecutionContext const exec_context{
                 .repo_config = repo_config,
diff --git a/test/buildtool/execution_engine/executor/executor_api_remote_bazel.test.cpp b/test/buildtool/execution_engine/executor/executor_api_remote_bazel.test.cpp
index a9fd6c10..a809d04f 100644
--- a/test/buildtool/execution_engine/executor/executor_api_remote_bazel.test.cpp
+++ b/test/buildtool/execution_engine/executor/executor_api_remote_bazel.test.cpp
@@ -12,10 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+#include <memory>
 #include <optional>
 
 #include "catch2/catch_test_macros.hpp"
-#include "gsl/gsl"
 #include "src/buildtool/auth/authentication.hpp"
 #include "src/buildtool/common/remote/remote_common.hpp"
 #include "src/buildtool/common/remote/retry_config.hpp"
@@ -47,13 +47,13 @@ TEST_CASE("Executor<BazelApi>: Upload blob", "[executor]") {
     HashFunction const hash_function{TestHashType::ReadFromEnvironment()};
 
     TestBlobUpload(&repo_config, [&] {
-        return BazelApi::Ptr{new BazelApi{"remote-execution",
+        return std::make_shared<BazelApi>("remote-execution",
                                           remote_config->remote_address->host,
                                           remote_config->remote_address->port,
                                           &*auth_config,
                                           &retry_config,
                                           config,
-                                          &hash_function}};
+                                          hash_function);
     });
 }
 
@@ -80,14 +80,14 @@ TEST_CASE("Executor<BazelApi>: Compile hello world", "[executor]") {
         &stats,
         &progress,
         [&] {
-            return BazelApi::Ptr{
-                new BazelApi{"remote-execution",
-                             remote_config->remote_address->host,
-                             remote_config->remote_address->port,
-                             &*auth_config,
-                             &retry_config,
-                             config,
-                             &hash_function}};
+            return std::make_shared<BazelApi>(
+                "remote-execution",
+                remote_config->remote_address->host,
+                remote_config->remote_address->port,
+                &*auth_config,
+                &retry_config,
+                config,
+                hash_function);
         },
         &*auth_config,
         false /* not hermetic */);
@@ -116,14 +116,14 @@ TEST_CASE("Executor<BazelApi>: Compile greeter", "[executor]") {
         &stats,
         &progress,
         [&] {
-            return BazelApi::Ptr{
-                new BazelApi{"remote-execution",
-                             remote_config->remote_address->host,
-                             remote_config->remote_address->port,
-                             &*auth_config,
-                             &retry_config,
-                             config,
-                             &hash_function}};
+            return std::make_shared<BazelApi>(
+                "remote-execution",
+                remote_config->remote_address->host,
+                remote_config->remote_address->port,
+                &*auth_config,
+                &retry_config,
+                config,
+                hash_function);
         },
         &*auth_config,
         false /* not hermetic */);
@@ -152,14 +152,14 @@ TEST_CASE("Executor<BazelApi>: Upload and download trees", "[executor]") {
         &stats,
         &progress,
         [&] {
-            return BazelApi::Ptr{
-                new BazelApi{"remote-execution",
-                             remote_config->remote_address->host,
-                             remote_config->remote_address->port,
-                             &*auth_config,
-                             &retry_config,
-                             config,
-                             &hash_function}};
+            return std::make_shared<BazelApi>(
+                "remote-execution",
+                remote_config->remote_address->host,
+                remote_config->remote_address->port,
+                &*auth_config,
+                &retry_config,
+                config,
+                hash_function);
         },
         &*auth_config,
         false /* not hermetic */);
@@ -188,14 +188,14 @@ TEST_CASE("Executor<BazelApi>: Retrieve output directories", "[executor]") {
         &stats,
         &progress,
         [&] {
-            return BazelApi::Ptr{
-                new BazelApi{"remote-execution",
-                             remote_config->remote_address->host,
-                             remote_config->remote_address->port,
-                             &*auth_config,
-                             &retry_config,
-                             config,
-                             &hash_function}};
+            return std::make_shared<BazelApi>(
+                "remote-execution",
+                remote_config->remote_address->host,
+                remote_config->remote_address->port,
+                &*auth_config,
+                &retry_config,
+                config,
+                hash_function);
         },
         &*auth_config,
         false /* not hermetic */);