From 447fb0b0da659b99ba713cdb53df9344a102cf32 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Thu, 7 Jan 2016 00:27:50 +0200 Subject: [PATCH] tests: Make eap_check_auth() error on missing selectedMethod clearer It was possible to hit an error case in ap_wpa2_eap_in_bridge where the selectedMethod STATUS field was not available. This resulted in not very helpful "'selectedMethod'" message in the test log file. Make this clearer by dumping all received STATUS fields and a clearer exception message indicating that selectedMethod was missing. Signed-off-by: Jouni Malinen --- tests/hwsim/test_ap_eap.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/hwsim/test_ap_eap.py b/tests/hwsim/test_ap_eap.py index 9efa73c4c..6a4eec66a 100644 --- a/tests/hwsim/test_ap_eap.py +++ b/tests/hwsim/test_ap_eap.py @@ -176,6 +176,9 @@ def eap_check_auth(dev, method, initial, rsn=True, sha256=False, if status["suppPortStatus"] != "Authorized": raise Exception("Port not authorized") + if "selectedMethod" not in status: + logger.info("Status: " + str(status)) + raise Exception("No selectedMethod in status") if method not in status["selectedMethod"]: raise Exception("Incorrect EAP method status") if sha256: