Merge pull request #298 from topcoder-platform/feature/project-product-templates-final-fixes

vikasrohit · web-flow · commit 40628db8d66e · 2019-04-30T14:32:05.000+05:30
Feature/project product templates final fixes
diff --git a/src/routes/productTemplates/create.js b/src/routes/productTemplates/create.js
@@ -23,8 +23,11 @@ const schema = {
       brief: Joi.string().max(45).required(),
       details: Joi.string().max(255).required(),
       aliases: Joi.array().required(),
-      template: Joi.object(),
-      form: Joi.object(),
+      template: Joi.object().empty(null),
+      form: Joi.object().keys({
+        key: Joi.string().required(),
+        version: Joi.number(),
+      }).empty(null),
       disabled: Joi.boolean().optional(),
       hidden: Joi.boolean().optional(),
       isAddOn: Joi.boolean().optional(),
@@ -34,7 +37,9 @@ const schema = {
       createdBy: Joi.any().strip(),
       updatedBy: Joi.any().strip(),
       deletedBy: Joi.any().strip(),
-    }).required(),
+    })
+      .xor('form', 'template')
+      .required(),
   },
 };
 
diff --git a/src/routes/productTemplates/create.spec.js b/src/routes/productTemplates/create.spec.js
@@ -89,13 +89,18 @@ describe('CREATE product template', () => {
       },
     };
 
-    const bodyWithForm = _.cloneDeep(body);
-    bodyWithForm.param.form = {
+    const bodyDefinedFormTemplate = _.cloneDeep(body);
+    bodyDefinedFormTemplate.param.form = {
       version: 1,
       key: 'dev',
     };
+
+    const bodyWithForm = _.cloneDeep(bodyDefinedFormTemplate);
     delete bodyWithForm.param.template;
 
+    const bodyMissingFormTemplate = _.cloneDeep(bodyWithForm);
+    delete bodyMissingFormTemplate.param.form;
+
     const bodyInvalidForm = _.cloneDeep(body);
     bodyInvalidForm.param.form = {
       version: 1,
@@ -277,5 +282,27 @@ describe('CREATE product template', () => {
         .send(bodyInvalidForm)
         .expect(422, done);
     });
+
+    it('should return 422 if both form or template field are defined', (done) => {
+      request(server)
+        .post('/v4/projects/metadata/productTemplates')
+        .set({
+          Authorization: `Bearer ${testUtil.jwts.admin}`,
+        })
+        .send(bodyDefinedFormTemplate)
+        .expect('Content-Type', /json/)
+        .expect(422, done);
+    });
+
+    it('should return 422 if both form or template field are missing', (done) => {
+      request(server)
+        .post('/v4/projects/metadata/productTemplates')
+        .set({
+          Authorization: `Bearer ${testUtil.jwts.admin}`,
+        })
+        .send(bodyMissingFormTemplate)
+        .expect('Content-Type', /json/)
+        .expect(422, done);
+    });
   });
 });
diff --git a/src/routes/productTemplates/update.js b/src/routes/productTemplates/update.js
@@ -26,8 +26,11 @@ const schema = {
       brief: Joi.string().max(45),
       details: Joi.string().max(255),
       aliases: Joi.array(),
-      template: Joi.object(),
-      form: Joi.object(),
+      template: Joi.object().empty(null),
+      form: Joi.object().keys({
+        key: Joi.string().required(),
+        version: Joi.number(),
+      }).empty(null),
       disabled: Joi.boolean().optional(),
       hidden: Joi.boolean().optional(),
       isAddOn: Joi.boolean().optional(),
@@ -37,7 +40,9 @@ const schema = {
       createdBy: Joi.any().strip(),
       updatedBy: Joi.any().strip(),
       deletedBy: Joi.any().strip(),
-    }).required(),
+    })
+      .xor('form', 'template')
+      .required(),
   },
 };
 
diff --git a/src/routes/productTemplates/update.spec.js b/src/routes/productTemplates/update.spec.js
@@ -133,12 +133,18 @@ describe('UPDATE product template', () => {
       },
     };
 
