From f575b4eacf27835da6fd3ba43ab452b43a8fde37 Mon Sep 17 00:00:00 2001 From: 0nko Date: Fri, 10 Jan 2025 20:36:36 +0100 Subject: [PATCH 1/2] Reduce the higher-count buckets and move them all to 81+ --- .../com/duckduckgo/app/tabs/store/TabStatsBucketing.kt | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/app/src/main/java/com/duckduckgo/app/tabs/store/TabStatsBucketing.kt b/app/src/main/java/com/duckduckgo/app/tabs/store/TabStatsBucketing.kt index d31686965504..447448941e86 100644 --- a/app/src/main/java/com/duckduckgo/app/tabs/store/TabStatsBucketing.kt +++ b/app/src/main/java/com/duckduckgo/app/tabs/store/TabStatsBucketing.kt @@ -46,12 +46,7 @@ interface TabStatsBucketing { 21..40, 41..60, 61..80, - 81..100, - 101..125, - 126..150, - 151..250, - 251..500, - 501..Int.MAX_VALUE, + 81..Int.MAX_VALUE, ) val ACTIVITY_BUCKETS = listOf( From 951fa84c78b849f6b850720465ecd15be594935b Mon Sep 17 00:00:00 2001 From: 0nko Date: Sat, 11 Jan 2025 12:50:33 +0100 Subject: [PATCH 2/2] Fix the unit tests --- .../app/tabs/store/TabStatsBucketingTest.kt | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/app/src/test/java/com/duckduckgo/app/tabs/store/TabStatsBucketingTest.kt b/app/src/test/java/com/duckduckgo/app/tabs/store/TabStatsBucketingTest.kt index 511ee02756d8..6fc9fe872820 100644 --- a/app/src/test/java/com/duckduckgo/app/tabs/store/TabStatsBucketingTest.kt +++ b/app/src/test/java/com/duckduckgo/app/tabs/store/TabStatsBucketingTest.kt @@ -94,42 +94,42 @@ class DefaultTabStatsBucketingTest { fun testGetNumberOfOpenTabs81To100() = runTest { whenever(tabRepository.getOpenTabCount()).thenReturn(90) val result = defaultTabStatsBucketing.getNumberOfOpenTabs() - assertEquals("81-100", result) + assertEquals("81+", result) } @Test fun testGetNumberOfOpenTabs101To125() = runTest { whenever(tabRepository.getOpenTabCount()).thenReturn(110) val result = defaultTabStatsBucketing.getNumberOfOpenTabs() - assertEquals("101-125", result) + assertEquals("81+", result) } @Test fun testGetNumberOfOpenTabs126To150() = runTest { whenever(tabRepository.getOpenTabCount()).thenReturn(130) val result = defaultTabStatsBucketing.getNumberOfOpenTabs() - assertEquals("126-150", result) + assertEquals("81+", result) } @Test fun testGetNumberOfOpenTabs151To250() = runTest { whenever(tabRepository.getOpenTabCount()).thenReturn(200) val result = defaultTabStatsBucketing.getNumberOfOpenTabs() - assertEquals("151-250", result) + assertEquals("81+", result) } @Test fun testGetNumberOfOpenTabs251To500() = runTest { whenever(tabRepository.getOpenTabCount()).thenReturn(300) val result = defaultTabStatsBucketing.getNumberOfOpenTabs() - assertEquals("251-500", result) + assertEquals("81+", result) } @Test fun testGetNumberOfOpenTabsMaxValue() = runTest { whenever(tabRepository.getOpenTabCount()).thenReturn(600) val result = defaultTabStatsBucketing.getNumberOfOpenTabs() - assertEquals("501+", result) + assertEquals("81+", result) } @Test