Merge branch 'dev' into code-cleanup/remove-experimentusercontroller-…

…issue-2084

backend/packages/Upgrade/src/api/DTO/ExperimentDTO.ts

@@ -10,6 +10,7 @@ import {
   IsObject,
   IsOptional,
   IsString,
+  IsUUID,
   MaxLength,
   MinLength,
   ValidateIf,
@@ -458,6 +459,12 @@ export class ExperimentDTO {
   public type: EXPERIMENT_TYPE;
 }
 
+export class ExperimentIdValidator {
+  @IsNotEmpty()
+  @IsUUID()
+  public id: string;
+}
+
 export interface ExperimentFile {
   fileName: string;
   fileContent: string;

backend/packages/Upgrade/src/api/controllers/ExperimentClientController.v5.ts

@@ -106,7 +106,7 @@ export class ExperimentClientController {
     public metricService: MetricService
   ) {}
 
-  private async checkIfUserExist(
+  public async checkIfUserExist(
     userId: string,
     logger: UpgradeLogger,
     api?: string

backend/packages/Upgrade/src/api/controllers/ExperimentController.ts

@@ -3,7 +3,6 @@ import {
   Get,
   JsonController,
   OnUndefined,
-  Param,
   Post,
   Put,
   Delete,
@@ -17,8 +16,6 @@ import { Experiment } from '../models/Experiment';
 import { ExperimentNotFoundError } from '../errors/ExperimentNotFoundError';
 import { ExperimentService } from '../services/ExperimentService';
 import { ExperimentAssignmentService } from '../services/ExperimentAssignmentService';
-import { SERVER_ERROR } from 'upgrade_types';
-import { isUUID } from 'class-validator';
 import { ExperimentCondition } from '../models/ExperimentCondition';
 import { ExperimentPaginatedParamsValidator } from './validators/ExperimentPaginatedParamsValidator';
 import { UserDTO } from '../DTO/UserDTO';
@@ -28,7 +25,7 @@ import { AppRequest, PaginationResponse } from '../../types';
 import { ExperimentDTO, ExperimentFile, ValidatedExperimentError } from '../DTO/ExperimentDTO';
 import { ExperimentIds } from './validators/ExperimentIdsValidator';
 import { NotFoundException } from '@nestjs/common/exceptions';
-import { IdValidator } from './validators/FeatureFlagValidator';
+import { ExperimentIdValidator } from '../DTO/ExperimentDTO';
 
 interface ExperimentPaginationInfo extends PaginationResponse {
   nodes: Experiment[];
@@ -634,7 +631,7 @@ export class ExperimentController {
    *            description: AuthorizationRequiredError
    */
   @Get()
-  public find(@Req() request: AppRequest): Promise<Experiment[]> {
+  public find(@Req() request: AppRequest): Promise<ExperimentDTO[]> {
     return this.experimentService.find(request.logger);
   }
 
@@ -743,14 +740,6 @@ export class ExperimentController {
     @Req()
     request: AppRequest
   ): Promise<ExperimentPaginationInfo> {
-    if (!paginatedParams) {
-      return Promise.reject(
-        new Error(
-          JSON.stringify({ type: SERVER_ERROR.MISSING_PARAMS, message: ' : paginatedParams should not be null.' })
-        )
-      );
-    }
-
     const [experiments, count] = await Promise.all([
       this.experimentService.findPaginated(
         paginatedParams.skip,
@@ -841,7 +830,10 @@ export class ExperimentController {
    */
   @Get('/single/:id')
   @OnUndefined(ExperimentNotFoundError)
-  public one(@Params({ validate: true }) { id }: IdValidator, @Req() request: AppRequest): Promise<Experiment> {
+  public one(
+    @Params({ validate: true }) { id }: ExperimentIdValidator,
+    @Req() request: AppRequest
+  ): Promise<ExperimentDTO> {
     return this.experimentService.getSingleExperiment(id, request.logger);
   }
 
@@ -920,7 +912,7 @@ export class ExperimentController {
   @Get('/conditions/:id')
   @OnUndefined(ExperimentNotFoundError)
   public async getCondition(
-    @Params({ validate: true }) { id }: IdValidator,
+    @Params({ validate: true }) { id }: ExperimentIdValidator,
     @Req() request: AppRequest
   ): Promise<ExperimentCondition[]> {
     return this.experimentService.getExperimentalConditions(id, request.logger);
@@ -1042,7 +1034,7 @@ export class ExperimentController {
 
   @Delete('/:id')
   public async delete(
-    @Params({ validate: true }) { id }: IdValidator,
+    @Params({ validate: true }) { id }: ExperimentIdValidator,
     @CurrentUser() currentUser: UserDTO,
     @Req() request: AppRequest
   ): Promise<Experiment | undefined> {
@@ -1140,21 +1132,14 @@ export class ExperimentController {
    */
   @Put('/:id')
   public update(
-    @Param('id') id: string,
+    @Params({ validate: true }) { id }: ExperimentIdValidator,
     @Body({ validate: true })
     experiment: ExperimentDTO,
     @CurrentUser() currentUser: UserDTO,
     @Req() request: AppRequest
-  ): Promise<Experiment> {
-    if (!isUUID(id)) {
-      return Promise.reject(
-        new Error(
-          JSON.stringify({ type: SERVER_ERROR.INCORRECT_PARAM_FORMAT, message: ' : id should be of type UUID.' })
-        )
-      );
-    }
+  ): Promise<ExperimentDTO> {
     request.logger.child({ user: currentUser });
-    return this.experimentService.update(experiment, currentUser, request.logger);
+    return this.experimentService.update({ ...experiment, id }, currentUser, request.logger);
   }
 
   /**
@@ -1345,7 +1330,7 @@ export class ExperimentController {
   @Get('/getGroupAssignmentStatus/:id')
   @OnUndefined(ExperimentNotFoundError)
   public async getGroupAssignmentStatus(
-    @Params({ validate: true }) { id }: IdValidator,
+    @Params({ validate: true }) { id }: ExperimentIdValidator,
     @Req() request: AppRequest
   ): Promise<number> | undefined {
     return this.experimentAssignmentService.getGroupAssignmentStatus(id, request.logger);

backend/packages/Upgrade/src/api/controllers/FeatureFlagController.ts

@@ -251,14 +251,6 @@ export class FeatureFlagsController {
     paginatedParams: FeatureFlagPaginatedParamsValidator,
     @Req() request: AppRequest
   ): Promise<FeatureFlagsPaginationInfo> {
-    if (!paginatedParams) {
-      return Promise.reject(
-        new Error(
-          JSON.stringify({ type: SERVER_ERROR.MISSING_PARAMS, message: ' : paginatedParams should not be null.' })
-        )
-      );
-    }
-
     const [featureFlags, count] = await Promise.all([
       this.featureFlagService.findPaginated(
         paginatedParams.skip,
@@ -453,7 +445,7 @@ export class FeatureFlagsController {
     @CurrentUser() currentUser: UserDTO,
     @Req() request: AppRequest
   ): Promise<FeatureFlag> {
-    return this.featureFlagService.update(flag, currentUser, request.logger);
+    return this.featureFlagService.update({ ...flag, id }, currentUser, request.logger);
   }
 
   /**

backend/packages/Upgrade/src/api/controllers/MetricController.ts

@@ -1,8 +1,8 @@
-import { Authorized, JsonController, Get, Delete, Param, Post, Req, Body, Params } from 'routing-controllers';
+import { Authorized, JsonController, Get, Delete, Post, Req, Body, Params } from 'routing-controllers';
 import { MetricService } from '../services/MetricService';
 import { IMetricUnit } from 'upgrade_types';
 import { AppRequest } from '../../types';
-import { MetricKeyValidator, MetricValidator } from './validators/MetricValidator';
+import { ContextValidator, MetricKeyValidator, MetricValidator } from './validators/MetricValidator';
 
 /**
  * @swagger
@@ -56,7 +56,10 @@ export class MetricController {
    *            description: Metrics Context not found
    */
   @Get('/:context')
-  public getMetricsByContext(@Param('context') context: string, @Req() request: AppRequest): Promise<IMetricUnit[]> {
+  public getMetricsByContext(
+    @Params({ validate: true }) { context }: ContextValidator,
+    @Req() request: AppRequest
+  ): Promise<IMetricUnit[]> {
     return this.metricService.getMetricsByContext(context, request.logger);
   }
 

backend/packages/Upgrade/src/api/controllers/PreviewUserController.ts

@@ -1,12 +1,22 @@
-import { JsonController, Get, OnUndefined, Param, Post, Put, Body, Authorized, Delete, Req } from 'routing-controllers';
+import {
+  JsonController,
+  Get,
+  OnUndefined,
+  Post,
+  Put,
+  Body,
+  Authorized,
+  Delete,
+  Req,
+  Params,
+} from 'routing-controllers';
 import { UserNotFoundError } from '../errors/UserNotFoundError';
-import { SERVER_ERROR } from 'upgrade_types';
 import { PreviewUserService } from '../services/PreviewUserService';
 import { PreviewUser } from '../models/PreviewUser';
-import { isString } from 'class-validator';
 import { PaginatedParamsValidator } from './validators/PaginatedParamsValidator';
 import { AppRequest, PaginationResponse } from '../../types';
 import { PreviewUserValidator } from './validators/PreviewUserValidator';
+import { IdValidator } from './validators/ExperimentUserValidator';
 
 interface PreviewUserPaginationInfo extends PaginationResponse {
   nodes: PreviewUser[];
@@ -83,14 +93,6 @@ export class PreviewUserController {
     @Body({ validate: true }) paginatedParams: PaginatedParamsValidator,
     @Req() request: AppRequest
   ): Promise<PreviewUserPaginationInfo> {
-    if (!paginatedParams) {
-      return Promise.reject(
-        new Error(
-          JSON.stringify({ type: SERVER_ERROR.MISSING_PARAMS, message: ' : paginatedParams should not be null.' })
-        )
-      );
-    }
-
     const [previewUsers, count] = await Promise.all([
       this.previewUserService.findPaginated(paginatedParams.skip, paginatedParams.take, request.logger),
       this.previewUserService.getTotalCount(request.logger),
@@ -126,14 +128,10 @@ export class PreviewUserController {
    */
   @Get('/:id')
   @OnUndefined(UserNotFoundError)
-  public one(@Param('id') id: string, @Req() request: AppRequest): Promise<PreviewUser | undefined> {
-    if (!isString(id)) {
-      return Promise.reject(
-        new Error(
-          JSON.stringify({ type: SERVER_ERROR.INCORRECT_PARAM_FORMAT, message: ' : id should be of type string.' })
-        )
-      );
-    }
+  public one(
+    @Params({ validate: true }) { id }: IdValidator,
+    @Req() request: AppRequest
+  ): Promise<PreviewUser | undefined> {
     return this.previewUserService.findOne(id, request.logger);
   }
 
@@ -161,7 +159,10 @@ export class PreviewUserController {
    *            description: New ExperimentUser is created
    */
   @Post()
-  public create(@Body({ validate: true }) users: PreviewUserValidator, @Req() request: AppRequest): Promise<PreviewUser> {
+  public create(
+    @Body({ validate: true }) users: PreviewUserValidator,
+    @Req() request: AppRequest
+  ): Promise<PreviewUser> {
     return this.previewUserService.create(users, request.logger);
   }
 
@@ -196,7 +197,7 @@ export class PreviewUserController {
    */
   @Put('/:id')
   public update(
-    @Param('id') id: string,
+    @Params({ validate: true }) { id }: IdValidator,
     @Body({ validate: true }) user: PreviewUserValidator,
     @Req() request: AppRequest
   ): Promise<PreviewUser> {
@@ -224,7 +225,10 @@ export class PreviewUserController {
    *            description: Delete User By Id
    */
   @Delete('/:id')
-  public delete(@Param('id') id: string, @Req() request: AppRequest): Promise<PreviewUser | undefined> {
+  public delete(
+    @Params({ validate: true }) { id }: IdValidator,
+    @Req() request: AppRequest
+  ): Promise<PreviewUser | undefined> {
     return this.previewUserService.delete(id, request.logger);
   }
 
@@ -252,7 +256,10 @@ export class PreviewUserController {
    *            description: Assignment is created
    */
   @Post('/assign')
-  public assign(@Body({ validate: true }) user: PreviewUserValidator, @Req() request: AppRequest): Promise<PreviewUser> {
+  public assign(
+    @Body({ validate: true }) user: PreviewUserValidator,
+    @Req() request: AppRequest
+  ): Promise<PreviewUser> {
     return this.previewUserService.upsertExperimentConditionAssignment(user, request.logger);
   }
 }

backend/packages/Upgrade/src/api/controllers/SegmentController.ts

@@ -1,24 +1,12 @@
-import {
-  JsonController,
-  Get,
-  Delete,
-  Param,
-  Authorized,
-  Post,
-  Req,
-  Body,
-  QueryParams,
-  Params,
-} from 'routing-controllers';
+import { JsonController, Get, Delete, Authorized, Post, Req, Body, QueryParams, Params } from 'routing-controllers';
 import { SegmentService, SegmentWithStatus } from '../services/SegmentService';
 import { Segment } from '../models/Segment';
-import { SERVER_ERROR } from 'upgrade_types';
-import { isUUID } from 'class-validator';
 import { AppRequest } from '../../types';
 import {
   IdValidator,
   SegmentFile,
   SegmentIds,
+  SegmentIdValidator,
   SegmentImportError,
   SegmentInputValidator,
 } from './validators/SegmentInputValidator';
@@ -383,17 +371,10 @@ export class SegmentController {
    *          description: Internal Server Error, SegmentId is not valid
    */
   @Delete('/:segmentId')
-  public deleteSegment(@Param('segmentId') segmentId: string, @Req() request: AppRequest): Promise<Segment> {
-    if (!segmentId) {
-      return Promise.reject(new Error(SERVER_ERROR.MISSING_PARAMS + ' : segmentId should not be null.'));
-    }
-    if (!isUUID(segmentId)) {
-      return Promise.reject(
-        new Error(
-          JSON.stringify({ type: SERVER_ERROR.INCORRECT_PARAM_FORMAT, message: ' : segmentId should be of type UUID.' })
-        )
-      );
-    }
+  public deleteSegment(
+    @Params({ validate: true }) { segmentId }: SegmentIdValidator,
+    @Req() request: AppRequest
+  ): Promise<Segment> {
     return this.segmentService.deleteSegment(segmentId, request.logger);
   }
 

backend/packages/Upgrade/src/api/controllers/StratificationController.ts

@@ -1,9 +1,12 @@
-import { JsonController, Get, Delete, Param, Authorized, Req, Res, Body, Post } from 'routing-controllers';
-import { SERVER_ERROR } from 'upgrade_types';
+import { JsonController, Get, Delete, Authorized, Req, Res, Body, Post, Params } from 'routing-controllers';
 import { AppRequest } from '../../types';
 import { UserStratificationFactor } from '../models/UserStratificationFactor';
 import { StratificationService } from '../services/StratificationService';
-import { FactorStrata, UploadedFilesArrayValidator } from './validators/StratificationValidator';
+import {
+  FactorStrata,
+  StratificationFactorValidator,
+  UploadedFilesArrayValidator,
+} from './validators/StratificationValidator';
 import { StratificationFactor } from '../models/StratificationFactor';
 import * as express from 'express';
 
@@ -110,16 +113,11 @@ export class StratificationController {
    */
   @Get('/download/:factor')
   public async getStratificationByFactor(
-    @Param('factor') factor: string,
+    @Params({ validate: true }) { factor }: StratificationFactorValidator,
     @Req() request: AppRequest,
     @Res() res: express.Response
   ): Promise<express.Response> {
-    if (!factor) {
-      return Promise.reject(new Error(SERVER_ERROR.MISSING_PARAMS + ' : stratification Factor should not be null.'));
-    }
-
     const csvData = await this.stratificationService.getCSVDataByFactor(factor, request.logger);
-
     // return csv file with appropriate headers to request;
     res.setHeader('Content-Type', 'text/csv; charset=UTF-8');
     res.setHeader('Content-Disposition', `attachment; filename="${factor}.csv"`);
@@ -190,12 +188,9 @@ export class StratificationController {
    */
   @Delete('/:factor')
   public deleteStratification(
-    @Param('factor') stratificationFactor: string,
+    @Params({ validate: true }) { factor }: StratificationFactorValidator,
     @Req() request: AppRequest
   ): Promise<StratificationFactor> {
-    if (!stratificationFactor) {
-      return Promise.reject(new Error(SERVER_ERROR.MISSING_PARAMS + ' : stratification Factor should not be null.'));
-    }
-    return this.stratificationService.deleteStratification(stratificationFactor, request.logger);
+    return this.stratificationService.deleteStratification(factor, request.logger);
   }
 }