From e8855c09e6ec109ec2464ed2258e188e4c30fdec Mon Sep 17 00:00:00 2001 From: Kate Glazko Date: Tue, 22 Sep 2020 12:54:21 -0700 Subject: [PATCH] For #15320: Ensure Mimetype Other PDF Shows PDF Icon --- app/src/main/java/org/mozilla/fenix/ext/DownloadItem.kt | 1 + app/src/test/java/org/mozilla/fenix/ext/DownloadItemKtTest.kt | 1 + 2 files changed, 2 insertions(+) diff --git a/app/src/main/java/org/mozilla/fenix/ext/DownloadItem.kt b/app/src/main/java/org/mozilla/fenix/ext/DownloadItem.kt index 93fa26aea4..df48545614 100644 --- a/app/src/main/java/org/mozilla/fenix/ext/DownloadItem.kt +++ b/app/src/main/java/org/mozilla/fenix/ext/DownloadItem.kt @@ -14,6 +14,7 @@ fun DownloadItem.getIcon(): Int { return when { fileName?.endsWith("apk") == true -> R.drawable.ic_file_type_apk fileName?.endsWith("zip") == true -> R.drawable.ic_file_type_zip + fileName?.endsWith("pdf") == true -> R.drawable.ic_file_type_document else -> R.drawable.ic_file_type_default } } diff --git a/app/src/test/java/org/mozilla/fenix/ext/DownloadItemKtTest.kt b/app/src/test/java/org/mozilla/fenix/ext/DownloadItemKtTest.kt index d4881dddbb..e5fdf24b4b 100644 --- a/app/src/test/java/org/mozilla/fenix/ext/DownloadItemKtTest.kt +++ b/app/src/test/java/org/mozilla/fenix/ext/DownloadItemKtTest.kt @@ -23,6 +23,7 @@ class DownloadItemKtTest { assertEquals(R.drawable.ic_file_type_zip, downloadItem.copy(contentType = "application/gzip").getIcon()) assertEquals(R.drawable.ic_file_type_apk, downloadItem.copy(contentType = null, fileName = "Fenix.apk").getIcon()) assertEquals(R.drawable.ic_file_type_zip, downloadItem.copy(contentType = null, fileName = "Fenix.zip").getIcon()) + assertEquals(R.drawable.ic_file_type_document, downloadItem.copy(contentType = null, fileName = "Fenix.pdf").getIcon()) assertEquals(R.drawable.ic_file_type_default, downloadItem.copy(contentType = null, fileName = null).getIcon()) } }