Skip to content
Snippets Groups Projects
principalNetworkDomain_test.go 23.4 KiB
Newer Older
  • Learn to ignore specific revisions
  • Manuel Kieweg's avatar
    Manuel Kieweg committed
    package nucleus
    
    Andre Sterba's avatar
    Andre Sterba committed
    	"errors"
    
    Andre Sterba's avatar
    Andre Sterba committed
    	"reflect"
    	"testing"
    
    
    	"code.fbi.h-da.de/danet/gosdn/api/go/gosdn/csbi"
    	ppb "code.fbi.h-da.de/danet/gosdn/api/go/gosdn/pnd"
    	spb "code.fbi.h-da.de/danet/gosdn/api/go/gosdn/southbound"
    	tpb "code.fbi.h-da.de/danet/gosdn/api/go/gosdn/transport"
    	"code.fbi.h-da.de/danet/gosdn/controller/interfaces/device"
    	"code.fbi.h-da.de/danet/gosdn/controller/interfaces/networkdomain"
    	"code.fbi.h-da.de/danet/gosdn/controller/interfaces/southbound"
    	"code.fbi.h-da.de/danet/gosdn/controller/mocks"
    	"code.fbi.h-da.de/danet/gosdn/controller/store"
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	"code.fbi.h-da.de/danet/yang-models/generated/openconfig"
    
    	"github.com/google/uuid"
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	gpb "github.com/openconfig/gnmi/proto/gnmi"
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	"github.com/openconfig/ygot/ygot"
    
    	log "github.com/sirupsen/logrus"
    
    	"github.com/stretchr/testify/mock"
    )
    
    func TestNewPND(t *testing.T) {
    
    	p := newPnd()
    
    	type args struct {
    		name        string
    		description string
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		pid         uuid.UUID
    
    		name    string
    		args    args
    
    		want    networkdomain.NetworkDomain
    
    		wantErr bool
    
    	}{
    		{
    			name: "default",
    			args: args{
    				name:        "default",
    				description: "default test pnd",
    
    				pid:         defaultPndID,
    
    			want:    &p,
    
    			wantErr: false,
    
    		},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			got, err := NewPND(tt.args.name, tt.args.description, tt.args.pid, nil, nil)
    
    			if (err != nil) != tt.wantErr {
    				t.Errorf("NewPND() error = %v, wantErr %v", err, tt.wantErr)
    				return
    			}
    
    			if got.GetName() != tt.want.GetName() {
    				t.Errorf("NewPND.GetName() = %v, want %v", got, tt.want)
    			}
    			if got.ID() != tt.want.ID() {
    				t.Errorf("NewPND.ID() = %v, want %v", got, tt.want)
    			}
    			if got.GetDescription() != tt.want.GetDescription() {
    				t.Errorf("NewPND.GetDescription() = %v, want %v", got, tt.want)
    
    			}
    		})
    	}
    }
    
    func Test_destroy(t *testing.T) {
    	tests := []struct {
    		name    string
    		wantErr bool
    	}{
    		{name: "dummy", wantErr: false},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    			if err := destroy(); (err != nil) != tt.wantErr {
    				t.Errorf("destroy() error = %v, wantErr %v", err, tt.wantErr)
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_AddDevice(t *testing.T) {
    	type args struct {
    
    Malte Bauch's avatar
    Malte Bauch committed
    		device interface{}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		name   string
    		opts   *tpb.TransportOption
    
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    		{
    			name: "default",
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				name: "fridolin",
    				opts: &tpb.TransportOption{
    					TransportOption: &tpb.TransportOption_GnmiTransportOption{
    						GnmiTransportOption: &tpb.GnmiTransportOption{},
    					},
    
    			wantErr: false,
    		},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			pnd := newPnd()
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			if err := pnd.addSbi(&OpenConfig{id: defaultSbiID}); err != nil {
    				t.Error(err)
    			}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			err := pnd.AddDevice(tt.args.name, tt.args.opts, defaultSbiID)
    
    			if (err != nil) != tt.wantErr {
    				t.Errorf("AddDevice() error = %v, wantErr %v", err, tt.wantErr)
    
    			if tt.name != "fails wrong type" {
    				if err == nil {
    
    					d, err := pnd.devices.Get(store.Query{Name: tt.args.name})
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    					if err != nil {
    						t.Errorf("AddDevice() error = %v", err)
    						return
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    					if d.Name() != tt.args.name {
    						t.Errorf("AddDevice() got = %v, want %v", d.Name(), tt.args.name)
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    					}
    
    					if err := pnd.devices.Delete(d); err != nil {
    
    // TODO: refactor test to use store interface instead if direct access.
    
    func Test_pndImplementation_AddSbi(t *testing.T) {
    	type args struct {
    
    		sbi southbound.SouthboundInterface
    
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    
    		{
    			name: "default",
    			args: args{
    				sbi: &OpenConfig{
    
    					id: defaultSbiID,
    
    		// {
    		// 	name: "already exists",
    		// 	args: args{
    		// 		sbi: &OpenConfig{
    		// 			id: defaultSbiID,
    		// 		},
    		// 	},
    		// 	wantErr: true,
    		// },
    
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			pnd := newPnd()
    
    			err := pnd.AddSbi(tt.args.sbi)
    			if (err != nil) != tt.wantErr {
    				t.Errorf("AddSbi() error = %v, wantErr %v", err, tt.wantErr)
    
    			if tt.name != "fails wrong type" {
    				if err == nil {
    
    					sbi, err := pnd.sbic.Get(store.Query{ID: defaultSbiID})
    					if err != nil {
    
    						t.Errorf("AddSbi() SBI %v not in device store %v",
    
    					if err := pnd.sbic.Delete(sbi); err != nil {
    
    		})
    	}
    }
    
    func Test_pndImplementation_Destroy(t *testing.T) {
    	type fields struct {
    		name        string
    		description string
    
    		sbi         southbound.SbiStore
    		devices     device.Store
    
    	}
    	tests := []struct {
    		name    string
    		fields  fields
    		wantErr bool
    	}{
    		{name: "dummy", fields: fields{}, wantErr: false},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    			pnd := &pndImplementation{
    
    				Name:        tt.fields.name,
    				Description: tt.fields.description,
    
    				sbic:        tt.fields.sbi,
    
    				devices:     tt.fields.devices,
    			}
    			if err := pnd.Destroy(); (err != nil) != tt.wantErr {
    				t.Errorf("Destroy() error = %v, wantErr %v", err, tt.wantErr)
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_GetDescription(t *testing.T) {
    	tests := []struct {
    		name string
    		want string
    	}{
    		{name: "default", want: "default test pnd"},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			pnd := newPnd()
    
    			if got := pnd.GetDescription(); got != tt.want {
    
    				t.Errorf("GetDescription() = %v, want %v", got, tt.want)
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_GetName(t *testing.T) {
    	tests := []struct {
    		name string
    		want string
    	}{
    		{name: "default", want: "default"},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			pnd := newPnd()
    
    			if got := pnd.GetName(); got != tt.want {
    
    				t.Errorf("GetName() = %v, want %v", got, tt.want)
    			}
    		})
    	}
    }
    
    
    // func Test_pndImplementation_GetSBIs(t *testing.T) {
    // 	pnd := newPnd()
    // 	tests := []struct {
    // 		name string
    // 		want []southbound.SouthboundInterface
    // 	}{
    // 		{name: "default", want: []southbound.SouthboundInterface{}},
    // 	}
    // 	for _, tt := range tests {
    // 		t.Run(tt.name, func(t *testing.T) {
    // 			if got, _ := pnd.GetSBIs(); !reflect.DeepEqual(got, tt.want) {
    // 				t.Errorf("GetSBIs() = %v, want %v", got, tt.want)
    // 			}
    // 		})
    // 	}
    // }
    
    
    func Test_pndImplementation_MarshalDevice(t *testing.T) {
    	type args struct {
    		uuid uuid.UUID
    	}
    	tests := []struct {
    		name    string
    		args    args
    		want    string
    		wantErr bool
    	}{
    
    Malte Bauch's avatar
    Malte Bauch committed
    		{
    			name:    "default",
    			args:    args{did},
    			want:    "{\n\t\"Acl\": null,\n\t\"Bgp\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"LocalRoutes\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}",
    			wantErr: false,
    		},
    
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			pnd := newPnd()
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			d := &CommonDevice{
    
    				UUID:      tt.args.uuid,
    
    				GoStruct:  &openconfig.Device{},
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				sbi:       nil,
    				transport: nil,
    
    			if err := pnd.addDevice(d); err != nil {
    				t.Error(err)
    			}
    
    			got, err := pnd.MarshalDevice(tt.args.uuid.String())
    
    			if (err != nil) != tt.wantErr {
    				t.Errorf("MarshalDevice() error = %v, wantErr %v", err, tt.wantErr)
    				return
    			}
    			if got != tt.want {
    				t.Errorf("MarshalDevice() got = %v, want %v", got, tt.want)
    			}
    
    			if err := pnd.devices.Delete(d); err != nil {
    
    		})
    	}
    }
    
    func Test_pndImplementation_RemoveDevice(t *testing.T) {
    	type args struct {
    		uuid uuid.UUID
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    		{name: "default", args: args{uuid: did}, wantErr: false},
    
    		{name: "fails", args: args{uuid: uuid.New()}, wantErr: true},
    		{name: "fails empty", args: args{uuid: did}, wantErr: true},
    
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			pnd := newPnd()
    			if tt.name != "fails empty" {
    
    				sbi, err := NewSBI(spb.Type_TYPE_OPENCONFIG)
    				if err != nil {
    					t.Error(err)
    					return
    				}
    
    Malte Bauch's avatar
    Malte Bauch committed
    				d := &CommonDevice{
    					UUID: did,
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    					sbi:  sbi,
    
    Malte Bauch's avatar
    Malte Bauch committed
    				}
    
    				if err := pnd.addDevice(d); err != nil {
    					t.Error(err)
    				}
    			}
    
    			if err := pnd.RemoveDevice(tt.args.uuid); (err != nil) != tt.wantErr {
    
    				t.Errorf("RemoveDevice() error = %v, wantErr %v", err, tt.wantErr)
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_RemoveSbi(t *testing.T) {
    
    	opts := &tpb.TransportOption{
    		TransportOption: &tpb.TransportOption_GnmiTransportOption{
    			GnmiTransportOption: &tpb.GnmiTransportOption{},
    		},
    	}
    
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    
    		{name: "default", args: args{id: defaultSbiID}, wantErr: false},
    
    		{name: "fails", args: args{id: uuid.New()}, wantErr: true},
    
    		{name: "fails empty", args: args{id: defaultSbiID}, wantErr: true},
    
    		{name: "exclusively remove associated devices", args: args{id: defaultSbiID}, wantErr: false},
    
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			pnd := &pndImplementation{
    
    				Name:        "test-remove-sbi",
    				Description: "test-remove-sbi",
    
    				sbic:        sbiStore,
    				devices:     NewDeviceStore(defaultPndID, sbiStore),
    
    			if tt.name != "fails empty" && tt.name != "fails" {
    
    				if err := pnd.addSbi(&OpenConfig{id: defaultSbiID}); err != nil {
    
    				if err := pnd.AddDevice("associatedDevice", opts, tt.args.id); err != nil {
    					t.Error(err)
    				}
    				if err := pnd.AddDevice("associatedDevice2", opts, tt.args.id); err != nil {
    					t.Error(err)
    				}
    				if tt.name == "exclusively remove associated devices" {
    					newID := uuid.New()
    					if err := pnd.addSbi(&OpenConfig{id: newID}); err != nil {
    						t.Error(err)
    					}
    					if err := pnd.AddDevice("associatedDevice2", opts, newID); err != nil {
    						t.Error(err)
    					}
    				}
    
    			if err := pnd.RemoveSbi(tt.args.id); (err != nil) != tt.wantErr {
    
    				t.Errorf("RemoveSbi() error = %v, wantErr %v", err, tt.wantErr)
    			}
    
    
    			sbi, _ := pnd.sbic.Get(store.Query{ID: tt.args.id})
    			if sbi != nil {
    
    				t.Errorf("RemoveSbi() SBI still in SBI store %v", pnd.sbic)
    			}
    
    			if tt.name == "exclusively remove associated devices" {
    
    				allDevices, _ := pnd.devices.GetAll()
    				if len(allDevices) != 1 {
    					t.Errorf("RemoveSbi() non associated devices should remain in the storage %v", allDevices)
    
    				allDevices, _ := pnd.devices.GetAll()
    				if len(allDevices) != 0 {
    					t.Errorf("RemoveSbi() associated devices have not been removed correctly %v", len(allDevices))
    
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_Request(t *testing.T) {
    	type args struct {
    		uuid uuid.UUID
    		path string
    		rErr error
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    		{
    			name: "default",
    			args: args{
    				uuid: mdid,
    				path: "",
    				rErr: nil,
    			},
    			wantErr: false,
    		},
    		{
    			name: "error",
    			args: args{
    				uuid: did,
    				path: "",
    				rErr: errors.New("deliberate test fail"),
    			},
    			wantErr: true,
    		},
    	}
    
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			deviceWithMockTransport := mockDevice()
    
    			pnd := newPnd()
    
    			transport := deviceWithMockTransport.Transport().(*mocks.Transport)
    			transport.On("Get", mockContext, mock.Anything).Return(&gpb.GetResponse{}, tt.args.rErr)
    			transport.On("ProcessResponse", mock.Anything, mock.Anything, mock.Anything).Return(tt.args.rErr)
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			_ = pnd.addDevice(deviceWithMockTransport)
    
    			_, err := pnd.Request(tt.args.uuid, tt.args.path)
    			if (err != nil) != tt.wantErr {
    
    				t.Errorf("Request() error = %v, wantErr %v", err, tt.wantErr)
    			}
    
    
    			device, _ := pnd.devices.Get(store.Query{ID: mdid})
    			if device == nil {
    				return
    			}
    			if err := pnd.devices.Delete(device); err != nil {
    
    		})
    	}
    }
    
    func Test_pndImplementation_RequestAll(t *testing.T) {
    	type args struct {
    		uuid uuid.UUID
    		path string
    		rErr error
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    		{
    			name: "default",
    			args: args{
    				uuid: mdid,
    				path: "",
    				rErr: nil,
    			},
    			wantErr: false,
    		},
    		{
    			name: "error",
    			args: args{
    
    				path: "",
    				rErr: errors.New("deliberate test fail"),
    			},
    			wantErr: true,
    		},
    	}
    
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			deviceWithMockTransport := mockDevice()
    
    			pnd := newPnd()
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			tr := deviceWithMockTransport.Transport().(*mocks.Transport)
    			tr.On("Get", mockContext, mock.Anything).Return(&gpb.GetResponse{}, tt.args.rErr)
    
    			tr.On("ProcessResponse", mock.Anything, mock.Anything, mock.Anything).Return(tt.args.rErr)
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			_ = pnd.addDevice(deviceWithMockTransport)
    
    			if err := pnd.RequestAll(tt.args.path); (err != nil) != tt.wantErr {
    
    				t.Errorf("RequestAll() error = %v, wantErr %v", err, tt.wantErr)
    			}
    
    
    			device, _ := pnd.devices.Get(store.Query{ID: mdid})
    			if device == nil {
    				return
    			}
    			if err := pnd.devices.Delete(device); err != nil {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    
    func Test_pndImplementation_ChangeOND(t *testing.T) {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	opts := &tpb.TransportOption{
    		TransportOption: &tpb.TransportOption_GnmiTransportOption{
    			GnmiTransportOption: &tpb.GnmiTransportOption{},
    		},
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	}
    	type args struct {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		operation ppb.ApiOperation
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		path      string
    		value     []string
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "update",
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_UPDATE,
    
    				path:      "/system/config/hostname",
    				value:     []string{"ceos3000"},
    			},
    			wantErr: false,
    		},
    		{
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "replace",
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_REPLACE,
    
    				path:      "/system/config/hostname",
    				value:     []string{"ceos3000"},
    			},
    			wantErr: false,
    		},
    		{
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "delete",
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_DELETE,
    
    				path:      "/system/config/hostname",
    			},
    			wantErr: false,
    		},
    		{
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "delete w/args",
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_DELETE,
    
    				path:      "/system/config/hostname",
    				value:     []string{"ceos3000"},
    			},
    			wantErr: false,
    		},
    
    		// Negative test cases
    		{
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "invalid operation",
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				operation: 54,
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "invalid arg count",
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_UPDATE,
    
    				path:      "/system/config/hostname",
    				value:     []string{"ceos3000", "ceos3001"},
    			},
    			wantErr: true,
    		},
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "invalid arg count - update, no args",
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			args: args{
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_UPDATE,
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				path:      "/system/config/hostname",
    			},
    			wantErr: true,
    		},
    		{
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "invalid arg count - replace, no args",
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			args: args{
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_UPDATE,
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				path:      "/system/config/hostname",
    			},
    			wantErr: true,
    		},
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			name: "device not found",
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    				operation: ppb.ApiOperation_API_OPERATION_UPDATE,
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			pnd := newPnd()
    			if err := pnd.addSbi(&OpenConfig{id: defaultSbiID}); err != nil {
    				t.Error(err)
    			}
    			if err := pnd.AddDevice("testdevice", opts, defaultSbiID); err != nil {
    
    			devices, err := pnd.devices.GetAll()
    			if err != nil {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				err := errors.New("error fetching device")
    				t.Error(err)
    				return
    			}
    
    
    			_, err = pnd.ChangeOND(devices[0].ID(), tt.args.operation, tt.args.path, tt.args.value...)
    
    			if (err != nil) != tt.wantErr {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				t.Errorf("ChangeOND() error = %v, wantErr %v", err, tt.wantErr)
    
    				if len(pnd.changes.Store) != 1 {
    					t.Errorf("ChangeOND() unexpected change count. got %v, want 1", len(pnd.changes.Store))
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			}
    		})
    	}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    
    func Test_pndImplementation_GetDevice(t *testing.T) {
    
    	pnd := newPnd()
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    	sbi, err := NewSBI(spb.Type_TYPE_OPENCONFIG)
    	if err != nil {
    		t.Errorf("NewSBI() error = %v", err)
    
    	d, err := NewDevice("", uuid.Nil, newGnmiTransportOptions(), sbi)
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	if err != nil {
    		t.Error(err)
    		return
    	}
    
    	if err = pnd.addDevice(d); err != nil {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		t.Error(err)
    		return
    	}
    	type args struct {
    		uuid uuid.UUID
    	}
    	tests := []struct {
    		name    string
    		args    args
    		want    ygot.GoStruct
    		wantErr bool
    	}{
    		{
    			name:    "default",
    			args:    args{uuid: d.ID()},
    			want:    sbi.Schema().Root,
    			wantErr: false,
    		},
    		{
    			name:    "device not found",
    			args:    args{uuid: mdid},
    			want:    nil,
    			wantErr: true,
    		},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    
    			foundDevice, err := pnd.GetDevice(tt.args.uuid.String())
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			if (err != nil) != tt.wantErr {
    				t.Errorf("GetDevice() error = %v, wantErr %v", err, tt.wantErr)
    				return
    			}
    
    			if foundDevice != nil {
    
    				if !reflect.DeepEqual(foundDevice.(device.Device).Model(), tt.want) {
    					t.Errorf("GetDevice() got = %v, want %v", foundDevice.(device.Device).Model(), tt.want)
    
    		})
    	}
    }
    
    func Test_pndImplementation_GetDeviceByName(t *testing.T) {
    	p := newPnd()
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    	sbi, err := NewSBI(spb.Type_TYPE_OPENCONFIG)
    	if err != nil {
    		t.Errorf("NewSBI() error = %v", err)
    
    	d, err := NewDevice("my-device", uuid.Nil, newGnmiTransportOptions(), sbi)
    
    	if err != nil {
    		t.Error(err)
    		return
    	}
    	if err = p.addDevice(d); err != nil {
    		t.Error(err)
    		return
    	}
    	type args struct {
    		name string
    	}
    	tests := []struct {
    		name    string
    		args    args
    		want    ygot.GoStruct
    		wantErr bool
    	}{
    		{
    			name:    "default",
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			args:    args{name: d.Name()},
    
    			want:    sbi.Schema().Root,
    			wantErr: false,
    		},
    		{
    			name:    "device not found",
    			args:    args{name: "test-device"},
    			want:    nil,
    			wantErr: true,
    		},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    			foundDevice, err := p.GetDevice(tt.args.name)
    			if (err != nil) != tt.wantErr {
    				t.Errorf("GetDeviceByName() error = %v, wantErr %v", err, tt.wantErr)
    				return
    			}
    			if foundDevice != nil {
    
    				if !reflect.DeepEqual(foundDevice.(device.Device).Model(), tt.want) {
    					t.Errorf("GetDeviceByName() got = %v, want %v", foundDevice.(device.Device).Model(), tt.want)
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		})
    	}
    }
    
    func Test_pndImplementation_Confirm(t *testing.T) {
    	tests := []struct {
    		name    string
    		wantErr bool
    	}{
    		{
    			name:    "default",
    			wantErr: false,
    		},
    		{
    			name:    "uncommitted",
    			wantErr: true,
    		},
    	}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    			pnd := newPnd()
    			d := mockDevice()
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			tr := d.Transport().(*mocks.Transport)
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			tr.On("Set", mockContext, mock.Anything, mock.Anything).Return(nil)
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			if err := pnd.addDevice(d); err != nil {
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				t.Error(err)
    				return
    			}
    
    Fabian Seidl's avatar
    Fabian Seidl committed
    			_, err := pnd.ChangeOND(d.ID(), ppb.ApiOperation_API_OPERATION_UPDATE, "system/config/hostname", "ceos3000")
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    				t.Error(err)
    				return
    			}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			u := pnd.PendingChanges()[0]
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			if tt.name != "uncommitted" {
    				if err := pnd.Commit(u); (err != nil) != tt.wantErr {
    					t.Errorf("Confirm() error = %v, wantErr %v", err, tt.wantErr)
    					return
    				}
    			}
    			if err := pnd.Confirm(u); (err != nil) != tt.wantErr {
    				t.Errorf("Confirm() error = %v, wantErr %v", err, tt.wantErr)
    			}
    		})
    	}
    }
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    
    func Test_pndImplementation_PendingChanges(t *testing.T) {
    
    	testName := t.Name()
    	callback := func(first ygot.GoStruct, second ygot.GoStruct) error {
    		log.Infof("callback in test %v", testName)
    		return nil
    	}
    
    
    	store := store.NewChangeStore()
    
    	pending := NewChange(did, &openconfig.Device{}, &openconfig.Device{}, callback)
    	if err := store.Add(pending); err != nil {
    		t.Error(err)
    		return
    	}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	tests := []struct {
    		name string
    		want []uuid.UUID
    	}{
    		{
    			name: "default",
    
    			want: []uuid.UUID{pending.cuid},
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    			pnd := newPnd()
    
    			pnd.changes = store
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			if got := pnd.PendingChanges(); !reflect.DeepEqual(got, tt.want) {
    				t.Errorf("pndImplementation.PendingChanges() = %v, want %v", got, tt.want)
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_CommittedChanges(t *testing.T) {
    
    	testName := t.Name()
    	callback := func(first ygot.GoStruct, second ygot.GoStruct) error {
    		log.Infof("callback in test %v", testName)
    		return nil
    	}
    
    
    	store := store.NewChangeStore()
    
    	committed := NewChange(did, &openconfig.Device{}, &openconfig.Device{}, callback)
    	if err := committed.Commit(); err != nil {
    		t.Error(err)
    		return
    	}
    	if err := store.Add(committed); err != nil {
    		t.Error(err)
    		return
    	}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	tests := []struct {
    		name string
    		want []uuid.UUID
    	}{
    		{
    			name: "default",
    
    			want: []uuid.UUID{committed.cuid},
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    			pnd := newPnd()
    
    			pnd.changes = store
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			if got := pnd.CommittedChanges(); !reflect.DeepEqual(got, tt.want) {
    				t.Errorf("pndImplementation.CommittedChanges() = %v, want %v", got, tt.want)
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_ConfirmedChanges(t *testing.T) {
    
    	testName := t.Name()
    	callback := func(first ygot.GoStruct, second ygot.GoStruct) error {
    		log.Infof("callback in test %v", testName)
    		return nil
    	}
    
    	store := store.NewChangeStore()
    
    	confirmed := NewChange(did, &openconfig.Device{}, &openconfig.Device{}, callback)
    	if err := confirmed.Commit(); err != nil {
    		t.Error(err)
    		return
    	}
    	if err := confirmed.Confirm(); err != nil {
    		t.Error(err)
    		return
    	}
    	if err := store.Add(confirmed); err != nil {
    		t.Error(err)
    		return
    	}
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    	tests := []struct {
    		name string
    		want []uuid.UUID
    	}{
    		{
    			name: "default",
    
    			want: []uuid.UUID{confirmed.cuid},
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    		},
    	}
    	for _, tt := range tests {
    		t.Run(tt.name, func(t *testing.T) {
    			pnd := newPnd()
    
    			pnd.changes = store
    
    Manuel Kieweg's avatar
    Manuel Kieweg committed
    			if got := pnd.ConfirmedChanges(); !reflect.DeepEqual(got, tt.want) {
    				t.Errorf("pndImplementation.ConfirmedChanges() = %v, want %v", got, tt.want)
    			}
    		})
    	}
    }
    
    func Test_pndImplementation_saveGoStructsToFile(t *testing.T) {
    	type genericGoStructClientArg struct {
    		fn    string
    		rtrn  []interface{}
    		times int
    	}
    	// Create a new mock for GenericGoStructClient. With
    	// genericGoStructClientArg it is possible to set the Return values of the
    	// mocks methods.
    	newGenericGoStructClient := func(args ...genericGoStructClientArg) *mocks.GenericGoStructClient {
    		ggsc := &mocks.GenericGoStructClient{}
    		for _, arg := range args {
    			ggsc.On(arg.fn).Return(arg.rtrn...).Times(arg.times)
    		}
    		ggsc.On("CloseSend").Return(nil)
    		return ggsc
    	}
    
    	type args struct {
    		id     uuid.UUID
    		client GenericGrpcClient
    	}
    	tests := []struct {
    		name    string
    		args    args
    		wantErr bool
    	}{
    		{
    			name: "default",
    			args: args{
    				id: uuid.New(),
    				client: newGenericGoStructClient(
    					[]genericGoStructClientArg{
    						{
    							fn: "Recv",
    							rtrn: []interface{}{
    								&csbi.Payload{Chunk: []byte("test")},
    								nil,
    							},
    							times: 3,
    						},
    						{
    							fn: "Recv",
    							rtrn: []interface{}{
    								&csbi.Payload{Chunk: nil},
    								io.EOF,
    							},
    							times: 1,
    						},
    					}...),
    			},
    			wantErr: false,
    		},
    		{
    			name: "unexpected EOF error",
    			args: args{
    				id: uuid.New(),
    				client: newGenericGoStructClient(
    					[]genericGoStructClientArg{
    						{
    							fn: "Recv",
    							rtrn: []interface{}{
    								&csbi.Payload{Chunk: nil},
    								io.ErrUnexpectedEOF,
    							},
    							times: 1,
    						},
    						{
    							fn: "CloseSend",
    							rtrn: []interface{}{
    								nil,