[Payments] Desktop: implement shipping address/option change

components/payments/content/payment_request_state_unittest.cc

 // 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.
 
 #include "components/payments/content/payment_request_state.h"
 
 #include <utility>
 
 #include "base/memory/weak_ptr.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 34 matching lines @@
 
   // PaymentRequestState::Observer:
   void OnSelectedInformationChanged() override {
     num_on_selected_information_changed_called_++;
   }
 
   // PaymentRequestState::Delegate:
   void OnPaymentResponseAvailable(mojom::PaymentResponsePtr response) override {
     payment_response_ = std::move(response);
   };
+  void OnShippingOptionIdSelected(std::string shipping_option_id) override {}
+  void OnShippingAddressSelected(mojom::PaymentAddressPtr address) override {}
 
   void RecreateStateWithOptionsAndDetails(
       mojom::PaymentOptionsPtr options,
       mojom::PaymentDetailsPtr details,
       std::vector<mojom::PaymentMethodDataPtr> method_data) {
     // The spec will be based on the |options| and |details| passed in.
     spec_ = base::MakeUnique<PaymentRequestSpec>(
         std::move(options), std::move(details), std::move(method_data), nullptr,
         "en-US");
     state_ = base::MakeUnique<PaymentRequestState>(
@@ skipping 129 matching lines @@
   method_data.push_back(std::move(entry));
   RecreateStateWithOptionsAndDetails(mojom::PaymentOptions::New(),
                                      mojom::PaymentDetails::New(),
                                      std::move(method_data));
 
   // CanMakePayment returns false because the method data supports mastercard,
   // and the user doesn't have such an instrument.
   EXPECT_FALSE(state()->CanMakePayment());
 }
 
-// Test that the last shipping option is selected.
-TEST_F(PaymentRequestStateTest, ShippingOptionsSelection) {
-  std::vector<mojom::PaymentShippingOptionPtr> shipping_options;
-  mojom::PaymentShippingOptionPtr option = mojom::PaymentShippingOption::New();
-  option->id = "option:1";
-  option->selected = false;
-  shipping_options.push_back(std::move(option));
-  mojom::PaymentShippingOptionPtr option2 = mojom::PaymentShippingOption::New();
-  option2->id = "option:2";
-  option2->selected = true;
-  shipping_options.push_back(std::move(option2));
-  mojom::PaymentDetailsPtr details = mojom::PaymentDetails::New();
-  details->shipping_options = std::move(shipping_options);
-
-  mojom::PaymentOptionsPtr options = mojom::PaymentOptions::New();
-  options->request_shipping = true;
-  RecreateStateWithOptionsAndDetails(std::move(options), std::move(details),
-                                     GetMethodDataForVisa());
-
-  EXPECT_EQ("option:2", state()->selected_shipping_option()->id);
-}
-
 TEST_F(PaymentRequestStateTest, ReadyToPay_DefaultSelections) {
   mojom::PaymentOptionsPtr options = mojom::PaymentOptions::New();
   options->request_shipping = true;
   options->request_payer_name = true;
   options->request_payer_phone = true;
   options->request_payer_email = true;
   RecreateStateWithOptions(std::move(options));
 
+  // Because there are shipping options, no address is selected by default.
+  // Therefore we are not ready to pay.
+  EXPECT_FALSE(state()->is_ready_to_pay());
+
+  state()->SetSelectedShippingProfile(test_address());
+  EXPECT_EQ(1, num_on_selected_information_changed_called());
+
   EXPECT_TRUE(state()->is_ready_to_pay());
 }
 
 // Testing that only supported intruments are shown. In this test the merchant
 // only supports Visa.
 TEST_F(PaymentRequestStateTest, UnsupportedCardAreNotAvailable) {
   // Default options.
   RecreateStateWithOptions(mojom::PaymentOptions::New());
 
   // Ready to pay because the default instrument is selected and supported.
@@ skipping 160 matching lines @@
   EXPECT_TRUE(state()->is_ready_to_pay());
   state()->GeneratePaymentResponse();
 
   // Check that the name was set, but not the other values.
   EXPECT_EQ("John H. Doe", response()->payer_name.value());
   EXPECT_FALSE(response()->payer_phone.has_value());
   EXPECT_FALSE(response()->payer_email.has_value());
 }
 
 }  // namespace payments