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:
[email protected]


Reply via email to