New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 681766 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Feature



Sign in to add a comment

Mock DownloadHistory in DownloadProvider tests.

Project Member Reported by vitaliii@chromium.org, Jan 17 2017

Issue description

Unfortunately, I am not aware of any way to create DownloadHistory in our tests, because its constructor requires being on UI thread. Thus, we cannot tests properly interaction between the provider and DownloadHistory. 

We (or DownloadHistory OWNERS) need to extract its interface and move implementation into DownloadHistoryImpl. Then add tests.
 
Labels: -Restrict-View-Google
Summary: Mock DownloadHistory in DownloadProvider tests. (was: Create DownloadHistory and test it in DownloadProvider tests.)
Actually it is possible to construct it, but the code looks quite back. 
It would be much better to be able to mock it.
Labels: -Pri-1 -M-58 Pri-3
Status: WontFix (was: Untriaged)

Sign in to add a comment