Fix build on some platforms (#1780)

* Add missed std::forward in mst_test_helpers.hpp and transaction_processor_test.cpp * Add missed target linkage for block_loader_test and synchronizer_test Signed-off-by: Kitsu <[email protected]>
hyperledger-iroha · Oct 14, 2018 · 44798fe · 44798fe
1 parent 5f1ac1e
commit 44798fe
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 3 deletions.
diff --git a/test/module/irohad/multi_sig_transactions/mst_test_helpers.hpp b/test/module/irohad/multi_sig_transactions/mst_test_helpers.hpp
@@ -58,7 +58,7 @@ auto addSignatures(Batch &&batch, int tx_number, Signatures... signatures) {
   mst_helpers_log_->info(
       "Number of signatures was inserted {}",
       boost::size(batch->transactions().at(tx_number)->signatures()));
-  return batch;
+  return std::forward<Batch>(batch);
 }
 
 template <typename Batch, typename... KeyPairs>
@@ -79,7 +79,7 @@ auto addSignaturesFromKeyPairs(Batch &&batch,
   // use unused variable
   (void)temp;
 
-  return batch;
+  return std::forward<Batch>(batch);
 }
 
 inline auto makeSignature(const std::string &sign,

diff --git a/test/module/irohad/network/CMakeLists.txt b/test/module/irohad/network/CMakeLists.txt
@@ -20,4 +20,5 @@ target_link_libraries(block_loader_test
     block_loader
     block_loader_service
     shared_model_cryptography
+    shared_model_default_builders
     )
diff --git a/test/module/irohad/synchronizer/CMakeLists.txt b/test/module/irohad/synchronizer/CMakeLists.txt
@@ -4,4 +4,6 @@ target_link_libraries(synchronizer_test
     shared_model_cryptography
     shared_model_proto_backend
     shared_model_stateless_validation
+    shared_model_interfaces_factories
+    shared_model_default_builders
     )
diff --git a/test/module/irohad/torii/processor/transaction_processor_test.cpp b/test/module/irohad/torii/processor/transaction_processor_test.cpp
@@ -90,7 +90,7 @@ class TransactionProcessorTest : public ::testing::Test {
     int temp[] = {(create_signature(std::forward<KeyPairs>(keypairs)), 0)...};
     (void)temp;
 
-    return tx;
+    return std::forward<Transaction>(tx);
   }
 
  protected: