Skip to content

Commit cd1099e

Browse files
committed
style fix
1 parent 404778d commit cd1099e

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

tests/integration/admin/test_describe_consumer_groups_compatability.py

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,13 @@
1313
# See the License for the specific language governing permissions and
1414
# limitations under the License.
1515

16-
import time
17-
import pytest
18-
19-
from confluent_kafka import Consumer, ConsumerGroupState, ConsumerGroupType, TopicPartition
20-
from confluent_kafka.admin import AdminClient
16+
from confluent_kafka import ConsumerGroupState, ConsumerGroupType, TopicPartition
2117
import uuid
2218

2319
from tests.common import TestUtils
2420

2521
topic_prefix = "test-topic"
26-
# Generate random group IDs
22+
2723

2824
def create_consumers(kafka_cluster, topic, group_id, client_id, Protocol):
2925
conf = {'group.id': group_id,
@@ -38,6 +34,7 @@ def create_consumers(kafka_cluster, topic, group_id, client_id, Protocol):
3834
consumer.poll(10)
3935
return consumer
4036

37+
4138
def verify_describe_consumer_groups(kafka_cluster, admin_client, topic):
4239

4340
group_id_new1 = f"test-group_new1-{uuid.uuid4()}"
@@ -51,7 +48,7 @@ def verify_describe_consumer_groups(kafka_cluster, admin_client, topic):
5148
client_id4 = "test-client4"
5249

5350
consumers = []
54-
51+
5552
# Create two groups with new group protocol
5653
consumers.append(create_consumers(kafka_cluster, topic, group_id_new1, client_id1, "consumer"))
5754
consumers.append(create_consumers(kafka_cluster, topic, group_id_new2, client_id2, "consumer"))
@@ -76,7 +73,7 @@ def verify_describe_consumer_groups(kafka_cluster, admin_client, topic):
7673
for member in result.members:
7774
assert member.client_id in [client_id1, client_id2]
7875
assert member.assignment.topic_partitions == partition
79-
76+
8077
fs2 = admin_client.describe_consumer_groups(group_ids=[group_id_old1, group_id_old2])
8178
for group_id, f in fs2.items():
8279
result = f.result()
@@ -88,7 +85,7 @@ def verify_describe_consumer_groups(kafka_cluster, admin_client, topic):
8885
for member in result.members:
8986
assert member.client_id in [client_id3, client_id4]
9087
assert member.assignment.topic_partitions == partition
91-
88+
9289
fs3 = admin_client.describe_consumer_groups(group_ids=[group_id_new1, group_id_new2, group_id_old1, group_id_old2])
9390
for group_id, f in fs3.items():
9491
result = f.result()
@@ -106,10 +103,11 @@ def verify_describe_consumer_groups(kafka_cluster, admin_client, topic):
106103
else:
107104
assert member.client_id in [client_id3, client_id4]
108105
assert member.assignment.topic_partitions == partition
109-
106+
110107
for consumer in consumers:
111108
consumer.close()
112109

110+
113111
def test_describe_consumer_groups_compatability(kafka_cluster):
114112

115113
admin_client = kafka_cluster.admin()
@@ -124,11 +122,11 @@ def test_describe_consumer_groups_compatability(kafka_cluster):
124122
},
125123
validate_only=False
126124
)
127-
125+
128126
if TestUtils.use_group_protocol_consumer():
129127
verify_describe_consumer_groups(kafka_cluster, admin_client, our_topic)
130128

131129
# Delete created topic
132130
fs = admin_client.delete_topics([our_topic])
133131
for topic, f in fs.items():
134-
f.result()
132+
f.result()

0 commit comments

Comments
 (0)