fix: don't fetch site user model, allow admin rebuild

Author: drewbo

api/admin/controllers/build.js

@@ -106,7 +106,7 @@ module.exports = wrapHandlers({
       include: [
         {
           model: Site,
-          include: [User, Domain, SiteBranchConfig],
+          include: [Domain, SiteBranchConfig],
         },
       ],
     });

test/api/admin/requests/build.test.js

@@ -10,14 +10,15 @@ const { authenticatedAdminOrSupportSession } = require('../../support/session');
 const factory = require('../../support/factory');
 const csrfToken = require('../../support/csrfToken');
 const config = require('../../../../config');
-const { Site, User, BuildLog } = require('../../../../api/models');
+const { Site, User, BuildLog, Organization } = require('../../../../api/models');
 const sessionConfig = require('../../../../api/admin/sessionConfig');
 const app = require('../../../../api/admin');
+const { createSiteUserOrg } = require('../../support/site-user');
 
 const itShouldRequireAdminAuthentication = (path, schema, method = 'get') => {
   it('should require admin authentication', async () => {
     const response = await request(app)[method](path).expect(401);
-    validateAgainstJSONSchema('GET', schema, 401, response.body);
+    validateAgainstJSONSchema(method, schema, 401, response.body);
     expect(response.body.message).to.equal('Unauthorized');
   });
 };
@@ -40,7 +41,7 @@ const buildResponseExpectations = (response, build) => {
 const s3Mock = mockClient(S3Client);
 
 describe('Admin - Site API', () => {
-  afterEach(() => Promise.all([User.truncate(), Site.truncate()]));
+  afterEach(() => Promise.all([User.truncate(), Site.truncate(), Organization.truncate()]));
 
   describe('GET /admin/builds', () => {
     itShouldRequireAdminAuthentication('/builds', '/site/{site_id}/build');
@@ -135,6 +136,27 @@ describe('Admin - Site API', () => {
     });
   });
 
+  describe('POST /builds', () => {
+    describe('rebuild', () => {
+      it('rebuilds a specified build', async () => {
+        const { site, user } = await createSiteUserOrg();
+        const build = await factory.build({ site });
+
+        const cookie = await authenticatedAdminOrSupportSession(user, sessionConfig);
+        await request(app)
+          .post(`/builds`)
+          .set('Cookie', cookie)
+          .set('Origin', config.app.adminHostname)
+          .set('x-csrf-token', csrfToken.getToken())
+          .send({
+            buildId: build.id,
+            siteId: site.id,
+          })
+          .expect(200);
+      });
+    });
+  });
+
   describe('GET /builds/:id/log', () => {
     itShouldRequireAdminAuthentication('/builds/1/log', '/build/{build_id}/log');