Make task tests more relevant

Author: bidoubiwa

tests/task.test.ts

@@ -94,32 +94,18 @@ describe.each([{ permission: 'Master' }, { permission: 'Private' }])(
 
       const tasks = await client.getTasks()
 
-      expect(tasks.results[0]).toHaveProperty(
-        'status',
-        TaskStatus.TASK_SUCCEEDED
-      )
-      expect(tasks.results[0].indexUid).toEqual(index.uid)
-      expect(tasks.results[0].status).toEqual(TaskStatus.TASK_SUCCEEDED)
-      expect(tasks.results[0].type).toEqual(
-        TaskTypes.DOCUMENTS_ADDITION_OR_UPDATE
-      )
-      expect(tasks.results[0].enqueuedAt).toBeDefined()
-      expect(tasks.results[0].uid).toBeDefined()
-      expect(tasks.results[0].type).toEqual(
-        TaskTypes.DOCUMENTS_ADDITION_OR_UPDATE
-      )
-      expect(tasks.results[0].duration).toBeDefined()
-      expect(tasks.results[0].finishedAt).toBeDefined()
-      expect(tasks.results[0].startedAt).toBeDefined()
+      expect(tasks.results).toBeInstanceOf(Array)
+      expect(tasks.results[0].uid).toEqual(enqueuedTask.taskUid)
     })
 
     test(`${permission} key: Get all tasks with type filter`, async () => {
       const client = await getClient(permission)
       const task1 = await client.index(index.uid).addDocuments([{ id: 1 }])
-      await client.waitForTask(task1.taskUid)
-
       const task2 = await client.index(index.uid).deleteDocument(1)
+      const task3 = await client.createIndex(index2.uid)
+      await client.waitForTask(task1.taskUid)
       await client.waitForTask(task2.taskUid)
+      await client.waitForTask(task3.taskUid)
 
       const tasks = await client.getTasks({
         type: [
@@ -144,19 +130,17 @@ describe.each([{ permission: 'Master' }, { permission: 'Private' }])(
       const tasks = await client.getTasks({
         status: [TaskStatus.TASK_SUCCEEDED, TaskStatus.TASK_FAILED],
       })
-      const onlySuccesFullTasks = new Set(
+      const onlySuccesfullTasks = new Set(
         tasks.results.map((task) => task.status)
       )
 
-      expect(onlySuccesFullTasks.size).toEqual(2)
+      expect(onlySuccesfullTasks.size).toEqual(2)
     })
 
     test(`${permission} key: Get all tasks with indexUid filter`, async () => {
       const client = await getClient(permission)
-      const task1 = await client.index(index.uid).addDocuments([{ id: 1 }])
-      const task2 = await client.index(index2.uid).addDocuments([{ id: 1 }])
-      await client.waitForTask(task1.taskUid)
-      await client.waitForTask(task2.taskUid)
+      await client.index(index.uid).addDocuments([{ id: 1 }])
+      await client.index(index2.uid).addDocuments([{ id: 1 }])
 
       const tasks = await client.getTasks({
         indexUid: [index.uid, index2.uid],
@@ -170,30 +154,15 @@ describe.each([{ permission: 'Master' }, { permission: 'Private' }])(
 
     test(`${permission} key: Get all indexes tasks with index instance`, async () => {
       const client = await getClient(permission)
-      const enqueuedTask = await client
-        .index(index.uid)
-        .addDocuments([{ id: 1 }])
-      await client.waitForTask(enqueuedTask.taskUid)
+      await client.index(index.uid).addDocuments([{ id: 1 }])
+      await client.index(index2.uid).addDocuments([{ id: 1 }])
 
       const tasks = await client.index(index.uid).getTasks()
-
-      expect(tasks.results[0]).toHaveProperty(
-        'status',
-        TaskStatus.TASK_SUCCEEDED
-      )
-      expect(tasks.results[0].indexUid).toEqual(index.uid)
-      expect(tasks.results[0].status).toEqual(TaskStatus.TASK_SUCCEEDED)
-      expect(tasks.results[0].type).toEqual(
-        TaskTypes.DOCUMENTS_ADDITION_OR_UPDATE
-      )
-      expect(tasks.results[0].enqueuedAt).toBeDefined()
-      expect(tasks.results[0].uid).toBeDefined()
-      expect(tasks.results[0].type).toEqual(
-        TaskTypes.DOCUMENTS_ADDITION_OR_UPDATE
+      const onlyTaskWithSameUid = new Set(
+        tasks.results.map((task) => task.indexUid)
       )
-      expect(tasks.results[0].duration).toBeDefined()
-      expect(tasks.results[0].finishedAt).toBeDefined()
-      expect(tasks.results[0].startedAt).toBeDefined()
+
+      expect(onlyTaskWithSameUid.size).toEqual(2)
     })
 
     test(`${permission} key: Try to get a task that does not exist`, async () => {