follow up minor fixes for connectivity manager test

Change-Id: I8f0606890b21d67130efb3bc02c6825823060139
This commit is contained in:
Guang Zhu
2014-08-20 17:40:18 -07:00
parent d21f1c1e4d
commit 4afd179a9a
2 changed files with 3 additions and 3 deletions

View File

@@ -520,7 +520,7 @@ public class ConnectivityManagerTestBase extends InstrumentationTestCase {
* @param length required length of the string
* @return
*/
protected boolean isHex(String input, int length) {
protected static boolean isHex(String input, int length) {
Pattern p = Pattern.compile(String.format("[0-9A-Fa-f]{%d}", length));
return p.matcher(input).matches();
}

View File

@@ -136,7 +136,7 @@ public class WifiAssociationTest extends ConnectivityManagerTestBase {
config.allowedProtocols.set(Protocol.WPA);
config.allowedPairwiseCiphers.set(PairwiseCipher.TKIP);
config.allowedGroupCiphers.set(GroupCipher.TKIP);
if (mPassword.matches("[0-9A-Fa-f]{64}")) {
if (isHex(mPassword, 64)) {
config.preSharedKey = mPassword;
} else {
config.preSharedKey = '"' + mPassword + '"';
@@ -150,7 +150,7 @@ public class WifiAssociationTest extends ConnectivityManagerTestBase {
config.allowedPairwiseCiphers.set(PairwiseCipher.CCMP);
config.allowedGroupCiphers.set(GroupCipher.CCMP);
config.allowedProtocols.set(Protocol.RSN);
if (mPassword.matches("[0-9A-Fa-f]{64}")) {
if (isHex(mPassword, 64)) {
config.preSharedKey = mPassword;
} else {
config.preSharedKey = '"' + mPassword + '"';