Skip to content

Commit eb834e2

Browse files
authored
Do not explicitly provide address for ServerSocket in tests (#1766)
1 parent ccae92d commit eb834e2

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

eclair-core/src/test/scala/fr/acinq/eclair/io/PeerSpec.scala

+3-3
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ import org.scalatest.funsuite.FixtureAnyFunSuiteLike
3737
import org.scalatest.{Outcome, ParallelTestExecution, Tag}
3838
import scodec.bits.ByteVector
3939

40-
import java.net.{InetAddress, ServerSocket, Socket}
40+
import java.net.{ServerSocket, Socket}
4141
import java.util.concurrent.Executors
4242
import scala.concurrent.duration._
4343
import scala.concurrent.{ExecutionContext, Future}
@@ -113,7 +113,7 @@ class PeerSpec extends TestKitBaseClass with FixtureAnyFunSuiteLike with Paralle
113113
system.actorOf(ClientSpawner.props(nodeParams.keyPair, nodeParams.socksProxy_opt, nodeParams.peerConnectionConf, TestProbe().ref, TestProbe().ref))
114114

115115
// we create a dummy tcp server and update bob's announcement to point to it
116-
val mockServer = new ServerSocket(0, 1, InetAddress.getLocalHost) // port will be assigned automatically
116+
val mockServer = new ServerSocket(0, 1) // port will be assigned automatically
117117
val mockAddress = HostAndPort.fromParts(mockServer.getInetAddress.getHostAddress, mockServer.getLocalPort)
118118

119119
val probe = TestProbe()
@@ -139,7 +139,7 @@ class PeerSpec extends TestKitBaseClass with FixtureAnyFunSuiteLike with Paralle
139139
system.actorOf(ClientSpawner.props(nodeParams.keyPair, nodeParams.socksProxy_opt, nodeParams.peerConnectionConf, TestProbe().ref, TestProbe().ref))
140140

141141
// we create a dummy tcp server and update bob's announcement to point to it
142-
val mockServer = new ServerSocket(0, 1, InetAddress.getLocalHost) // port will be assigned automatically
142+
val mockServer = new ServerSocket(0, 1) // port will be assigned automatically
143143
val mockAddress = NodeAddress.fromParts(mockServer.getInetAddress.getHostAddress, mockServer.getLocalPort).get
144144

145145
// we put the server address in the node db

eclair-core/src/test/scala/fr/acinq/eclair/io/ReconnectionTaskSpec.scala

+2-2
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ import fr.acinq.eclair.wire.protocol.{Color, NodeAddress, NodeAnnouncement}
2525
import org.scalatest.funsuite.FixtureAnyFunSuiteLike
2626
import org.scalatest.{Outcome, ParallelTestExecution, Tag}
2727

28-
import java.net.{InetAddress, ServerSocket}
28+
import java.net.ServerSocket
2929
import scala.concurrent.duration._
3030

3131
class ReconnectionTaskSpec extends TestKitBaseClass with FixtureAnyFunSuiteLike with ParallelTestExecution {
@@ -222,7 +222,7 @@ class ReconnectionTaskSpec extends TestKitBaseClass with FixtureAnyFunSuiteLike
222222
import f._
223223

224224
// we create a dummy tcp server and update bob's announcement to point to it
225-
val mockServer = new ServerSocket(0, 1, InetAddress.getLocalHost) // port will be assigned automatically
225+
val mockServer = new ServerSocket(0, 1) // port will be assigned automatically
226226
val mockAddress = NodeAddress.fromParts(mockServer.getInetAddress.getHostAddress, mockServer.getLocalPort).get
227227
val bobAnnouncement = NodeAnnouncement(randomBytes64, Features.empty, 1, remoteNodeId, Color(100.toByte, 200.toByte, 300.toByte), "node-alias", mockAddress :: Nil)
228228
nodeParams.db.network.addNode(bobAnnouncement)

0 commit comments

Comments
 (0)