fix(tests): port-allocate tests to avoid conflicts

hqpo · Aug 3, 2022 · 0f8bee5 · 0f8bee5
1 parent f32877f
commit 0f8bee5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/crates/sui-core/src/unit_tests/consensus_adapter_tests.rs b/crates/sui-core/src/unit_tests/consensus_adapter_tests.rs
@@ -41,8 +41,8 @@ async fn listen_to_sequenced_transaction() {
 
 #[tokio::test]
 async fn submit_transaction_to_consensus() {
-    // TODO [issue #932]: Use a port allocator to avoid port conflicts.
-    let consensus_address = "127.0.0.1:12456".parse().unwrap();
+    let port = sui_config::utils::get_available_port();
+    let consensus_address = format!("127.0.0.1:{port}").parse().unwrap();
     let (tx_consensus_listener, mut rx_consensus_listener) = channel(1);
 
     // Initialize an authority with a (owned) gas object and a shared object; then

diff --git a/crates/sui-core/src/unit_tests/consensus_tests.rs b/crates/sui-core/src/unit_tests/consensus_tests.rs
@@ -140,8 +140,8 @@ async fn listen_to_sequenced_transaction() {
 
 #[tokio::test]
 async fn submit_transaction_to_consensus() {
-    // TODO [issue #932]: Use a port allocator to avoid port conflicts.
-    let consensus_address: Multiaddr = "/dns/localhost/tcp/12456/http".parse().unwrap();
+    let port = sui_config::utils::get_available_port();
+    let consensus_address: Multiaddr = format!("/dns/localhost/tcp/{port}/http").parse().unwrap();
     let (tx_consensus_listener, mut rx_consensus_listener) = channel(1);
 
     // Initialize an authority with a (owned) gas object and a shared object; then