Skip to content

Commit 8ae868b

Browse files
committed
Adds more test cases in TestRunBindPlugins for the scheduler
1 parent 789dc87 commit 8ae868b

File tree

1 file changed

+40
-0
lines changed

1 file changed

+40
-0
lines changed

pkg/scheduler/framework/v1alpha1/framework_test.go

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1064,6 +1064,46 @@ func TestRunBindPlugins(t *testing.T) {
10641064
injects: []Code{Skip, Success, Error},
10651065
wantStatus: Success,
10661066
},
1067+
{
1068+
name: "no bind plugin, returns default binder",
1069+
injects: []Code{},
1070+
wantStatus: Success,
1071+
},
1072+
{
1073+
name: "invalid status",
1074+
injects: []Code{Unschedulable},
1075+
wantStatus: Error,
1076+
},
1077+
{
1078+
name: "simple error",
1079+
injects: []Code{Error},
1080+
wantStatus: Error,
1081+
},
1082+
{
1083+
name: "success on second, returns success",
1084+
injects: []Code{Skip, Success},
1085+
wantStatus: Success,
1086+
},
1087+
{
1088+
name: "invalid status, returns error",
1089+
injects: []Code{Skip, UnschedulableAndUnresolvable},
1090+
wantStatus: Error,
1091+
},
1092+
{
1093+
name: "error after success status, returns success",
1094+
injects: []Code{Success, Error},
1095+
wantStatus: Success,
1096+
},
1097+
{
1098+
name: "success before invalid status, returns success",
1099+
injects: []Code{Success, Error},
1100+
wantStatus: Success,
1101+
},
1102+
{
1103+
name: "success after error status, returns error",
1104+
injects: []Code{Error, Success},
1105+
wantStatus: Error,
1106+
},
10671107
}
10681108
for _, tt := range tests {
10691109
t.Run(tt.name, func(t *testing.T) {

0 commit comments

Comments
 (0)