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

Unified Diff: ios/chrome/browser/payments/payment_request_unittest.mm

Issue 2971003002: [Payment Request] Queries application locale from payments::PaymentRequest (Closed)
Patch Set: rebase 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/payments/payment_request_unittest.mm
diff --git a/ios/chrome/browser/payments/payment_request_unittest.mm b/ios/chrome/browser/payments/payment_request_unittest.mm
index 7eeee580078faa953a88eae4e790ce3031c94c54..d406b7faf4d77ca244fc4ffd20e44f97f21aee29 100644
--- a/ios/chrome/browser/payments/payment_request_unittest.mm
+++ b/ios/chrome/browser/payments/payment_request_unittest.mm
@@ -84,7 +84,6 @@ class PaymentRequestTest : public testing::Test {
// Tests that the payments::CurrencyFormatter is constructed with the correct
// currency code and currency system.
TEST_F(PaymentRequestTest, CreatesCurrencyFormatterCorrectly) {
- ASSERT_EQ("en", GetApplicationContext()->GetApplicationLocale());
web::PaymentRequest web_payment_request;
autofill::TestPersonalDataManager personal_data_manager;
@@ -93,6 +92,7 @@ TEST_F(PaymentRequestTest, CreatesCurrencyFormatterCorrectly) {
TestPaymentRequest payment_request1(web_payment_request,
chrome_browser_state_.get(), &web_state_,
&personal_data_manager);
+ ASSERT_EQ("en", payment_request1.GetApplicationLocale());
CurrencyFormatter* currency_formatter =
payment_request1.GetOrCreateCurrencyFormatter();
EXPECT_EQ(base::UTF8ToUTF16("$55.00"), currency_formatter->Format("55.00"));
@@ -102,6 +102,7 @@ TEST_F(PaymentRequestTest, CreatesCurrencyFormatterCorrectly) {
TestPaymentRequest payment_request2(web_payment_request,
chrome_browser_state_.get(), &web_state_,
&personal_data_manager);
+ ASSERT_EQ("en", payment_request2.GetApplicationLocale());
currency_formatter = payment_request2.GetOrCreateCurrencyFormatter();
EXPECT_EQ(base::UTF8ToUTF16("¥55"), currency_formatter->Format("55.00"));
EXPECT_EQ("JPY", currency_formatter->formatted_currency_code());
@@ -112,6 +113,7 @@ TEST_F(PaymentRequestTest, CreatesCurrencyFormatterCorrectly) {
TestPaymentRequest payment_request3(web_payment_request,
chrome_browser_state_.get(), &web_state_,
&personal_data_manager);
+ ASSERT_EQ("en", payment_request3.GetApplicationLocale());
currency_formatter = payment_request3.GetOrCreateCurrencyFormatter();
EXPECT_EQ(base::UTF8ToUTF16("55.00"), currency_formatter->Format("55.00"));
EXPECT_EQ("USD", currency_formatter->formatted_currency_code());
« no previous file with comments | « ios/chrome/browser/payments/payment_request.mm ('k') | ios/chrome/browser/ui/payments/address_edit_coordinator.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698