[Payments] Use <meta-data> tag instead of intent filter data to detect supported payment methods.

chrome/android/junit/src/org/chromium/chrome/browser/payments/AndroidPaymentAppFinderTest.java

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.payments;
 
 import android.content.Intent;
 import android.content.pm.ActivityInfo;
 import android.content.pm.PackageInfo;
 import android.content.pm.ResolveInfo;
 import android.content.pm.ServiceInfo;
 import android.content.pm.Signature;
-import android.net.Uri;
 
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.ArgumentMatcher;
 import org.mockito.ArgumentMatchers;
 import org.mockito.Mockito;
 import org.robolectric.RobolectricTestRunner;
 import org.robolectric.annotation.Config;
 
 import org.chromium.chrome.browser.payments.PaymentAppFactory.PaymentAppCreatedCallback;
 import org.chromium.components.payments.PaymentManifestDownloader;
 import org.chromium.components.payments.PaymentManifestParser;
 import org.chromium.content_public.browser.WebContents;
 import org.chromium.payments.mojom.PaymentManifestSection;
 
 import java.net.URI;
 import java.util.ArrayList;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 
 /** Tests for the native Android payment app finder. */
 @RunWith(RobolectricTestRunner.class)
 @Config(sdk = 21, manifest = Config.NONE)
 public class AndroidPaymentAppFinderTest {
+    private static final IntentArgumentMatcher sPayIntentArgumentMatcher =
+            new IntentArgumentMatcher(new Intent("org.chromium.intent.action.PAY"));
+
     public AndroidPaymentAppFinderTest() {}
 
     /**
      * Argument matcher that matches Intents using |filterEquals| method.
      */
     private static class IntentArgumentMatcher implements ArgumentMatcher<Intent> {
 
         private final Intent mIntent;
 
         public IntentArgumentMatcher(Intent intent) {
@@ skipping 23 matching lines @@
                 Mockito.mock(PaymentManifestDownloader.class),
                 Mockito.mock(PaymentManifestParser.class),
                 Mockito.mock(PackageManagerDelegate.class), callback);
 
         Mockito.verify(callback, Mockito.never())
                 .onPaymentAppCreated(Mockito.any(PaymentApp.class));
         Mockito.verify(callback).onAllPaymentAppsCreated();
     }
 
     @Test
-    public void testQueryBasicCardsWithoutApps() {
+    public void testQueryWithoutApps() {
         PackageManagerDelegate packageManagerDelegate = Mockito.mock(PackageManagerDelegate.class);
         Mockito.when(packageManagerDelegate.getActivitiesThatCanRespondToIntent(
-                ArgumentMatchers.argThat(new IntentArgumentMatcher(
-                        new Intent(AndroidPaymentAppFinder.ACTION_PAY_BASIC_CARD)))))
+                             ArgumentMatchers.argThat(sPayIntentArgumentMatcher)))
                 .thenReturn(new ArrayList<ResolveInfo>());
         Set<String> methodNames = new HashSet<>();
         methodNames.add(AndroidPaymentAppFinder.BASIC_CARD_PAYMENT_METHOD);
         PaymentAppCreatedCallback callback = Mockito.mock(PaymentAppCreatedCallback.class);
 
         AndroidPaymentAppFinder.find(Mockito.mock(WebContents.class), methodNames, false,
                 Mockito.mock(PaymentManifestDownloader.class),
                 Mockito.mock(PaymentManifestParser.class), packageManagerDelegate, callback);
 
+        Mockito.verify(packageManagerDelegate, Mockito.never())
+                .getStringArrayMetaData(ArgumentMatchers.anyString(), ArgumentMatchers.anyString());
+        Mockito.verify(packageManagerDelegate, Mockito.never())
+                .getStringMetaData(ArgumentMatchers.anyString(), ArgumentMatchers.anyString());
         Mockito.verify(callback, Mockito.never())
                 .onPaymentAppCreated(Mockito.any(PaymentApp.class));
         Mockito.verify(callback).onAllPaymentAppsCreated();
     }
 
     @Test
