Skip to content

Commit 0259502

Browse files
Update conftest.py to use request.node.originalname instead for legal topic naming (dpkp#172)
* Update conftest.py to use request.node.originalname instead for legal topic naming Otherwise parametrization doesn't work. * Update test/conftest.py Co-authored-by: code-review-doctor[bot] <72320148+code-review-doctor[bot]@users.noreply.github.com> --------- Co-authored-by: code-review-doctor[bot] <72320148+code-review-doctor[bot]@users.noreply.github.com>
1 parent 2f2ccb1 commit 0259502

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ def factory(**kafka_admin_client_params):
119119
@pytest.fixture
120120
def topic(kafka_broker, request):
121121
"""Return a topic fixture"""
122-
topic_name = '%s_%s' % (request.node.name, random_string(10))
122+
topic_name = f'{request.node.originalname}_{random_string(10)}'
123123
kafka_broker.create_topics([topic_name])
124124
return topic_name
125125

0 commit comments

Comments
 (0)