Pass IExecutionApi to IExecutionApi by reference

...instead of not_null const ptr.

1 files changed, 2 insertions, 3 deletions

diff --git a/test/buildtool/execution_engine/executor/executor.test.cpp b/test/buildtool/execution_engine/executor/executor.test.cpp
index 9b1c8263..26fd9b7d 100644
--- a/test/buildtool/execution_engine/executor/executor.test.cpp
+++ b/test/buildtool/execution_engine/executor/executor.test.cpp
@@ -149,7 +149,7 @@ class TestApi : public IExecutionApi {
     [[nodiscard]] auto RetrieveToPaths(
         std::vector<Artifact::ObjectInfo> const& /*unused*/,
         std::vector<std::filesystem::path> const& /*unused*/,
-        std::optional<gsl::not_null<IExecutionApi*>> const& /* unused */)
+        std::optional<gsl::not_null<const IExecutionApi*>> const& /* unused */)
         const noexcept -> bool final {
         return false;  // not needed by Executor
     }
@@ -161,8 +161,7 @@ class TestApi : public IExecutionApi {
     }
     [[nodiscard]] auto RetrieveToCas(
         std::vector<Artifact::ObjectInfo> const& unused,
-        gsl::not_null<IExecutionApi*> const& /*unused*/) const noexcept
-        -> bool final {
+        IExecutionApi const& /*unused*/) const noexcept -> bool final {
         // Note that a false-positive "free-nonheap-object" warning is thrown by
         // gcc 12.2 with GNU libstdc++, if the caller passes a temporary vector
         // that is not used by this function. Therefore, we explicitly use this