From aa3cb2270347a825ae1e50daa6db242b2bb535b4 Mon Sep 17 00:00:00 2001
From: Tom Moor <tom.moor@gmail.com>
Date: Mon, 3 May 2021 21:39:01 -0700
Subject: [PATCH] test: Fix tests around utils.gc

---
 server/api/utils.test.js | 41 +++++++++++++++++++++++++++++++++++-----
 1 file changed, 36 insertions(+), 5 deletions(-)

diff --git a/server/api/utils.test.js b/server/api/utils.test.js
index 752e8e25e..56f105fe4 100644
--- a/server/api/utils.test.js
+++ b/server/api/utils.test.js
@@ -20,6 +20,37 @@ beforeEach(() => flushdb());
 afterAll(() => server.close());
 
 describe("#utils.gc", () => {
+  it("should not destroy documents not deleted", async () => {
+    await buildDocument({
+      publishedAt: new Date(),
+    });
+
+    const res = await server.post("/api/utils.gc", {
+      body: {
+        token: process.env.UTILS_SECRET,
+      },
+    });
+
+    expect(res.status).toEqual(200);
+    expect(await Document.unscoped().count({ paranoid: false })).toEqual(1);
+  });
+
+  it("should not destroy documents deleted less than 30 days ago", async () => {
+    await buildDocument({
+      publishedAt: new Date(),
+      deletedAt: subDays(new Date(), 25),
+    });
+
+    const res = await server.post("/api/utils.gc", {
+      body: {
+        token: process.env.UTILS_SECRET,
+      },
+    });
+
+    expect(res.status).toEqual(200);
+    expect(await Document.unscoped().count({ paranoid: false })).toEqual(1);
+  });
+
   it("should destroy documents deleted more than 30 days ago", async () => {
     await buildDocument({
       publishedAt: new Date(),
@@ -33,7 +64,7 @@ describe("#utils.gc", () => {
     });
 
     expect(res.status).toEqual(200);
-    expect(await Document.scope().count()).toEqual(0);
+    expect(await Document.unscoped().count({ paranoid: false })).toEqual(0);
   });
 
   it("should destroy attachments no longer referenced", async () => {
@@ -58,7 +89,7 @@ describe("#utils.gc", () => {
 
     expect(res.status).toEqual(200);
     expect(await Attachment.count()).toEqual(0);
-    expect(await Document.scope().count()).toEqual(0);
+    expect(await Document.unscoped().count({ paranoid: false })).toEqual(0);
   });
 
   it("should handle unknown attachment ids", async () => {
@@ -88,7 +119,7 @@ describe("#utils.gc", () => {
 
     expect(res.status).toEqual(200);
     expect(await Attachment.count()).toEqual(0);
-    expect(await Document.scope().count()).toEqual(0);
+    expect(await Document.unscoped().count({ paranoid: false })).toEqual(0);
   });
 
   it("should not destroy attachments referenced in other documents", async () => {
@@ -121,7 +152,7 @@ describe("#utils.gc", () => {
 
     expect(res.status).toEqual(200);
     expect(await Attachment.count()).toEqual(1);
-    expect(await Document.scope().count()).toEqual(1);
+    expect(await Document.unscoped().count({ paranoid: false })).toEqual(1);
   });
 
   it("should destroy draft documents deleted more than 30 days ago", async () => {
@@ -136,7 +167,7 @@ describe("#utils.gc", () => {
       },
     });
     expect(res.status).toEqual(200);
-    expect(await Document.scope().count()).toEqual(0);
+    expect(await Document.unscoped().count({ paranoid: false })).toEqual(0);
   });
 
   it("should require authentication", async () => {