feat(pmp): edit repository in settings - backend

refs #107

libs/server/repository/api-rest/src/lib/controllers/repository.controller.ts

@@ -1,4 +1,4 @@
-import { Body, Controller, Delete, Get, Param, Post, UseGuards } from '@nestjs/common';
+import { Body, Controller, Delete, Get, Param, Post, Put, UseGuards } from '@nestjs/common';
 import { ApiBearerAuth, ApiTags } from '@nestjs/swagger';
 import {
   AuthGuard,
@@ -10,12 +10,14 @@ import {
 import {
   AddRepositoryCommand,
   DeleteRepositoryCommand,
+  EditRepositoryCommand,
   RepositoryFacade
 } from '@pimp-my-pr/server/repository/core/application-services';
 import { RepositoryEntity } from '@pimp-my-pr/server/repository/core/domain';
 import { extractFullName } from '@pimp-my-pr/server/shared/util-repository';
 import { AddRepositoryDto } from '../dtos/add-repository.dto';
 import { UserRepositoryGuard } from '../guards/user-repository.guard';
+import { EditRepositoryDto } from '../dtos/edit-repository.dto';
 
 @ApiTags('repository')
 @ApiBearerAuth()
@@ -52,4 +54,19 @@ export class RepositoryController {
   delete(@Param('repositoryId') repositoryId: string): Promise<void> {
     return this.repositoryFacade.deleteRepository(new DeleteRepositoryCommand(repositoryId));
   }
+
+  @UseGuards(UserRepositoryGuard)
+  @Put(':repositoryId')
+  edit(
+    @Param('repositoryId') repositoryId: string,
+    @Body() editRepositoryDto: EditRepositoryDto
+  ): Promise<void> {
+    return this.repositoryFacade.editRepository(
+      new EditRepositoryCommand(
+        repositoryId,
+        editRepositoryDto.maxLines,
+        editRepositoryDto.maxWaitingTime
+      )
+    );
+  }
 }

libs/server/repository/api-rest/src/lib/dtos/edit-repository.dto.ts

@@ -0,0 +1,14 @@
+import { ApiProperty } from '@nestjs/swagger';
+import { IsNumber, IsOptional } from 'class-validator';
+
+export class EditRepositoryDto {
+  @ApiProperty()
+  @IsNumber()
+  @IsOptional()
+  maxLines?: number;
+
+  @ApiProperty()
+  @IsNumber()
+  @IsOptional()
+  maxWaitingTime?: number;
+}

libs/server/repository/core/application-services/src/index.ts

@@ -8,3 +8,4 @@ export * from './lib/queries/list-repositories-statistics/repositories-statistic
 export * from './lib/read-models/reviewer-model-with-pr.interface';
 export * from './lib/commands/add-repository/add-repository.command';
 export * from './lib/commands/delete-repository/delete-repository.command';
+export * from './lib/commands/edit-repository/edit-repository.command';

libs/server/repository/core/application-services/src/lib/commands/edit-repository/edit-repository.command.ts

@@ -0,0 +1,7 @@
+export class EditRepositoryCommand {
+  constructor(
+    public repositoryId: string,
+    public maxLines?: number,
+    public maxWaitingTime?: number
+  ) {}
+}

libs/server/repository/core/application-services/src/lib/commands/edit-repository/edit-repository.handler.spec.ts

@@ -0,0 +1,51 @@
+import { EditRepositoryHandler } from './edit-repository.handler';
+import { EditRepositoryCommand } from './edit-repository.command';
+import { RepositoryRepository } from '@pimp-my-pr/server/repository/core/domain-services';
+import { Test } from '@nestjs/testing';
+import { RepositoryEntity } from '@pimp-my-pr/server/repository/core/domain';
+import Mocked = jest.Mocked;
+
+describe('Edit Repository Handler', () => {
+  let editRepositoryHandler: EditRepositoryHandler;
+  let repositoryRepository: Mocked<RepositoryRepository>;
+  const repository = new RepositoryEntity('123', 'test', 'test', 'test', '1');
+
+  beforeEach(async () => {
+    const module = await Test.createTestingModule({
+      providers: [
+        {
+          provide: RepositoryRepository,
+          useValue: {}
+        },
+        EditRepositoryHandler
+      ]
+    }).compile();
+
+    repositoryRepository = module.get(RepositoryRepository);
+    editRepositoryHandler = module.get<EditRepositoryHandler>(EditRepositoryHandler);
+  });
+
+  describe('update', () => {
+    it('execute save repository', async () => {
+      const command: EditRepositoryCommand = {
+        repositoryId: '123',
+        maxLines: 2,
+        maxWaitingTime: 3
+      };
+
+      repositoryRepository.getById = jest.fn();
+      repositoryRepository.getById.mockResolvedValue(repository);
+
+      repository.edit = jest.fn();
+      repositoryRepository.save = jest.fn();
+
+      await editRepositoryHandler.execute(command);
+      expect(repository.edit).toHaveBeenCalledWith({
+        maxLines: command.maxLines,
+        maxWaitingTime: command.maxWaitingTime
+      });
+
+      expect(repositoryRepository.save).toHaveBeenCalledWith(repository);
+    });
+  });
+});

libs/server/repository/core/application-services/src/lib/commands/edit-repository/edit-repository.handler.ts

@@ -0,0 +1,18 @@
+import { CommandHandler, ICommandHandler } from '@nestjs/cqrs';
+import { RepositoryRepository } from '@pimp-my-pr/server/repository/core/domain-services';
+import { EditRepositoryCommand } from './edit-repository.command';
+
+@CommandHandler(EditRepositoryCommand)
+export class EditRepositoryHandler implements ICommandHandler<EditRepositoryCommand> {
+  constructor(private repositoryRepository: RepositoryRepository) {}
+
+  async execute(command: EditRepositoryCommand): Promise<void> {
+    const { repositoryId, maxLines, maxWaitingTime } = command;
+
+    const repository = await this.repositoryRepository.getById(repositoryId);
+
+    repository.edit({ maxLines, maxWaitingTime });
+
+    return this.repositoryRepository.save(repository);
+  }
+}

libs/server/repository/core/application-services/src/lib/repository.facade.ts

@@ -3,6 +3,7 @@ import { CommandBus, QueryBus } from '@nestjs/cqrs';
 
 import {
   DeleteRepositoryCommand,
+  EditRepositoryCommand,
   RepositoriesStatisticsItemReadModel,
   ReviewersStatisticsItemReadModel,
   ReviewerStatisticsReadModel
@@ -28,6 +29,10 @@ export class RepositoryFacade {
     return this.commandBus.execute(command);
   }
 
+  editRepository(command: EditRepositoryCommand): Promise<void> {
+    return this.commandBus.execute(command);
+  }
+
   getRepositoryStatistics(
     repositoryId: string,
     token: string,

libs/server/repository/core/application-services/src/lib/server-repository-core-application-services.module.ts

@@ -8,10 +8,12 @@ import { ListRepositoriesHandler } from './queries/list-repositories/list-reposi
 import { ListReviewersStatisticsHandler } from './queries/list-reviewers-statistics/list-reviewers-statistics.handler';
 import { RepositoryFacade } from './repository.facade';
 import { DeleteRepositoryHandler } from './commands/delete-repository/delete-repository.handler';
+import { EditRepositoryHandler } from './commands/edit-repository/edit-repository.handler';
 
 const QueryHandlers = [
   AddRepositoryHandler,
   DeleteRepositoryHandler,
+  EditRepositoryHandler,
   GetRepositoryStatisticsHandler,
   GetReviewerStatisticsHandler,
   ListRepositoriesStatisticsHandler,

libs/server/repository/core/domain/src/lib/entities/repository.entity.ts

@@ -1,5 +1,6 @@
 import { f } from '@marcj/marshal';
 import { PrEntity } from './pr.entity';
+import { RepositoryEditWriteModel } from '@pimp-my-pr/shared/domain';
 
 export class RepositoryEntity {
   @f.primary()
@@ -50,4 +51,9 @@ export class RepositoryEntity {
     this.maxWaitingTime = maxWaitingTime;
     this.maxLines = maxLines;
   }
+
+  edit(writeModel: RepositoryEditWriteModel): void {
+    this.maxLines = writeModel.maxLines;
+    this.maxWaitingTime = writeModel.maxWaitingTime;
+  }
 }

libs/shared/domain/src/index.ts

@@ -9,6 +9,7 @@ export * from './lib/interfaces/pr-author.interface';
 export * from './lib/interfaces/pr-reviewer.interface';
 export * from './lib/interfaces/pr.statistics';
 export * from './lib/interfaces/repository.model';
+export * from './lib/interfaces/repository-edit-write.model';
 export * from './lib/interfaces/repository-statistics.interface';
 export * from './lib/read-models/user.read-model';
 export * from './lib/responses/api-repository-statistics.response';

libs/shared/domain/src/lib/interfaces/repository-edit-write.model.ts

@@ -0,0 +1,4 @@
+export interface RepositoryEditWriteModel {
+  maxLines?: number;
+  maxWaitingTime?: number;
+}