Commit d2954428 authored by Alice Wang's avatar Alice Wang Committed by Commit Bot

[Android][WebSignin] Test the sign-in metrics in bottom sheet

This CL tests two sign-in metrics in the web sign-in bottom sheet.

Bug: 1134874
Change-Id: Ie2a4b213856226f7e8004b17a55f7a097a405ddd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2475700Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Commit-Queue: Alice Wang <aliceywang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#817500}
parent 1c33b55b
...@@ -46,12 +46,15 @@ import org.chromium.base.Callback; ...@@ -46,12 +46,15 @@ import org.chromium.base.Callback;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.Batch; import org.chromium.base.test.util.Batch;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.MetricsUtils;
import org.chromium.base.test.util.MetricsUtils.HistogramDelta;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeTabbedActivity; import org.chromium.chrome.browser.ChromeTabbedActivity;
import org.chromium.chrome.browser.flags.ChromeFeatureList; import org.chromium.chrome.browser.flags.ChromeFeatureList;
import org.chromium.chrome.browser.flags.ChromeSwitches; import org.chromium.chrome.browser.flags.ChromeSwitches;
import org.chromium.chrome.browser.incognito.IncognitoUtils; import org.chromium.chrome.browser.incognito.IncognitoUtils;
import org.chromium.chrome.browser.incognito.interstitial.IncognitoInterstitialDelegate; import org.chromium.chrome.browser.incognito.interstitial.IncognitoInterstitialDelegate;
import org.chromium.chrome.browser.signin.account_picker.AccountConsistencyPromoAction;
import org.chromium.chrome.browser.signin.account_picker.AccountPickerBottomSheetCoordinator; import org.chromium.chrome.browser.signin.account_picker.AccountPickerBottomSheetCoordinator;
import org.chromium.chrome.browser.signin.account_picker.AccountPickerDelegate; import org.chromium.chrome.browser.signin.account_picker.AccountPickerDelegate;
import org.chromium.chrome.test.ChromeJUnit4ClassRunner; import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
...@@ -310,18 +313,26 @@ public class AccountPickerBottomSheetTest { ...@@ -310,18 +313,26 @@ public class AccountPickerBottomSheetTest {
@Test @Test
@MediumTest @MediumTest
public void testSignInDefaultAccountOnCollapsedSheet() { public void testSignInDefaultAccountOnCollapsedSheet() {
MetricsUtils.HistogramDelta accountConsistencyHistogram =
new HistogramDelta("Signin.AccountConsistencyPromoAction",
AccountConsistencyPromoAction.SIGNED_IN_WITH_DEFAULT_ACCOUNT);
buildAndShowCollapsedBottomSheet(); buildAndShowCollapsedBottomSheet();
clickContinueButtonAndCheckSignInInProgressSheet(); clickContinueButtonAndCheckSignInInProgressSheet();
Assert.assertEquals(1, accountConsistencyHistogram.getDelta());
} }
@Test @Test
@MediumTest @MediumTest
public void testSignInAnotherAccount() { public void testSignInAnotherAccount() {
MetricsUtils.HistogramDelta accountConsistencyHistogram =
new HistogramDelta("Signin.AccountConsistencyPromoAction",
AccountConsistencyPromoAction.SIGNED_IN_WITH_NON_DEFAULT_ACCOUNT);
buildAndShowExpandedBottomSheet(); buildAndShowExpandedBottomSheet();
onView(withText(PROFILE_DATA2.getAccountName())).perform(click()); onView(withText(PROFILE_DATA2.getAccountName())).perform(click());
CriteriaHelper.pollUiThread(mCoordinator.getBottomSheetViewForTesting().findViewById( CriteriaHelper.pollUiThread(mCoordinator.getBottomSheetViewForTesting().findViewById(
R.id.account_picker_continue_as_button)::isShown); R.id.account_picker_continue_as_button)::isShown);
clickContinueButtonAndCheckSignInInProgressSheet(); clickContinueButtonAndCheckSignInInProgressSheet();
Assert.assertEquals(1, accountConsistencyHistogram.getDelta());
} }
@Test @Test
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment