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

Unified Diff: ios/chrome/browser/ui/payments/payment_items_display_view_controller_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/payment_items_display_view_controller_unittest.mm
diff --git a/ios/chrome/browser/ui/payments/payment_items_display_view_controller_unittest.mm b/ios/chrome/browser/ui/payments/payment_items_display_view_controller_unittest.mm
index 5d3a6794189532f39799e364bbf56efc866191b4..ef7d488be919094011c70205f85a7d5639a5be09 100644
--- a/ios/chrome/browser/ui/payments/payment_items_display_view_controller_unittest.mm
+++ b/ios/chrome/browser/ui/payments/payment_items_display_view_controller_unittest.mm
@@ -9,6 +9,7 @@
#include "base/test/scoped_task_environment.h"
#include "components/autofill/core/browser/test_personal_data_manager.h"
#include "components/strings/grit/components_strings.h"
+#include "ios/chrome/browser/browser_state/test_chrome_browser_state.h"
#include "ios/chrome/browser/payments/payment_request.h"
#import "ios/chrome/browser/payments/payment_request_test_util.h"
#include "ios/chrome/browser/payments/test_payment_request.h"
@@ -17,6 +18,7 @@
#import "ios/chrome/browser/ui/payments/payment_items_display_view_controller_data_source.h"
#include "ios/chrome/grit/ios_strings.h"
#include "ios/web/public/payments/payment_request.h"
+#import "ios/web/public/test/fakes/test_web_state.h"
#include "testing/gtest/include/gtest/gtest.h"
#if !defined(__has_feature) || !__has_feature(objc_arc)
@@ -45,10 +47,13 @@
class PaymentRequestPaymentItemsDisplayViewControllerTest
: public CollectionViewControllerTest {
protected:
+ PaymentRequestPaymentItemsDisplayViewControllerTest()
+ : chrome_browser_state_(TestChromeBrowserState::Builder().Build()) {}
+
CollectionViewController* InstantiateController() override {
payment_request_ = base::MakeUnique<payments::TestPaymentRequest>(
payment_request_test_util::CreateTestWebPaymentRequest(),
- &personal_data_manager_);
+ chrome_browser_state_.get(), &web_state_, &personal_data_manager_);
mediator_ = [[TestPaymentItemsDisplayMediator alloc] init];
PaymentItemsDisplayViewController* viewController = [
[PaymentItemsDisplayViewController alloc] initWithPayButtonEnabled:YES];
@@ -63,7 +68,9 @@ class PaymentRequestPaymentItemsDisplayViewControllerTest
base::test::ScopedTaskEnvironment scoped_task_evironment_;
+ web::TestWebState web_state_;
autofill::TestPersonalDataManager personal_data_manager_;
+ std::unique_ptr<TestChromeBrowserState> chrome_browser_state_;
std::unique_ptr<payments::TestPaymentRequest> payment_request_;
TestPaymentItemsDisplayMediator* mediator_ = nil;
};

Powered by Google App Engine
This is Rietveld 408576698