Enables external attachments in DocumentUsage

Author: Spoffy

test/nbrowser/DocumentUsage.ts

@@ -14,9 +14,18 @@ describe('DocumentUsage', function() {
   let api: UserAPI;
   let session: gu.Session;
 
-  before(async function () {
-    session = await gu.session().user(ownerUser).login();
+  gu.enableExternalAttachments();
+
+  async function makeSessionAndLogin() {
+    // login() needs an options object passing to bypass an optimization that causes .login()
+    // to think we're already logged in when we're not after using `server.restart()`.
+    // Without this we end up with bad credentials.
+    session = await gu.session().user(ownerUser).login({ retainExistingLogin: false });
     api = session.createHomeApi();
+  }
+
+  before(async function () {
+    await makeSessionAndLogin();
   });
 
   it('shows usage stats on the raw data page', async function() {
@@ -105,11 +114,15 @@ describe('DocumentUsage', function() {
     });
   });
 
-  describe('attachment usage', function() {
+  function testAttachments(docSuffix: string) {
     let docId: string;
 
     before(async () => {
-      docId = await session.tempNewDoc(cleanup, "AttachmentUsageTestDoc");
+      docId = await session.tempNewDoc(cleanup, `AttachmentUsageTestDoc - ${docSuffix}`);
+    });
+
+    it('updates attachments size usage when uploading attachments', async function () {
+      docId = await session.tempNewDoc(cleanup, `AttachmentUsageTestDoc - ${docSuffix}`);
     });
 
     it('updates attachments size usage when uploading attachments', async function () {
@@ -147,6 +160,14 @@ describe('DocumentUsage', function() {
       await assertDataSize('0.00');
       await assertAttachmentsSize('0.00');
     });
+  }
+
+  describe('attachment usage without external attachments', function() {
+    testAttachments("without external attachments");
+  });
+
+  describe('attachment usage with external attachments', function() {
+    testAttachments("with external attachments");
   });
 
   describe('doc usage access', function() {