utils: T9177: fix ruff findings in touched files

Same class of pre-existing whole-file ruff findings as vyos-1x#5374's
477dd4d6b: bare except:, one unused except-clause variable, one
multi-statement if, and one unused loop variable in
python/vyos/utils/network.py.

Also renames a duplicate test_is_ipv6_link_local() in
test_utils_network.py to test_is_loopback_addr() (its actual
subject) - the duplicate name silently shadowed the first test
method, so is_ipv6_link_local() was never actually exercised.

No behavior changes to application code.

1 files changed, 8 insertions, 8 deletions

diff --git a/python/vyos/utils/network.py b/python/vyos/utils/network.py
index ef8205b97..9eecf97f4 100644
--- a/python/vyos/utils/network.py
+++ b/python/vyos/utils/network.py
@@ -131,7 +131,7 @@ def get_vrf_members(vrf: str) -> list:
                 # Skip PIM interfaces which appears in VRF
                 if 'pim' not in data.get('ifname'):
                     interfaces.append(data.get('ifname'))
-    except:
+    except Exception:
         pass
     return interfaces
 
@@ -219,7 +219,8 @@ def get_interface_namespace(interface: str):
     """
     # Bail out early if netns does not exist
     tmp = cmdl(['ip', '--json', 'netns', 'ls'])
-    if not tmp: return None
+    if not tmp:
+        return None
 
     for ns in loads(tmp):
         netns = f'{ns["name"]}'
@@ -339,7 +340,7 @@ def mac2eui64(mac, prefix=None):
             net = ip_network(prefix, strict=False)
             euil = int('0x{0}'.format(eui64), 16)
             return str(net[euil])
-        except:  # pylint: disable=bare-except
+        except Exception:
             return
 
 
@@ -381,7 +382,7 @@ def check_port_availability(address: str = None, port: int = 0,
     protocol = socket.SOCK_STREAM if protocol == 'tcp' else socket.SOCK_DGRAM
     try:
         addr_info = socket.getaddrinfo(address, port, socket.AF_UNSPEC, protocol)
-    except socket.gaierror as e:
+    except socket.gaierror:
         print(f'Invalid address: {address}')
         return False
 
@@ -500,7 +501,6 @@ def is_intf_addr_assigned(ifname: str, addr: str, netns: str=None) -> bool:
         json_out = loads(out)
         addresses = jmespath.search("[].addr_info[].{family: family, address: local, prefixlen: prefixlen}", json_out)
         for address_info in addresses:
-            family = address_info['family']
             address = address_info['address']
             prefixlen = address_info['prefixlen']
             # Remove the interface name if present in the given address
@@ -688,7 +688,7 @@ def ipv6_prefix_length(low, high):
     try:
         lo = bytearray(socket.inet_pton(socket.AF_INET6, low))
         hi = bytearray(socket.inet_pton(socket.AF_INET6, high))
-    except:
+    except Exception:
         return None
 
     xor = bytearray(a ^ b for a, b in zip(lo, hi))
@@ -752,7 +752,7 @@ def is_valid_ipv4_address_or_range(addr: str) -> bool:
             return is_valid_ipv4_address_or_range(split[0]) and is_valid_ipv4_address_or_range(split[1])
         else:
             return ip_network(addr).version == 4
-    except:
+    except Exception:
         return False
 
 def is_valid_ipv6_address_or_range(addr: str) -> bool:
@@ -768,7 +768,7 @@ def is_valid_ipv6_address_or_range(addr: str) -> bool:
             return is_valid_ipv6_address_or_range(split[0]) and is_valid_ipv6_address_or_range(split[1])
         else:
             return ip_network(addr).version == 6
-    except:
+    except Exception:
         return False