Add test to check that group isn't mutated

grafana · Jan 10, 2025 · b33ec96 · b33ec96
1 parent 13b96f4
commit b33ec96
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pkg/ruler/rule_concurrency_test.go b/pkg/ruler/rule_concurrency_test.go
@@ -244,6 +244,10 @@ func TestSplitGroupIntoBatches(t *testing.T) {
 
 			batches := controller.SplitGroupIntoBatches(context.Background(), group)
 			requireConcurrentRulesEqual(t, tc.expectedGroups, batches)
+
+			// Make sure the group is not mutated and will still return the same batches when called again.
+			batches = controller.SplitGroupIntoBatches(context.Background(), group)
+			requireConcurrentRulesEqual(t, tc.expectedGroups, batches)
 		})
 	}
 }