winning submission from challenge 30088714 - Topcoder Connect - Add add-ons flag

Author: maxceem

migrations/20190418_productTemplates_isAddOn.sql

@@ -0,0 +1,12 @@
+--
+-- UPDATE EXISTING TABLES:
+--   product_templates:
+--     added column `isAddOn`
+
+--
+-- product_templates
+
+-- Add new column
+ALTER TABLE product_templates ADD COLUMN "isAddOn" boolean DEFAULT false;
+-- Update new column
+UPDATE product_templates SET "isAddOn"='true' WHERE "subCategory" != "category";

src/models/productTemplate.js

@@ -18,6 +18,7 @@ module.exports = (sequelize, DataTypes) => {
     deletedAt: DataTypes.DATE,
     disabled: { type: DataTypes.BOOLEAN, defaultValue: false },
     hidden: { type: DataTypes.BOOLEAN, defaultValue: false },
+    isAddOn: { type: DataTypes.BOOLEAN, defaultValue: false },
     createdAt: { type: DataTypes.DATE, defaultValue: DataTypes.NOW },
     updatedAt: { type: DataTypes.DATE, defaultValue: DataTypes.NOW },
     deletedBy: DataTypes.BIGINT,

src/routes/productTemplates/create.js

@@ -26,6 +26,7 @@ const schema = {
       template: Joi.object().required(),
       disabled: Joi.boolean().optional(),
       hidden: Joi.boolean().optional(),
+      isAddOn: Joi.boolean().optional(),
       createdAt: Joi.any().strip(),
       updatedAt: Joi.any().strip(),
       deletedAt: Joi.any().strip(),

src/routes/productTemplates/create.spec.js

@@ -42,6 +42,7 @@ describe('CREATE product template', () => {
         aliases: ['product key 1', 'product_key_1'],
         disabled: true,
         hidden: true,
+        isAddOn: true,
         template: {
           template1: {
             name: 'template 1',

src/routes/productTemplates/list.spec.js

@@ -15,7 +15,7 @@ const validateProductTemplates = (count, resJson, expectedTemplates) => {
   resJson.should.have.length(count);
   resJson.forEach((pt, idx) => {
     pt.should.have.all.keys('id', 'name', 'productKey', 'category', 'subCategory', 'icon', 'brief', 'details',
-    'aliases', 'template', 'disabled', 'hidden', 'createdBy', 'createdAt', 'updatedBy', 'updatedAt');
+    'aliases', 'template', 'disabled', 'hidden', 'isAddOn', 'createdBy', 'createdAt', 'updatedBy', 'updatedAt');
     pt.should.not.have.all.keys('deletedAt', 'deletedBy');
     pt.name.should.be.eql(expectedTemplates[idx].name);
     pt.productKey.should.be.eql(expectedTemplates[idx].productKey);
@@ -30,6 +30,7 @@ const validateProductTemplates = (count, resJson, expectedTemplates) => {
     pt.updatedBy.should.be.eql(expectedTemplates[idx].updatedBy);
     pt.disabled.should.be.eql(_.get(expectedTemplates[idx], 'disabled', false));
     pt.hidden.should.be.eql(_.get(expectedTemplates[idx], 'hidden', false));
+    pt.isAddOn.should.be.eql(_.get(expectedTemplates[idx], 'isAddOn', false));
   });
 };
 
@@ -52,6 +53,7 @@ describe('LIST product templates', () => {
       },
       disabled: true,
       hidden: true,
+      isAddOn: true,
       template: {
         template1: {
           name: 'template 1',

src/routes/productTemplates/update.js

@@ -29,6 +29,7 @@ const schema = {
       template: Joi.object(),
       disabled: Joi.boolean().optional(),
       hidden: Joi.boolean().optional(),
+      isAddOn: Joi.boolean().optional(),
       createdAt: Joi.any().strip(),
       updatedAt: Joi.any().strip(),
       deletedAt: Joi.any().strip(),

src/routes/productTemplates/update.spec.js

@@ -22,6 +22,7 @@ describe('UPDATE product template', () => {
     aliases: ['productTemplate-1', 'productTemplate_1'],
     disabled: true,
     hidden: true,
+    isAddOn: true,
     template: {
       template1: {
         name: 'template 1',

swagger.yaml

@@ -3703,6 +3703,9 @@ definitions:
       template:
         type: object
         description: the product template template
+      isAddOn:
+        type: boolean
+        description: the flag that shows if the product template is an add on
   ProductTemplateBodyParam:
     title: Product template body param
     type: object