-    public void testQueryBasicCardsWithTwoApps() {
+    public void testQueryWithoutMetaData() {
         List<ResolveInfo> activities = new ArrayList<>();
         ResolveInfo alicePay = new ResolveInfo();
         alicePay.activityInfo = new ActivityInfo();
         alicePay.activityInfo.packageName = "com.alicepay.app";
         alicePay.activityInfo.name = "com.alicepay.app.WebPaymentActivity";
         activities.add(alicePay);
-        ResolveInfo bobPay = new ResolveInfo();
-        bobPay.activityInfo = new ActivityInfo();
-        bobPay.activityInfo.packageName = "com.bobpay.app";
-        bobPay.activityInfo.name = "com.bobpay.app.WebPaymentActivity";
-        activities.add(bobPay);
+
         PackageManagerDelegate packageManagerDelegate = Mockito.mock(PackageManagerDelegate.class);
-        Mockito.when(packageManagerDelegate.getAppLabel(Mockito.any(ResolveInfo.class)))
-                .thenReturn("A non-empty label");
         Mockito.when(packageManagerDelegate.getActivitiesThatCanRespondToIntent(
-                ArgumentMatchers.argThat(new IntentArgumentMatcher(
-                        new Intent(AndroidPaymentAppFinder.ACTION_PAY_BASIC_CARD)))))
+                             ArgumentMatchers.argThat(sPayIntentArgumentMatcher)))
                 .thenReturn(activities);
-        Mockito.when(packageManagerDelegate.getServicesThatCanRespondToIntent(
-                ArgumentMatchers.argThat(new IntentArgumentMatcher(
-                        new Intent(AndroidPaymentAppFinder.ACTION_IS_READY_TO_PAY)))))
-                .thenReturn(new ArrayList<ResolveInfo>());
+        Mockito.when(packageManagerDelegate.getStringArrayMetaData(
+                        ArgumentMatchers.matches("com.alicepay.app"),
+                        ArgumentMatchers.matches(
+                                AndroidPaymentAppFinder.META_DATA_NAME_OF_PAYMENT_METHOD_NAMES)))
+                .thenReturn(null);
+        Mockito.when(packageManagerDelegate.getStringMetaData(
+                             ArgumentMatchers.matches("com.alicepay.app"),
+                             ArgumentMatchers.matches(
+                                     AndroidPaymentAppFinder
+                                             .META_DATA_NAME_OF_DEFAULT_PAYMENT_METHOD_NAME)))
+                .thenReturn(null);
+
         Set<String> methodNames = new HashSet<>();
         methodNames.add(AndroidPaymentAppFinder.BASIC_CARD_PAYMENT_METHOD);
         PaymentAppCreatedCallback callback = Mockito.mock(PaymentAppCreatedCallback.class);
 
         AndroidPaymentAppFinder.find(Mockito.mock(WebContents.class), methodNames, false,
                 Mockito.mock(PaymentManifestDownloader.class),
                 Mockito.mock(PaymentManifestParser.class), packageManagerDelegate, callback);
 
-        Mockito.verify(callback).onPaymentAppCreated(
-                ArgumentMatchers.argThat(Matches.paymentAppIdentifier("com.alicepay.app")));
-        Mockito.verify(callback).onPaymentAppCreated(
-                ArgumentMatchers.argThat(Matches.paymentAppIdentifier("com.bobpay.app")));
+        Mockito.verify(callback, Mockito.never())
+                .onPaymentAppCreated(Mockito.any(PaymentApp.class));
         Mockito.verify(callback).onAllPaymentAppsCreated();
     }
 
     @Test
