Fix proguard flags
Removed the *ForTest exception. It caused issues with guava. Now that
we have NeededForTesting, we don't really need it.
(But didn't change method names from XxxForTest to Xxx, because I still
think using this prefix for test-only methods is a good idea, as it'll
make it easier to catch the use of methods in the main apk that are
not supposed to be used.)
Change-Id: Idccfd7175372b1a2253b19161ae572dae7e1e952
diff --git a/proguard.flags b/proguard.flags
index 3117066..308513b 100644
--- a/proguard.flags
+++ b/proguard.flags
@@ -7,13 +7,10 @@
int getCount();
}
-# Any methods whose name is '*ForTest' are preserved.
--keep class ** {
- *** *ForTest(...);
-}
-
# Any class or method annotated with NeededForTesting.
-keep @com.android.providers.contacts.util.NeededForTesting class *
-keepclassmembers class * {
@com.android.providers.contacts.util.NeededForTesting *;
}
+
+-verbose
\ No newline at end of file
diff --git a/src/com/android/providers/contacts/ContactsDatabaseHelper.java b/src/com/android/providers/contacts/ContactsDatabaseHelper.java
index bfa317d..fa197bd 100644
--- a/src/com/android/providers/contacts/ContactsDatabaseHelper.java
+++ b/src/com/android/providers/contacts/ContactsDatabaseHelper.java
@@ -17,6 +17,7 @@
package com.android.providers.contacts;
import com.android.common.content.SyncStateContentProviderHelper;
+import com.android.providers.contacts.util.NeededForTesting;
import com.google.android.collect.Sets;
import android.content.ContentResolver;
@@ -5207,6 +5208,7 @@
return mCountryMonitor.getCountryIso();
}
+ @NeededForTesting
/* package */ String querySearchIndexContentForTest(long contactId) {
return DatabaseUtils.stringForQuery(getReadableDatabase(),
"SELECT " + SearchIndexColumns.CONTENT +
@@ -5215,6 +5217,7 @@
new String[] { String.valueOf(contactId) });
}
+ @NeededForTesting
/* package */ String querySearchIndexTokensForTest(long contactId) {
return DatabaseUtils.stringForQuery(getReadableDatabase(),
"SELECT " + SearchIndexColumns.TOKENS +
diff --git a/src/com/android/providers/contacts/ContactsProvider2.java b/src/com/android/providers/contacts/ContactsProvider2.java
index 4dbfbcb..23d7d2d 100644
--- a/src/com/android/providers/contacts/ContactsProvider2.java
+++ b/src/com/android/providers/contacts/ContactsProvider2.java
@@ -8004,6 +8004,7 @@
/**
* @return the currently active {@link ContactsDatabaseHelper} for the current thread.
*/
+ @NeededForTesting
public ContactsDatabaseHelper getThreadActiveDatabaseHelperForTest() {
return mDbHelper.get();
}
diff --git a/tests/src/com/android/providers/contacts/ContactsMockResources.java b/tests/src/com/android/providers/contacts/ContactsMockResources.java
index d1ec817..248d6da 100644
--- a/tests/src/com/android/providers/contacts/ContactsMockResources.java
+++ b/tests/src/com/android/providers/contacts/ContactsMockResources.java
@@ -16,9 +16,8 @@
package com.android.providers.contacts;
-import com.google.common.collect.Maps;
+import com.google.android.collect.Maps;
-import android.content.res.Resources.NotFoundException;
import android.test.mock.MockResources;
import java.util.Map;