From ae3d83eb5c81d2a178bf9e2ca7e1e52dbecc6565 Mon Sep 17 00:00:00 2001
From: Mathieu Marchois <mathieu.marchois@gmail.com>
Date: Fri, 5 Mar 2021 16:12:58 +0100
Subject: [PATCH] Remove leave request

---
 client/i18n/fr.json                           |  1 +
 client/src/components/links/DeleteLink.svelte | 21 +++++
 .../leaves/requests/_Table.svelte             | 13 ++-
 .../leaves/requests/index.svelte              | 15 ++-
 .../Command/DeleteLeaveRequestCommand.ts      |  9 ++
 .../DeleteLeaveRequestCommandHandler.spec.ts  | 94 +++++++++++++++++++
 .../DeleteLeaveRequestCommandHandler.ts       | 33 +++++++
 .../LeaveRequestCantBeRemovedException.ts     |  5 +
 .../Repository/ILeaveRequestRepository.ts     |  1 +
 .../CanLeaveRequestBeRemoved.spec.ts          | 41 ++++++++
 .../Specification/CanLeaveRequestBeRemoved.ts |  8 ++
 .../Leave/Action/DeleteLeaveRequestAction.ts  | 39 ++++++++
 .../Repository/LeaveRequestRepository.ts      |  4 +
 .../HumanResource/humanResource.module.ts     |  8 +-
 14 files changed, 288 insertions(+), 4 deletions(-)
 create mode 100644 client/src/components/links/DeleteLink.svelte
 create mode 100644 server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommand.ts
 create mode 100644 server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.spec.ts
 create mode 100644 server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.ts
 create mode 100644 server/src/Domain/HumanResource/Leave/Exception/LeaveRequestCantBeRemovedException.ts
 create mode 100644 server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.spec.ts
 create mode 100644 server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.ts
 create mode 100644 server/src/Infrastructure/HumanResource/Leave/Action/DeleteLeaveRequestAction.ts

diff --git a/client/i18n/fr.json b/client/i18n/fr.json
index 42cbd401..49812193 100644
--- a/client/i18n/fr.json
+++ b/client/i18n/fr.json
@@ -261,6 +261,7 @@
         "moderate_by": "Modéré par",
         "end_date": "Date de fin",
         "start_date": "Date de début",
+        "confirm": "Etes-vous sûr de vouloir supprimer cette demande de congé ?",
         "add": {
           "title": "Faire une demande de congé"
         },
diff --git a/client/src/components/links/DeleteLink.svelte b/client/src/components/links/DeleteLink.svelte
new file mode 100644
index 00000000..ab4b0494
--- /dev/null
+++ b/client/src/components/links/DeleteLink.svelte
@@ -0,0 +1,21 @@
+<script>
+  import { _ } from 'svelte-i18n';
+  import { createEventDispatcher } from 'svelte';
+  import TrashIcon from '../icons/TrashIcon.svelte';
+
+  export let confirmMessage = 'common.confirm.remove';
+  const dispatch = createEventDispatcher();
+
+  const handleClick = () => {
+    if (confirm($_(confirmMessage))) {
+      dispatch('confirm');
+    }
+  }
+</script>
+
+<button
+  on:click={handleClick}
+  class="flex items-center justify-between px-2 text-sm font-medium leading-5 text-purple-600 rounded-lg dark:text-gray-400 focus:outline-none focus:shadow-outline-gray"
+  aria-label={$_('common.form.remove')}>
+  <TrashIcon className={'w-5 h-5'} />
+</button>
diff --git a/client/src/routes/human_resources/leaves/requests/_Table.svelte b/client/src/routes/human_resources/leaves/requests/_Table.svelte
index 2f7807eb..7bdf4b11 100644
--- a/client/src/routes/human_resources/leaves/requests/_Table.svelte
+++ b/client/src/routes/human_resources/leaves/requests/_Table.svelte
@@ -1,16 +1,22 @@
 <script>
+  import { stores } from '@sapper/app';
   import { format } from 'date-fns';
+  import { createEventDispatcher } from 'svelte';
   import { _ } from 'svelte-i18n';
   import SeeLink from '../../../../components/links/SeeLink.svelte';
   import RedBadge from '../../../../components/badges/RedBadge.svelte';
   import OrangeBadge from '../../../../components/badges/OrangeBadge.svelte';
   import GrayBadge from '../../../../components/badges/GrayBadge.svelte';
+  import DeleteLink from '../../../../components/links/DeleteLink.svelte';
+
+  export let items;
 
   const formatDate = (date) => {
     return format(new Date(date), 'dd/MM/yyyy');
   };
 
-  export let items;
+  const dispatch = createEventDispatcher();
+  const { session } = stores();
 </script>
 
 <table class="w-full whitespace-no-wrap">
@@ -53,6 +59,11 @@
         <td class="px-4 py-3">
           <div class="flex items-center space-x-4 text-sm">
             <SeeLink href={`/human_resources/leaves/requests/${id}`} />
+            {#if $session.user && $session.user.id === user.id}
+              <DeleteLink 
+                on:confirm={() => dispatch('delete', id)} 
+                confirmMessage={$_("human_resources.leaves.requests.confirm")} />
+            {/if}
           </div>
         </td>
       </tr>
diff --git a/client/src/routes/human_resources/leaves/requests/index.svelte b/client/src/routes/human_resources/leaves/requests/index.svelte
index 7a0d3926..43398120 100644
--- a/client/src/routes/human_resources/leaves/requests/index.svelte
+++ b/client/src/routes/human_resources/leaves/requests/index.svelte
@@ -9,7 +9,7 @@
 <script>
   import { onMount } from 'svelte';
   import { _ } from 'svelte-i18n';
-  import { get } from '../../../../utils/axios';
+  import { get, del } from '../../../../utils/axios';
   import { historyPushState } from '../../../../utils/url';
   import { errorNormalizer } from '../../../../normalizer/errors';
   import H4Title from '../../../../components/H4Title.svelte';
@@ -39,6 +39,17 @@
     fetchLeaveRequests();
   };
 
