change scorecardid to support uuid

Author: ThomasKranitsas

src/services/ReviewService.js

@@ -98,7 +98,7 @@ const listReviewsQuerySchema = {
   score: joi.score(),
   typeId: joi.string().uuid(),
   reviewerId: joi.alternatives().try(joi.id(), joi.string().uuid()),
-  scoreCardId: joi.id(),
+  scoreCardId: joi.alternatives().try(joi.id(), joi.string().uuid()),
   submissionId: joi.string().uuid(),
   status: joi.reviewStatus(),
   page: joi.id(),
@@ -206,7 +206,7 @@ createReview.schema = {
         .error(errors => ({
           message: '"reviewerId" must be a number or a string'
         })),
-      scoreCardId: joi.id().required(),
+      scoreCardId: joi.alternatives().try(joi.id().required(), joi.string().uuid().required()),
       submissionId: joi
         .string()
         .uuid()
@@ -338,7 +338,7 @@ updateReview.schema = {
         .alternatives()
         .try(joi.id(), joi.string().uuid())
         .required(),
-      scoreCardId: joi.id().required(),
+      scoreCardId: joi.alternatives().try(joi.id().required(), joi.string().uuid().required()),
       submissionId: joi
         .string()
         .uuid()
@@ -371,7 +371,7 @@ patchReview.schema = {
     score: joi.score(),
     typeId: joi.string().uuid(),
     reviewerId: joi.alternatives().try(joi.id(), joi.string().uuid()),
-    scoreCardId: joi.id(),
+    scoreCardId: joi.alternatives().try(joi.id(), joi.string().uuid()),
     submissionId: joi.string().uuid(),
     status: joi.reviewStatus(),
     metadata: joi.object(),

src/services/ReviewSummationService.js

@@ -59,7 +59,7 @@ function * listReviewSummations (query) {
 }
 
 const listReviewSummationsQuerySchema = {
-  scoreCardId: joi.id(),
+  scoreCardId: joi.alternatives().try(joi.id(), joi.string().uuid()),
   submissionId: joi.string().uuid(),
   aggregateScore: joi.score(),
   isPassing: joi.boolean(),
@@ -138,7 +138,7 @@ function * createReviewSummation (authUser, entity) {
 createReviewSummation.schema = {
   authUser: joi.object().required(),
   entity: joi.object().keys({
-    scoreCardId: joi.id().required(),
+    scoreCardId: joi.alternatives().try(joi.id().required(), joi.string().uuid().required()),
     submissionId: joi.string().uuid().required(),
     aggregateScore: joi.score().required(),
     isPassing: joi.boolean().required(),
@@ -263,7 +263,7 @@ updateReviewSummation.schema = {
   authUser: joi.object().required(),
   reviewSummationId: joi.string().uuid().required(),
   entity: joi.object().keys({
-    scoreCardId: joi.id().required(),
+    scoreCardId: joi.alternatives().try(joi.id().required(), joi.string().uuid().required()),
     submissionId: joi.string().uuid().required(),
     aggregateScore: joi.score().required(),
     isPassing: joi.boolean().required(),
@@ -288,7 +288,7 @@ patchReviewSummation.schema = {
   authUser: joi.object().required(),
   reviewSummationId: joi.string().uuid().required(),
   entity: joi.object().keys({
-    scoreCardId: joi.id(),
+    scoreCardId: joi.alternatives().try(joi.id(), joi.string().uuid()),
     submissionId: joi.string().uuid(),
     aggregateScore: joi.score(),
     isPassing: joi.boolean(),