Skip to content

Commit 2b54d40

Browse files
authored
Remove CTA validation variants (#720)
1 parent 2e755d9 commit 2b54d40

File tree

2 files changed

+0
-40
lines changed

2 files changed

+0
-40
lines changed

app/src/androidTest/java/com/duckduckgo/app/statistics/VariantManagerTest.kt

Lines changed: 0 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -68,40 +68,6 @@ class VariantManagerTest {
6868
assertTrue(variant.hasFeature(SuppressDefaultBrowserCta))
6969
}
7070

71-
// CTA Validation experiments
72-
73-
@Test
74-
fun ctaControlVariantIsInactiveAndHasNoFeatures() {
75-
val variant = variants.firstOrNull { it.key == "mq" }
76-
assertEqualsDouble(0.0, variant!!.weight)
77-
assertEquals(0, variant!!.features.size)
78-
}
79-
80-
@Test
81-
fun ctaSuppressDefaultBrowserVariantIsInactiveAndHasSuppressDefaultBrowserFeature() {
82-
val variant = variants.firstOrNull { it.key == "mr" }
83-
assertEqualsDouble(0.0, variant!!.weight)
84-
assertEquals(1, variant!!.features.size)
85-
assertTrue(variant.hasFeature(SuppressDefaultBrowserCta))
86-
}
87-
88-
@Test
89-
fun ctaSuppressWidgetVariantIsInactiveAndHasSuppressWidgetCtaFeature() {
90-
val variant = variants.firstOrNull { it.key == "ms" }
91-
assertEqualsDouble(0.0, variant!!.weight)
92-
assertEquals(1, variant!!.features.size)
93-
assertTrue(variant.hasFeature(SuppressWidgetCta))
94-
}
95-
96-
@Test
97-
fun ctaSuppressAllVariantIsInactiveAndHasSuppressCtaFeatures() {
98-
val variant = variants.firstOrNull { it.key == "mt" }
99-
assertEqualsDouble(0.0, variant!!.weight)
100-
assertEquals(2, variant!!.features.size)
101-
assertTrue(variant.hasFeature(SuppressDefaultBrowserCta))
102-
assertTrue(variant.hasFeature(SuppressWidgetCta))
103-
}
104-
10571
// CTA on Concept Test experiments
10672

10773
@Test

app/src/main/java/com/duckduckgo/app/statistics/VariantManager.kt

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,12 +57,6 @@ interface VariantManager {
5757
filterBy = { isEnglishLocale() }),
5858
Variant(key = "md", weight = 0.0, features = listOf(SuppressWidgetCta, SuppressDefaultBrowserCta), filterBy = { isEnglishLocale() }),
5959

60-
// Validate CTAs experiment
61-
Variant(key = "mq", weight = 0.0, features = emptyList(), filterBy = { noFilter() }),
62-
Variant(key = "mr", weight = 0.0, features = listOf(SuppressDefaultBrowserCta), filterBy = { noFilter() }),
63-
Variant(key = "ms", weight = 0.0, features = listOf(SuppressWidgetCta), filterBy = { noFilter() }),
64-
Variant(key = "mt", weight = 0.0, features = listOf(SuppressWidgetCta, SuppressDefaultBrowserCta), filterBy = { noFilter() }),
65-
6660
// Insert CTAs on Concept test experiment
6761
Variant(key = "mu", weight = 1.0, features = emptyList(), filterBy = { isEnglishLocale() }),
6862
Variant(key = "mv", weight = 1.0,

0 commit comments

Comments
 (0)