[Payments] Move PaymentResponse logic to PaymentResponseHelper.

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 279 matching lines @@
 
   EXPECT_FALSE(state()->is_ready_to_pay());
 
   state()->SetSelectedContactProfile(test_address());
   EXPECT_EQ(2, num_on_selected_information_changed_called());
 
   // Ready to pay!
   EXPECT_TRUE(state()->is_ready_to_pay());
 }
 
+// TODO(crbug.com/710160): Move the PaymentResponse tests to
+// payment_response_helper_unittest.cc.
 // Test generating a PaymentResponse.
 TEST_F(PaymentRequestStateTest, GeneratePaymentResponse_SupportedMethod) {
   // Default options (no shipping, no contact info).
   RecreateStateWithOptions(mojom::PaymentOptions::New());
   state()->SetSelectedInstrument(state()->available_instruments()[0].get());
   EXPECT_EQ(1, num_on_selected_information_changed_called());
   EXPECT_TRUE(state()->is_ready_to_pay());
 
   // TODO(mathp): Currently synchronous, when async will need a RunLoop.
   // "visa" is specified directly in the supportedMethods so it is returned
@@ skipping 11 matching lines @@
       "\"recipient\":\"John H. Doe\","
       "\"region\":\"CA\"},"
       "\"cardNumber\":\"4111111111111111\","
       "\"cardSecurityCode\":\"123\","
       "\"cardholderName\":\"Test User\","
       "\"expiryMonth\":\"11\","
       "\"expiryYear\":\"2017\"}",
       response()->stringified_details);
 }
 
+// TODO(crbug.com/710160): Move the PaymentResponse tests to
+// payment_response_helper_unittest.cc.
 // Test generating a PaymentResponse when the method is specified through
 // "basic-card".
 TEST_F(PaymentRequestStateTest, GeneratePaymentResponse_BasicCard) {
   // The method data supports visa through basic-card.
   mojom::PaymentMethodDataPtr entry = mojom::PaymentMethodData::New();
   entry->supported_methods.push_back("basic-card");
   entry->supported_networks.push_back(mojom::BasicCardNetwork::VISA);
   std::vector<mojom::PaymentMethodDataPtr> method_data;
   method_data.push_back(std::move(entry));
   RecreateStateWithOptionsAndDetails(mojom::PaymentOptions::New(),
@@ skipping 17 matching lines @@
       "\"recipient\":\"John H. Doe\","
       "\"region\":\"CA\"},"
       "\"cardNumber\":\"4111111111111111\","
       "\"cardSecurityCode\":\"123\","
       "\"cardholderName\":\"Test User\","
       "\"expiryMonth\":\"11\","
       "\"expiryYear\":\"2017\"}",
       response()->stringified_details);
 }
 
+// TODO(crbug.com/710160): Move the PaymentResponse tests to
+// payment_response_helper_unittest.cc.
 // Tests the the generated PaymentResponse has the correct values for the
 // shipping address.
 TEST_F(PaymentRequestStateTest, GeneratePaymentResponse_ShippingAddress) {
   // Setup so that a shipping address is requested.
   std::vector<mojom::PaymentShippingOptionPtr> shipping_options;
   mojom::PaymentShippingOptionPtr option = mojom::PaymentShippingOption::New();
   option->id = "option:1";
   option->selected = true;
   shipping_options.push_back(std::move(option));
   mojom::PaymentDetailsPtr details = mojom::PaymentDetails::New();
@@ skipping 14 matching lines @@
   EXPECT_EQ("Elysium", response()->shipping_address->city);
   EXPECT_EQ("", response()->shipping_address->dependent_locality);
   EXPECT_EQ("91111", response()->shipping_address->postal_code);
   EXPECT_EQ("", response()->shipping_address->sorting_code);
   EXPECT_EQ("", response()->shipping_address->language_code);
   EXPECT_EQ("Underworld", response()->shipping_address->organization);
   EXPECT_EQ("John H. Doe", response()->shipping_address->recipient);
   EXPECT_EQ("16502111111", response()->shipping_address->phone);
 }
 
+// TODO(crbug.com/710160): Move the PaymentResponse tests to
+// payment_response_helper_unittest.cc.
 // Tests the the generated PaymentResponse has the correct values for the
 // contact details when all values are requested.
 TEST_F(PaymentRequestStateTest, GeneratePaymentResponse_ContactDetails_All) {
   // Request all contact detail values.
   mojom::PaymentOptionsPtr options = mojom::PaymentOptions::New();
   options->request_payer_name = true;
   options->request_payer_phone = true;
   options->request_payer_email = true;
   RecreateStateWithOptions(std::move(options));
 
   EXPECT_TRUE(state()->is_ready_to_pay());
   state()->GeneratePaymentResponse();
 
   // Check that all the expected values were set.
   EXPECT_EQ("John H. Doe", response()->payer_name.value());
   EXPECT_EQ("16502111111", response()->payer_phone.value());
   EXPECT_EQ("johndoe@hades.com", response()->payer_email.value());
 }
 
+// TODO(crbug.com/710160): Move the PaymentResponse tests to
+// payment_response_helper_unittest.cc.
 // Tests the the generated PaymentResponse has the correct values for the
 // contact details when all values are requested.
 TEST_F(PaymentRequestStateTest, GeneratePaymentResponse_ContactDetails_Some) {
   // Request one contact detail value.
   mojom::PaymentOptionsPtr options = mojom::PaymentOptions::New();
   options->request_payer_name = true;
   RecreateStateWithOptions(std::move(options));
 
   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