modify testGetServByName so it tries a few different protocols. In this day

Skip Montanaro · Skip Montanaro · commit d3c884d4ea6f · 2002-08-02T02:19:46.000Z
and age of rampant computer breakins I imagine there are plenty of systems
with telnet disabled.  Successful check of at least one getservbyname() call
is required for success
diff --git a/Lib/test/test_socket.py b/Lib/test/test_socket.py
@@ -260,11 +260,23 @@ def testNtoH(self):
     def testGetServByName(self):
         """Testing getservbyname()."""
         if hasattr(socket, 'getservbyname'):
-            socket.getservbyname('telnet', 'tcp')
-            try:
-                socket.getservbyname('telnet', 'udp')
-            except socket.error:
-                pass
+            # try a few protocols - not everyone has telnet enabled
+            found = 0
+            for proto in ("telnet", "ssh", "www", "ftp"):
+                try:
+                    socket.getservbyname(proto, 'tcp')
+                    found = 1
+                    break
+                except socket.error:
+                    pass
+                try:
+                    socket.getservbyname(proto, 'udp')
+                    found = 1
+                    break
+                except socket.error:
+                    pass
+                if not found:
+                    raise socket.error
 
     def testDefaultTimeout(self):
         """Testing default timeout."""
