kgiusti commented on a change in pull request #1097:
URL: https://github.com/apache/qpid-dispatch/pull/1097#discussion_r609023607
##########
File path: tests/system_tests_http1_base.py
##########
@@ -0,0 +1,1187 @@
+#
Review comment:
Hmmm... I think this file shouldn't be a "system_tests_*" file since it
does not define any TestCases. It defines a set of HTTP/1.x tests that are
shared between the http1_adaptor TestCases and http1_over_tcp TestCases.
Name this fine something like http1_tests.py instead otherwise I think
people will be tempted to add HTTP/1.x TestCases here instead of in the proper
system_tests_*.py files.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org
