From 403105a8ec14f37110a7899172c15d359b4974d7 Mon Sep 17 00:00:00 2001
From: Dimitrios Siganos <dimitris@siganos.org>
Date: Tue, 15 Nov 2022 16:59:36 +0200
Subject: [PATCH] Fix slow_test compilation (#3997)

The slow_test target was broken by the previous commit. This fixes it to at least compile.

* rename request_interval_ms to request_interval_ms
* active_container::roots became private so some slow tests need to become friends of the class to access
---
 nano/node/active_transactions.hpp |  3 +++
 nano/slow_test/node.cpp           | 11 ++++++++++-
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/nano/node/active_transactions.hpp b/nano/node/active_transactions.hpp
index c98ab351..f422d115 100644
--- a/nano/node/active_transactions.hpp
+++ b/nano/node/active_transactions.hpp
@@ -250,6 +250,9 @@ private:
 public: // Tests
 	void clear ();
 
+	friend class node_fork_storm_Test;
+	friend class system_block_sequence_Test;
+	friend class node_mass_block_new_Test;
 	friend class active_transactions_vote_replays_Test;
 	friend class frontiers_confirmation_prioritize_frontiers_Test;
 	friend class frontiers_confirmation_prioritize_frontiers_max_optimistic_elections_Test;
diff --git a/nano/slow_test/node.cpp b/nano/slow_test/node.cpp
index 7e882369..c9bb6f74 100644
--- a/nano/slow_test/node.cpp
+++ b/nano/slow_test/node.cpp
@@ -238,6 +238,8 @@ TEST (store, load)
 	}
 }
 
+namespace nano
+{
 TEST (node, fork_storm)
 {
 	// WIP against issue #3709
@@ -342,6 +344,7 @@ TEST (node, fork_storm)
 	}
 	ASSERT_TRUE (true);
 }
+} // namespace nano
 
 namespace
 {
@@ -1942,13 +1945,15 @@ TEST (node, mass_epoch_upgrader)
 	perform_test (std::numeric_limits<size_t>::max ());
 }
 
+namespace nano
+{
 TEST (node, mass_block_new)
 {
 	nano::test::system system;
 	nano::node_config node_config (nano::test::get_available_port (), system.logging);
 	node_config.frontiers_confirmation = nano::frontiers_confirmation_mode::disabled;
 	auto & node = *system.add_node (node_config);
-	node.network_params.network.request_interval_ms = 500;
+	node.network_params.network.aec_loop_interval_ms = 500;
 
 #ifndef NDEBUG
 	auto const num_blocks = 5000;
@@ -2069,6 +2074,7 @@ TEST (node, mass_block_new)
 	process_all (receive_blocks);
 	std::cout << "Receive blocks time: " << timer.stop ().count () << " " << timer.unit () << "\n\n";
 }
+}
 
 TEST (node, wallet_create_block_confirm_conflicts)
 {
@@ -2125,6 +2131,8 @@ TEST (node, wallet_create_block_confirm_conflicts)
 	}
 }
 
+namespace nano
+{
 /**
  * This test creates a small network of evenly weighted PRs and ensures a sequence of blocks from the genesis account to random accounts are able to be processed
  * Ongoing bootstrap is disabled to directly test election activation. A failure to activate a block on any PR will cause the test to stall
@@ -2228,3 +2236,4 @@ TEST (system, block_sequence)
 		system.poll ();
 	}
 }
+} // namespace nano