Archives/iceraven-browser
2
0
Fork 0
mirror of https://github.com/fork-maintainers/iceraven-browser synced 2024-11-03 23:15:31 +00:00
Code Issues Releases Wiki Activity

For #14772: Crash in Downloads

Browse Source
This commit is contained in:
Kate Glazko 2020-11-11 09:00:17 -08:00 committed by kglazko
parent 729f46ec1f
commit 6c68e79a22
2 changed files with 43 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/src/main/java/org/mozilla/fenix/library/downloads/DownloadFragment.kt
View File

@ -91,7 +91,7 @@ class DownloadFragment : LibraryPageFragment<DownloadItem>(), UserInteractionHan
it.id,
it.fileName,
it.filePath,
it.contentLength.toString(),
it.contentLength?.toString() ?: "0",
it.contentType,
it.status
)

45
app/src/test/java/org/mozilla/fenix/library/downloads/DownloadFragmentTest.kt
View File

@ -48,7 +48,7 @@ class DownloadFragmentTest {
"3",
"3.pdf",
downloadedFile3.path,
"null",
"0",
null,
DownloadState.Status.COMPLETED
),
@ -56,7 +56,7 @@ class DownloadFragmentTest {
"2",
"2.pdf",
downloadedFile2.path,
"null",
"0",
null,
DownloadState.Status.COMPLETED
),
@ -64,7 +64,7 @@ class DownloadFragmentTest {
"1",
"1.pdf",
downloadedFile1.path,
"null",
"0",
null,
DownloadState.Status.COMPLETED
)
@ -104,4 +104,43 @@ class DownloadFragmentTest {
downloadedFile2.delete()
downloadedFile3.delete()
}
@Test
fun `downloads with null content length don't crash`() {
val downloadedFile0 = File(
Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS),
"1.pdf"
)
downloadedFile0.createNewFile()
val fragment = DownloadFragment()
val expectedList = listOf(
DownloadItem(
"1",
"1.pdf",
downloadedFile0.path,
"0",
null,
DownloadState.Status.COMPLETED
)
)
val state: BrowserState = mockk(relaxed = true)
every { state.downloads } returns mapOf(
"1" to DownloadState(
id = "1",
createdTime = 1,
url = "url",
fileName = "1.pdf",
contentLength = null,
status = DownloadState.Status.COMPLETED
)
)
val list = fragment.provideDownloads(state)
assertEquals(expectedList[0].size, list[0].size)
}
}