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

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

Issue 2970233002: [Payment Request] Adds a JourneyLogger instance to the PaymentRequest (Closed)
Patch Set: Initial 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/contact_info_edit_coordinator_unittest.mm
diff --git a/ios/chrome/browser/ui/payments/contact_info_edit_coordinator_unittest.mm b/ios/chrome/browser/ui/payments/contact_info_edit_coordinator_unittest.mm
index 95adbe58b81218c0626de69a7092229f79d8f98f..8c63d72ffa6ac3538b7969d2af55dccd38e555fe 100644
--- a/ios/chrome/browser/ui/payments/contact_info_edit_coordinator_unittest.mm
+++ b/ios/chrome/browser/ui/payments/contact_info_edit_coordinator_unittest.mm
@@ -17,11 +17,13 @@
#include "components/payments/core/payments_profile_comparator.h"
#include "components/prefs/pref_service.h"
#include "ios/chrome/browser/application_context.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_edit_view_controller.h"
#import "ios/chrome/browser/ui/payments/payment_request_editor_field.h"
#import "ios/chrome/test/scoped_key_window.h"
+#import "ios/web/public/test/fakes/test_web_state.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "testing/platform_test.h"
@@ -52,8 +54,12 @@ class MockPaymentsProfileComparator
class MockTestPaymentRequest : public payments::TestPaymentRequest {
public:
MockTestPaymentRequest(web::PaymentRequest web_payment_request,
+ ios::ChromeBrowserState* browser_state,
+ web::WebState* web_state,
autofill::PersonalDataManager* personal_data_manager)
: payments::TestPaymentRequest(web_payment_request,
+ browser_state,
+ web_state,
personal_data_manager) {}
MOCK_METHOD1(AddAutofillProfile,
autofill::AutofillProfile*(const autofill::AutofillProfile&));
@@ -96,9 +102,13 @@ class PaymentRequestContactInfoEditCoordinatorTest : public PlatformTest {
PaymentRequestContactInfoEditCoordinatorTest()
: pref_service_(autofill::test::PrefServiceForTesting()) {
personal_data_manager_.SetTestingPrefService(pref_service_.get());
+
+ TestChromeBrowserState::Builder test_cbs_builder;
+ chrome_browser_state_ = test_cbs_builder.Build();
+
payment_request_ = base::MakeUnique<MockTestPaymentRequest>(
payment_request_test_util::CreateTestWebPaymentRequest(),
- &personal_data_manager_);
+ chrome_browser_state_.get(), &web_state_, &personal_data_manager_);
profile_comparator_ = base::MakeUnique<MockPaymentsProfileComparator>(
GetApplicationContext()->GetApplicationLocale(),
@@ -115,11 +125,13 @@ class PaymentRequestContactInfoEditCoordinatorTest : public PlatformTest {
base::test::ScopedTaskEnvironment scoped_task_evironment_;
+ web::TestWebState web_state_;
std::unique_ptr<PrefService> pref_service_;
MockTestPersonalDataManager personal_data_manager_;
autofill::TestRegionDataLoader test_region_data_loader_;
std::unique_ptr<MockPaymentsProfileComparator> profile_comparator_;
std::unique_ptr<MockTestPaymentRequest> payment_request_;
+ std::unique_ptr<TestChromeBrowserState> chrome_browser_state_;
please use gerrit instead 2017/07/06 14:52:25 Ditto
Moe 2017/07/07 02:24:54 Done.
};
// Tests that invoking start and stop on the coordinator presents and dismisses

Powered by Google App Engine
This is Rietveld 408576698