X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=E2Manager%2Fservices%2Frnib_data_service_test.go;h=5460608b701bf291d09958c98a5cf98d2d48d077;hb=refs%2Fchanges%2F19%2F4519%2F1;hp=ab67a32041ad9c02d734ba63415195fbbf5aa6f8;hpb=372a275602ae05da22130a4601709291c7fbbaa6;p=ric-plt%2Fe2mgr.git

diff --git a/E2Manager/services/rnib_data_service_test.go b/E2Manager/services/rnib_data_service_test.go
index ab67a32..5460608 100644
--- a/E2Manager/services/rnib_data_service_test.go
+++ b/E2Manager/services/rnib_data_service_test.go
@@ -17,7 +17,6 @@
 //  This source code is part of the near-RT RIC (RAN Intelligent Controller)
 //  platform project (RICP).
 
-
 package services
 
 import (
@@ -43,14 +42,12 @@ func setupRnibDataServiceTestWithMaxAttempts(t *testing.T, maxAttempts int) (*rN
 		t.Errorf("#... - failed to initialize logger, error: %s", err)
 	}
 
-	config := &configuration.Configuration{RnibRetryIntervalMs: 10, MaxRnibConnectionAttempts: maxAttempts}
+	config := &configuration.Configuration{RnibRetryIntervalMs: 10, MaxRnibConnectionAttempts: maxAttempts, RnibWriter: configuration.RnibWriterConfig{RanManipulationMessageChannel: "RAN_MANIPULATION", StateChangeMessageChannel: "RAN_CONNECTION_STATUS_CHANGE"}}
 
 	readerMock := &mocks.RnibReaderMock{}
 
-
 	writerMock := &mocks.RnibWriterMock{}
 
-
 	rnibDataService := NewRnibDataService(logger, config, readerMock, writerMock)
 	assert.NotNil(t, rnibDataService)
 
@@ -61,10 +58,9 @@ func TestSuccessfulSaveNodeb(t *testing.T) {
 	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
 
 	nodebInfo := &entities.NodebInfo{}
-	nbIdentity := &entities.NbIdentity{}
-	writerMock.On("SaveNodeb", nbIdentity, nodebInfo).Return(nil)
+	writerMock.On("SaveNodeb", nodebInfo).Return(nil)
 
-	rnibDataService.SaveNodeb(nbIdentity, nodebInfo)
+	rnibDataService.SaveNodeb(nodebInfo)
 	writerMock.AssertNumberOfCalls(t, "SaveNodeb", 1)
 }
 
@@ -72,11 +68,10 @@ func TestConnFailureSaveNodeb(t *testing.T) {
 	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
 
 	nodebInfo := &entities.NodebInfo{}
-	nbIdentity := &entities.NbIdentity{}
 	mockErr := &common.InternalError{Err: &net.OpError{Err: fmt.Errorf("connection error")}}
-	writerMock.On("SaveNodeb", nbIdentity, nodebInfo).Return(mockErr)
+	writerMock.On("SaveNodeb", nodebInfo).Return(mockErr)
 
-	rnibDataService.SaveNodeb(nbIdentity, nodebInfo)
+	rnibDataService.SaveNodeb(nodebInfo)
 	writerMock.AssertNumberOfCalls(t, "SaveNodeb", 3)
 }
 
@@ -84,11 +79,10 @@ func TestNonConnFailureSaveNodeb(t *testing.T) {
 	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
 
 	nodebInfo := &entities.NodebInfo{}
-	nbIdentity := &entities.NbIdentity{}
 	mockErr := &common.InternalError{Err: fmt.Errorf("non connection failure")}
-	writerMock.On("SaveNodeb", nbIdentity, nodebInfo).Return(mockErr)
+	writerMock.On("SaveNodeb", nodebInfo).Return(mockErr)
 
-	rnibDataService.SaveNodeb(nbIdentity, nodebInfo)
+	rnibDataService.SaveNodeb(nodebInfo)
 	writerMock.AssertNumberOfCalls(t, "SaveNodeb", 1)
 }
 
@@ -159,7 +153,7 @@ func TestConnFailureGetNodeb(t *testing.T) {
 
 	res, err := rnibDataService.GetNodeb(invName)
 	readerMock.AssertNumberOfCalls(t, "GetNodeb", 3)
-	assert.True(t, strings.Contains(err.Error(), "connection error", ))
+	assert.True(t, strings.Contains(err.Error(), "connection error"))
 	assert.Equal(t, nodeb, res)
 }
 
@@ -184,7 +178,7 @@ func TestConnFailureGetNodebIdList(t *testing.T) {
 
 	res, err := rnibDataService.GetListNodebIds()
 	readerMock.AssertNumberOfCalls(t, "GetListNodebIds", 3)
-	assert.True(t, strings.Contains(err.Error(), "connection error", ))
+	assert.True(t, strings.Contains(err.Error(), "connection error"))
 	assert.Equal(t, nodeIds, res)
 }
 
@@ -200,12 +194,12 @@ func TestConnFailureTwiceGetNodebIdList(t *testing.T) {
 
 	res, err := rnibDataService.GetListNodebIds()
 	readerMock.AssertNumberOfCalls(t, "GetListNodebIds", 3)
-	assert.True(t, strings.Contains(err.Error(), "connection error", ))
+	assert.True(t, strings.Contains(err.Error(), "connection error"))
 	assert.Equal(t, nodeIds, res)
 
 	res2, err := rnibDataService.GetNodeb(invName)
 	readerMock.AssertNumberOfCalls(t, "GetNodeb", 3)
-	assert.True(t, strings.Contains(err.Error(), "connection error", ))
+	assert.True(t, strings.Contains(err.Error(), "connection error"))
 	assert.Equal(t, nodeb, res2)
 }
 
@@ -218,7 +212,7 @@ func TestConnFailureWithAnotherConfig(t *testing.T) {
 
 	res, err := rnibDataService.GetListNodebIds()
 	readerMock.AssertNumberOfCalls(t, "GetListNodebIds", 5)
-	assert.True(t, strings.Contains(err.Error(), "connection error", ))
+	assert.True(t, strings.Contains(err.Error(), "connection error"))
 	assert.Equal(t, nodeIds, res)
 }
 
@@ -257,20 +251,28 @@ func TestPingRnibOkOtherError(t *testing.T) {
 	assert.True(t, res)
 }
 
-//func TestConnFailureThenSuccessGetNodebIdList(t *testing.T) {
-//	rnibDataService, readerMock, _ := setupRnibDataServiceTest(t)
-//
-//	var nilNodeIds []*entities.NbIdentity = nil
-//	nodeIds := []*entities.NbIdentity{}
-//	mockErr := &common.InternalError{Err: &net.OpError{Err:fmt.Errorf("connection error")}}
-//	//readerMock.On("GetListNodebIds").Return(nilNodeIds, mockErr)
-//	//readerMock.On("GetListNodebIds").Return(nodeIds, nil)
-//
-//	res, err := rnibDataService.GetListNodebIds()
-//	readerMock.AssertNumberOfCalls(t, "GetListNodebIds", 2)
-//	assert.True(t, strings.Contains(err.Error(),"connection failure", ))
-//	assert.Equal(t, nodeIds, res)
-//}
+func TestSuccessfulUpdateNodebInfoOnConnectionStatusInversion(t *testing.T) {
+	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
+	event := "event"
+
+	nodebInfo := &entities.NodebInfo{}
+	writerMock.On("UpdateNodebInfoOnConnectionStatusInversion", nodebInfo, event).Return(nil)
+
+	rnibDataService.UpdateNodebInfoOnConnectionStatusInversion(nodebInfo, event)
+	writerMock.AssertNumberOfCalls(t, "UpdateNodebInfoOnConnectionStatusInversion", 1)
+}
+
+func TestConnFailureUpdateNodebInfoOnConnectionStatusInversion(t *testing.T) {
+	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
+	event := "event"
+
+	nodebInfo := &entities.NodebInfo{}
+	mockErr := &common.InternalError{Err: &net.OpError{Err: fmt.Errorf("connection error")}}
+	writerMock.On("UpdateNodebInfoOnConnectionStatusInversion", nodebInfo, event).Return(mockErr)
+
+	rnibDataService.UpdateNodebInfoOnConnectionStatusInversion(nodebInfo, event)
+	writerMock.AssertNumberOfCalls(t, "UpdateNodebInfoOnConnectionStatusInversion", 3)
+}
 
 func TestGetE2TInstanceConnFailure(t *testing.T) {
 	rnibDataService, readerMock, _ := setupRnibDataServiceTest(t)
@@ -312,3 +314,38 @@ func TestGetE2TInstanceOkOtherError(t *testing.T) {
 	assert.Nil(t, res)
 	assert.NotNil(t, err)
 }
+
+func TestRemoveEnbConnFailure(t *testing.T) {
+	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
+
+	mockErr := &common.InternalError{Err: &net.OpError{Err: fmt.Errorf("connection error")}}
+	nodebInfo := &entities.NodebInfo{}
+	writerMock.On("RemoveEnb", nodebInfo).Return(mockErr)
+
+	err := rnibDataService.RemoveEnb(nodebInfo)
+	writerMock.AssertNumberOfCalls(t, "RemoveEnb", 3)
+	assert.NotNil(t, err)
+}
+
+func TestRemoveEnbOkNoError(t *testing.T) {
+	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
+
+	nodebInfo := &entities.NodebInfo{}
+	writerMock.On("RemoveEnb", nodebInfo).Return(nil)
+
+	err := rnibDataService.RemoveEnb(nodebInfo)
+	writerMock.AssertNumberOfCalls(t, "RemoveEnb", 1)
+	assert.Nil(t, err)
+}
+
+func TestRemoveEnbOtherError(t *testing.T) {
+	rnibDataService, _, writerMock := setupRnibDataServiceTest(t)
+
+	mockErr := &common.InternalError{Err: fmt.Errorf("non connection error")}
+	nodebInfo := &entities.NodebInfo{}
+	writerMock.On("RemoveEnb", nodebInfo).Return(mockErr)
+
+	err := rnibDataService.RemoveEnb(nodebInfo)
+	writerMock.AssertNumberOfCalls(t, "RemoveEnb", 1)
+	assert.NotNil(t, err)
+}