todo clean up

controller/api/initialise_test.go

@@ -47,8 +47,6 @@ import (
 	pb "google.golang.org/protobuf/proto"
 )
 
-// TODO(PND): fix tests "json: unsupported value: encountered a cycle via []*mock.Call", probably because of pndService interface missing change stuff?!
-
 /*
 Based on this StackOverflow answer: https://stackoverflow.com/a/52080545/4378176
 */
@@ -196,8 +194,6 @@ func bootstrapUnitTest() {
 	networkElementStore := nucleus.NewNetworkElementStore()
 	networkElementService := nucleus.NewNetworkElementService(networkElementStore, pluginService, eventService)
 
-	//TODO(PND): check/fix api_test: getdevice/adddevice, maybe the mne here is not necessary!
-
 	mne, _ := nucleus.NewNetworkElement("test", mneUUID, &tpb.TransportOption{
 		Address:  "test",
 		Username: "test",

controller/northbound/server/networkElement_test.go

@@ -131,7 +131,6 @@ package server
 // 	}
 // }
 
-// // TODO(PND): fix test, problem here is either missing port in transport address or unreachable when address:port is provided.
 // // Somehow, provided mock transport does not get used?
 // func TestNetworkElementServer_getPath(t *testing.T) {
 // 	mneServer := &NetworkElementServer{}
@@ -357,8 +356,6 @@ package server
 // 	}
 // }
 
-// // TODO(PND): Need to fix test. Probably same problem as in GetPath Test.
-// // error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp: missing address"
 // func TestNetworkElementServer_Update(t *testing.T) {
 // 	mneServer := &NetworkElementServer{}
 // 	createTestNetworkElementServer(t, mneServer)

controller/northbound/server/pnd_test.go

@@ -16,7 +16,6 @@ package server
 // 	"github.com/stretchr/testify/mock"
 // )
 
-// // TODO(PND): this whole file needs to be reworked!
 // func getTestPndServer(t *testing.T) *PndServer {
 // 	var err error
 // 	pndUUID, err = uuid.Parse(pndID)

controller/test/integration/nucleusIntegration_test.go

@@ -162,7 +162,6 @@ package integration
 // 	}
 // }
 
-// // TODO(PND): test compilable for now, not working. Changes pnd to n within t.Run method
 // func TestGnmi_SetValidIntegration(t *testing.T) {
 // 	if testing.Short() {
 // 		t.Skip("skipping integration test")