Skip to content

Commit c4bf77f

Browse files
committed
Wait for ABTest deletion in TestSuite
1 parent 620ecf5 commit c4bf77f

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

src/commonTest/kotlin/suite/Extensions.kt

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ internal fun compareVariant(actual: ResponseVariant, expected: Variant) {
3030
}
3131
}
3232

33-
internal suspend fun cleanABTest(suffix: String, now: Boolean = false) {
33+
internal suspend fun cleanABTest(clientSearch: ClientSearch, suffix: String, now: Boolean = false) {
3434
val regex = Regex("kotlin-(.*)-$username-$suffix")
3535

3636
clientAnalytics.browseAllABTests().forEach {
@@ -42,7 +42,11 @@ internal suspend fun cleanABTest(suffix: String, now: Boolean = false) {
4242
val difference = Time.getCurrentTimeMillis() - DateFormat.parse(date)
4343

4444
if (difference >= dayInMillis || now) {
45-
clientAnalytics.deleteABTest(abTest.abTestID)
45+
val deletion = clientAnalytics.deleteABTest(abTest.abTestID)
46+
47+
clientSearch.initIndex(deletion.indexName).apply {
48+
deletion.wait()
49+
}
4650
}
4751
}
4852
}

src/commonTest/kotlin/suite/TestSuiteAATest.kt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import com.algolia.search.model.search.Query
1212
import com.algolia.search.model.task.TaskStatus
1313
import com.algolia.search.serialize.KeyObjectID
1414
import dayInMillis
15-
import kotlinx.coroutines.delay
1615
import kotlinx.serialization.json.json
1716
import runBlocking
1817
import shouldEqual
@@ -41,7 +40,7 @@ internal class TestSuiteAATest {
4140
@BeforeTest
4241
fun clean() {
4342
runBlocking {
44-
cleanABTest(suffix)
43+
cleanABTest(clientAdmin1, suffix)
4544
cleanIndex(clientAdmin1, suffix)
4645
}
4746
}
@@ -54,7 +53,6 @@ internal class TestSuiteAATest {
5453
val response = clientAnalytics.addABTest(abTest)
5554

5655
response.wait() shouldEqual TaskStatus.Published
57-
delay(2000L)
5856
clientAnalytics.getABTest(response.abTestID).let {
5957
it.name shouldEqual abTest.name
6058
it.endAt shouldEqual abTest.endAt

src/commonTest/kotlin/suite/TestSuiteABTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ internal class TestSuiteABTest {
4141
@BeforeTest
4242
fun clean() {
4343
runBlocking {
44-
cleanABTest(suffix)
44+
cleanABTest(clientAdmin1, suffix)
4545
cleanIndex(clientAdmin1, suffix)
4646
}
4747
}

0 commit comments

Comments
 (0)