PaymentHandler: Implement PaymentInstruments.has().

content/browser/payments/payment_manager_unittest.cc

Index: content/browser/payments/payment_manager_unittest.cc
diff --git a/content/browser/payments/payment_manager_unittest.cc b/content/browser/payments/payment_manager_unittest.cc
index 0180d826ddda2180d8277b02ae1f158c09d06424..d5cc68cdfa2729a5b610a3c1cb3800bef1698c5c 100644
--- a/content/browser/payments/payment_manager_unittest.cc
+++ b/content/browser/payments/payment_manager_unittest.cc
@@ -50,6 +50,11 @@ void SetPaymentInstrumentCallback(PaymentHandlerStatus* out_status,
   *out_status = status;
 }
 
+void HasPaymentInstrumentCallback(PaymentHandlerStatus* out_status,
+                                  PaymentHandlerStatus status) {
+  *out_status = status;
+}
+
 void GetPaymentInstrumentCallback(PaymentInstrumentPtr* out_instrument,
                                   PaymentHandlerStatus* out_status,
                                   PaymentInstrumentPtr instrument,
@@ -87,6 +92,13 @@ class PaymentManagerTest : public PaymentAppContentUnitTestBase {
     base::RunLoop().RunUntilIdle();
   }
 
+  void HasPaymentInstrument(const std::string& instrument_key,
+                            PaymentHandlerStatus* out_status) {
+    manager_->HasPaymentInstrument(
+        instrument_key, base::Bind(&HasPaymentInstrumentCallback, out_status));
+    base::RunLoop().RunUntilIdle();
+  }
+
   void GetPaymentInstrument(const std::string& instrument_key,
                             PaymentInstrumentPtr* out_instrument,
                             PaymentHandlerStatus* out_status) {
@@ -223,4 +235,23 @@ TEST_F(PaymentManagerTest, DeletePaymentInstrument) {
   ASSERT_EQ(PaymentHandlerStatus::NOT_FOUND, read_status);
 }
 
+TEST_F(PaymentManagerTest, HasPaymentInstrument) {
+  PaymentHandlerStatus write_status = PaymentHandlerStatus::NOT_FOUND;
+  PaymentInstrumentPtr write_details = PaymentInstrument::New();
+  write_details->name = "Visa ending ****4756",
+  write_details->enabled_methods.push_back("visa");
+  write_details->stringified_capabilities = "{}";
+  ASSERT_EQ(PaymentHandlerStatus::NOT_FOUND, write_status);

[please use gerrit instead, 2017/04/26 15:18:26]

Here and everywhere: There's no need to ASSERT_EQ(success, status) immediately
after "status = success" statement.

+  SetPaymentInstrument("test_key", std::move(write_details), &write_status);
+  ASSERT_EQ(PaymentHandlerStatus::SUCCESS, write_status);
+
+  PaymentHandlerStatus has_status = PaymentHandlerStatus::NOT_FOUND;
+  ASSERT_EQ(PaymentHandlerStatus::NOT_FOUND, has_status);
+  HasPaymentInstrument("test_key", &has_status);
+  ASSERT_EQ(PaymentHandlerStatus::SUCCESS, has_status);
+
+  HasPaymentInstrument("unstored_test_key", &has_status);
+  ASSERT_EQ(PaymentHandlerStatus::NOT_FOUND, has_status);
+}
+
 }  // namespace content