Commit 3ed2915c authored by goni.kim's avatar goni.kim
Browse files

Fix bugs regarding ShortCodeTest that fails without "112" in ECC List.



1. Google takes "112" as an Emergency Number in native code and it is passed only when CATEGORY_NOT_SHORT_CODE is returned.
On the other hand, Operator likes VZW doesn't take "112" as an Emergency Number.

2. The mobile phones which don't have "112" in ECC List(eg Non-sim model) don't take "112" as an Emergency Number.
So, those test cases are returned to CATEGORY_POSSIBLE_PREMIUM_SHORT_CODE matched by the pattern defined in sms_short_codes.xml.
In conclusion, the CTS Test result is fail.

3. CDMA only devices is using 3G CDMA network and CDMA is not 3GPP but 3GPP2 spec. 3GPP is only for GSM/WCDMA/LTE network.
3GPP (GSM/WCDMA/LTE) required to use 112 as ECC list, but not in 3GPP2.

4. That's why I added exception handling codes in order to return CATEGORY_NOT_SHORT_CODE when PhoneType is "CDMA" and destAddress is "112"

Conflicts:
	tests/tests/telephony/src/android/telephony/cts/SmsUsageMonitorShortCodeTest.java

Change-Id: I27d31533dbb2afc846fa9ba163cd0444a8cbba6d
Signed-off-by: default avatarJaehyun Kim <goni.kim@lge.com>
parent aefedc95
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment