Skip to content

Commit 41d7976

Browse files
committed
Improve Resilience of Delete Test
Sometimes it would fail stating the task was still there, I believe this was because it hadn't finished deleting. Hopefully it'll now wait. Passed 50 iterations locally.
1 parent aeb6f44 commit 41d7976

File tree

1 file changed

+18
-10
lines changed

1 file changed

+18
-10
lines changed

Tests/MeiliSearchIntegrationTests/TaskTests.swift

Lines changed: 18 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -187,17 +187,25 @@ class TasksTests: XCTestCase {
187187
addDocuments(client: self.client, uid: self.uid, primaryKey: nil) { result in
188188
switch result {
189189
case .success(let task):
190-
self.client.deleteTasks(filter: .init(uids: [task.uid])) { _ in
191-
self.client.getTasks { result in
192-
switch result {
193-
case .success(let tasks):
194-
XCTAssertEqual(tasks.results[0].type, .taskDeletion)
195-
XCTAssertNotEqual(tasks.results[1].uid, task.uid)
196-
case .failure(let error):
197-
dump(error)
198-
XCTFail("Failed to get tasks")
199-
}
190+
self.client.deleteTasks(filter: .init(uids: [task.uid])) { result in
191+
switch result {
192+
case .success(let taskInfo):
193+
self.client.waitForTask(task: taskInfo) { _ in
194+
self.client.getTasks { result in
195+
switch result {
196+
case .success(let tasks):
197+
XCTAssertEqual(tasks.results[0].type, .taskDeletion)
198+
XCTAssertNotEqual(tasks.results[1].uid, task.uid)
199+
case .failure(let error):
200+
dump(error)
201+
XCTFail("Failed to get tasks")
202+
}
200203

204+
addDocExpectation.fulfill()
205+
}
206+
}
207+
case .failure:
208+
XCTFail("Failed to delete document")
201209
addDocExpectation.fulfill()
202210
}
203211
}

0 commit comments

Comments
 (0)