commit 71e35742798ca292d2fb727dd3a73e11b0f9b09d
author Paul Stewart <pstew@google.com> Thu Feb 18 13:27:34 2016 -0800
committer Paul Stewart <pstew@google.com> Thu Feb 25 09:47:23 2016 -0800
tree d1fc3e1246a2bba3bf19a1e59143e956c8b810c8
parent a2580260f0c3308c6394aa85c9837e3f249c7027

WifiStateMachine: Only respond once to multiple EAP-SIM challenges

The commit in 5ebc35f19feab2755a13cf7d5acaeff4bbc78994 made an
attempt to fail the EAPOL process due to intermediate failures
with TelephonyManager.  It's not clear how well that worked,
but it's clear from inspection that when multiple challenges
are supplied that the code then sent multiple simAuthResponse
messages -- one per challenge.  This is clearly wrong.

This CL refactors the code to clearly separate the function
that generates the complete response string so it can be tested,
and leaves the handleGsmAuthRequest() method to interact with
mWifiNative in a hopefully more readable manner.

BUG: 26758602
Change-Id: Ibf8f85893e4c18195eb08114b5169dcff05dc045
TEST: New unit test for the getGsmSimAuthResponse method