Browse Source

fix broken tests

Darien Raymond 6 years ago
parent
commit
4613008148

+ 3 - 2
testing/scenarios/command_test.go

@@ -3,6 +3,7 @@ package scenarios
 import (
 	"context"
 	"fmt"
+	"io"
 	"testing"
 	"time"
 
@@ -264,8 +265,8 @@ func TestCommanderAddRemoveUser(t *testing.T) {
 	common.Must(err)
 	defer CloseAllServers(servers)
 
-	if err := testTCPConn(clientPort, 1024, time.Second*5)(); err != nil {
-		t.Fatal(err)
+	if err := testTCPConn(clientPort, 1024, time.Second*5)(); err != io.EOF {
+		t.Fatal("expected error: ", err)
 	}
 
 	cmdConn, err := grpc.Dial(fmt.Sprintf("127.0.0.1:%d", cmdPort), grpc.WithInsecure(), grpc.WithBlock())

+ 1 - 1
testing/scenarios/feature_test.go

@@ -537,7 +537,7 @@ func TestUDPConnection(t *testing.T) {
 
 	servers, err := InitializeServerConfigs(clientConfig)
 	common.Must(err)
-	CloseAllServers(servers)
+	defer CloseAllServers(servers)
 
 	if err := testUDPConn(clientPort, 1024, time.Second*5)(); err != nil {
 		t.Error(err)

+ 1 - 1
testing/scenarios/vmess_test.go

@@ -452,7 +452,7 @@ func TestVMessGCMUDP(t *testing.T) {
 
 	servers, err := InitializeServerConfigs(serverConfig, clientConfig)
 	common.Must(err)
-	CloseAllServers(servers)
+	defer CloseAllServers(servers)
 
 	var errg errgroup.Group
 	for i := 0; i < 10; i++ {