-    const bodyWithForm = _.cloneDeep(body);
-    bodyWithForm.param.form = {
+    const bodyDefinedFormTemplate = _.cloneDeep(body);
+    bodyDefinedFormTemplate.param.form = {
       version: 1,
       key: 'dev',
     };
 
+    const bodyWithForm = _.cloneDeep(bodyDefinedFormTemplate);
+    delete bodyWithForm.param.template;
+
+    const bodyMissingFormTemplate = _.cloneDeep(bodyWithForm);
+    delete bodyMissingFormTemplate.param.form;
+
     const bodyInvalidForm = _.cloneDeep(body);
     bodyInvalidForm.param.form = {
       version: 1,
@@ -312,5 +318,25 @@ describe('UPDATE product template', () => {
         .send(bodyInvalidForm)
         .expect(422, done);
     });
+
+    it('should return 422 if both form or template field are defined', (done) => {
+      request(server)
+        .patch(`/v4/projects/metadata/productTemplates/${templateId}`)
+        .set({
+          Authorization: `Bearer ${testUtil.jwts.admin}`,
+        })
+        .send(bodyDefinedFormTemplate)
+        .expect(422, done);
+    });
+
+    it('should return 422 if both form or template field are missing', (done) => {
+      request(server)
+        .patch(`/v4/projects/metadata/productTemplates/${templateId}`)
+        .set({
+          Authorization: `Bearer ${testUtil.jwts.admin}`,
+        })
+        .send(bodyMissingFormTemplate)
+        .expect(422, done);
+    });
   });
 });
diff --git a/src/routes/projectTemplates/create.js b/src/routes/projectTemplates/create.js
@@ -22,11 +22,20 @@ const schema = {
       question: Joi.string().max(255).required(),
       info: Joi.string().max(255).required(),
       aliases: Joi.array().required(),
-      scope: Joi.object().optional().allow(null),
-      phases: Joi.object().optional().allow(null),
-      form: Joi.object().optional().allow(null),
-      planConfig: Joi.object().optional().allow(null),
-      priceConfig: Joi.object().optional().allow(null),
+      scope: Joi.object().empty(null),
+      phases: Joi.object().empty(null),
+      form: Joi.object().keys({
+        key: Joi.string().required(),
+        version: Joi.number(),
+      }).empty(null),
+      planConfig: Joi.object().keys({
+        key: Joi.string().required(),
+        version: Joi.number(),
+      }).empty(null),
+      priceConfig: Joi.object().keys({
+        key: Joi.string().required(),
+        version: Joi.number(),
+      }).empty(null),
       disabled: Joi.boolean().optional(),
       hidden: Joi.boolean().optional(),
       createdAt: Joi.any().strip(),
@@ -35,7 +44,11 @@ const schema = {
       createdBy: Joi.any().strip(),
       updatedBy: Joi.any().strip(),
       deletedBy: Joi.any().strip(),
-    }).required(),
+    })
+      .xor('form', 'scope')
+      .xor('phases', 'planConfig')
+      .nand('priceConfig', 'scope')
+      .required(),
   },
 };
 
