Skip to content

Commit 8689842

Browse files
bneradtbneradt
andauthored
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 <bneradt@yahooinc.com>
1 parent 636772a commit 8689842

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/gold_tests/autest-site/curl.test.ext

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ Tools to help with TestRun commands
3030
#
3131

3232

33-
def spawn_curl_commands(self, cmdstr, count, retcode=0, use_default=True, ts):
33+
def spawn_curl_commands(self, cmdstr, count, retcode=0, use_default=True, ts=None):
3434
ret = []
3535

3636
if self.Variables.get("CurlUds", False):
@@ -48,7 +48,7 @@ def spawn_curl_commands(self, cmdstr, count, retcode=0, use_default=True, ts):
4848
return ret
4949

5050

51-
def curl_command(self, cmd, ts, p=None):
51+
def curl_command(self, cmd, ts=None, p=None):
5252
if p == None:
5353
p = self.Processes.Default
5454
if self.Variables.get("CurlUds", False):
@@ -58,7 +58,7 @@ def curl_command(self, cmd, ts, p=None):
5858
return p
5959

6060

61-
def curl_multiple_commands(self, cmd, ts):
61+
def curl_multiple_commands(self, cmd, ts=None):
6262
p = self.Processes.Default
6363
if self.Variables.get("CurlUds", False):
6464
p.Command = cmd.format(curl=f'curl --unix-socket {ts.Variables.uds_path}', curl_base='curl')

0 commit comments

Comments
 (0)