From 2567b8b6b77eaaa3ef0731f93732da4a8d408133 Mon Sep 17 00:00:00 2001
From: Malte Bauch <malte.bauch@stud.h-da.de>
Date: Fri, 26 Feb 2021 13:49:19 +0100
Subject: [PATCH] added testcase for NewGnmiTransport

currently this fails and it has to do with the gnmi connection attached
to a GNMI struct. Ref #78
---
 nucleus/gnmi_transport_test.go | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/nucleus/gnmi_transport_test.go b/nucleus/gnmi_transport_test.go
index 095731339..3cd5cc2d7 100644
--- a/nucleus/gnmi_transport_test.go
+++ b/nucleus/gnmi_transport_test.go
@@ -37,15 +37,6 @@ func testSetupGnmi() {
 		Encoding: gpb.Encoding_PROTO,
 	}
 
-	gnmiTransportOptions = &GnmiTransportOptions{
-		Username: "test",
-		Password: "test",
-		Addr:     "localhost:13371",
-		SetNode:  nil,
-		RespChan: make(chan *gpb.SubscribeResponse),
-		Encoding: gpb.Encoding_PROTO,
-	}
-
 	startGnmiTarget = make(chan string)
 	stopGnmiTarget = make(chan bool)
 	go targetRunner()
@@ -64,13 +55,24 @@ func mockTransport() Gnmi {
 	return Gnmi{
 		SetNode:  nil,
 		RespChan: make(chan *gpb.SubscribeResponse),
+		Options:  newGnmiTransportOptions(),
 		config:   gnmiConfig,
 		client:   &mocks.GNMIClient{},
 	}
 }
 
+func newGnmiTransportOptions() *GnmiTransportOptions {
+	return &GnmiTransportOptions{
+		Username: "test",
+		Password: "test",
+		Addr:     "localhost:13371",
+		SetNode:  nil,
+		RespChan: make(chan *gpb.SubscribeResponse),
+		Encoding: gpb.Encoding_PROTO,
+	}
+}
+
 var gnmiConfig *gnmi.Config
-var gnmiTransportOptions *GnmiTransportOptions
 var startGnmiTarget chan string
 var stopGnmiTarget chan bool
 var mockContext = mock.MatchedBy(func(ctx context.Context) bool { return true })
@@ -89,7 +91,7 @@ func TestNewGnmiTransport(t *testing.T) {
 		{
 			name: "default",
 			args: args{
-				opts: gnmiTransportOptions,
+				opts: newGnmiTransportOptions(),
 			},
 			want:    &gnmiTransport,
 			wantErr: false,
-- 
GitLab