Use variadic positional parameters for plural install_* methods

It saves an indent and brackets on the call sites.

Also sort the handlers alphabetically where their order doesn't matter
and split the fallback handlers into a separate call to signify that
their position in the end matters.

(cherry picked from commit 7e587201a4)

bin/tests/system/chain/ans3/ans.py

@@ -114,7 +114,7 @@ class Cve202125215(DomainHandler):
 
 def main() -> None:
     server = AsyncDnsServer(acknowledge_manual_dname_handling=True, default_aa=True)
-    server.install_response_handlers([CnameThenDnameHandler(), Cve202125215()])
+    server.install_response_handlers(CnameThenDnameHandler(), Cve202125215())
     server.run()
 
 

bin/tests/system/cookie/cookie_ans.py

@@ -194,13 +194,11 @@ def cookie_server(evil: bool) -> AsyncDnsServer:
         keyring=KEYRING, default_aa=True, default_rcode=dns.rcode.NOERROR
     )
     server.install_response_handlers(
-        [
-            NsHandler(evil),
-            GlueHandler(evil),
-            TcpAHandler(),
-            WithtsigUdpAHandler(),
-            UdpAHandler(),
-            FallbackHandler(),
-        ]
+        NsHandler(evil),
+        GlueHandler(evil),
+        TcpAHandler(),
+        WithtsigUdpAHandler(),
+        UdpAHandler(),
     )
+    server.install_response_handler(FallbackHandler())
     return server

bin/tests/system/digdelv/ans7/ans.py

@@ -63,11 +63,9 @@ class SilentThenServfailHandler(DomainHandler):
 def main() -> None:
     server = AsyncDnsServer()
     server.install_response_handlers(
-        [
-            CloseHandler(),
-            SilentHandler(),
-            SilentThenServfailHandler(),
-        ]
+        CloseHandler(),
+        SilentHandler(),
+        SilentThenServfailHandler(),
     )
     server.run()
 

bin/tests/system/dnssec/ans10/ans.py

@@ -57,7 +57,7 @@ class AddNsecToTxtHandler(ResponseHandler):
 
 def main() -> None:
     server = AsyncDnsServer()
-    server.install_response_handlers([AddRrsigToAHandler(), AddNsecToTxtHandler()])
+    server.install_response_handlers(AddNsecToTxtHandler(), AddRrsigToAHandler())
     server.run()
 
 

bin/tests/system/isctest/asyncserver.py

@@ -887,7 +887,7 @@ class AsyncDnsServer(AsyncServer):
         else:
             self._response_handlers.append(handler)
 
-    def install_response_handlers(self, handlers: List[ResponseHandler]) -> None:
+    def install_response_handlers(self, *handlers: ResponseHandler) -> None:
         for handler in handlers:
             self.install_response_handler(handler)
 
@@ -1380,7 +1380,7 @@ class ControllableAsyncDnsServer(AsyncDnsServer):
     def _commands(self) -> Dict[dns.name.Name, "ControlCommand"]:
         return {}
 
-    def install_control_commands(self, commands: List["ControlCommand"]) -> None:
+    def install_control_commands(self, *commands: "ControlCommand") -> None:
         for command in commands:
             self.install_control_command(command)
 

bin/tests/system/qmin/ans2/ans.py

@@ -100,13 +100,11 @@ class StaleHandler(DomainHandler):
 def main() -> None:
     server = AsyncDnsServer()
     server.install_response_handlers(
-        [
-            QueryLogger(),
-            BadHandler(),
-            UglyHandler(),
-            SlowHandler(),
-            StaleHandler(),
-        ]
+        QueryLogger(),
+        BadHandler(),
+        UglyHandler(),
+        SlowHandler(),
+        StaleHandler(),
     )
     server.run()
 

bin/tests/system/qmin/ans3/ans.py

@@ -40,12 +40,10 @@ class ZoopBoingSlowHandler(DelayedResponseHandler):
 def main() -> None:
     server = AsyncDnsServer()
     server.install_response_handlers(
-        [
-            QueryLogger(),
-            ZoopBoingBadHandler(),
-            ZoopBoingUglyHandler(),
-            ZoopBoingSlowHandler(),
-        ]
+        QueryLogger(),
+        ZoopBoingBadHandler(),
+        ZoopBoingUglyHandler(),
+        ZoopBoingSlowHandler(),
     )
     server.run()
 

bin/tests/system/qmin/ans4/ans.py

@@ -87,13 +87,11 @@ class IckyPtangZoopBoingSlowHandler(DelayedResponseHandler):
 def main() -> None:
     server = AsyncDnsServer()
     server.install_response_handlers(
-        [
-            QueryLogger(),
-            StaleHandler(),
-            IckyPtangZoopBoingBadHandler(),
-            IckyPtangZoopBoingUglyHandler(),
-            IckyPtangZoopBoingSlowHandler(),
-        ]
+        QueryLogger(),
+        StaleHandler(),
+        IckyPtangZoopBoingBadHandler(),
+        IckyPtangZoopBoingUglyHandler(),
+        IckyPtangZoopBoingSlowHandler(),
     )
     server.run()
 

bin/tests/system/rpzrecurse/ans5/ans.py

@@ -52,7 +52,7 @@ class IgnoreNs(ResponseHandler):
 
 def main() -> None:
     server = AsyncDnsServer(default_aa=True, default_rcode=dns.rcode.NOERROR)
-    server.install_response_handlers([ReplyA(), IgnoreNs()])
+    server.install_response_handlers(ReplyA(), IgnoreNs())
     server.run()
 
 

bin/tests/system/statistics/ans4/ans.py

@@ -162,18 +162,16 @@ class FallbackHandler(ResponseHandler):
 def main() -> None:
     server = AsyncDnsServer(default_rcode=dns.rcode.NOERROR)
     server.install_response_handlers(
-        [
-            BadGoodCnameHandler(),
-            Cname1Handler(),
-            Cname2Handler(),
-            ExampleHandler(),
-            FooInfoHandler(),
-            NoDataHandler(),
-            NxdomainHandler(),
-            SubHandler(),
-            FallbackHandler(),
-        ]
+        BadGoodCnameHandler(),
+        Cname1Handler(),
+        Cname2Handler(),
+        ExampleHandler(),
+        FooInfoHandler(),
+        NoDataHandler(),
+        NxdomainHandler(),
+        SubHandler(),
     )
+    server.install_response_handler(FallbackHandler())
     server.run()