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

Unified Diff: ios/chrome/browser/ui/payments/billing_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/billing_address_selection_coordinator_unittest.mm
diff --git a/ios/chrome/browser/ui/payments/billing_address_selection_coordinator_unittest.mm b/ios/chrome/browser/ui/payments/billing_address_selection_coordinator_unittest.mm
index d7fb8e700e5798679b29cabf04cb730cebb69f42..3558927ab569964bf6b0bee65191bd174826d9ce 100644
--- a/ios/chrome/browser/ui/payments/billing_address_selection_coordinator_unittest.mm
+++ b/ios/chrome/browser/ui/payments/billing_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 PaymentRequestBillingAddressSelectionCoordinatorTest
PaymentRequestBillingAddressSelectionCoordinatorTest()
: 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 PaymentRequestBillingAddressSelectionCoordinatorTest
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_);
@@ -79,9 +83,11 @@ class PaymentRequestBillingAddressSelectionCoordinatorTest
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