This is an automated email from the ASF dual-hosted git repository.

bneradt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 8689842cfe Fix the curl.test.ext spawn_curl_commands ts parameter 
(#12358)
8689842cfe is described below

commit 8689842cfe06a9f4e7ec7c4107f5039d71b64e64
Author: Brian Neradt <brian.ner...@gmail.com>
AuthorDate: Sat Jul 12 17:57:03 2025 -0400

    Fix the curl.test.ext spawn_curl_commands ts parameter (#12358)
    
    This fixes:
    
    ```
      File 
"/home/jenkins/trafficserver/tests/gold_tests_filtered/autest-site/curl.test.ext",
 line 33
        def spawn_curl_commands(self, cmdstr, count, retcode=0, 
use_default=True, ts):
                                                                                
  ^^
    SyntaxError: parameter without a default follows parameter with a default
    ```
    
    Co-authored-by: bneradt <bner...@yahooinc.com>
---
 tests/gold_tests/autest-site/curl.test.ext | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/gold_tests/autest-site/curl.test.ext 
b/tests/gold_tests/autest-site/curl.test.ext
index 69b0cb9d5b..b1e3528942 100644
--- a/tests/gold_tests/autest-site/curl.test.ext
+++ b/tests/gold_tests/autest-site/curl.test.ext
@@ -30,7 +30,7 @@ Tools to help with TestRun commands
 #
 
 
-def spawn_curl_commands(self, cmdstr, count, retcode=0, use_default=True, ts):
+def spawn_curl_commands(self, cmdstr, count, retcode=0, use_default=True, 
ts=None):
     ret = []
 
     if self.Variables.get("CurlUds", False):
@@ -48,7 +48,7 @@ def spawn_curl_commands(self, cmdstr, count, retcode=0, 
use_default=True, ts):
     return ret
 
 
-def curl_command(self, cmd, ts, p=None):
+def curl_command(self, cmd, ts=None, p=None):
     if p == None:
         p = self.Processes.Default
     if self.Variables.get("CurlUds", False):
@@ -58,7 +58,7 @@ def curl_command(self, cmd, ts, p=None):
     return p
 
 
-def curl_multiple_commands(self, cmd, ts):
+def curl_multiple_commands(self, cmd, ts=None):
     p = self.Processes.Default
     if self.Variables.get("CurlUds", False):
         p.Command = cmd.format(curl=f'curl --unix-socket 
{ts.Variables.uds_path}', curl_base='curl')

Reply via email to