From f1b92cf45f0ea07f22965ab6d9bca5fcd97e03a8 Mon Sep 17 00:00:00 2001
From: Maksim Denisov <denisov.maksim@huawei.com>
Date: Thu, 20 Jun 2024 16:30:38 +0200
Subject: Pass IExecutionApi to IExecutionApi by reference

...instead of not_null const ptr.
---
 test/buildtool/execution_engine/executor/executor.test.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 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 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
-- 
cgit v1.2.3