Merge pull request #102 from topcoder-platform/feature/update-job-start-date-format

support Job Start Date format without time

Author: maxceem

package-lock.json

@@ -18,12 +18,13 @@
     "superagent": "^5.1.0"
   },
   "dependencies": {
-    "@hapi/joi": "^15.1.0",
+    "@joi/date": "^2.1.0",
     "async-mutex": "^0.2.4",
     "bluebird": "^3.5.5",
     "config": "^3.1.0",
     "dotenv": "^10.0.0",
     "get-parameter-names": "^0.3.0",
+    "joi": "^17.4.2",
     "lodash": "^4.17.20",
     "no-kafka": "^3.4.3",
     "tc-core-library-js": "^2.4.1",

package.json

@@ -1,4 +1,4 @@
-const Joi = require('@hapi/joi')
+const Joi = require('joi')
 const config = require('config')
 const constants = require('./common/constants')
 

src/bootstrap.js

@@ -3,7 +3,7 @@
  */
 
 const _ = require('lodash')
-const Joi = require('@hapi/joi')
+const Joi = require('joi')
 const util = require('util')
 const config = require('config')
 const getParams = require('get-parameter-names')

src/common/logger.js

@@ -2,7 +2,7 @@
  * Jobcandidate Processor Service
  */
 
-const Joi = require('@hapi/joi')
+const Joi = require('joi')
 const logger = require('../common/logger')
 const helper = require('../common/helper')
 const constants = require('../common/constants')
@@ -74,8 +74,9 @@ async function processUpdate (message, transactionId) {
   })
 }
 
-processUpdate.schema = {
-  message: Joi.object()
+processUpdate.schema = Joi.object()
+  .keys({
+    message: Joi.object()
     .keys({
       topic: Joi.string().required(),
       originator: Joi.string().required(),
@@ -100,8 +101,8 @@ processUpdate.schema = {
         .required()
     })
     .required(),
-  transactionId: Joi.string().required()
-}
+    transactionId: Joi.string().required()
+  })
 
 module.exports = {
   processUpdate

src/services/JobCandidateProcessorService.js

@@ -2,7 +2,7 @@
  * Job Processor Service
  */
 
-const Joi = require('@hapi/joi')
+const Joi = require('joi').extend(require('@joi/date'))
 const logger = require('../common/logger')
 const helper = require('../common/helper')
 const constants = require('../common/constants')
@@ -50,49 +50,50 @@ async function processCreate (message, transactionId) {
   })
 }
 
-processCreate.schema = {
-  message: Joi.object()
-    .keys({
-      topic: Joi.string().required(),
-      originator: Joi.string().required(),
-      timestamp: Joi.date().required(),
-      'mime-type': Joi.string().required(),
-      payload: Joi.object()
-        .keys({
-          id: Joi.string().uuid().required(),
-          projectId: Joi.number().integer().required(),
-          externalId: Joi.string().allow(null),
-          description: Joi.stringAllowEmpty().allow(null),
-          title: Joi.title().required(),
-          startDate: Joi.date().allow(null),
-          duration: Joi.number().integer().min(1).allow(null),
-          numPositions: Joi.number().integer().min(1).required(),
-          resourceType: Joi.stringAllowEmpty().allow(null),
-          rateType: Joi.rateType().allow(null),
-          workload: Joi.workload().allow(null),
-          skills: Joi.array().items(Joi.string().uuid()).required(),
-          roles: Joi.array().items(Joi.string().uuid()).allow(null),
-          createdAt: Joi.date().required(),
-          createdBy: Joi.string().uuid().required(),
-          updatedAt: Joi.date().allow(null),
-          updatedBy: Joi.string().uuid().allow(null),
-          status: Joi.jobStatus().required(),
-          isApplicationPageActive: Joi.boolean().required(),
-          minSalary: Joi.number().integer().allow(null),
-          maxSalary: Joi.number().integer().allow(null),
-          hoursPerWeek: Joi.number().integer().allow(null),
-          jobLocation: Joi.stringAllowEmpty().allow(null),
-          jobTimezone: Joi.stringAllowEmpty().allow(null),
-          currency: Joi.stringAllowEmpty().allow(null),
-          roleIds: Joi.array()
-            .items(Joi.string().uuid().required())
-            .allow(null)
-        })
-        .required()
-    })
-    .required(),
-  transactionId: Joi.string().required()
-}
+processCreate.schema = Joi.object()
+  .keys({
+    message: Joi.object()
+      .keys({
+        topic: Joi.string().required(),
+        originator: Joi.string().required(),
+        timestamp: Joi.date().required(),
+        'mime-type': Joi.string().required(),
+        payload: Joi.object()
+          .keys({
+            id: Joi.string().uuid().required(),
+            projectId: Joi.number().integer().required(),
+            externalId: Joi.string().allow(null),
+            description: Joi.stringAllowEmpty().allow(null),
+            title: Joi.title().required(),
+            startDate: Joi.date().format('YYYY-MM-DD').allow(null),
+            duration: Joi.number().integer().min(1).allow(null),
+            numPositions: Joi.number().integer().min(1).required(),
+            resourceType: Joi.stringAllowEmpty().allow(null),
+            rateType: Joi.rateType().allow(null),
+            workload: Joi.workload().allow(null),
+            skills: Joi.array().items(Joi.string().uuid()).required(),
+            roles: Joi.array().items(Joi.string().uuid()).allow(null),
+            createdAt: Joi.date().required(),
+            createdBy: Joi.string().uuid().required(),
+            updatedAt: Joi.date().allow(null),
+            updatedBy: Joi.string().uuid().allow(null),
+            status: Joi.jobStatus().required(),
+            isApplicationPageActive: Joi.boolean().required(),
+            minSalary: Joi.number().integer().allow(null),
+            maxSalary: Joi.number().integer().allow(null),
+            hoursPerWeek: Joi.number().integer().allow(null),
+            jobLocation: Joi.stringAllowEmpty().allow(null),
+            jobTimezone: Joi.stringAllowEmpty().allow(null),
+            currency: Joi.stringAllowEmpty().allow(null),
+            roleIds: Joi.array()
+              .items(Joi.string().uuid().required())
+              .allow(null)
+          })
+          .required()
+      })
+      .required(),
+    transactionId: Joi.string().required()
+  })
 
 /**
  * Process update entity message