+  const handleDelete = async (event) => {
+    const id = event.detail;
+
+    try {
+      await del(`leave-requests/${id}`);
+      response.items = response.items.filter((leaveRequest) => leaveRequest.id !== id);
+    } catch (e) {
+      errors = errorNormalizer(e);
+    }
+  }
+
   const fetchLeaveRequests = async () => {
     try {
       response = (await get('leave-requests', { params: { page } })).data;
@@ -67,7 +78,7 @@
 </div>
 <div class="w-full overflow-hidden rounded-lg shadow-xs">
   <div class="w-full overflow-x-auto">
-    <Table items="{response.items}" />
+    <Table items="{response.items}" on:delete={handleDelete} />
   </div>
   <Pagination
     on:change="{changePage}"
diff --git a/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommand.ts b/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommand.ts
new file mode 100644
index 00000000..397e49c1
--- /dev/null
+++ b/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommand.ts
@@ -0,0 +1,9 @@
+import { ICommand } from 'src/Application/ICommand';
+import { User } from 'src/Domain/HumanResource/User/User.entity';
+
+export class DeleteLeaveRequestCommand implements ICommand {
+  constructor(
+    public readonly id: string,
+    public readonly owner: User
+  ) {}
+}
diff --git a/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.spec.ts b/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.spec.ts
new file mode 100644
index 00000000..da289ada
--- /dev/null
+++ b/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.spec.ts
@@ -0,0 +1,94 @@
+import { mock, instance, when, verify, anything, deepEqual } from 'ts-mockito';
+import { DeleteLeaveRequestCommandHandler } from './DeleteLeaveRequestCommandHandler';
+import { DeleteLeaveRequestCommand } from './DeleteLeaveRequestCommand';
+import { User } from 'src/Domain/HumanResource/User/User.entity';
+import { LeaveRequestNotFoundException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestNotFoundException';
+import { CanLeaveRequestBeRemoved } from 'src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved';
+import { LeaveRequest } from 'src/Domain/HumanResource/Leave/LeaveRequest.entity';
+import { LeaveRequestRepository } from 'src/Infrastructure/HumanResource/Leave/Repository/LeaveRequestRepository';
+import { LeaveRequestCantBeRemovedException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestCantBeRemovedException';
+
+describe('DeleteLeaveRequestCommandHandler', () => {
+  let leaveRequestRepository: LeaveRequestRepository;
+  let canLeaveRequestBeRemoved: CanLeaveRequestBeRemoved;
+  let handler: DeleteLeaveRequestCommandHandler;
+
+  const user = mock(User);
+  const leaveRequest = mock(LeaveRequest);
+  const command = new DeleteLeaveRequestCommand(
+    'cfdd06eb-cd71-44b9-82c6-46110b30ce05',
+    instance(user)
+  );
+
+  beforeEach(() => {
+    leaveRequestRepository = mock(LeaveRequestRepository);
+    canLeaveRequestBeRemoved = mock(CanLeaveRequestBeRemoved);
+
+    handler = new DeleteLeaveRequestCommandHandler(
+      instance(leaveRequestRepository),
+      instance(canLeaveRequestBeRemoved),
+    );
+  });
+
+  it('testLeaveRequestNotNotFound', async () => {
+    when(
+      leaveRequestRepository.findOneById('cfdd06eb-cd71-44b9-82c6-46110b30ce05')
+    ).thenResolve(null);
+
+    try {
+      expect(await handler.execute(command)).toBeUndefined();
+    } catch (e) {
+      expect(e).toBeInstanceOf(LeaveRequestNotFoundException);
+      expect(e.message).toBe('human_resources.leaves.requests.errors.not_found');
+      verify(
+        leaveRequestRepository.findOneById('cfdd06eb-cd71-44b9-82c6-46110b30ce05')
+      ).once();
+      verify(
+        canLeaveRequestBeRemoved.isSatisfiedBy(anything(), anything())
+      ).never();
+      verify(leaveRequestRepository.remove(anything())).never();
+    }
+  });
+
+  it('testLeaveRequestCantBeRemoved', async () => {
+    when(
+      leaveRequestRepository.findOneById('cfdd06eb-cd71-44b9-82c6-46110b30ce05')
+    ).thenResolve(instance(leaveRequest));
+    when(
+      canLeaveRequestBeRemoved.isSatisfiedBy(instance(leaveRequest), instance(user))
+    ).thenReturn(false);
+
+    try {
+      expect(await handler.execute(command)).toBeUndefined();
+    } catch (e) {
+      expect(e).toBeInstanceOf(LeaveRequestCantBeRemovedException);
+      expect(e.message).toBe('human_resources.leaves.requests.errors.cant_be_removed');
+      verify(
+        canLeaveRequestBeRemoved.isSatisfiedBy(instance(leaveRequest), instance(user))
+      ).once();
+      verify(
+        leaveRequestRepository.findOneById('cfdd06eb-cd71-44b9-82c6-46110b30ce05')
+      ).once();
+      verify(leaveRequestRepository.remove(anything())).never();
+    }
+  });
+
+  it('testLeaveSuccessfullyDeleted', async () => {
+    when(
+      leaveRequestRepository.findOneById('cfdd06eb-cd71-44b9-82c6-46110b30ce05')
+    ).thenResolve(instance(leaveRequest));
+    when(
+      canLeaveRequestBeRemoved.isSatisfiedBy(instance(leaveRequest), instance(user))
+    ).thenReturn(true);
+
+    expect(await handler.execute(command)).toBeUndefined();
+
+    verify(
+      canLeaveRequestBeRemoved.isSatisfiedBy(instance(leaveRequest), instance(user))
+    ).once();
+    verify(
+      leaveRequestRepository.findOneById('cfdd06eb-cd71-44b9-82c6-46110b30ce05')
+    ).once();
+    verify(leaveRequestRepository.remove(instance(leaveRequest))).once();
+  });
+});
diff --git a/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.ts b/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.ts
new file mode 100644
index 00000000..e12651ce
--- /dev/null
+++ b/server/src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler.ts
@@ -0,0 +1,33 @@
+import { Inject } from '@nestjs/common';
+import { CommandHandler } from '@nestjs/cqrs';
+import { DeleteLeaveRequestCommand } from './DeleteLeaveRequestCommand';
+import { ILeaveRequestRepository } from 'src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository';
+import { LeaveRequestNotFoundException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestNotFoundException';
+import { CanLeaveRequestBeRemoved } from 'src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved';
+import { LeaveRequestCantBeRemovedException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestCantBeRemovedException';
+
+@CommandHandler(DeleteLeaveRequestCommand)
+export class DeleteLeaveRequestCommandHandler {
+  constructor(
+    @Inject('ILeaveRequestRepository')
+    private readonly leaveRequestRepository: ILeaveRequestRepository,
+    private readonly canLeaveRequestBeRemoved: CanLeaveRequestBeRemoved
+  ) {}
+
+  public async execute(command: DeleteLeaveRequestCommand): Promise<void> {
+    const { owner, id } = command;
+
+    const leaveRequest = await this.leaveRequestRepository.findOneById(id);
+    if (!leaveRequest) {
+      throw new LeaveRequestNotFoundException();
+    }
+
+    if (
+      false === this.canLeaveRequestBeRemoved.isSatisfiedBy(leaveRequest, owner)
+    ) {
+      throw new LeaveRequestCantBeRemovedException();
+    }
+
+    this.leaveRequestRepository.remove(leaveRequest);
+  }
+}
diff --git a/server/src/Domain/HumanResource/Leave/Exception/LeaveRequestCantBeRemovedException.ts b/server/src/Domain/HumanResource/Leave/Exception/LeaveRequestCantBeRemovedException.ts
new file mode 100644
index 00000000..c79a6f7c
--- /dev/null
+++ b/server/src/Domain/HumanResource/Leave/Exception/LeaveRequestCantBeRemovedException.ts
@@ -0,0 +1,5 @@
+export class LeaveRequestCantBeRemovedException extends Error {
+  constructor() {
+    super('human_resources.leaves.requests.errors.cant_be_removed');
+  }
+}
diff --git a/server/src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository.ts b/server/src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository.ts
index 4c1816e6..518a4da5 100644
--- a/server/src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository.ts
+++ b/server/src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository.ts
@@ -3,6 +3,7 @@ import { User } from '../../User/User.entity';
 
 export interface ILeaveRequestRepository {
   save(leaveRequest: LeaveRequest): Promise<LeaveRequest>;
+  remove(leaveRequest: LeaveRequest): void;
   findOneById(id: string): Promise<LeaveRequest | undefined>;
   findLeaveRequests(page: number, status?: Status): Promise<[LeaveRequest[], number]>;
   findExistingLeaveRequestsByUserAndPeriod(
diff --git a/server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.spec.ts b/server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.spec.ts
new file mode 100644
index 00000000..f35486b4
--- /dev/null
+++ b/server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.spec.ts
@@ -0,0 +1,41 @@
+import { mock, instance, when } from 'ts-mockito';
+import { User } from 'src/Domain/HumanResource/User/User.entity';
+import { LeaveRequest } from '../LeaveRequest.entity';
+import { CanLeaveRequestBeRemoved } from './CanLeaveRequestBeRemoved';
+
+describe('CanLeaveRequestBeRemoved', () => {
+  let canLeaveRequestBeRemoved: CanLeaveRequestBeRemoved;
+  const user = mock(User);
+  const leaveRequest = mock(LeaveRequest);
+  const owner = mock(User);
+
+  beforeEach(() => {
+    canLeaveRequestBeRemoved = new CanLeaveRequestBeRemoved();
+  });
+
+  it('testLeaveRequestCantBeRemoved', async () => {
+    when(user.getId()).thenReturn('cfdd06eb-cd71-44b9-82c6-46110b30ce05');
+    when(owner.getId()).thenReturn('50e624ef-3609-4053-a437-f74844a2d2de');
+    when(leaveRequest.getUser()).thenReturn(instance(user));
+
+    expect(
+      await canLeaveRequestBeRemoved.isSatisfiedBy(
+        instance(leaveRequest),
+        instance(owner)
+      )
+    ).toBe(false);
+  });
+
+  it('testLeaveRequestCanBeRemoved', async () => {
+    when(user.getId()).thenReturn('cfdd06eb-cd71-44b9-82c6-46110b30ce05');
+    when(owner.getId()).thenReturn('cfdd06eb-cd71-44b9-82c6-46110b30ce05');
+    when(leaveRequest.getUser()).thenReturn(instance(user));
+
+    expect(
+      await canLeaveRequestBeRemoved.isSatisfiedBy(
+        instance(leaveRequest),
+        instance(owner)
+      )
+    ).toBe(true);
+  });
+});
diff --git a/server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.ts b/server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.ts
new file mode 100644
index 00000000..299058c8
--- /dev/null
+++ b/server/src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved.ts
@@ -0,0 +1,8 @@
+import { User } from '../../User/User.entity';
+import { LeaveRequest } from '../LeaveRequest.entity';
+
+export class CanLeaveRequestBeRemoved {
+  public isSatisfiedBy(leaveRequest: LeaveRequest, user: User): boolean {
+    return leaveRequest.getUser().getId() === user.getId();
+  }
+}
diff --git a/server/src/Infrastructure/HumanResource/Leave/Action/DeleteLeaveRequestAction.ts b/server/src/Infrastructure/HumanResource/Leave/Action/DeleteLeaveRequestAction.ts
new file mode 100644
index 00000000..833ddc62
--- /dev/null
+++ b/server/src/Infrastructure/HumanResource/Leave/Action/DeleteLeaveRequestAction.ts
@@ -0,0 +1,39 @@
+import {
+  Controller,
+  Inject,
+  BadRequestException,
+  UseGuards,
+  Param,
+  Delete
+} from '@nestjs/common';
+import { AuthGuard } from '@nestjs/passport';
+import { ApiTags, ApiBearerAuth, ApiOperation } from '@nestjs/swagger';
+import { IdDTO } from 'src/Infrastructure/Common/DTO/IdDTO';
+import { Roles } from 'src/Infrastructure/HumanResource/User/Decorator/Roles';
+import { UserRole, User } from 'src/Domain/HumanResource/User/User.entity';
+import { RolesGuard } from 'src/Infrastructure/HumanResource/User/Security/RolesGuard';
+import { LoggedUser } from '../../User/Decorator/LoggedUser';
+import { ICommandBus } from 'src/Application/ICommandBus';
+import { DeleteLeaveRequestCommand } from 'src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommand';
+
+@Controller('leave-requests')
+@ApiTags('Human Resource')
+@ApiBearerAuth()
+@UseGuards(AuthGuard('bearer'), RolesGuard)
+export class DeleteLeaveRequestAction {
+  constructor(
+    @Inject('ICommandBus')
+    private readonly commandBus: ICommandBus
+  ) {}
+
+  @Delete(':id')
+  @Roles(UserRole.COOPERATOR)
+  @ApiOperation({summary: 'Delete leave request'})
+  public async index(@Param() { id }: IdDTO, @LoggedUser() user: User) {
+    try {
+      await this.commandBus.execute(new DeleteLeaveRequestCommand(id, user));
+    } catch (e) {
+      throw new BadRequestException(e.message);
+    }
+  }
+}
diff --git a/server/src/Infrastructure/HumanResource/Leave/Repository/LeaveRequestRepository.ts b/server/src/Infrastructure/HumanResource/Leave/Repository/LeaveRequestRepository.ts
index 7a68632f..cc8d9f3d 100644
--- a/server/src/Infrastructure/HumanResource/Leave/Repository/LeaveRequestRepository.ts
+++ b/server/src/Infrastructure/HumanResource/Leave/Repository/LeaveRequestRepository.ts
@@ -15,6 +15,10 @@ export class LeaveRequestRepository implements ILeaveRequestRepository {
     return this.repository.save(leaveRequest);
   }
 
+  public remove(leaveRequest: LeaveRequest): void {
+    this.repository.delete(leaveRequest.getId());
+  }
+
   public findOneById(id: string): Promise<LeaveRequest | undefined> {
     return this.repository
       .createQueryBuilder('leaveRequest')
diff --git a/server/src/Infrastructure/HumanResource/humanResource.module.ts b/server/src/Infrastructure/HumanResource/humanResource.module.ts
index 3acd28e1..71820fb0 100644
--- a/server/src/Infrastructure/HumanResource/humanResource.module.ts
+++ b/server/src/Infrastructure/HumanResource/humanResource.module.ts
@@ -59,6 +59,9 @@ import { GetUserAction } from './User/Action/GetUserAction';
 import { UpdateUserCommandHandler } from 'src/Application/HumanResource/User/Command/UpdateUserCommandHandler';
 import { GetUserAdministrativeByIdQueryHandler } from 'src/Application/HumanResource/User/Query/GetUserAdministrativeByIdQueryHandler';
 import { GetLeavesAction } from './Leave/Action/GetLeavesAction';
+import { CanLeaveRequestBeRemoved } from 'src/Domain/HumanResource/Leave/Specification/CanLeaveRequestBeRemoved';
+import { DeleteLeaveRequestCommandHandler } from 'src/Application/HumanResource/Leave/Command/DeleteLeaveRequestCommandHandler';
+import { DeleteLeaveRequestAction } from './Leave/Action/DeleteLeaveRequestAction';
 
 @Module({
   imports: [
@@ -91,7 +94,8 @@ import { GetLeavesAction } from './Leave/Action/GetLeavesAction';
     GetLeaveRequestAction,
     CreateLeaveRequestAction,
     RefuseLeaveRequestAction,
-    AcceptLeaveRequestAction
+    AcceptLeaveRequestAction,
+    DeleteLeaveRequestAction
   ],
   providers: [
     {provide: 'IUserRepository', useClass: UserRepository},
@@ -131,6 +135,8 @@ import { GetLeavesAction } from './Leave/Action/GetLeavesAction';
     GetLeaveRequestsQueryHandler,
     GetLeaveRequestByIdQueryHandler,
     DoesEventsOrLeaveExistForPeriod,
+    CanLeaveRequestBeRemoved,
+    DeleteLeaveRequestCommandHandler
   ]
 })
 export class HumanResourceModule {}