build fix

Signed-off-by: iceseer <[email protected]>
qdrvm · Apr 1, 2024 · 7bdfca4 · 7bdfca4
1 parent 70cfba1
commit 7bdfca4
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 37 deletions.
diff --git a/core/network/impl/peer_manager_impl.cpp b/core/network/impl/peer_manager_impl.cpp
@@ -735,30 +735,29 @@ namespace kagome::network {
 
       log_->trace("Try to open outgoing validation protocol.(peer={})",
                   peer_info.id);
-      openOutgoing(
-          stream_engine_,
-          validation_protocol,
-          peer_info,
-          [validation_protocol, peer_info, wptr{weak_from_this()}](
-              outcome::result<std::shared_ptr<Stream>> stream_result) {
-            auto self = wptr.lock();
-            if (not self) {
-              return;
-            }
-
-            auto &peer_id = peer_info.id;
-            if (!stream_result.has_value()) {
-              SL_TRACE(self->log_,
-                       "Unable to create stream {} with {}: {}",
-                       validation_protocol->protocolName(),
-                       peer_id,
-                       stream_result.error().message());
-              return;
-            }
-
-            self->stream_engine_->addOutgoing(stream_result.value(),
-                                              validation_protocol);
-          });
+      openOutgoing(stream_engine_,
+                   validation_protocol,
+                   peer_info,
+                   [validation_protocol, peer_info, wptr{weak_from_this()}](
+                       outcome::result<std::shared_ptr<Stream>> stream_result) {
+                     auto self = wptr.lock();
+                     if (not self) {
+                       return;
+                     }
+
+                     auto &peer_id = peer_info.id;
+                     if (!stream_result.has_value()) {
+                       SL_TRACE(self->log_,
+                                "Unable to create stream {} with {}: {}",
+                                validation_protocol->protocolName(),
+                                peer_id,
+                                stream_result.error().message());
+                       return;
+                     }
+
+                     self->stream_engine_->addOutgoing(stream_result.value(),
+                                                       validation_protocol);
+                   });
     }
   }
 
@@ -837,10 +836,11 @@ namespace kagome::network {
   }
 
   void PeerManagerImpl::reserveStatusStreams(const PeerId &peer_id) const {
-    if (auto ps = getPeerState(peer_id); ps && ps->get().roles.flags.authority) {
+    if (auto ps = getPeerState(peer_id);
+        ps && ps->get().roles.flags.authority) {
       auto proto_val_vstaging = router_->getValidationProtocolVStaging();
       BOOST_ASSERT_MSG(proto_val_vstaging,
-                      "Router did not provide validation protocol vstaging");
+                       "Router did not provide validation protocol vstaging");
 
       stream_engine_->reserveStreams(peer_id, proto_val_vstaging);
     }

diff --git a/core/runtime/common/runtime_instances_pool.hpp b/core/runtime/common/runtime_instances_pool.hpp
@@ -14,7 +14,6 @@
 #include <shared_mutex>
 #include <unordered_set>
 
-#include "injector/inject.hpp"
 #include "runtime/common/stack_limiter.hpp"
 #include "runtime/module_factory.hpp"
 #include "utils/lru.hpp"
@@ -34,10 +33,6 @@ namespace kagome::runtime {
         std::shared_ptr<InstrumentWasm> instrument,
         size_t capacity = DEFAULT_MODULES_CACHE_SIZE);
 
-    explicit RuntimeInstancesPoolImpl(
-        Inject, std::shared_ptr<ModuleFactory> module_factory, ...)
-        : RuntimeInstancesPoolImpl{std::move(module_factory)} {}
-
     outcome::result<std::shared_ptr<ModuleInstance>> instantiateFromCode(
         const CodeHash &code_hash,
         common::BufferView code_zstd,

diff --git a/core/runtime/module_factory.hpp b/core/runtime/module_factory.hpp
@@ -40,10 +40,6 @@ namespace kagome::runtime {
 
     virtual outcome::result<std::shared_ptr<Module>, CompilationError> make(
         common::BufferView code) const = 0;
-
-    virtual bool testDontInstrument() const {
-      return false;
-    }
   };
 
 }  // namespace kagome::runtime
diff --git a/test/mock/core/network/peer_manager_mock.hpp b/test/mock/core/network/peer_manager_mock.hpp
@@ -61,6 +61,11 @@ namespace kagome::network {
                 (const PeerId &),
                 (override));
 
+    MOCK_METHOD(std::optional<std::reference_wrapper<const PeerState>>,
+                getPeerState,
+                (const PeerId &),
+                (const, override));
+
     MOCK_METHOD(size_t, activePeersNumber, (), (const, override));
 
     MOCK_METHOD(void,

diff --git a/test/mock/core/runtime/module_factory_mock.hpp b/test/mock/core/runtime/module_factory_mock.hpp
@@ -20,7 +20,5 @@ namespace kagome::runtime {
                 make,
                 (common::BufferView),
                 (const, override));
-
-    MOCK_METHOD(bool, testDontInstrument, (), (const, override));
   };
 }  // namespace kagome::runtime