Skip to content
Snippets Groups Projects
Commit 688d4801 authored by Malte Bauch's avatar Malte Bauch
Browse files

Update test cases

parent b538aafd
Branches
Tags
1 merge request!401Change the current gosdn plugin implementation from Go's internal plugin system to hashicorp's go-plugins
Pipeline #135929 failed
...@@ -131,7 +131,7 @@ func bootstrapUnitTest() { ...@@ -131,7 +131,7 @@ func bootstrapUnitTest() {
pluginMock.On("ID").Return(defaultPluginID) pluginMock.On("ID").Return(defaultPluginID)
pluginMock.On("Unmarshal", mock.Anything, mock.Anything).Return(nil) pluginMock.On("Unmarshal", mock.Anything, mock.Anything).Return(nil)
pluginMock.On("GetNode", mock.Anything).Return([]*gpb.Notification{}, nil) pluginMock.On("GetNode", mock.Anything).Return([]*gpb.Notification{}, nil)
pluginMock.On("Model").Return([]byte( pluginMock.On("Model", mock.Anything).Return([]byte(
"{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"), "{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"),
nil, nil,
) )
......
...@@ -98,7 +98,7 @@ func mockPlugin(t testing.TB) plugin.Plugin { ...@@ -98,7 +98,7 @@ func mockPlugin(t testing.TB) plugin.Plugin {
mockPlugin := &mocks.Plugin{} mockPlugin := &mocks.Plugin{}
mockPlugin.On("ID").Return(defaultPluginID) mockPlugin.On("ID").Return(defaultPluginID)
mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil) mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil)
mockPlugin.On("Model").Return([]byte( mockPlugin.On("Model", mock.Anything).Return([]byte(
"{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"), "{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"),
nil, nil,
) )
...@@ -156,7 +156,7 @@ func mockNetworkElement() networkelement.NetworkElement { ...@@ -156,7 +156,7 @@ func mockNetworkElement() networkelement.NetworkElement {
mockPlugin := &mocks.Plugin{} mockPlugin := &mocks.Plugin{}
mockPlugin.On("ID").Return(defaultPluginID) mockPlugin.On("ID").Return(defaultPluginID)
mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil) mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil)
mockPlugin.On("Model").Return([]byte( mockPlugin.On("Model", mock.Anything).Return([]byte(
"{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"), "{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"),
nil, nil,
) )
...@@ -169,12 +169,11 @@ func mockNetworkElement() networkelement.NetworkElement { ...@@ -169,12 +169,11 @@ func mockNetworkElement() networkelement.NetworkElement {
} }
} }
// TODO: return an error for pluginservice add!!
func newPnd() (*pndImplementation, error) { func newPnd() (*pndImplementation, error) {
mockPlugin := &mocks.Plugin{} mockPlugin := &mocks.Plugin{}
mockPlugin.On("ID").Return(defaultPluginID) mockPlugin.On("ID").Return(defaultPluginID)
mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil) mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil)
mockPlugin.On("Model").Return([]byte( mockPlugin.On("Model", mock.Anything).Return([]byte(
"{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"), "{\n\t\"Acl\": null,\n\t\"Bfd\": null,\n\t\"Components\": null,\n\t\"Interfaces\": null,\n\t\"Keychains\": null,\n\t\"Lldp\": null,\n\t\"Messages\": null,\n\t\"NetworkInstances\": null,\n\t\"RoutingPolicy\": null,\n\t\"System\": null\n}"),
nil, nil,
) )
......
...@@ -9,6 +9,7 @@ import ( ...@@ -9,6 +9,7 @@ import (
"code.fbi.h-da.de/danet/gosdn/controller/mocks" "code.fbi.h-da.de/danet/gosdn/controller/mocks"
"code.fbi.h-da.de/danet/gosdn/controller/store" "code.fbi.h-da.de/danet/gosdn/controller/store"
"github.com/google/uuid" "github.com/google/uuid"
"github.com/stretchr/testify/mock"
) )
func returnBasicTransportOption() tpb.TransportOption { func returnBasicTransportOption() tpb.TransportOption {
...@@ -37,7 +38,7 @@ func TestAddNetworkElement(t *testing.T) { ...@@ -37,7 +38,7 @@ func TestAddNetworkElement(t *testing.T) {
plugin1 := &mocks.Plugin{} plugin1 := &mocks.Plugin{}
plugin1.On("ID").Return(pluginID1) plugin1.On("ID").Return(pluginID1)
plugin1.On("Model").Return([]byte{}, nil) plugin1.On("Model", mock.Anything).Return([]byte{}, nil)
networkElementStore := NewNetworkElementStore(pndID) networkElementStore := NewNetworkElementStore(pndID)
mne, _ := NewNetworkElement("testNetworkElement", mneID, &trop, plugin1, conflict.Metadata{}) mne, _ := NewNetworkElement("testNetworkElement", mneID, &trop, plugin1, conflict.Metadata{})
...@@ -57,7 +58,7 @@ func TestGetAllNetworkElements(t *testing.T) { ...@@ -57,7 +58,7 @@ func TestGetAllNetworkElements(t *testing.T) {
pluginID, _ := uuid.Parse("ssssssss-ssss-ssss-ssss-ssssssssssss") pluginID, _ := uuid.Parse("ssssssss-ssss-ssss-ssss-ssssssssssss")
plugin := &mocks.Plugin{} plugin := &mocks.Plugin{}
plugin.On("ID").Return(pluginID) plugin.On("ID").Return(pluginID)
plugin.On("Model").Return([]byte{}, nil) plugin.On("Model", mock.Anything).Return([]byte{}, nil)
mneID1, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa") mneID1, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa")
mneID2, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaab") mneID2, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaab")
...@@ -112,7 +113,7 @@ func TestGetNetworkElement(t *testing.T) { ...@@ -112,7 +113,7 @@ func TestGetNetworkElement(t *testing.T) {
pluginID, _ := uuid.Parse("ssssssss-ssss-ssss-ssss-ssssssssssss") pluginID, _ := uuid.Parse("ssssssss-ssss-ssss-ssss-ssssssssssss")
plugin := &mocks.Plugin{} plugin := &mocks.Plugin{}
plugin.On("ID").Return(pluginID) plugin.On("ID").Return(pluginID)
plugin.On("Model").Return([]byte{}, nil) plugin.On("Model", mock.Anything).Return([]byte{}, nil)
mneID1, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa") mneID1, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa")
mneID2, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaab") mneID2, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaab")
...@@ -164,7 +165,7 @@ func TestUpdateNetworkElement(t *testing.T) { ...@@ -164,7 +165,7 @@ func TestUpdateNetworkElement(t *testing.T) {
plugin1 := &mocks.Plugin{} plugin1 := &mocks.Plugin{}
plugin1.On("ID").Return(pluginID1) plugin1.On("ID").Return(pluginID1)
plugin1.On("Model").Return([]byte{}, nil) plugin1.On("Model", mock.Anything).Return([]byte{}, nil)
networkElementStore := NewNetworkElementStore(pndID) networkElementStore := NewNetworkElementStore(pndID)
mne, _ := NewNetworkElement("testNetworkElement", mneID, &trop, plugin1, conflict.Metadata{}) mne, _ := NewNetworkElement("testNetworkElement", mneID, &trop, plugin1, conflict.Metadata{})
...@@ -203,7 +204,7 @@ func TestDeleteNetworkElement(t *testing.T) { ...@@ -203,7 +204,7 @@ func TestDeleteNetworkElement(t *testing.T) {
pluginID, _ := uuid.Parse("ssssssss-ssss-ssss-ssss-ssssssssssss") pluginID, _ := uuid.Parse("ssssssss-ssss-ssss-ssss-ssssssssssss")
plugin := &mocks.Plugin{} plugin := &mocks.Plugin{}
plugin.On("ID").Return(pluginID) plugin.On("ID").Return(pluginID)
plugin.On("Model").Return([]byte{}, nil) plugin.On("Model", mock.Anything).Return([]byte{}, nil)
mneID1, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa") mneID1, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa")
mneID2, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaab") mneID2, _ := uuid.Parse("aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaab")
......
...@@ -27,7 +27,7 @@ func getNetworkElementTestStores(t *testing.T, mneID uuid.UUID) (networkelement. ...@@ -27,7 +27,7 @@ func getNetworkElementTestStores(t *testing.T, mneID uuid.UUID) (networkelement.
} }
mockPlugin := &mocks.Plugin{} mockPlugin := &mocks.Plugin{}
mockPlugin.On("ID").Return(mockPluginID) mockPlugin.On("ID").Return(mockPluginID)
mockPlugin.On("Model").Return([]byte("hello"), nil) mockPlugin.On("Model", mock.Anything).Return([]byte("hello"), nil)
mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil) mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil)
eventService := eventservice.NewMockEventService() eventService := eventservice.NewMockEventService()
pluginStore := NewMemoryPluginStore() pluginStore := NewMemoryPluginStore()
......
...@@ -91,7 +91,7 @@ func Test_pndImplementation_AddNetworkElement(t *testing.T) { ...@@ -91,7 +91,7 @@ func Test_pndImplementation_AddNetworkElement(t *testing.T) {
requestPluginFn: func(u uuid.UUID) (plugin.Plugin, error) { requestPluginFn: func(u uuid.UUID) (plugin.Plugin, error) {
mockPlugin := &mocks.Plugin{} mockPlugin := &mocks.Plugin{}
mockPlugin.On("ID").Return(u) mockPlugin.On("ID").Return(u)
mockPlugin.On("Model").Return([]byte("mockModel"), nil) mockPlugin.On("Model", mock.Anything).Return([]byte("mockModel"), nil)
return mockPlugin, nil return mockPlugin, nil
}, },
}, },
...@@ -102,7 +102,7 @@ func Test_pndImplementation_AddNetworkElement(t *testing.T) { ...@@ -102,7 +102,7 @@ func Test_pndImplementation_AddNetworkElement(t *testing.T) {
t.Run(tt.name, func(t *testing.T) { t.Run(tt.name, func(t *testing.T) {
mockPlugin := &mocks.Plugin{} mockPlugin := &mocks.Plugin{}
mockPlugin.On("ID").Return(defaultPluginID) mockPlugin.On("ID").Return(defaultPluginID)
mockPlugin.On("Model").Return([]byte("mockModel"), nil) mockPlugin.On("Model", mock.Anything).Return([]byte("mockModel"), nil)
mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil) mockPlugin.On("Unmarshal", mock.Anything, mock.Anything).Return(nil)
pnd, err := newPnd() pnd, err := newPnd()
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment