For EXP-3625: Take getCoenrollingFeatureIds out of the interfaces

mozilla · Jul 10, 2023 · c2353d0 · c2353d0
1 parent 955aaa0
commit c2353d0
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 36 deletions.
diff --git a/components/nimbus/android/src/main/java/org/mozilla/experiments/nimbus/NimbusInterface.kt b/components/nimbus/android/src/main/java/org/mozilla/experiments/nimbus/NimbusInterface.kt
@@ -182,13 +182,6 @@ interface NimbusInterface : FeaturesInterface, GleanPlumbInterface, NimbusEventS
      */
     fun resetTelemetryIdentifiers() = Unit
 
-    /**
-     * Get a list of feature ids where the feature allows co-enrollment.
-     *
-     * @return A list of experiment ids that allow co-enrollment.
-     */
-    fun getCoenrollingFeatureIds(): List<String> = listOf()
-
     /**
      * Control the opt out for all experiments at once. This is likely a user action.
      */

diff --git a/components/nimbus/android/src/test/java/org/mozilla/experiments/nimbus/NimbusTests.kt b/components/nimbus/android/src/test/java/org/mozilla/experiments/nimbus/NimbusTests.kt
@@ -644,29 +644,6 @@ class NimbusTests {
 
         assertTrue(observed)
     }
-
-    @Test
-    fun `test coenrolling features list is returned`() {
-        val expected = listOf("abc", "def", "ghi")
-        val observer = object : NimbusInterface.Observer {
-            override fun onUpdatesApplied(updated: List<org.mozilla.experiments.nimbus.internal.EnrolledExperiment>) {
-                runBlocking {
-                    delay(250)
-                }
-            }
-        }
-        val nimbus = Nimbus(
-            context = context,
-            appInfo = appInfo,
-            coenrollingFeatureIds = expected,
-            server = null,
-            deviceInfo = deviceInfo,
-            observer = observer,
-            delegate = nimbusDelegate,
-        )
-
-        assertEquals(expected, nimbus.getCoenrollingFeatureIds())
-    }
 }
 
 // Mocking utilities, from mozilla.components.support.test

diff --git a/components/nimbus/ios/Nimbus/NimbusApi.swift b/components/nimbus/ios/Nimbus/NimbusApi.swift
@@ -167,12 +167,6 @@ public protocol NimbusUserConfiguration {
     /// - Returns  A list of `AvailableExperiment`s
     ///
     func getAvailableExperiments() -> [AvailableExperiment]
-
-    /// Get a list of feature ids where the feature allows co-enrollment.
-    ///
-    /// - Returns  A list of experiment ids that allow co-enrollment.
-    ///
-    func getCoenrollingFeatureIds() -> [String]
 }
 
 public protocol NimbusEventStore {