Skip to content

Commit 6406f42

Browse files
committed
operator: Explain intention behind unit test assertion
1 parent a869471 commit 6406f42

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

operator/internal/controller/redpanda/redpanda_controller_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -385,6 +385,8 @@ func (s *RedpandaControllerSuite) TestClusterSettings() {
385385
return int(a.ConfigVersion - b.ConfigVersion)
386386
}).ConfigVersion
387387

388+
// Only operator should change cluster configuration once. If there is any other party that changes
389+
// Redpanda cluster configuration, it is unexpected and should be investigated.
388390
assert.Equal(t, initialVersion+1, currVersion, "current config version should increase only by one")
389391

390392
assert.False(t, slices.ContainsFunc(st, func(cs rpadmin.ConfigStatus) bool {

0 commit comments

Comments
 (0)