Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(291)

Unified Diff: ios/chrome/browser/ui/payments/shipping_address_selection_coordinator_unittest.mm

Issue 2970233002: [Payment Request] Adds a JourneyLogger instance to the PaymentRequest (Closed)
Patch Set: Addressed comments Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/ui/payments/shipping_address_selection_coordinator_unittest.mm
diff --git a/ios/chrome/browser/ui/payments/shipping_address_selection_coordinator_unittest.mm b/ios/chrome/browser/ui/payments/shipping_address_selection_coordinator_unittest.mm
index 4eb69572897c99101c873ea901ead77259c3b6d1..7c94a2ee61a243c0ef0e2c4b0b463b9564162c94 100644
--- a/ios/chrome/browser/ui/payments/shipping_address_selection_coordinator_unittest.mm
+++ b/ios/chrome/browser/ui/payments/shipping_address_selection_coordinator_unittest.mm
@@ -15,9 +15,11 @@
#include "components/autofill/core/browser/test_personal_data_manager.h"
#include "components/autofill/core/browser/test_region_data_loader.h"
#include "components/prefs/pref_service.h"
+#include "ios/chrome/browser/browser_state/test_chrome_browser_state.h"
#include "ios/chrome/browser/payments/payment_request_test_util.h"
#include "ios/chrome/browser/payments/test_payment_request.h"
#import "ios/chrome/browser/ui/payments/payment_request_selector_view_controller.h"
+#import "ios/web/public/test/fakes/test_web_state.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "testing/platform_test.h"
#include "third_party/ocmock/OCMock/OCMock.h"
@@ -33,7 +35,8 @@ class PaymentRequestShippingAddressSelectionCoordinatorTest
PaymentRequestShippingAddressSelectionCoordinatorTest()
: autofill_profile1_(autofill::test::GetFullProfile()),
autofill_profile2_(autofill::test::GetFullProfile2()),
- pref_service_(autofill::test::PrefServiceForTesting()) {
+ pref_service_(autofill::test::PrefServiceForTesting()),
+ chrome_browser_state_(TestChromeBrowserState::Builder().Build()) {
personal_data_manager_.SetTestingPrefService(pref_service_.get());
// Add testing profiles to autofill::TestPersonalDataManager. Make the less
// frequently used one incomplete.
@@ -44,9 +47,10 @@ class PaymentRequestShippingAddressSelectionCoordinatorTest
autofill::AutofillType(autofill::PHONE_HOME_WHOLE_NUMBER),
base::string16(), "en-US");
personal_data_manager_.AddTestingProfile(&autofill_profile2_);
+
payment_request_ = base::MakeUnique<payments::TestPaymentRequest>(
payment_request_test_util::CreateTestWebPaymentRequest(),
- &personal_data_manager_);
+ chrome_browser_state_.get(), &web_state_, &personal_data_manager_);
test_region_data_loader_.set_synchronous_callback(true);
payment_request_->SetRegionDataLoader(&test_region_data_loader_);
@@ -60,9 +64,11 @@ class PaymentRequestShippingAddressSelectionCoordinatorTest
autofill::AutofillProfile autofill_profile1_;
autofill::AutofillProfile autofill_profile2_;
+ web::TestWebState web_state_;
std::unique_ptr<PrefService> pref_service_;
autofill::TestPersonalDataManager personal_data_manager_;
autofill::TestRegionDataLoader test_region_data_loader_;
+ std::unique_ptr<TestChromeBrowserState> chrome_browser_state_;
std::unique_ptr<payments::TestPaymentRequest> payment_request_;
};

Powered by Google App Engine
This is Rietveld 408576698