github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2026-07-18 04:54:54 +00:00
Code Issues Projects Releases Wiki Activity

Remove test for unhandled case.

Browse Source 
Assume placement will always contain nodes that are present in the
snapshot
This commit is contained in:
Bartosz
2026-03-04 10:11:57 +00:00
parent 1140b62b68
commit 09379fbba7
1 changed files with 0 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
pkg/scheduler/backend/cache/snapshot_test.go vendored
View File

@@ -695,37 +695,6 @@ func TestSnapshot_Placement(t *testing.T) {
}
},
},
{
name: "When placement contains nodes from outside the snapshot, AssumePlacement returns error and placement is not set",
initialNodes: []string{"n1", "n2", "n3"},
placementNodes: []string{"n2", "n3", "n4"},
testFn: func(t *testing.T, snapshot *Snapshot, placement *fwk.Placement) {
err := snapshot.AssumePlacement(placement)
if err == nil {
t.Fatalf("expected AssumePlacement to return error due to placement using a node from outside the snapshot but got nil")
}
// ensure the placement is cleared
numNodes := snapshot.NumNodesInPlacement()
if numNodes != 3 {
t.Errorf("unexpected number of nodes from NumNodesInPlacement, want: %v, got: %v", 3, numNodes)
}
nodes, err := snapshot.ListNodesInPlacement()
if err != nil {
t.Fatalf("unexpected error from ListNodesInPlacement %v", err)
}
if len(nodes) != 3 {
t.Errorf("unexpected number of nodes from ListNodesInPlacement, want: %v, got: %v", 3, len(nodes))
}
_, err = snapshot.GetNodeInPlacement("n1")
if err != nil {
t.Errorf("expected GetNodeInPlacement to find node but instead got unexpected error %v", err)
}
_, err = snapshot.GetNodeInPlacement("n4")
if err == nil {
t.Errorf("expected GetNodeInPlacement not to find node but instead got nil error")
}
},
},
{
name: "When placement uses nodes that point to a different instance than snapshot, AssumePlacement returns error and placement is not set",
initialNodes: []string{"n1", "n2", "n3"},