autotest: stop passing defaults in via --defaults to custom_sitl_comm…

…andline
ArduPilot · Apr 18, 2024 · 61a7460 · 61a7460
1 parent b78b79f
commit 61a7460
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 5 deletions.
diff --git a/Tools/autotest/arducopter.py b/Tools/autotest/arducopter.py
@@ -7652,7 +7652,8 @@ def BaroWindCorrection(self):
         ex = None
         try:
             self.customise_SITL_commandline(
-                ["--defaults", ','.join(self.model_defaults_filepath('Callisto'))],
+                [],
+                defaults_filepath=self.model_defaults_filepath('Callisto'),
                 model="octa-quad:@ROMFS/models/Callisto.json",
                 wipe=True,
             )
@@ -8847,7 +8848,8 @@ def GPSBlending(self):
     def Callisto(self):
         '''Test Callisto'''
         self.customise_SITL_commandline(
-            ["--defaults", ','.join(self.model_defaults_filepath('Callisto')), ],
+            [],
+            defaults_filepath=self.model_defaults_filepath('Callisto'),
             model="octa-quad:@ROMFS/models/Callisto.json",
             wipe=True,
         )

diff --git a/Tools/autotest/arduplane.py b/Tools/autotest/arduplane.py
@@ -3917,7 +3917,8 @@ def FlyEachFrame(self):
             if not isinstance(defaults, list):
                 defaults = [defaults]
             self.customise_SITL_commandline(
-                ["--defaults", ','.join(defaults), ],
+                [],
+                defaults_filepath=defaults,
                 model=model,
                 wipe=True,
             )

diff --git a/Tools/autotest/helicopter.py b/Tools/autotest/helicopter.py
@@ -199,7 +199,8 @@ def FlyEachFrame(self):
             if not isinstance(defaults, list):
                 defaults = [defaults]
             self.customise_SITL_commandline(
-                ["--defaults", ','.join(defaults), ],
+                [],
+                defaults_filepath=defaults,
                 model=model,
                 wipe=True,
             )
@@ -210,7 +211,8 @@ def FlyEachFrame(self):
     def governortest(self):
         '''Test Heli Internal Throttle Curve and Governor'''
         self.customise_SITL_commandline(
-            ["--defaults", ','.join(self.model_defaults_filepath('heli-gas')), ],
+            [],
+            defaults_filepath=self.model_defaults_filepath('heli-gas'),
             model="heli-gas",
             wipe=True,
         )

diff --git a/Tools/autotest/pysim/util.py b/Tools/autotest/pysim/util.py
@@ -542,6 +542,9 @@ def start_SITL(binary,
 
     cmd.extend(customisations)
 
+    if "--defaults" in customisations:
+        raise ValueError("--defaults must be passed in via defaults_filepath keyword argument, not as part of customisation list")  # noqa
+
     pexpect_logfile_prefix = stdout_prefix
     if pexpect_logfile_prefix is None:
         pexpect_logfile_prefix = os.path.basename(binary)