Minor fixes

Azure · Sep 28, 2023 · e8acfbb · e8acfbb
1 parent e50c44d
commit e8acfbb
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/pkg/scheduler/framework/status.go b/pkg/scheduler/framework/status.go
@@ -38,7 +38,7 @@ const (
 	Skip
 )
 
-var statusCodeNames = []string{"Success", "InternalError", "ClusterUnschedulable", "Skip"}
+var statusCodeNames = []string{"Success", "InternalError", "ClusterUnschedulable", "ClusterAlreadySelected", "Skip"}
 
 // Name returns the name of a status code.
 func (sc StatusCode) Name() string {

diff --git a/pkg/scheduler/framework/status_test.go b/pkg/scheduler/framework/status_test.go
@@ -51,11 +51,17 @@ func TestNonNilStatusMethods(t *testing.T) {
 			sourcePlugin: dummyPlugin,
 		},
 		{
-			name:         "status preskip",
+			name:         "status skip",
 			statusCode:   Skip,
 			reasons:      dummyReasons,
 			sourcePlugin: dummyPlugin,
 		},
+		{
+			name:         "status cluster already selected",
+			statusCode:   ClusterAlreadySelected,
+			reasons:      dummyReasons,
+			sourcePlugin: dummyPlugin,
+		},
 	}
 
 	for _, tc := range testCases {
@@ -73,6 +79,7 @@ func TestNonNilStatusMethods(t *testing.T) {
 				status.IsSuccess,
 				status.IsInteralError,
 				status.IsClusterUnschedulable,
+				status.IsClusterAlreadySelected,
 				status.IsSkip,
 			}
 			for idx, checkFunc := range checkFuncs {