chore(groups): fix tests after delete controller modification

Author: QuentinRousselet

plugin/lib/modules/group/GroupsService.ts

@@ -339,7 +339,7 @@ export class GroupsService extends BaseService {
     await this.sdk.document.mDelete(
       engineId,
       InternalCollection.GROUPS,
-      [...childrenGroups.map((g) => g._id), _id],
+      childrenGroups.map((g) => g._id),
       { strict: true, triggerEvents: true },
     );
 

plugin/tests/scenario/modules/groups/group-permissions.test.ts

@@ -30,7 +30,6 @@ import {
   ApiGroupSearchResult,
   ApiGroupUpdateRequest,
 } from "lib/modules/group/types/GroupsApi";
-import { GroupContent } from "lib/modules/group/types/GroupContent";
 
 describe("GroupsController", () => {
   const sdk = setupHooks();
@@ -188,17 +187,15 @@ describe("GroupsController", () => {
       _id: groupTestParentId1,
     });
 
-    const { _source: childrenGroup } = await sdk.document.get<GroupContent>(
-      "engine-ayse",
-      InternalCollection.GROUPS,
-      groupTestChildrenId1,
+    const deletedChildQuery = {
+      controller: "device-manager/groups",
+      action: "get",
+      engineId: "engine-ayse",
+      _id: groupTestChildrenId1,
+    };
+    await expect(sdk.query(deletedChildQuery)).rejects.toThrow(
+      /test-children-1/,
     );
-
-    expect(childrenGroup).toMatchObject({
-      path: groupTestChildrenId1,
-    });
-    expect(childrenGroup.lastUpdate).toBeGreaterThanOrEqual(now);
-
     await sdk.query<ApiGroupDeleteRequest>({
       controller: "device-manager/groups",
       engineId: "engine-ayse",
@@ -212,7 +209,7 @@ describe("GroupsController", () => {
       "Container-grouped",
     );
 
-    expect(assetGrouped).toMatchObject({
+    expect(assetGrouped).not.toMatchObject({
       groups: [
         {
           path: groupChildrenWithAssetId,

plugin/tests/scenario/modules/groups/group.test.ts

@@ -385,18 +385,16 @@ describe("GroupsController", () => {
       action: "delete",
       _id: groupTestParentId1,
     });
-
-    const { _source: childrenGroup } = await sdk.document.get<GroupContent>(
-      "engine-ayse",
-      InternalCollection.GROUPS,
-      groupTestChildrenId1,
+    await sdk.collection.refresh("engine-ayse", "groups");
+    const deletedChildQuery = {
+      controller: "device-manager/groups",
+      action: "get",
+      engineId: "engine-ayse",
+      _id: groupTestChildrenId1,
+    };
+    await expect(sdk.query(deletedChildQuery)).rejects.toThrow(
+      /test-children-1/,
     );
-
-    expect(childrenGroup).toMatchObject({
-      path: groupTestChildrenId1,
-    });
-    expect(childrenGroup.lastUpdate).toBeGreaterThanOrEqual(now);
-
     await sdk.query<ApiGroupDeleteRequest>({
       controller: "device-manager/groups",
       engineId: "engine-ayse",
@@ -410,7 +408,7 @@ describe("GroupsController", () => {
       "Container-grouped",
     );
 
-    expect(assetGrouped).toMatchObject({
+    expect(assetGrouped).not.toMatchObject({
       groups: [
         {
           path: groupChildrenWithAssetId,