driazati commented on code in PR #12436:
URL: https://github.com/apache/tvm/pull/12436#discussion_r954601652
##########
tests/python/ci/test_ci.py:
##########
@@ -33,6 +34,168 @@ def parameterize_named(*values):
return pytest.mark.parametrize(",".join(keys), [tuple(d.values()) for d in
values])
[email protected]_if_wheel_test
[email protected](
+ # pylint: disable=line-too-long
+
"main_xml_file,main_xml_content,pr_xml_file,pr_xml_content,target_url,s3_prefix,"
Review Comment:
also the test names here that pytest autogenerates and shows in test reports
are crazy long since they include the whole junit string, could you also pass
through `ids=[...]` with a short name for each case? i.e. something similar to
https://github.com/apache/tvm/blob/main/tests/python/ci/test_tvmbot.py#L141-L145
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