-    public void testQueryBobPayWithoutApps() {
-        Intent bobPayIntent = new Intent("org.chromium.intent.action.PAY");
-        bobPayIntent.setData(Uri.parse("https://bobpay.com"));
+    public void testQueryWithUnsupportedPaymentMethod() {
+        List<ResolveInfo> activities = new ArrayList<>();
+        ResolveInfo alicePay = new ResolveInfo();
+        alicePay.activityInfo = new ActivityInfo();
+        alicePay.activityInfo.packageName = "com.alicepay.app";
+        alicePay.activityInfo.name = "com.alicepay.app.WebPaymentActivity";
+        activities.add(alicePay);
+
         PackageManagerDelegate packageManagerDelegate = Mockito.mock(PackageManagerDelegate.class);
-        Mockito.when(packageManagerDelegate.getActivitiesThatCanRespondToIntent(bobPayIntent))
-                .thenReturn(new ArrayList<ResolveInfo>());
+        Mockito.when(packageManagerDelegate.getActivitiesThatCanRespondToIntent(
+                             ArgumentMatchers.argThat(sPayIntentArgumentMatcher)))
+                .thenReturn(activities);
+        Mockito.when(packageManagerDelegate.getStringArrayMetaData(
+                        ArgumentMatchers.matches("com.alicepay.app"),
+                        ArgumentMatchers.matches(
+                                AndroidPaymentAppFinder.META_DATA_NAME_OF_PAYMENT_METHOD_NAMES)))
+                .thenReturn(null);
+        Mockito.when(packageManagerDelegate.getStringMetaData(
+                             ArgumentMatchers.matches("com.alicepay.app"),
+                             ArgumentMatchers.matches(
+                                     AndroidPaymentAppFinder
+                                             .META_DATA_NAME_OF_DEFAULT_PAYMENT_METHOD_NAME)))
+                .thenReturn("https://bobpay.com");
+
         Set<String> methodNames = new HashSet<>();
-        methodNames.add("https://bobpay.com");
+        methodNames.add(AndroidPaymentAppFinder.BASIC_CARD_PAYMENT_METHOD);
         PaymentAppCreatedCallback callback = Mockito.mock(PaymentAppCreatedCallback.class);
 
         AndroidPaymentAppFinder.find(Mockito.mock(WebContents.class), methodNames, false,
                 Mockito.mock(PaymentManifestDownloader.class),
                 Mockito.mock(PaymentManifestParser.class), packageManagerDelegate, callback);
 
         Mockito.verify(callback, Mockito.never())
                 .onPaymentAppCreated(Mockito.any(PaymentApp.class));
         Mockito.verify(callback).onAllPaymentAppsCreated();
     }
 
     @Test
+    public void testQueryBasicCardsWithTwoApps() {
+        List<ResolveInfo> activities = new ArrayList<>();
+        ResolveInfo alicePay = new ResolveInfo();
+        alicePay.activityInfo = new ActivityInfo();
+        alicePay.activityInfo.packageName = "com.alicepay.app";
+        alicePay.activityInfo.name = "com.alicepay.app.WebPaymentActivity";
+        activities.add(alicePay);
+        ResolveInfo bobPay = new ResolveInfo();
+        bobPay.activityInfo = new ActivityInfo();
+        bobPay.activityInfo.packageName = "com.bobpay.app";
+        bobPay.activityInfo.name = "com.bobpay.app.WebPaymentActivity";
+        activities.add(bobPay);
+
+        PackageManagerDelegate packageManagerDelegate = Mockito.mock(PackageManagerDelegate.class);
+        Mockito.when(packageManagerDelegate.getAppLabel(Mockito.any(ResolveInfo.class)))
+                .thenReturn("A non-empty label");
+        Mockito.when(packageManagerDelegate.getActivitiesThatCanRespondToIntent(
+                             ArgumentMatchers.argThat(sPayIntentArgumentMatcher)))
+                .thenReturn(activities);
+        Mockito.when(packageManagerDelegate.getServicesThatCanRespondToIntent(
+                             ArgumentMatchers.argThat(new IntentArgumentMatcher(
+                                     new Intent(AndroidPaymentAppFinder.ACTION_IS_READY_TO_PAY)))))
+                .thenReturn(new ArrayList<ResolveInfo>());
+        Mockito.when(packageManagerDelegate.getStringArrayMetaData(
+                        ArgumentMatchers.matches("com.alicepay.app"),
+                        ArgumentMatchers.matches(
+                                AndroidPaymentAppFinder.META_DATA_NAME_OF_PAYMENT_METHOD_NAMES)))
+                .thenReturn(new String[] {
+                        "https://alicepay.com", AndroidPaymentAppFinder.BASIC_CARD_PAYMENT_METHOD});
+        Mockito.when(packageManagerDelegate.getStringMetaData(
+                             ArgumentMatchers.matches("com.alicepay.app"),
+                             ArgumentMatchers.matches(
+                                     AndroidPaymentAppFinder
+                                             .META_DATA_NAME_OF_DEFAULT_PAYMENT_METHOD_NAME)))
+                .thenReturn("https://alicepay.com");
+        Mockito.when(packageManagerDelegate.getStringArrayMetaData(
+                        ArgumentMatchers.matches("com.bobpay.app"),
+                        ArgumentMatchers.matches(
+                                AndroidPaymentAppFinder.META_DATA_NAME_OF_PAYMENT_METHOD_NAMES)))
+                .thenReturn(new String[] {
+                        "https://bobpay.com", AndroidPaymentAppFinder.BASIC_CARD_PAYMENT_METHOD});
+        Mockito.when(packageManagerDelegate.getStringMetaData(
+                             ArgumentMatchers.matches("com.bobpay.app"),
+                             ArgumentMatchers.matches(
+                                     AndroidPaymentAppFinder
+                                             .META_DATA_NAME_OF_DEFAULT_PAYMENT_METHOD_NAME)))
+                .thenReturn("https://bobpay.com");
+
+        Set<String> methodNames = new HashSet<>();
+        methodNames.add(AndroidPaymentAppFinder.BASIC_CARD_PAYMENT_METHOD);
+        PaymentAppCreatedCallback callback = Mockito.mock(PaymentAppCreatedCallback.class);
+
+        AndroidPaymentAppFinder.find(Mockito.mock(WebContents.class), methodNames, false,
+                Mockito.mock(PaymentManifestDownloader.class),
+                Mockito.mock(PaymentManifestParser.class), packageManagerDelegate, callback);
+
+        Mockito.verify(callback).onPaymentAppCreated(
+                ArgumentMatchers.argThat(Matches.paymentAppIdentifier("com.alicepay.app")));
+        Mockito.verify(callback).onPaymentAppCreated(
+                ArgumentMatchers.argThat(Matches.paymentAppIdentifier("com.bobpay.app")));
+        Mockito.verify(callback).onAllPaymentAppsCreated();
+    }
+
+    @Test
     public void testQueryBobPayWithOneAppThatHasIsReadyToPayService() {
         List<ResolveInfo> activities = new ArrayList<>();
         ResolveInfo bobPay = new ResolveInfo();
         bobPay.activityInfo = new ActivityInfo();
         bobPay.activityInfo.packageName = "com.bobpay.app";
         bobPay.activityInfo.name = "com.bobpay.app.WebPaymentActivity";
         activities.add(bobPay);
-        Intent bobPayIntent = new Intent("org.chromium.intent.action.PAY");
-        bobPayIntent.setData(Uri.parse("https://bobpay.com"));
+
         PackageManagerDelegate packageManagerDelegate = Mockito.mock(PackageManagerDelegate.class);
         Mockito.when(packageManagerDelegate.getAppLabel(Mockito.any(ResolveInfo.class)))
                 .thenReturn("A non-empty label");
         Mockito.when(packageManagerDelegate.getActivitiesThatCanRespondToIntent(
-                ArgumentMatchers.argThat(new IntentArgumentMatcher(bobPayIntent))))
+                             ArgumentMatchers.argThat(sPayIntentArgumentMatcher)))
                 .thenReturn(activities);
