Merge pull request #99 from Gekko0114/minor-fix

k8s-ci-robot · web-flow · commit 30ca499de602 · 2024-01-25T03:32:09.000+01:00
Fix minor bug about %v
diff --git a/scheduler/plugin/plugin_test.go b/scheduler/plugin/plugin_test.go
@@ -483,7 +483,7 @@ wasm stack trace:
 				t.Fatalf("unexpected node names: want %v, have %v", want, have)
 			}
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if want, have := tc.expectedStatusMessage, status.Message(); want != have {
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)
@@ -569,7 +569,7 @@ wasm stack trace:
 			ni.SetNode(tc.node)
 			s := p.(framework.FilterPlugin).Filter(ctx, nil, tc.pod, ni)
 			if want, have := tc.expectedStatusCode, s.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if want, have := tc.expectedStatusMessage, s.Message(); want != have {
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)
@@ -677,7 +677,7 @@ wasm stack trace:
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)
 			}
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 		})
 	}
@@ -772,7 +772,7 @@ wasm stack trace:
 
 			status := p.(framework.PreScorePlugin).PreScore(ctx, nil, tc.pod, tc.nodes)
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if want, have := tc.expectedStatusMessage, status.Message(); want != have {
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)
@@ -881,7 +881,7 @@ wasm stack trace:
 				t.Fatalf("unexpected score: want %v, have %v", want, have)
 			}
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if want, have := tc.expectedStatusMessage, status.Message(); want != have {
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)
@@ -975,7 +975,7 @@ wasm stack trace:
 				t.Fatalf("unexpected status message: want \n%v, have \n%v", want, have)
 			}
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if tc.expectedNodeScoreList != nil {
 				if tc.expectedNodeScoreList[0] != tc.nodeScoreList[0] {
@@ -1061,7 +1061,7 @@ func TestReserve(t *testing.T) {
 			cycleState := framework.NewCycleState()
 			status := p.(framework.ReservePlugin).Reserve(ctx, cycleState, tc.pod, tc.nodeName)
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if want, have := tc.expectedStatusMessage, status.Message(); want != have {
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)
@@ -1162,7 +1162,7 @@ wasm stack trace:
 
 			status := p.(framework.PreBindPlugin).PreBind(ctx, nil, tc.pod, tc.nodeName)
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if want, have := tc.expectedStatusMessage, status.Message(); want != have {
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)
@@ -1247,7 +1247,7 @@ wasm stack trace:
 
 			status := p.(framework.BindPlugin).Bind(ctx, nil, tc.pod, tc.nodeName)
 			if want, have := tc.expectedStatusCode, status.Code(); want != have {
-				t.Fatalf("unexpected status code: want %v, have %v", want, have)
+				t.Fatalf("unexpected status code: want %d, have %d", want, have)
 			}
 			if want, have := tc.expectedStatusMessage, status.Message(); want != have {
 				t.Fatalf("unexpected status message: want %v, have %v", want, have)

Original file line number	Diff line number	Diff line change
`@@ -483,7 +483,7 @@ wasm stack trace:`
`483`	`483`	`t.Fatalf("unexpected node names: want %v, have %v", want, have)`
`484`	`484`	`}`
`485`	`485`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`486`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`486`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`487`	`487`	`}`
`488`	`488`	`if want, have := tc.expectedStatusMessage, status.Message(); want != have {`
`489`	`489`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`
`@@ -569,7 +569,7 @@ wasm stack trace:`
`569`	`569`	`ni.SetNode(tc.node)`
`570`	`570`	`s := p.(framework.FilterPlugin).Filter(ctx, nil, tc.pod, ni)`
`571`	`571`	`if want, have := tc.expectedStatusCode, s.Code(); want != have {`
`572`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`572`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`573`	`573`	`}`
`574`	`574`	`if want, have := tc.expectedStatusMessage, s.Message(); want != have {`
`575`	`575`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`
`@@ -677,7 +677,7 @@ wasm stack trace:`
`677`	`677`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`
`678`	`678`	`}`
`679`	`679`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`680`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`680`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`681`	`681`	`}`
`682`	`682`	`})`
`683`	`683`	`}`
`@@ -772,7 +772,7 @@ wasm stack trace:`
`772`	`772`
`773`	`773`	`status := p.(framework.PreScorePlugin).PreScore(ctx, nil, tc.pod, tc.nodes)`
`774`	`774`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`775`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`775`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`776`	`776`	`}`
`777`	`777`	`if want, have := tc.expectedStatusMessage, status.Message(); want != have {`
`778`	`778`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`
`@@ -881,7 +881,7 @@ wasm stack trace:`
`881`	`881`	`t.Fatalf("unexpected score: want %v, have %v", want, have)`
`882`	`882`	`}`
`883`	`883`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`884`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`884`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`885`	`885`	`}`
`886`	`886`	`if want, have := tc.expectedStatusMessage, status.Message(); want != have {`
`887`	`887`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`
`@@ -975,7 +975,7 @@ wasm stack trace:`
`975`	`975`	`t.Fatalf("unexpected status message: want \n%v, have \n%v", want, have)`
`976`	`976`	`}`
`977`	`977`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`978`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`978`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`979`	`979`	`}`
`980`	`980`	`if tc.expectedNodeScoreList != nil {`
`981`	`981`	`if tc.expectedNodeScoreList[0] != tc.nodeScoreList[0] {`
`@@ -1061,7 +1061,7 @@ func TestReserve(t *testing.T) {`
`1061`	`1061`	`cycleState := framework.NewCycleState()`
`1062`	`1062`	`status := p.(framework.ReservePlugin).Reserve(ctx, cycleState, tc.pod, tc.nodeName)`
`1063`	`1063`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`1064`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`1064`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`1065`	`1065`	`}`
`1066`	`1066`	`if want, have := tc.expectedStatusMessage, status.Message(); want != have {`
`1067`	`1067`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`
`@@ -1162,7 +1162,7 @@ wasm stack trace:`
`1162`	`1162`
`1163`	`1163`	`status := p.(framework.PreBindPlugin).PreBind(ctx, nil, tc.pod, tc.nodeName)`
`1164`	`1164`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`1165`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`1165`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`1166`	`1166`	`}`
`1167`	`1167`	`if want, have := tc.expectedStatusMessage, status.Message(); want != have {`
`1168`	`1168`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`
`@@ -1247,7 +1247,7 @@ wasm stack trace:`
`1247`	`1247`
`1248`	`1248`	`status := p.(framework.BindPlugin).Bind(ctx, nil, tc.pod, tc.nodeName)`
`1249`	`1249`	`if want, have := tc.expectedStatusCode, status.Code(); want != have {`
`1250`		`- t.Fatalf("unexpected status code: want %v, have %v", want, have)`
	`1250`	`+ t.Fatalf("unexpected status code: want %d, have %d", want, have)`
`1251`	`1251`	`}`
`1252`	`1252`	`if want, have := tc.expectedStatusMessage, status.Message(); want != have {`
`1253`	`1253`	`t.Fatalf("unexpected status message: want %v, have %v", want, have)`