wakeonlan: coverage near 100% and check_mode++ (#26683)

This makes check_mode run more code (without sending out a packet.

And should get coverage to 98%.
pull/26703/head
Dag Wieers 7 years ago committed by John R Barker
parent 112cae73df
commit d4c0124d04

@ -102,18 +102,20 @@ def wakeonlan(module, mac, broadcast, port):
for i in range(0, len(padding), 2): for i in range(0, len(padding), 2):
data = b''.join([data, struct.pack('B', int(padding[i: i + 2], 16))]) data = b''.join([data, struct.pack('B', int(padding[i: i + 2], 16))])
# Broadcast payload to network
sock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
sock.setsockopt(socket.SOL_SOCKET, socket.SO_BROADCAST, 1)
if not module.check_mode: if not module.check_mode:
# Broadcast payload to network
sock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
sock.setsockopt(socket.SOL_SOCKET, socket.SO_BROADCAST, 1)
try: try:
sock.sendto(data, (broadcast, port)) sock.sendto(data, (broadcast, port))
except socket.error: except socket.error:
e = get_exception() e = get_exception()
sock.close() sock.close()
module.fail_json(msg=str(e)) module.fail_json(msg=str(e))
sock.close()
sock.close()
def main(): def main():

Loading…
Cancel
Save