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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShippingAddressAndOptionTest.java

Issue 2914503002: align shipping option name to left and shipping cost to right on bottom sheet (Closed)
Patch Set: formatting Created 3 years, 7 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: chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShippingAddressAndOptionTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShippingAddressAndOptionTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShippingAddressAndOptionTest.java
index 299b6fc3107599d1b75fe08423793ddfe7dff270..4e451bbf34342a189e5ac0e3d5c014d3e9866312 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShippingAddressAndOptionTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShippingAddressAndOptionTest.java
@@ -74,8 +74,10 @@ public class PaymentRequestShippingAddressAndOptionTest implements MainActivityS
// Make sure shipping option summary on bottom sheet is displayed in a single line
// as expected.
- Assert.assertEquals("Free global shipping, $0.00",
- mPaymentRequestTestRule.getShippingOptionSummaryLabel());
+ Assert.assertEquals(
+ "Free global shipping", mPaymentRequestTestRule.getShippingOptionSummaryLabel());
gogerald1 2017/05/31 20:34:47 one space line below
wuandy1 2017/06/01 17:17:57 Done.
+ // Make sure shipping cost is aligned to the right on bottom sheet.
gogerald1 2017/05/31 20:34:47 You are not checking alignment here. Suggest: "//
wuandy1 2017/06/01 17:17:57 Done.
+ Assert.assertEquals("$0.00", mPaymentRequestTestRule.getShippingOptionCostSummaryLabel());
}
/** Verifies that the shipping address format in fullsheet mode is as expected. */

Powered by Google App Engine
This is Rietveld 408576698