ygrette
/
Digitalisation_EPA_Serveur
2
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Optimisation du code du service des notes et mise à jour et validation des tests unitaires

Browse Source
develop
Yanaël GRETTE 4 years ago
parent f13fcc25b7
commit db66711b77
2 changed files with 55 additions and 46 deletions
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Show Stats Download Patch File Download Diff File
  1. 51
      EPAServeur.Tests/Services/NoteServiceTests.cs
  2. 50
      EPAServeur/Services/NoteService.cs

51
EPAServeur.Tests/Services/NoteServiceTests.cs
Unescape View File

@ -415,15 +415,16 @@ namespace EPAServeur.Tests.Services
note.Titre = "Titre à jour " + id; note.Titre = "Titre à jour " + id;
note.Texte= "Texte à jour " + id; note.Texte= "Texte à jour " + id;
DetailsNoteDTO noteUpdate = await noteService.UpdateNoteAsync(id, note); DetailsNoteDTO noteUpdate = await noteService.UpdateNoteAsync(id, note);
Note noteUpdated = context.Note.Find(noteUpdate.Id);
Assert.IsNotNull(noteUpdate); Assert.IsNotNull(noteUpdate);
Assert.AreEqual(note.Id, noteUpdate.Id); Assert.IsNotNull(noteUpdated);
Assert.AreEqual(note.IdAuteur, noteUpdate.IdAuteur); Assert.AreEqual(noteUpdated.IdNote, noteUpdate.Id);
Assert.AreEqual(note.Collaborateur.Id, noteUpdate.Collaborateur.Id); Assert.AreEqual(noteUpdated.IdAuteur, noteUpdate.IdAuteur);
Assert.AreEqual(note.DateCreation, noteUpdate.DateCreation); Assert.AreEqual(noteUpdated.IdCollaborateur, noteUpdate.Collaborateur.Id);
Assert.AreEqual(note.Titre, noteUpdate.Titre); Assert.AreEqual(noteUpdated.DateCreation, noteUpdate.DateCreation);
Assert.AreEqual(note.Texte, noteUpdate.Texte); Assert.AreEqual(noteUpdated.Titre, noteUpdate.Titre);
Assert.AreNotEqual(note.DateMiseAjour, noteUpdate.DateMiseAjour); Assert.AreEqual(noteUpdated.Texte, noteUpdate.Texte);
Assert.AreEqual(noteUpdated.DateMiseAJour, noteUpdated.DateMiseAJour);
} }
[TestCase(1)] [TestCase(1)]
@ -436,15 +437,16 @@ namespace EPAServeur.Tests.Services
DetailsNoteDTO note = await noteService.GetNoteByIdAsync(id); DetailsNoteDTO note = await noteService.GetNoteByIdAsync(id);
note.Texte = "Texte à jour " + id; note.Texte = "Texte à jour " + id;
DetailsNoteDTO noteUpdate = await noteService.UpdateNoteAsync(id, note); DetailsNoteDTO noteUpdate = await noteService.UpdateNoteAsync(id, note);
Note noteUpdated = context.Note.Find(noteUpdate.Id);
Assert.IsNotNull(noteUpdate); Assert.IsNotNull(noteUpdate);
Assert.AreEqual(note.Id, noteUpdate.Id); Assert.IsNotNull(noteUpdated);
Assert.AreEqual(note.IdAuteur, noteUpdate.IdAuteur); Assert.AreEqual(noteUpdated.IdNote, noteUpdate.Id);
Assert.AreEqual(note.Collaborateur.Id, noteUpdate.Collaborateur.Id); Assert.AreEqual(noteUpdated.IdAuteur, noteUpdate.IdAuteur);
Assert.AreEqual(note.DateCreation, noteUpdate.DateCreation); Assert.AreEqual(noteUpdated.IdCollaborateur, noteUpdate.Collaborateur.Id);
Assert.AreEqual(note.Titre, noteUpdate.Titre); Assert.AreEqual(noteUpdated.DateCreation, noteUpdate.DateCreation);
Assert.AreEqual(note.Texte, noteUpdate.Texte); Assert.AreEqual(noteUpdated.Titre, noteUpdate.Titre);
Assert.AreNotEqual(note.DateMiseAjour, noteUpdate.DateMiseAjour); Assert.AreEqual(noteUpdated.Texte, noteUpdate.Texte);
Assert.AreEqual(noteUpdated.DateMiseAJour, noteUpdated.DateMiseAJour);
} }
[TestCase(1)] [TestCase(1)]
@ -457,15 +459,16 @@ namespace EPAServeur.Tests.Services
DetailsNoteDTO note = await noteService.GetNoteByIdAsync(id); DetailsNoteDTO note = await noteService.GetNoteByIdAsync(id);
note.Titre = "Titre à jour " + id; note.Titre = "Titre à jour " + id;
DetailsNoteDTO noteUpdate = await noteService.UpdateNoteAsync(id, note); DetailsNoteDTO noteUpdate = await noteService.UpdateNoteAsync(id, note);
Note noteUpdated = context.Note.Find(noteUpdate.Id);
Assert.IsNotNull(noteUpdate); Assert.IsNotNull(noteUpdate);
Assert.AreEqual(note.Id, noteUpdate.Id); Assert.IsNotNull(noteUpdated);
Assert.AreEqual(note.IdAuteur, noteUpdate.IdAuteur); Assert.AreEqual(noteUpdated.IdNote, noteUpdate.Id);
Assert.AreEqual(note.Collaborateur.Id, noteUpdate.Collaborateur.Id); Assert.AreEqual(noteUpdated.IdAuteur, noteUpdate.IdAuteur);
Assert.AreEqual(note.DateCreation, noteUpdate.DateCreation); Assert.AreEqual(noteUpdated.IdCollaborateur, noteUpdate.Collaborateur.Id);
Assert.AreEqual(note.Titre, noteUpdate.Titre); Assert.AreEqual(noteUpdated.DateCreation, noteUpdate.DateCreation);
Assert.AreEqual(note.Texte, noteUpdate.Texte); Assert.AreEqual(noteUpdated.Titre, noteUpdate.Titre);
Assert.AreNotEqual(note.DateMiseAjour, noteUpdate.DateMiseAjour); Assert.AreEqual(noteUpdated.Texte, noteUpdate.Texte);
Assert.AreEqual(noteUpdated.DateMiseAJour, noteUpdated.DateMiseAJour);
} }
[TestCase(1,2)] [TestCase(1,2)]

