fix: remove hard-coded TestBackend implementation

Signed-off-by: Minh Huy Tran <huy@perun.network>

Author: NhoxxKienn

wire/account.go

@@ -40,8 +40,6 @@ type Account interface {
 	Sign(msg []byte) ([]byte, error)
 }
 
-const testBackendID = 0
-
 var _ Msg = (*AuthResponseMsg)(nil)
 
 // AuthResponseMsg is the response message in the peer authentication protocol.

wire/net/simple/address.go

@@ -28,9 +28,6 @@ import (
 	"perun.network/go-perun/wire"
 )
 
-// testBackendID is the identifier for the simulated Backend.
-const testBackendID = 0
-
 // Address is a wire address.
 type Address struct {
 	Name      string
@@ -208,18 +205,22 @@ func NewRandomAddress(rng *rand.Rand) *Address {
 }
 
 // NewRandomAddresses returns a new random peer address.
-func NewRandomAddresses(rng *rand.Rand) map[wallet.BackendID]wire.Address {
+func NewRandomAddresses(rng *rand.Rand, backendID []wallet.BackendID) map[wallet.BackendID]wire.Address {
 	const addrLen = 32
 	l := rng.Intn(addrLen)
 	d := make([]byte, l)
 	if _, err := rng.Read(d); err != nil {
 		panic(err)
 	}
 
-	a := Address{
-		Name: string(d),
+	addresses := make(map[wallet.BackendID]wire.Address)
+	for _, id := range backendID {
+		a := Address{
+			Name: string(d),
+		}
+		addresses[id] = &a
 	}
-	return map[wallet.BackendID]wire.Address{testBackendID: &a}
+	return addresses
 }
 
 // Verify verifies a message signature.

wire/net/simple/dialer_internal_test.go

@@ -139,7 +139,7 @@ func TestDialer_Dial(t *testing.T) {
 	})
 
 	t.Run("unknown host", func(t *testing.T) {
-		noHostAddr := NewRandomAddresses(rng)
+		noHostAddr := NewRandomAddresses(rng, []wallet.BackendID{wiretest.TestBackendID})
 		d.Register(noHostAddr, "no such host")
 
 		ctxtest.AssertTerminates(t, timeout, func() {
@@ -151,7 +151,7 @@ func TestDialer_Dial(t *testing.T) {
 
 	t.Run("unknown address", func(t *testing.T) {
 		ctxtest.AssertTerminates(t, timeout, func() {
-			unkownAddr := NewRandomAddresses(rng)
+			unkownAddr := NewRandomAddresses(rng, []wallet.BackendID{wiretest.TestBackendID})
 			conn, err := d.Dial(context.Background(), unkownAddr, ser)
 			assert.Error(t, err)
 			assert.Nil(t, conn)

wire/net/simple/simple_exchange_addr_test.go

@@ -27,6 +27,7 @@ import (
 	"time"
 
 	"perun.network/go-perun/channel"
+	"perun.network/go-perun/wallet"
 
 	"github.com/stretchr/testify/assert"
 
@@ -107,8 +108,8 @@ func newPipeConnPair() (a wirenet.Conn, b wirenet.Conn) {
 // recipient generated using randomness from rng.
 func newRandomEnvelope(rng *rand.Rand, m wire.Msg) *wire.Envelope {
 	return &wire.Envelope{
-		Sender:    NewRandomAddresses(rng),
-		Recipient: NewRandomAddresses(rng),
+		Sender:    NewRandomAddresses(rng, []wallet.BackendID{wiretest.TestBackendID}),
+		Recipient: NewRandomAddresses(rng, []wallet.BackendID{wiretest.TestBackendID}),
 		Msg:       m,
 	}
 }