From 52ada0131a70cbd96bfe27e293e6cc6abbd413dd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Mart=C3=AD?= Date: Fri, 13 Nov 2015 18:45:59 +0100 Subject: [PATCH] Deduplicate types in tests --- F-Droid/test/src/mock/MockEmptyPackageManager.java | 2 +- .../src/org/fdroid/fdroid/ApkProviderHelperTest.java | 2 +- .../test/src/org/fdroid/fdroid/ApkProviderTest.java | 12 ++++++------ F-Droid/test/src/org/fdroid/fdroid/TestUtils.java | 6 +++--- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/F-Droid/test/src/mock/MockEmptyPackageManager.java b/F-Droid/test/src/mock/MockEmptyPackageManager.java index 39fdee310..463864179 100644 --- a/F-Droid/test/src/mock/MockEmptyPackageManager.java +++ b/F-Droid/test/src/mock/MockEmptyPackageManager.java @@ -10,7 +10,7 @@ public class MockEmptyPackageManager extends MockPackageManager { @Override public List getInstalledPackages(int flags) { - return new ArrayList(); + return new ArrayList<>(); } } diff --git a/F-Droid/test/src/org/fdroid/fdroid/ApkProviderHelperTest.java b/F-Droid/test/src/org/fdroid/fdroid/ApkProviderHelperTest.java index 6ef702801..f332c492b 100644 --- a/F-Droid/test/src/org/fdroid/fdroid/ApkProviderHelperTest.java +++ b/F-Droid/test/src/org/fdroid/fdroid/ApkProviderHelperTest.java @@ -51,7 +51,7 @@ public class ApkProviderHelperTest extends BaseApkProviderTest { new MockApk("info.example", 2), }; - List apksToCheck = new ArrayList(known.length + unknown.length); + List apksToCheck = new ArrayList<>(known.length + unknown.length); Collections.addAll(apksToCheck, known); Collections.addAll(apksToCheck, unknown); diff --git a/F-Droid/test/src/org/fdroid/fdroid/ApkProviderTest.java b/F-Droid/test/src/org/fdroid/fdroid/ApkProviderTest.java index 39068432a..f7a2bce6e 100644 --- a/F-Droid/test/src/org/fdroid/fdroid/ApkProviderTest.java +++ b/F-Droid/test/src/org/fdroid/fdroid/ApkProviderTest.java @@ -33,7 +33,7 @@ public class ApkProviderTest extends BaseApkProviderTest { assertInvalidUri(ApkProvider.getAuthority()); assertInvalidUri(RepoProvider.getContentUri()); - List apks = new ArrayList(3); + List apks = new ArrayList<>(3); for (int i = 0; i < 10; i++) { apks.add(new MockApk("com.example." + i, i)); } @@ -46,7 +46,7 @@ public class ApkProviderTest extends BaseApkProviderTest { assertValidUri(ApkProvider.getContentUri("org.fdroid.fdroid", 100)); assertValidUri(ApkProvider.getRepoUri(1000)); - List manyApks = new ArrayList(PublicApkProvider.MAX_APKS_TO_QUERY - 5); + List manyApks = new ArrayList<>(PublicApkProvider.MAX_APKS_TO_QUERY - 5); for (int i = 0; i < PublicApkProvider.MAX_APKS_TO_QUERY - 1; i++) { manyApks.add(new MockApk("com.example." + i, i)); } @@ -100,7 +100,7 @@ public class ApkProviderTest extends BaseApkProviderTest { public void testInvalidUpdateUris() { Apk apk = new MockApk("org.fdroid.fdroid", 10); - List apks = new ArrayList(); + List apks = new ArrayList<>(); apks.add(apk); assertCantUpdate(ApkProvider.getContentUri()); @@ -137,14 +137,14 @@ public class ApkProviderTest extends BaseApkProviderTest { }; List all = ApkProvider.Helper.findByRepo(getSwappableContext(), new MockRepo(10), ApkProvider.DataColumns.ALL); - List actualIds = new ArrayList(); + List actualIds = new ArrayList<>(); for (Apk apk : all) { actualIds.add(apk.id); } TestUtils.assertContainsOnly(actualIds, expectedIds); - List toDelete = new ArrayList(3); + List toDelete = new ArrayList<>(3); toDelete.add(two); toDelete.add(three); toDelete.add(four); @@ -153,7 +153,7 @@ public class ApkProviderTest extends BaseApkProviderTest { assertTotalApkCount(2); List allRemaining = ApkProvider.Helper.findByRepo(getSwappableContext(), new MockRepo(10), ApkProvider.DataColumns.ALL); - List actualRemainingIds = new ArrayList(); + List actualRemainingIds = new ArrayList<>(); for (Apk apk : allRemaining) { actualRemainingIds.add(apk.id); } diff --git a/F-Droid/test/src/org/fdroid/fdroid/TestUtils.java b/F-Droid/test/src/org/fdroid/fdroid/TestUtils.java index 0869383de..70ad5a9a5 100644 --- a/F-Droid/test/src/org/fdroid/fdroid/TestUtils.java +++ b/F-Droid/test/src/org/fdroid/fdroid/TestUtils.java @@ -35,19 +35,19 @@ public class TestUtils { private static final String TAG = "TestUtils"; public static void assertContainsOnly(List actualList, T[] expectedArray) { - List expectedList = new ArrayList(expectedArray.length); + List expectedList = new ArrayList<>(expectedArray.length); Collections.addAll(expectedList, expectedArray); assertContainsOnly(actualList, expectedList); } public static void assertContainsOnly(T[] actualArray, List expectedList) { - List actualList = new ArrayList(actualArray.length); + List actualList = new ArrayList<>(actualArray.length); Collections.addAll(actualList, actualArray); assertContainsOnly(actualList, expectedList); } public static void assertContainsOnly(T[] actualArray, T[] expectedArray) { - List expectedList = new ArrayList(expectedArray.length); + List expectedList = new ArrayList<>(expectedArray.length); Collections.addAll(expectedList, expectedArray); assertContainsOnly(actualArray, expectedList); }