ignore bgp test in ci

controller/bgp_test.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	"net"
+	"os"
 	"testing"
 
 	"github.com/golang/protobuf/ptypes"
@@ -18,8 +19,6 @@ type BgpListener struct {
 	recvdPaths chan string
 }
 
-var listener *BgpListener
-
 // NewBgpListener starts a local BGP server for testing purposes
 func NewBgpListener(localAS int) (*BgpListener, error) {
 	s := gobgp.NewBgpServer()
@@ -71,6 +70,14 @@ func (l *BgpListener) Shutdown() error {
 // if the test timeouts are very small. It also needs to be run as
 // root (sudo)
 func TestBgpNew(t *testing.T) {
+	if os.Getenv("CI") != "" {
+		t.Skip("Skipping testing in CI environment")
+	}
+	listener, err := NewBgpListener(22222)
+	if err != nil {
+		panic(err)
+	}
+	defer listener.Shutdown()
 	a := assert.New(t)
 	c := config.BgpConfig{
 		LocalAS:     11111,

controller/system_test.go

@@ -54,17 +54,10 @@ func TestMain(m *testing.M) {
 	case "test_add_fail":
 		os.Exit(1)
 	default:
-		fmt.Println("success")
+		break
 	}
 	if os.Getenv("test_name") != "" {
 		return
 	}
-	var err error
-	listener, err = NewBgpListener(22222)
-	if err != nil {
-		panic(err)
-	}
-	code := m.Run()
-	listener.Shutdown()
-	os.Exit(code)
+	os.Exit(m.Run())
 }