WIP addAssets

kuzzleio · Jun 2, 2023 · e943ed6 · e943ed6
1 parent d7b6dc4
commit e943ed6
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 9 deletions.
diff --git a/lib/modules/asset/AssetsGroupsController.ts b/lib/modules/asset/AssetsGroupsController.ts
@@ -225,13 +225,15 @@ export class AssetsGroupsController {
       _id
     );
 
-    // TODO implement addAsset on database
-    // eslint-disable-next-line no-console
-    console.debug({
-      _id,
-      action: request.getAction(),
-      body,
+    return await this.sdk.document.mUpdate(
       engineId,
-    });
+      InternalCollection.ASSETS,
+      body.assetId.map((assetId) => ({
+        _id: assetId,
+        body: {
+          group: _id,
+        },
+      }))
+    );
   }
 }
diff --git a/lib/modules/asset/collections/assetsMappings.ts b/lib/modules/asset/collections/assetsMappings.ts
@@ -14,6 +14,10 @@ export const assetsMappings = {
       type: "keyword",
       fields: { text: { type: "text" } },
     },
+    group: {
+      type: "keyword",
+      fields: { text: { type: "text" } },
+    },
 
     metadata: {
       properties: {

diff --git a/lib/modules/asset/types/AssetGroupsAPI.ts b/lib/modules/asset/types/AssetGroupsAPI.ts
@@ -1,4 +1,10 @@
-import { JSONObject, KDocument, KHit, SearchResult } from "kuzzle-sdk";
+import {
+  JSONObject,
+  KDocument,
+  KHit,
+  SearchResult,
+  mUpdateResponse,
+} from "kuzzle-sdk";
 import { AssetsGroupsBody, AssetsGroupContent } from "./AssetGroupContent";
 
 // Make "parent" property to optionnal for request
@@ -57,4 +63,4 @@ export interface ApiGroupAddAssetsRequest extends GroupControllerRequest {
     assetId: string[];
   };
 }
-export type ApiGroupAddAssetsResult = void;
+export type ApiGroupAddAssetsResult = mUpdateResponse;
diff --git a/tests/scenario/modules/assets/asset-group.test.ts b/tests/scenario/modules/assets/asset-group.test.ts
@@ -316,6 +316,7 @@ describe("AssetsGroupsController", () => {
       },
     });
 
+    // TODO improve tests
     expect({ error, status }).toStrictEqual({
       error: null,
       status: 200,