fix: Remove data from Mongo on deactivation (#1118)

open-amt-cloud-toolkit · Oct 13, 2023 · 40926e3 · 40926e3
1 parent af74746
commit 40926e3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/data/mongo/collections/device.test.ts b/src/data/mongo/collections/device.test.ts
@@ -64,13 +64,13 @@ describe('MongoDeviceTable', () => {
     expect(collection.findOne).toHaveBeenCalledWith({ guid: 'someId', tenantId: 'someTenantId' })
   })
 
-  it('should delete document by name and tenantId', async () => {
+  it('should delete document by guid and tenantId', async () => {
     collection.deleteOne.mockResolvedValue({ deletedCount: 1 } as any)
 
-    const result = await mongoDeviceTable.delete('someName', 'someTenantId')
+    const result = await mongoDeviceTable.delete('someGuid', 'someTenantId')
 
     expect(result).toBe(true)
-    expect(collection.deleteOne).toHaveBeenCalledWith({ friendlyName: 'someName', tenantId: 'someTenantId' })
+    expect(collection.deleteOne).toHaveBeenCalledWith({ guid: 'someGuid', tenantId: 'someTenantId' })
   })
 
   it('should insert a device', async () => {

diff --git a/src/data/mongo/collections/device.ts b/src/data/mongo/collections/device.ts
@@ -33,8 +33,8 @@ export class MongoDeviceTable implements IDeviceTable {
     return this.collection.findOne({ guid: id, tenantId }) as unknown as WithId<Device>
   }
 
-  async delete (name: string, tenantId: string = ''): Promise<boolean> {
-    const result = await this.collection.deleteOne({ friendlyName: name, tenantId })
+  async delete (id: string, tenantId: string = ''): Promise<boolean> {
+    const result = await this.collection.deleteOne({ guid: id, tenantId })
     return result.deletedCount && result.deletedCount > 0
   }