Issue#2494: disassociate device on delete (#125)

* disassociate device on delete

* refactor: fixing the promise to disassociate each device Id from all certificates it is bound to

* add promise.all on disassociateCertificates

Co-authored-by: luisfelipefurlan <[email protected]>

Author: LeonamMoura

src/operations/device/mutation.deleteDevices.js

@@ -1,15 +1,29 @@
-import LOG from '../../utils/Log.js';
-import * as service from '../../services/service.device.js';
+import LOG from "../../utils/Log.js";
+import * as service from "../../services/service.device.js";
+import * as securityService from "../../services/service.security";
 
 const deleteDevices = async (_, { deviceIds }, { token }) => {
   try {
-    const promises = deviceIds.map(
-      async (deviceId) => {
-        await service.deleteDevice(token, deviceId);
-      },
-    );
-    await Promise.all(promises);
-    return 'ok';
+    const disassociateCertificatesPromise = deviceIds.map(async (deviceId) => {
+      const { data: certificateData } = await securityService
+        .getAllCertificates(
+          token,
+          undefined,
+          undefined,
+          deviceId,
+        );
+      const { certificates } = certificateData;
+      const disassociateLinkedCertificates = certificates.map(async (certificate) => {
+        await securityService.disassociateCertificate(token, certificate.fingerprint);
+      });
+      await Promise.all(disassociateLinkedCertificates);
+    });
+    const deleteDevicesPromise = deviceIds.map(async (deviceId) => {
+      await service.deleteDevice(token, deviceId);
+    });
+    await Promise.all(disassociateCertificatesPromise);
+    await Promise.all(deleteDevicesPromise);
+    return "ok";
   } catch (error) {
     LOG.error(error.stack || error);
     throw error;