[GitHub] [trafficserver] dragon512 commented on a change in pull request #7063: Add an autest testcase for HTTP3

[GitBox <gi...@apache.org>]

dragon512 commented on a change in pull request #7063:
URL: https://github.com/apache/trafficserver/pull/7063#discussion_r486611867



##########
File path: tests/gold_tests/timeout/active_timeout.test.py
##########
@@ -60,3 +60,8 @@
 tr3= Test.AddTestRun("tr")
 tr3.Processes.Default.Command = 'curl -k -i --http2 https://127.0.0.1:{0}/file'.format(ts.Variables.ssl_port)
 tr3.Processes.Default.Streams.stdout = Testers.ContainsExpression("Activity Timeout", "Request should fail with active timeout")
+
+if Condition.HasCurlFeature('http3'):

Review comment:
       I am good with this.. the general question is if @zwoop is concerned with a test being skipped with no message that it was. I know he has a feature request in AuTest to have an option to force skipped test to be reported as an error




----------------------------------------------------------------
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:
users@infra.apache.org