extern/netbird
1
0
Fork 0
mirror of https://github.com/netbirdio/netbird.git synced 2025-08-22 20:21:04 +02:00
Code Issues Packages Projects Releases Wiki Activity

Revert error handling behaviour in test to support auto net.Conn creation

Browse Source
This commit is contained in:
Zoltan Papp
2025-07-16 21:16:07 +02:00
parent 04619fe54f
commit fcb06d1a23
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
relay/client/client_test.go
View File

@@ -314,8 +314,8 @@ func TestBindToUnavailabePeer(t *testing.T) {
t.Errorf("failed to connect to server: %s", err) t.Errorf("failed to connect to server: %s", err)
} }
_, err = clientAlice.OpenConn(ctx, "bob") _, err = clientAlice.OpenConn(ctx, "bob")
if err == nil { if err != nil {
t.Errorf("expected error when binding to unavailable peer, got nil") t.Errorf("failed to open bob: %s", err)
} }
log.Infof("closing client") log.Infof("closing client")

8
relay/client/manager_test.go
View File

@@ -285,15 +285,15 @@ func TestForeginAutoClose(t *testing.T) {
} }
t.Log("open connection to another peer") t.Log("open connection to another peer")
if _, err = mgr.OpenConn(ctx, toURL(srvCfg2)[0], "anotherpeer"); err == nil { if _, err = mgr.OpenConn(ctx, toURL(srvCfg2)[0], "anotherpeer"); err != nil {
t.Fatalf("should have failed to open connection to another peer") t.Fatalf("failed to open connection: %s", err)
} }
timeout := relayCleanupInterval + keepUnusedServerTime + 1*time.Second timeout := relayCleanupInterval + keepUnusedServerTime + 1*time.Second
t.Logf("waiting for relay cleanup: %s", timeout) t.Logf("waiting for relay cleanup: %s", timeout)
time.Sleep(timeout) time.Sleep(timeout)
if len(mgr.relayClients) != 0 { if len(mgr.relayClients) != 1 {
t.Errorf("expected 0, got %d", len(mgr.relayClients)) t.Errorf("expected 1 relay client, got %d", len(mgr.relayClients))
} }
t.Logf("closing manager") t.Logf("closing manager")