50
EPAServeur/Services/NoteService.cs
Unescape View File

@ -42,7 +42,7 @@ namespace EPAServeur.Services
} }
#endregion #endregion
#region Services Async #region Services Async
/// <summary> /// <summary>
/// Ajouter une nouvelle note dans la base de données de manière async /// Ajouter une nouvelle note dans la base de données de manière async
@ -53,14 +53,17 @@ namespace EPAServeur.Services
{ {
//vérifier qu'il n'y a aucune valeur null //vérifier qu'il n'y a aucune valeur null
CheckNoteValide(nouvelleNote); CheckNoteValide(nouvelleNote);
List<Guid?> guids = new List<Guid?>();
guids.AddRange(new Guid?[]{nouvelleNote.IdAuteur, nouvelleNote.Collaborateur.Id});
IEnumerable<Collaborateur> collaborateurs = await collaborateurApi.ChercherCollabAsync(guids);
//vérifier que le référent existe //vérifier que le référent existe
Collaborateur auteur = await collaborateurApi.ChercherCollabIdAsync((Guid?)nouvelleNote.IdAuteur); CheckReferent(collaborateurs.Where(c => c.Id.Equals(nouvelleNote.IdAuteur)).FirstOrDefault());
CheckReferent(auteur);
//vérifier que le collaborateur existe //vérifier que le collaborateur existe
Collaborateur collaborateur = await collaborateurApi.ChercherCollabIdAsync(nouvelleNote.Collaborateur.Id); CheckCollaborateur(collaborateurs.Where(c => c.Id.Equals(nouvelleNote.Collaborateur.Id)).FirstOrDefault());
CheckCollaborateur(collaborateur);
//transformer la note DTO en Note //transformer la note DTO en Note
Note note = DetailsNoteDTOToNouvelleNote(nouvelleNote); Note note = DetailsNoteDTOToNouvelleNote(nouvelleNote);
@ -70,6 +73,7 @@ namespace EPAServeur.Services
await context.SaveChangesAsync(); await context.SaveChangesAsync();
nouvelleNote.Id = note.IdNote; nouvelleNote.Id = note.IdNote;
return nouvelleNote; return nouvelleNote;
} }
/// <summary> /// <summary>
@ -107,7 +111,7 @@ namespace EPAServeur.Services
if (auteur == null) if (auteur == null)
throw new ReferentNotFoundException("L'auteur de la note n'existe pas"); throw new ReferentNotFoundException("L'auteur de la note n'existe pas");
return await NoteToDetailSDTOAsync(note); return NoteToDetailSDTO(note, await collaborateurService.GetCollaborateurByIdAsync(note.IdCollaborateur));
} }
/// <summary> /// <summary>
@ -122,17 +126,20 @@ namespace EPAServeur.Services
/// <returns>Retour la liste des notes à afficher</returns> /// <returns>Retour la liste des notes à afficher</returns>
public async Task<IEnumerable<AffichageNoteDTO>> GetNotesByAuteurAsync(Guid? idAuteur, bool? asc, int? numPage, int? parPage, string texte, string tri) public async Task<IEnumerable<AffichageNoteDTO>> GetNotesByAuteurAsync(Guid? idAuteur, bool? asc, int? numPage, int? parPage, string texte, string tri)
{ {
Collaborateur auteur = await collaborateurApi.ChercherCollabIdAsync(idAuteur); Collaborateur auteur = await collaborateurApi.ChercherCollabIdAsync(idAuteur);
if (auteur == null) if (auteur == null)
throw new ReferentNotFoundException("L'auteur de la note n'existe pas"); throw new ReferentNotFoundException("L'auteur de la note n'existe pas");
IEnumerable<Note> notes = context.Note.Where(n => n.IdAuteur == idAuteur);
List<Guid?> guids = notes.Select(n => (Guid?)n.IdCollaborateur).ToList();
IEnumerable<Collaborateur> collaborateurs = await collaborateurApi.ChercherCollabAsync(guids);
IEnumerable<AffichageNoteDTO> AffichageNoteDTO = notes.Select(note => NoteToAffichageDTO(note, collaborateurs));
int skip = (numPage.Value - 1) * parPage.Value; int skip = (numPage.Value - 1) * parPage.Value;
int take = parPage.Value; int take = parPage.Value;
IEnumerable<Note> notes = context.Note.Where(n => n.IdAuteur == idAuteur);
var task = notes.Select(n => NoteToAffichageDTOAsync(n));
IEnumerable<AffichageNoteDTO> AffichageNoteDTO = await Task.WhenAll(task);
if (texte != null) if (texte != null)
{ {
@ -148,6 +155,7 @@ namespace EPAServeur.Services
AffichageNoteDTO = AffichageNoteDTO.Reverse(); AffichageNoteDTO = AffichageNoteDTO.Reverse();
} }
return AffichageNoteDTO.Skip(skip).Take(take); return AffichageNoteDTO.Skip(skip).Take(take);
} }
/// <summary> /// <summary>
@ -161,11 +169,11 @@ namespace EPAServeur.Services
Collaborateur auteur = await collaborateurApi.ChercherCollabIdAsync(idAuteur); Collaborateur auteur = await collaborateurApi.ChercherCollabIdAsync(idAuteur);
if (auteur == null) if (auteur == null)
throw new ReferentNotFoundException("L'auteur de la note n'existe pas"); throw new ReferentNotFoundException("L'auteur de la note n'existe pas");
IEnumerable<Note> notes = context.Note.Where(n => n.IdAuteur == idAuteur); IEnumerable<Note> notes = context.Note.Where(n => n.IdAuteur == idAuteur);
var task = notes.Select(n => NoteToAffichageDTOAsync(n)); List<Guid?> guids = notes.Select(n => (Guid?)n.IdCollaborateur).ToList();
IEnumerable<AffichageNoteDTO> AffichageNoteDTO = await Task.WhenAll(task); IEnumerable<Collaborateur> collaborateurs = await collaborateurApi.ChercherCollabAsync(guids);
IEnumerable<AffichageNoteDTO> AffichageNoteDTO = notes.Select(note => NoteToAffichageDTO(note, collaborateurs));
if (texte != null) if (texte != null)
{ {
@ -196,9 +204,8 @@ namespace EPAServeur.Services
noteToUpdate.Titre = note.Titre; noteToUpdate.Titre = note.Titre;
noteToUpdate.Texte = note.Texte; noteToUpdate.Texte = note.Texte;
noteToUpdate.DateMiseAJour = DateTime.Now; noteToUpdate.DateMiseAJour = DateTime.Now;
note.DateMiseAjour = DateTime.Now; note.DateMiseAjour = noteToUpdate.DateMiseAJour;
await context.SaveChangesAsync(); return note;
return await NoteToDetailSDTOAsync(noteToUpdate);
} }
#endregion #endregion
@ -276,12 +283,12 @@ namespace EPAServeur.Services
/// <param name="note">Note à transformer</param> /// <param name="note">Note à transformer</param>
/// <param name="collaborateurService">Service collaborateur pour récupérer les informations des collaborateurs</param> /// <param name="collaborateurService">Service collaborateur pour récupérer les informations des collaborateurs</param>
/// <returns>La note transformée pour être affichée</returns> /// <returns>La note transformée pour être affichée</returns>
private async Task<AffichageNoteDTO> NoteToAffichageDTOAsync(Note note) private AffichageNoteDTO NoteToAffichageDTO(Note note, IEnumerable<Collaborateur> collaborateurs)
{ {
Collaborateur collaborateur = collaborateurs.FirstOrDefault(c => c.Id.Equals(note.IdCollaborateur));
string infoCollab = "Aucun collaborateur lié"; string infoCollab = "Aucun collaborateur lié";
Collaborateur collaborateur = await collaborateurApi.ChercherCollabIdAsync((Guid?)note.IdCollaborateur);
if (collaborateur != null) if (collaborateur != null)
infoCollab = collaborateur.Prenom + " " + collaborateur.Nom; infoCollab = collaborateur.Nom + " " + collaborateur.Prenom;
AffichageNoteDTO affichage = new AffichageNoteDTO() AffichageNoteDTO affichage = new AffichageNoteDTO()
{ {
Id = note.IdNote, Id = note.IdNote,
@ -300,9 +307,8 @@ namespace EPAServeur.Services
/// </summary> /// </summary>
/// <param name="note">Note à transformer</param> /// <param name="note">Note à transformer</param>
/// <returns>Note transformer en DetailsNoteDTO</returns> /// <returns>Note transformer en DetailsNoteDTO</returns>
private async Task<DetailsNoteDTO> NoteToDetailSDTOAsync(Note note) private DetailsNoteDTO NoteToDetailSDTO(Note note, CollaborateurDTO collaborateur)
{ {
CollaborateurDTO collaborateur = await collaborateurService.GetCollaborateurByIdAsync(note.IdCollaborateur);
if (collaborateur == null) if (collaborateur == null)
throw new CollaborateurNotFoundException("Il est impossible de récupérer une note donc le collaborateur n'existe plus"); throw new CollaborateurNotFoundException("Il est impossible de récupérer une note donc le collaborateur n'existe plus");
DetailsNoteDTO details = new DetailsNoteDTO() DetailsNoteDTO details = new DetailsNoteDTO()

Write Preview
Loading…
Cancel
Save