autotest: correct duplicate-test-name problems

Tools/autotest/common.py

@@ -13401,7 +13401,11 @@ SERIAL5_BAUD 128
             disabled = {}
         skip_list = []
         tests = []
+        seen_test_name = set()
         for test in all_tests:
+            if test.name in seen_test_name:
+                raise ValueError("Duplicate test name %s" % test.name)
+            seen_test_name.add(test.name)
             if test.name in disabled:
                 self.progress("##### %s is skipped: %s" % (test, disabled[test.name]))
                 skip_list.append((test, disabled[test.name]))

Tools/autotest/rover.py

@@ -1195,7 +1195,7 @@ Brakes have negligible effect (with=%0.2fm without=%0.2fm delta=%0.2fm)
         self.do_set_mode_via_command_long("HOLD")
         self.do_set_mode_via_command_long("MANUAL")
 
-    def InitialMode(self):
+    def RoverInitialMode(self):
         '''test INITIAL_MODE parameter works'''
         # from mavproxy_rc.py
         self.wait_ready_to_arm()
@@ -6314,7 +6314,6 @@ Brakes have negligible effect (with=%0.2fm without=%0.2fm delta=%0.2fm)
             self.DriveRTL,
             self.SmartRTL,
             self.DriveSquare,
-            self.DriveMaxRCIN,
             self.DriveMission,
             # self.DriveBrake,  # disabled due to frequent failures
             self.GetBanner,
@@ -6372,7 +6371,7 @@ Brakes have negligible effect (with=%0.2fm without=%0.2fm delta=%0.2fm)
             self.AutoDock,
             self.PrivateChannel,
             self.GCSFailsafe,
-            self.InitialMode,
+            self.RoverInitialMode,
             self.DriveMaxRCIN,
             self.NoArmWithoutMissionItems,
             self.CompassPrearms,