@@ -47,54 +60,12 @@ module.exports = [
     const param = req.body.param;
     const { form, priceConfig, planConfig } = param;
 
-    const checkModel = (keyInfo, modelName, model) => {
-      let errorMessage = '';
-      if (keyInfo == null) {
-        return Promise.resolve(null);
-      }
-      if ((keyInfo.version != null) && (keyInfo.key != null)) {
-        errorMessage = `${modelName} with key ${keyInfo.key} and version ${keyInfo.version}`
-          + ' referred in the project template is not found';
-        return (model.findOne({
-          where: {
-            key: keyInfo.key,
-            version: keyInfo.version,
-          },
-        })).then((record) => {
-          if (record == null) {
-            return Promise.resolve(errorMessage);
-          }
-          return Promise.resolve(null);
-        });
-      } else if ((keyInfo.version == null) && (keyInfo.key != null)) {
-        errorMessage = `${modelName} with key ${keyInfo.key}`
-          + ' referred in the project template is not found';
-        return model.findOne({
-          where: {
-            key: keyInfo.key,
-          },
-        }).then((record) => {
-          if (record == null) {
-            return Promise.resolve(errorMessage);
-          }
-          return Promise.resolve(null);
-        });
-      }
-      return Promise.resolve(null);
-    };
-
     return Promise.all([
-      checkModel(form, 'Form', models.Form, next),
-      checkModel(priceConfig, 'PriceConfig', models.PriceConfig, next),
-      checkModel(planConfig, 'PlanConfig', models.PlanConfig, next),
+      util.checkModel(form, 'Form', models.Form, 'project template'),
+      util.checkModel(priceConfig, 'PriceConfig', models.PriceConfig, 'project template'),
+      util.checkModel(planConfig, 'PlanConfig', models.PlanConfig, 'project template'),
     ])
-      .then((errorMessages) => {
-        const errorMessage = errorMessages.find(e => e && e.length > 0);
-        if (errorMessage) {
-          const apiErr = new Error(errorMessage);
-          apiErr.status = 422;
-          throw apiErr;
-        }
+      .then(() => {
         const entity = _.assign(req.body.param, {
           createdBy: req.authUser.userId,
           updatedBy: req.authUser.userId,
diff --git a/src/routes/projectTemplates/create.spec.js b/src/routes/projectTemplates/create.spec.js
@@ -12,23 +12,51 @@ import testUtil from '../../tests/util';
 const should = chai.should();
 
 describe('CREATE project template', () => {
-  before((done) => {
-    testUtil.clearDb()
-      .then(() => models.ProjectType.bulkCreate([
-        {
-          key: 'generic',
-          displayName: 'Generic',
-          icon: 'http://example.com/icon1.ico',
-          question: 'question 1',
-          info: 'info 1',
-          aliases: ['key-1', 'key_1'],
-          metadata: {},
-          createdBy: 1,
-          updatedBy: 1,
-        },
-      ]))
-      .then(() => done());
-  });
+  before(() => testUtil.clearDb()
+    .then(() => models.ProjectType.bulkCreate([
+      {
+        key: 'generic',
+        displayName: 'Generic',
+        icon: 'http://example.com/icon1.ico',
+        question: 'question 1',
+        info: 'info 1',
+        aliases: ['key-1', 'key_1'],
+        metadata: {},
+        createdBy: 1,
+        updatedBy: 1,
+      },
+    ]))
+    .then(() => models.Form.create({
+      key: 'test',
+      config: {
+        test: 'test1',
+      },
+      version: 1,
+      revision: 1,
+      createdBy: 1,
+      updatedBy: 1,
+    }))
+    .then(() => models.PlanConfig.create({
+      key: 'test',
+      config: {
+        test: 'test1',
+      },
+      version: 1,
+      revision: 1,
+      createdBy: 1,
+      updatedBy: 1,
+    }))
+    .then(() => models.PriceConfig.create({
+      key: 'test',
+      config: {
+        test: 'test1',
+      },
+      version: 1,
+      revision: 1,
+      createdBy: 1,
+      updatedBy: 1,
+    })),
+  );
 
   describe('POST /projects/metadata/projectTemplates', () => {
     const body = {
@@ -80,34 +108,29 @@ describe('CREATE project template', () => {
         disabled: true,
         hidden: true,
         form: {
-          scope1: {
-            subScope1A: 1,
-            subScope1B: 2,
-          },
-          scope2: [1, 2, 3],
+          key: 'test',
+          version: 1,
         },
         priceConfig: {
-          first: '$800',
+          key: 'test',
         },
         planConfig: {
-          phase1: {
-            name: 'phase 1',
-            details: {
-              anyDetails: 'any details 1',
-            },
-            others: ['others 11', 'others 12'],
-          },
-          phase2: {
-            name: 'phase 2',
-            details: {
-              anyDetails: 'any details 2',
-            },
-            others: ['others 21', 'others 22'],
-          },
+          key: 'test',
         },
       },
     };
 
+    const bodyDefinedFormScope = _.cloneDeep(body);
+    bodyDefinedFormScope.param.form = {
+      scope1: {
+        subScope1A: 1,
+        subScope1B: 2,
+      },
+      scope2: [1, 2, 3],
+    };
+    const bodyMissingFormScope = _.cloneDeep(body);
+    delete bodyMissingFormScope.param.scope;
+
     it('should return 403 if user is not authenticated', (done) => {
       request(server)
         .post('/v4/projects/metadata/projectTemplates')
@@ -270,5 +293,27 @@ describe('CREATE project template', () => {
           done();
         });
     });
+
+    it('should return 422 if both scope and form are defined', (done) => {
+      request(server)
+        .post('/v4/projects/metadata/projectTemplates')
+        .set({
+          Authorization: `Bearer ${testUtil.jwts.admin}`,
+        })
+        .send(bodyDefinedFormScope)
+        .expect('Content-Type', /json/)
+        .expect(422, done);
+    });
+
+    it('should return 422 if both scope and form are missing', (done) => {
+      request(server)
+        .post('/v4/projects/metadata/projectTemplates')
+        .set({
+          Authorization: `Bearer ${testUtil.jwts.admin}`,
+        })
+        .send(bodyMissingFormScope)
+        .expect('Content-Type', /json/)
+        .expect(422, done);
+    });
   });
 });
diff --git a/src/routes/projectTemplates/update.js b/src/routes/projectTemplates/update.js
diff --git a/src/routes/projectTemplates/update.spec.js b/src/routes/projectTemplates/update.spec.js
diff --git a/src/routes/projectTemplates/upgrade.js b/src/routes/projectTemplates/upgrade.js
