Amend runtime handler test for stable order

EricMountain · k8s-infra-cherrypick-robot · commit 57db13d50de6 · 2025-07-02T00:52:55.000Z
Signed-off-by: Eric Mountain &lt;eric.mountain@datadoghq.com&gt;
diff --git a/internal/cri/server/status_test.go b/internal/cri/server/status_test.go
@@ -134,7 +134,6 @@ func TestStatusRuntimeHandlersOrdering(t *testing.T) {
 		}
 	}
 
-	// The test will fail if the order is the same, showing that the ordering is stable
-	// In the current implementation, order is not stable
-	assert.False(t, sameOrder, "RuntimeHandlers order is stable across Status() calls - unexpected")
+	// Fail if runtime handlers order varies across calls to Status()
+	assert.True(t, sameOrder, "RuntimeHandlers order is unstable across Status() calls")
 }

Original file line number	Diff line number	Diff line change
`@@ -134,7 +134,6 @@ func TestStatusRuntimeHandlersOrdering(t *testing.T) {`
`134`	`134`	`}`
`135`	`135`	`}`
`136`	`136`
`137`		`- // The test will fail if the order is the same, showing that the ordering is stable`
`138`		`- // In the current implementation, order is not stable`
`139`		`- assert.False(t, sameOrder, "RuntimeHandlers order is stable across Status() calls - unexpected")`
	`137`	`+ // Fail if runtime handlers order varies across calls to Status()`
	`138`	`+ assert.True(t, sameOrder, "RuntimeHandlers order is unstable across Status() calls")`
`140`	`139`	`}`