X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=dmaap-mediator-producer%2Finternal%2Fserver%2Fserver_test.go;h=6fe4d7a70a0bf89e09dd19b101838a2ef6cd2ca2;hb=6f5d3d1eccb8a1857c645ba6bd0b5e1b89ca7088;hp=5c2027aa9d237a6c92a07ed67572f5b587345f0d;hpb=c658d388f93467cb1a1229d11a0e01616cd97e3c;p=nonrtric.git

diff --git a/dmaap-mediator-producer/internal/server/server_test.go b/dmaap-mediator-producer/internal/server/server_test.go
index 5c2027aa..6fe4d7a7 100644
--- a/dmaap-mediator-producer/internal/server/server_test.go
+++ b/dmaap-mediator-producer/internal/server/server_test.go
@@ -34,20 +34,20 @@ import (
 	"github.com/stretchr/testify/mock"
 	"github.com/stretchr/testify/require"
 	"oransc.org/nonrtric/dmaapmediatorproducer/internal/jobs"
-	"oransc.org/nonrtric/dmaapmediatorproducer/mocks/jobhandler"
+	"oransc.org/nonrtric/dmaapmediatorproducer/mocks/jobshandler"
 )
 
 func TestNewRouter(t *testing.T) {
 	assertions := require.New(t)
 
-	r := NewRouter(nil)
-	statusRoute := r.Get("status")
+	r := NewRouter(nil, nil)
+	statusRoute := r.Get("health_check")
 	assertions.NotNil(statusRoute)
 	supportedMethods, err := statusRoute.GetMethods()
 	assertions.Equal([]string{http.MethodGet}, supportedMethods)
 	assertions.Nil(err)
 	path, _ := statusRoute.GetPathTemplate()
-	assertions.Equal("/status", path)
+	assertions.Equal("/health_check", path)
 
 	addJobRoute := r.Get("add")
 	assertions.NotNil(addJobRoute)
@@ -55,7 +55,7 @@ func TestNewRouter(t *testing.T) {
 	assertions.Equal([]string{http.MethodPost}, supportedMethods)
 	assertions.Nil(err)
 	path, _ = addJobRoute.GetPathTemplate()
-	assertions.Equal("/jobs", path)
+	assertions.Equal("/info_job", path)
 
 	deleteJobRoute := r.Get("delete")
 	assertions.NotNil(deleteJobRoute)
@@ -63,7 +63,7 @@ func TestNewRouter(t *testing.T) {
 	assertions.Equal([]string{http.MethodDelete}, supportedMethods)
 	assertions.Nil(err)
 	path, _ = deleteJobRoute.GetPathTemplate()
-	assertions.Equal("/jobs/{infoJobId}", path)
+	assertions.Equal("/info_job/{infoJobId}", path)
 
 	notFoundHandler := r.NotFoundHandler
 	handler := http.HandlerFunc(notFoundHandler.ServeHTTP)
@@ -78,22 +78,17 @@ func TestNewRouter(t *testing.T) {
 	handler.ServeHTTP(responseRecorder, newRequest(http.MethodPut, "/status", nil, t))
 	assertions.Equal(http.StatusMethodNotAllowed, responseRecorder.Code)
 	assertions.Contains(responseRecorder.Body.String(), "Method is not supported.")
-}
-
-func TestStatusHandler(t *testing.T) {
-	assertions := require.New(t)
 
-	handler := http.HandlerFunc(statusHandler)
-	responseRecorder := httptest.NewRecorder()
-	r := newRequest(http.MethodGet, "/status", nil, t)
-
-	handler.ServeHTTP(responseRecorder, r)
-
-	assertions.Equal(http.StatusOK, responseRecorder.Code)
-	assertions.Equal("", responseRecorder.Body.String())
+	setLogLevelRoute := r.Get("setLogLevel")
+	assertions.NotNil(setLogLevelRoute)
+	supportedMethods, err = setLogLevelRoute.GetMethods()
+	assertions.Equal([]string{http.MethodPut}, supportedMethods)
+	assertions.Nil(err)
+	path, _ = setLogLevelRoute.GetPathTemplate()
+	assertions.Equal("/admin/log", path)
 }
 
-func TestAddInfoJobHandler(t *testing.T) {
+func TestAddInfoJobToJobsHandler(t *testing.T) {
 	assertions := require.New(t)
 
 	type args struct {
@@ -107,22 +102,21 @@ func TestAddInfoJobHandler(t *testing.T) {
 		wantedBody   string
 	}{
 		{
-			name: "AddInfoJobHandler with correct job, should return OK",
+			name: "AddInfoJobToJobsHandler with correct job, should return OK",
 			args: args{
 				job: jobs.JobInfo{
 					Owner:            "owner",
 					LastUpdated:      "now",
 					InfoJobIdentity:  "jobId",
 					TargetUri:        "target",
-					InfoJobData:      "{}",
+					InfoJobData:      jobs.Parameters{},
 					InfoTypeIdentity: "type",
 				},
 			},
 			wantedStatus: http.StatusOK,
-			wantedBody:   "",
 		},
 		{
-			name: "AddInfoJobHandler with incorrect job info, should return BadRequest",
+			name: "AddInfoJobToJobsHandler with incorrect job info, should return BadRequest",
 			args: args{
 				job: jobs.JobInfo{
 					Owner: "bad",
@@ -135,10 +129,10 @@ func TestAddInfoJobHandler(t *testing.T) {
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			jobHandlerMock := jobhandler.JobHandler{}
-			jobHandlerMock.On("AddJob", tt.args.job).Return(tt.args.mockReturn)
+			jobsHandlerMock := jobshandler.JobsHandler{}
+			jobsHandlerMock.On("AddJobFromRESTCall", tt.args.job).Return(tt.args.mockReturn)
 
-			callbackHandlerUnderTest := NewProducerCallbackHandler(&jobHandlerMock)
+			callbackHandlerUnderTest := NewProducerCallbackHandler(&jobsHandlerMock)
 
 			handler := http.HandlerFunc(callbackHandlerUnderTest.addInfoJobHandler)
 			responseRecorder := httptest.NewRecorder()
@@ -148,17 +142,17 @@ func TestAddInfoJobHandler(t *testing.T) {
 
 			assertions.Equal(tt.wantedStatus, responseRecorder.Code, tt.name)
 			assertions.Contains(responseRecorder.Body.String(), tt.wantedBody, tt.name)
-			jobHandlerMock.AssertCalled(t, "AddJob", tt.args.job)
+			jobsHandlerMock.AssertCalled(t, "AddJobFromRESTCall", tt.args.job)
 		})
 	}
 }
 
 func TestDeleteJob(t *testing.T) {
 	assertions := require.New(t)
-	jobHandlerMock := jobhandler.JobHandler{}
-	jobHandlerMock.On("DeleteJob", mock.Anything).Return(nil)
+	jobsHandlerMock := jobshandler.JobsHandler{}
+	jobsHandlerMock.On("DeleteJobFromRESTCall", mock.Anything).Return(nil)
 
-	callbackHandlerUnderTest := NewProducerCallbackHandler(&jobHandlerMock)
+	callbackHandlerUnderTest := NewProducerCallbackHandler(&jobsHandlerMock)
 
 	responseRecorder := httptest.NewRecorder()
 	r := mux.SetURLVars(newRequest(http.MethodDelete, "/jobs/", nil, t), map[string]string{"infoJobId": "job1"})
@@ -168,7 +162,51 @@ func TestDeleteJob(t *testing.T) {
 
 	assertions.Equal("", responseRecorder.Body.String())
 
-	jobHandlerMock.AssertCalled(t, "DeleteJob", "job1")
+	jobsHandlerMock.AssertCalled(t, "DeleteJobFromRESTCall", "job1")
+}
+
+func TestSetLogLevel(t *testing.T) {
+	assertions := require.New(t)
+
+	type args struct {
+		logLevel string
+	}
+	tests := []struct {
+		name         string
+		args         args
+		wantedStatus int
+		wantedBody   string
+	}{
+		{
+			name: "Set to valid log level, should return OK",
+			args: args{
+				logLevel: "Debug",
+			},
+			wantedStatus: http.StatusOK,
+		},
+		{
+			name: "Set to invalid log level, should return BadRequest",
+			args: args{
+				logLevel: "bad",
+			},
+			wantedStatus: http.StatusBadRequest,
+			wantedBody:   "Invalid log level: bad",
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			callbackHandlerUnderTest := NewProducerCallbackHandler(nil)
+
+			handler := http.HandlerFunc(callbackHandlerUnderTest.setLogLevel)
+			responseRecorder := httptest.NewRecorder()
+			r, _ := http.NewRequest(http.MethodPut, "/admin/log?level="+tt.args.logLevel, nil)
+
+			handler.ServeHTTP(responseRecorder, r)
+
+			assertions.Equal(tt.wantedStatus, responseRecorder.Code, tt.name)
+			assertions.Contains(responseRecorder.Body.String(), tt.wantedBody, tt.name)
+		})
+	}
 }
 
 func newRequest(method string, url string, jobInfo *jobs.JobInfo, t *testing.T) *http.Request {