From 6467d015fa96d914be760ebd39fe9c5ee68d3e54 Mon Sep 17 00:00:00 2001 From: yan Garthwaite Date: Wed, 3 Jul 2024 06:53:11 +0000 Subject: [PATCH] Replaced try-catch with if-else + key in list check Addresses PR comments. --- ansible/roles/test/files/ptftests/py3/advanced-reboot.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ansible/roles/test/files/ptftests/py3/advanced-reboot.py b/ansible/roles/test/files/ptftests/py3/advanced-reboot.py index 9fe5432f59..af7bf45be0 100644 --- a/ansible/roles/test/files/ptftests/py3/advanced-reboot.py +++ b/ansible/roles/test/files/ptftests/py3/advanced-reboot.py @@ -2005,10 +2005,10 @@ def examine_flow(self, filename=None): prev_pkt_pt = prev_payload + 1 prev_sent_packet_time = None while prev_pkt_pt < received_payload: - try: + if prev_pkt_pt in sent_packets: prev_sent_packet_time = sent_packets[prev_pkt_pt] break # Found it - except KeyError: + else: if prev_pkt_pt not in received_but_not_sent_packets: missing_sent_and_received_pkt_count += 1 prev_pkt_pt += 1