From a783047bb57858e2a405b5400fbd9e997aee7c72 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Yana=C3=ABl=20GRETTE?= Date: Tue, 2 Feb 2021 16:51:40 +0100 Subject: [PATCH] =?UTF-8?q?Mise=20=C3=A0=20jour=20des=20guid=20utilis?= =?UTF-8?q?=C3=A9s=20par=20rapport=20aux=20guid=20des=20donn=C3=A9es=20du?= =?UTF-8?q?=20serveur=20collaborateur=20cr=C3=A9=C3=A9?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- EPAServeur.Tests/Controllers/NoteApiTests.cs | 4 +- .../Controllers/ReferentEPApiTests.cs | 29 +++- EPAServeur.Tests/Services/NoteServiceTests.cs | 18 +-- EPAServeur.Tests/Services/ReferentEPTests.cs | 2 +- EPAServeur/Context/DataSeeder.cs | 129 +++++++++++++++--- EPAServeur/Services/NoteService.cs | 2 +- 6 files changed, 144 insertions(+), 40 deletions(-) diff --git a/EPAServeur.Tests/Controllers/NoteApiTests.cs b/EPAServeur.Tests/Controllers/NoteApiTests.cs index a124691..a4e5e36 100644 --- a/EPAServeur.Tests/Controllers/NoteApiTests.cs +++ b/EPAServeur.Tests/Controllers/NoteApiTests.cs @@ -57,8 +57,8 @@ namespace EPAServeur.Tests.Controllers noteService = new NoteService(collaborateurApi, collaborateurService, context); logger = new NullLogger(); - auteur1 = new Guid("ecf528c3-e509-402f-87bb-c8821467e350"); - collaborateur1 = new Guid("92b29b9c-40a4-4aa0-9412-bc97a379e52f"); + auteur1 = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"); + collaborateur1 = new Guid("842650db-a548-4472-a3af-4c5fff3c1ab8"); } #endregion diff --git a/EPAServeur.Tests/Controllers/ReferentEPApiTests.cs b/EPAServeur.Tests/Controllers/ReferentEPApiTests.cs index 7e71af8..4f789a5 100644 --- a/EPAServeur.Tests/Controllers/ReferentEPApiTests.cs +++ b/EPAServeur.Tests/Controllers/ReferentEPApiTests.cs @@ -22,7 +22,7 @@ namespace EPAServeur.Tests.Controllers private EpContext context; private ICollaborateurApi collaborateurApi; private IReferentEPService referentEPService; - private Guid? referent, collaborateur; + private Guid? referent, collaborateur1, collaborateur2, collaborateur3; #endregion #region SetUp @@ -47,8 +47,10 @@ namespace EPAServeur.Tests.Controllers collaborateurApi = new CollaborateurApi(); referentEPService = new ReferentEPService(context, collaborateurApi); - referent = new Guid("ecf528c3-e509-402f-87bb-c8821467e350"); - collaborateur = new Guid("006226f6-51b2-4a02-a302-7447f7fccc04"); + referent = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"); + collaborateur1 = new Guid("301ba7f3-095e-4912-8998-a7c942dc5f23"); + collaborateur2 = new Guid("a43b6f4f-f199-4dd0-93b6-a1cb2c0a0d14"); + collaborateur3 = new Guid("b5254c6c-7caa-435f-a4bb-e0cf92559832"); } #endregion @@ -63,8 +65,25 @@ namespace EPAServeur.Tests.Controllers IdReferent = referent, IdsCollaborateur = new List() }; - referentEPDTO.IdsCollaborateur.Add(collaborateur); - Task task = referentsEPApi.UpdateReferentCollaborateur(referentEPDTO, collaborateur); + referentEPDTO.IdsCollaborateur.Add(collaborateur1); + Task task = referentsEPApi.UpdateReferentCollaborateur(referentEPDTO, collaborateur1); + OkObjectResult result = task.Result as OkObjectResult; + Assert.AreEqual(200, result.StatusCode); + Assert.IsInstanceOf(result.Value); + } + + public void UpdateCollaborateursReferent() + { + ReferentsEPApiController referentsEPApi = new ReferentsEPApiController(referentEPService, new NullLogger()); + ReferentEPDTO referentEPDTO = new ReferentEPDTO() + { + IdReferent = referent, + IdsCollaborateur = new List() + }; + referentEPDTO.IdsCollaborateur.Add(collaborateur1); + referentEPDTO.IdsCollaborateur.Add(collaborateur2); + referentEPDTO.IdsCollaborateur.Add(collaborateur3); + Task task = referentsEPApi.UpdateCollaborateursReferent(referentEPDTO, referent); OkObjectResult result = task.Result as OkObjectResult; Assert.AreEqual(200, result.StatusCode); Assert.IsInstanceOf(result.Value); diff --git a/EPAServeur.Tests/Services/NoteServiceTests.cs b/EPAServeur.Tests/Services/NoteServiceTests.cs index c2992f5..3ae3813 100644 --- a/EPAServeur.Tests/Services/NoteServiceTests.cs +++ b/EPAServeur.Tests/Services/NoteServiceTests.cs @@ -54,17 +54,17 @@ namespace EPAServeur.Tests.Services collaborateurService = new CollaborateurService(collaborateurApi, context); - auteur1 = new Guid("ecf528c3-e509-402f-87bb-c8821467e350"); - auteur2 = new Guid("6aa62dcb-f7c9-4c0c-af40-e934a4d6a7eb"); - auteur3 = new Guid("571463f3-b286-4a21-9eab-0707dc506dec"); + auteur1 = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"); + auteur2 = new Guid("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d"); + auteur3 = new Guid("eb8b0f33-f529-4985-861e-1207f3312bb5"); - collaborateur1 = new Guid("92b29b9c-40a4-4aa0-9412-bc97a379e52f"); - collaborateur2 = new Guid("006226f6-51b2-4a02-a302-7447f7fccc04"); - collaborateur3 = new Guid("e5d36da4-df16-4d19-8a11-1ba2f6efc80c"); + collaborateur1 = new Guid("842650db-a548-4472-a3af-4c5fff3c1ab8"); + collaborateur2 = new Guid("b5254c6c-7caa-435f-a4bb-e0cf92559832"); + collaborateur3 = new Guid("de98a866-736f-4295-a669-92a8694e2ee3"); auteurNonExistant = collaborateurNonExistant = new Guid("397f616e-44ff-11eb-b378-0242ac130002"); - collaborateurParti = new Guid("3c99214d-0a5e-4bb6-b7b2-7d9bb8143c50"); - referentParti = new Guid("0021430f-9cb5-4d7e-a395-904c95421920"); + collaborateurParti = new Guid("13fbe621-1bc9-4f04-afde-b54ca076e239"); + referentParti = new Guid("ea027734-ff0f-4308-8879-133a09fb3c46"); } #endregion @@ -186,7 +186,7 @@ namespace EPAServeur.Tests.Services Assert.AreEqual(nouvelleNote4.Texte, noteAjoutee4.Texte); Assert.AreEqual(nouvelleNote4.DateMiseAjour, noteAjoutee4.DateMiseAJour); Assert.AreEqual(nouvelleNote4.DateCreation, noteAjoutee4.DateCreation); - + } diff --git a/EPAServeur.Tests/Services/ReferentEPTests.cs b/EPAServeur.Tests/Services/ReferentEPTests.cs index b376aac..b59bca1 100644 --- a/EPAServeur.Tests/Services/ReferentEPTests.cs +++ b/EPAServeur.Tests/Services/ReferentEPTests.cs @@ -51,7 +51,7 @@ namespace EPAServeur.Tests.Services collaborateurService = new CollaborateurService(collaborateurApi, context); - referent1 = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"); + referent1 = new Guid("c"); collaborateur1 = new Guid("301ba7f3-095e-4912-8998-a7c942dc5f23"); collaborateur2 = new Guid("a43b6f4f-f199-4dd0-93b6-a1cb2c0a0d14"); diff --git a/EPAServeur/Context/DataSeeder.cs b/EPAServeur/Context/DataSeeder.cs index cda13b0..72e2c2b 100644 --- a/EPAServeur/Context/DataSeeder.cs +++ b/EPAServeur/Context/DataSeeder.cs @@ -143,7 +143,7 @@ namespace EPAServeur.Context ep3 = new Ep { - IdCollaborateur = Guid.Parse("006226f6-51b2-4a02-a302-7447f7fccc04"), + IdCollaborateur = Guid.Parse("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), IdReferent = Guid.Parse("ecf528c3-e509-402f-87bb-c8821467e350"), IdBu = 2, Fonction = "Dev", @@ -161,7 +161,7 @@ namespace EPAServeur.Context ep4 = new Ep { IdCollaborateur = Guid.Parse("f960f851-16b6-4631-952e-d05f38d8d80f"), - IdReferent = Guid.Parse("6aa62dcb-f7c9-4c0c-af40-e934a4d6a7eb"), + IdReferent = Guid.Parse("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d"), IdBu = 2, Fonction = "Dev", TypeEP = TypeEp.EPA, @@ -828,8 +828,8 @@ namespace EPAServeur.Context IdNote = 1, Titre = "Titre1", Texte = "Texte1", - IdAuteur = new Guid("ecf528c3-e509-402f-87bb-c8821467e350"), - IdCollaborateur = new Guid("fc1d7593-83c3-44de-b3f5-53dbe53c0830"), + IdAuteur = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"), + IdCollaborateur = new Guid("842650db-a548-4472-a3af-4c5fff3c1ab8"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -841,8 +841,8 @@ namespace EPAServeur.Context IdNote = 2, Titre = "Titre2", Texte = "Texte2", - IdAuteur = new Guid("ecf528c3-e509-402f-87bb-c8821467e350"), - IdCollaborateur = new Guid("fc1d7593-83c3-44de-b3f5-53dbe53c0830"), + IdAuteur = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"), + IdCollaborateur = new Guid("842650db-a548-4472-a3af-4c5fff3c1ab8"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -854,8 +854,8 @@ namespace EPAServeur.Context IdNote = 3, Titre = "Titre3", Texte = "Texte3", - IdAuteur = new Guid("073a4fb0-45c3-11eb-b378-0242ac130002"), - IdCollaborateur = new Guid("fc1d7593-83c3-44de-b3f5-53dbe53c0830"), + IdAuteur = new Guid(), + IdCollaborateur = new Guid("842650db-a548-4472-a3af-4c5fff3c1ab8"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -867,8 +867,8 @@ namespace EPAServeur.Context IdNote = 4, Titre = "Titre4", Texte = "Texte4", - IdAuteur = new Guid("ecf528c3-e509-402f-87bb-c8821467e350"), - IdCollaborateur = new Guid("006226f6-51b2-4a02-a302-7447f7fccc04"), + IdAuteur = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"), + IdCollaborateur = new Guid("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -880,8 +880,8 @@ namespace EPAServeur.Context IdNote = 5, Titre = "Titre5", Texte = "Texte5", - IdAuteur = new Guid("ecf528c3-e509-402f-87bb-c8821467e350"), - IdCollaborateur = new Guid("006226f6-51b2-4a02-a302-7447f7fccc04"), + IdAuteur = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696"), + IdCollaborateur = new Guid("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -893,8 +893,8 @@ namespace EPAServeur.Context IdNote = 6, Titre = "Titre6", Texte = "Texte6", - IdAuteur = new Guid("6aa62dcb-f7c9-4c0c-af40-e934a4d6a7eb"), - IdCollaborateur = new Guid("006226f6-51b2-4a02-a302-7447f7fccc04"), + IdAuteur = new Guid("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d"), + IdCollaborateur = new Guid("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -906,8 +906,8 @@ namespace EPAServeur.Context IdNote = 7, Titre = "Titre7", Texte = "Texte7", - IdAuteur = new Guid("073a4fb0-45c3-11eb-b378-0242ac130002"), - IdCollaborateur = new Guid("006226f6-51b2-4a02-a302-7447f7fccc04"), + IdAuteur = new Guid(), + IdCollaborateur = new Guid("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -919,8 +919,8 @@ namespace EPAServeur.Context IdNote = 8, Titre = "Titre8", Texte = "Texte8", - IdAuteur = new Guid("571463f3-b286-4a21-9eab-0707dc506dec"), - IdCollaborateur = new Guid("227e0e1a-45c3-11eb-b378-0242ac130002"), + IdAuteur = new Guid("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d"), + IdCollaborateur = new Guid(), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -932,8 +932,8 @@ namespace EPAServeur.Context IdNote = 9, Titre = "Titre9", Texte = "Texte9", - IdAuteur = new Guid("571463f3-b286-4a21-9eab-0707dc506dec"), - IdCollaborateur = new Guid("006226f6-51b2-4a02-a302-7447f7fccc04"), + IdAuteur = new Guid("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d"), + IdCollaborateur = new Guid("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -945,8 +945,8 @@ namespace EPAServeur.Context IdNote = 10, Titre = "Titre10", Texte = "Texte10", - IdAuteur = new Guid("6aa62dcb-f7c9-4c0c-af40-e934a4d6a7eb"), - IdCollaborateur = new Guid("694ffef2-45c3-11eb-b378-0242ac130002"), + IdAuteur = new Guid("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d"), + IdCollaborateur = new Guid(), DateCreation = DateTime.Now, DateMiseAJour = DateTime.Now @@ -955,5 +955,90 @@ namespace EPAServeur.Context epContext.SaveChanges(); } + + + /// + /// Insérer un jeu de données fictif pour les référents EP + /// + /// + public static void AddRereferentEP(EpContext epContext) + { + ReferentEP referentEP; + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("842650db-a548-4472-a3af-4c5fff3c1ab8"), + IdReferent = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid(), + IdReferent = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), + IdReferent = new Guid("aa36f34c-9041-42f5-9db3-6536fe7f1696") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("4f3fcd23-a1e4-4c9e-afa2-d06ca9216491"), + IdReferent = new Guid("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("0968ccd3-1ef5-4041-83f3-1c76afb02bbf"), + IdReferent = new Guid("01ee85ff-d7f3-494b-b1de-26ced8fbfa0d") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("1429be5b-9125-482c-80c4-c1d34afbd8d2"), + IdReferent = new Guid("e7820f92-eab1-42f5-ae96-5c16e71ff1e6") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("13fbe621-1bc9-4f04-afde-b54ca076e239"), + IdReferent = new Guid("e7820f92-eab1-42f5-ae96-5c16e71ff1e6") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("b5254c6c-7caa-435f-a4bb-e0cf92559832"), + IdReferent = new Guid("de98a866-736f-4295-a669-92a8694e2ee3") + }; + + epContext.ReferentEP.Add(referentEP); + + referentEP = new ReferentEP() + { + IdCollaborateur = new Guid("17b87130-0e9d-4b78-b0e3-a11e5f70318d"), + IdReferent = new Guid("de98a866-736f-4295-a669-92a8694e2ee3") + }; + + epContext.ReferentEP.Add(referentEP); + + epContext.SaveChanges(); + + } } } diff --git a/EPAServeur/Services/NoteService.cs b/EPAServeur/Services/NoteService.cs index 86c12fe..68a1a62 100644 --- a/EPAServeur/Services/NoteService.cs +++ b/EPAServeur/Services/NoteService.cs @@ -246,7 +246,7 @@ namespace EPAServeur.Services if (collaborateur == null) throw new CollaborateurNotFoundException("Le collaborateur pour qui la note est écrite n'a pas été trouvé"); if (collaborateur.DateDepart.HasValue) - throw new CollaborateurPartiException("Vous ne pouvez pas créer ou modifier de note si vous avez quitté l'entreprise"); + throw new CollaborateurPartiException("Vous ne pouvez pas créer ou modifier de notes pour un collaborateur qui a quitté l'entreprise"); } #endregion