New issue
Advanced search Search tips

Issue 908124 link

Starred by 2 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Implement unit test for GetAccountsForDicePromos() (signin_ui_util.h)

Project Member Reported by ma...@igalia.com, Nov 23

Issue description

The unit test DiceSigninUiUtilTest.GetAccountsForDicePromos does not really test much, only that the method returns an empty list after startup, which cause some confusion in CL1346409 because a test that was wrongly migrated to the Identity service was not failing (when it should).

Filing this ticket to track this down, so it's not forgotten.
 
Owner: tangltom@chromium.org
Status: Started (was: Untriaged)
Thanks for filing! This somehow got lost. I will add a test asap.
Labels: -Pri-3 Pri-2

Sign in to add a comment