Merge pull request #955 from cloudnativelabs/fix-build-break

fix lint error: minor fix to catch the error from .bgpServer.Stop()
cloudnativelabs · Jul 17, 2020 · 3ab31ab · 3ab31ab
2 parents 031a992 + bb35b9a
commit 3ab31ab
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/pkg/controllers/routing/network_routes_controller_test.go b/pkg/controllers/routing/network_routes_controller_test.go
@@ -1523,7 +1523,11 @@ func Test_routeReflectorConfiguration(t *testing.T) {
 
 			err := testcase.nrc.startBgpServer()
 			if err == nil {
-				defer testcase.nrc.bgpServer.Stop()
+				defer func() {
+					if err := testcase.nrc.bgpServer.Stop(); err != nil {
+						t.Fatalf("failed to stop BGP server : %s", err)
+					}
+				}()
 			}
 
 			if testcase.expectedBgpToStart {