+        Mockito.when(packageManagerDelegate.getStringArrayMetaData(
+                        ArgumentMatchers.matches("com.bobpay.app"),
+                        ArgumentMatchers.matches(
+                                AndroidPaymentAppFinder.META_DATA_NAME_OF_PAYMENT_METHOD_NAMES)))
+                .thenReturn(new String[] {
+                        "https://bobpay.com", AndroidPaymentAppFinder.BASIC_CARD_PAYMENT_METHOD});
+        Mockito.when(packageManagerDelegate.getStringMetaData(
+                             ArgumentMatchers.matches("com.bobpay.app"),
+                             ArgumentMatchers.matches(
+                                     AndroidPaymentAppFinder
+                                             .META_DATA_NAME_OF_DEFAULT_PAYMENT_METHOD_NAME)))
+                .thenReturn("https://bobpay.com");
 
         List<ResolveInfo> services = new ArrayList<>();
         ResolveInfo isBobPayReadyToPay = new ResolveInfo();
         isBobPayReadyToPay.serviceInfo = new ServiceInfo();
         isBobPayReadyToPay.serviceInfo.packageName = "com.bobpay.app";
         isBobPayReadyToPay.serviceInfo.name = "com.bobpay.app.IsReadyToWebPay";
         services.add(isBobPayReadyToPay);
         Intent isReadyToPayIntent = new Intent(AndroidPaymentAppFinder.ACTION_IS_READY_TO_PAY);
         Mockito.when(packageManagerDelegate.getServicesThatCanRespondToIntent(
-                ArgumentMatchers.argThat(new IntentArgumentMatcher(isReadyToPayIntent))))
+                        ArgumentMatchers.argThat(new IntentArgumentMatcher(isReadyToPayIntent))))
                 .thenReturn(services);
 
         PackageInfo bobPayPackageInfo = new PackageInfo();
         bobPayPackageInfo.versionCode = 10;
         bobPayPackageInfo.signatures = new Signature[1];
         bobPayPackageInfo.signatures[0] = new Signature("01020304050607080900");
         Mockito.when(packageManagerDelegate.getPackageInfoWithSignatures("com.bobpay.app"))
                 .thenReturn(bobPayPackageInfo);
 
         PaymentManifestDownloader downloader = new PaymentManifestDownloader(null) {
@@ skipping 57 matching lines @@
                 String expectedAppIdentifier) {
             return new Matches(expectedAppIdentifier);
         }
 
         @Override
         public boolean matches(PaymentApp app) {
             return app.getAppIdentifier().equals(mExpectedAppIdentifier);
         }
     }
 }