Issue 664095 link

Starred by 1 user

Issue metadata

Status:	Archived
Owner:	poromov@chromium.org
Closed:	Apr 2017
Cc:	█ pelets...@chromium.org hidehiko@chromium.org
EstimatedDays:	----
NextAction:	----
OS:	Chrome
Pri:	2
Type:	Bug
Test-Functional VerifyIn-60 VerifyIn-61

Missing unit tests for ArcRobotAuth

Project Member Reported by pelets...@chromium.org, Nov 10 2016

Issue description

There is no unit test for ArcRobotAuth class. It is tested in browser test, but good practice is to have unit tests as well.

Comment 1 by poromov@chromium.org, Apr 12 2017

Cc: hidehiko@chromium.org pelets...@chromium.org

I think it was already fixed in http://crrev.com/2507073002 ? After refactoring there are dedicated browser tests for ArcRobotAuthCodeFetcher into which ArcRobotAuth was refactored.

Hidehiko, Alexandr, WDYT?