add read only test

segmentio · Nov 21, 2023 · 2f593fa · 2f593fa
1 parent 434d602
commit 2f593fa
Showing 1 changed file with 21 additions and 0 deletions.
diff --git a/pkg/admin/brokerclient_test.go b/pkg/admin/brokerclient_test.go
@@ -868,3 +868,24 @@ func TestBrokerClientCreateACLReadOnly(t *testing.T) {
 	assert.Equal(t, err, errors.New("Cannot create ACL in read-only mode"))
 
 }
+
+func TestBrokerClientDeleteACLReadOnly(t *testing.T) {
+	if !util.CanTestBrokerAdminSecurity() {
+		t.Skip("Skipping because KAFKA_TOPICS_TEST_BROKER_ADMIN_SECURITY is not set")
+	}
+
+	ctx := context.Background()
+	client, err := NewBrokerAdminClient(
+		ctx,
+		BrokerAdminClientConfig{
+			ConnectorConfig: ConnectorConfig{
+				BrokerAddr: util.TestKafkaAddr(),
+			},
+			ReadOnly: true,
+		},
+	)
+	require.NoError(t, err)
+	_, err = client.DeleteACLs(ctx, []kafka.DeleteACLsFilter{})
+
+	assert.Equal(t, errors.New("Cannot delete ACL in read-only mode"), err)
+}