From 8bf726df8041d07c03c1aabe93773d4aacbf27ee Mon Sep 17 00:00:00 2001
From: Yoan <yoan2004.yb@gmail.com>
Date: Mon, 12 Jun 2023 18:08:50 +0200
Subject: [PATCH 1/5] Tests unitaires

---
 Sources/Model/Classes/Equipage.cs          |   2 +-
 Sources/Model/Classes/Personnage.cs        |   2 +-
 Sources/Model/Managers/Manager.cs          |  11 +-
 Sources/Model/Serializer/XML_Serializer.cs |   4 +-
 Sources/Ohara/PageFavoris.xaml             |  16 +-
 Sources/Ohara/PageInfoBateau.xaml          |   5 +-
 Sources/Ohara/PageInfoEquipage.xaml        |  30 +++-
 Sources/Ohara/PageInfoPersonnage.xaml      |  12 ++
 Sources/Ohara/PagePersonnage.xaml          |   4 +-
 Sources/Ohara/Resources/Styles/Styles.xaml |   7 +-
 Sources/TestConsole/Program.cs             |   2 +-
 Sources/TestProject1/TestEquipage.cs       |  19 ++-
 Sources/TestProject1/TestManager.cs        | 189 ++++++++++++++++++++-
 Sources/TestProject1/TestPersonnage.cs     |  19 ++-
 Sources/TestProject1/XMLSerializerTests.cs |   8 +
 15 files changed, 301 insertions(+), 29 deletions(-)

diff --git a/Sources/Model/Classes/Equipage.cs b/Sources/Model/Classes/Equipage.cs
index ef7f611..e7b6fbd 100644
--- a/Sources/Model/Classes/Equipage.cs
+++ b/Sources/Model/Classes/Equipage.cs
@@ -132,7 +132,7 @@ namespace Model.Classes
             }
             if (premierChap < 0)
             {
-                premierChap = 0;
+                PremierChap = 0;
             }
             else
             {
diff --git a/Sources/Model/Classes/Personnage.cs b/Sources/Model/Classes/Personnage.cs
index 594c7f0..564de80 100644
--- a/Sources/Model/Classes/Personnage.cs
+++ b/Sources/Model/Classes/Personnage.cs
@@ -158,7 +158,7 @@ namespace Model.Classes
 
         public override int GetHashCode()
         {
-            return HashCode.Combine(Prime, Epithete, Age, Origine,Biographie, Citation,Equipage,Fruit);
+            return HashCode.Combine(Prime, Epithete, Age, Taille, Origine,Biographie, Citation,Equipage);
         }
 
         public override string ToString()
diff --git a/Sources/Model/Managers/Manager.cs b/Sources/Model/Managers/Manager.cs
index b8c7d22..800d77c 100644
--- a/Sources/Model/Managers/Manager.cs
+++ b/Sources/Model/Managers/Manager.cs
@@ -113,17 +113,15 @@ namespace Model.Managers
         /// \return La liste filtrée des fruits du démon.
         public List<FruitDuDemon> FiltrerFDD(string type)
         {
-            List<FruitDuDemon> fdd = new List<FruitDuDemon>();
-            fdd=Fruits.Where(p=>p.Type==type).ToList();
+            List<FruitDuDemon> fdd =Fruits.Where(p=>p.Type==type).ToList();
             return fdd;
         }
         /// \brief Filtre les bateaux par nom d'affiliation.
         /// \param nom Le nom de l'équipage affilié au(x) bateau(x).
         /// \return La liste filtrée des bateaux.
         public List<Bateau> FiltrerBateau(string nom)
-        {
-            List<Bateau> bateau = new List<Bateau>();
-            bateau = Bateaux.Where(p => p.Affiliation?.Nom == nom).ToList();
+        {           
+           List<Bateau> bateau = Bateaux.Where(p => p.Affiliation?.Nom == nom).ToList();
             return bateau;
         }
         /// \brief Filtre les îles par région.
@@ -131,8 +129,7 @@ namespace Model.Managers
         /// \return La liste filtrée des îles.
         public List<Ile> FiltrerIle(string region)
         {
-            List<Ile> ile = new List<Ile>();
-            ile = Iles.Where(p => p.Region == region).ToList();
+            List<Ile> ile = Iles.Where(p => p.Region == region).ToList();
             return ile;
         }
         /// \brief Recherche les objets Ohara correspondant au texte donné.
diff --git a/Sources/Model/Serializer/XML_Serializer.cs b/Sources/Model/Serializer/XML_Serializer.cs
index 86e4259..75d8ffd 100644
--- a/Sources/Model/Serializer/XML_Serializer.cs
+++ b/Sources/Model/Serializer/XML_Serializer.cs
@@ -143,7 +143,7 @@ namespace Model.Serializer
             }
         }
 
-        public void SetBateau(List<Bateau> listeBateau)
+        public void SetBateau(List<Bateau> listeBateaux)
         {
             var serializer = new DataContractSerializer(typeof(List<Bateau>));
             string xmlFile = "bateau.xml";
@@ -155,7 +155,7 @@ namespace Model.Serializer
                 using (XmlWriter writer = XmlWriter.Create(tw, settings))
                 {
 
-                    serializer.WriteObject(writer, listeBateau);
+                    serializer.WriteObject(writer, listeBateaux);
 
                 }
             }
diff --git a/Sources/Ohara/PageFavoris.xaml b/Sources/Ohara/PageFavoris.xaml
index aa6bdf1..b82477d 100644
--- a/Sources/Ohara/PageFavoris.xaml
+++ b/Sources/Ohara/PageFavoris.xaml
@@ -13,7 +13,8 @@
             
             </Grid>
             <FlexLayout x:Name="listeFav" AlignItems="Center" Wrap="Wrap"
-                    HorizontalOptions="Center" JustifyContent="SpaceEvenly">
+                    HorizontalOptions="Center" JustifyContent="SpaceEvenly"
+                   >
                 <BindableLayout.ItemTemplate>
                     <DataTemplate>
                         <Frame Style="{StaticResource frameObjet}" Margin="0,10,0,0" >
@@ -37,6 +38,19 @@
                         </Frame>
                     </DataTemplate>
                 </BindableLayout.ItemTemplate>
+                <BindableLayout.EmptyView>
+                    <ContentView>
+                        <StackLayout HorizontalOptions="CenterAndExpand" VerticalOptions="CenterAndExpand">
+                            <Label Text="Vous n'avez pas encore ajoutez d'objets à vos favoris ..."                                     
+                                           TextColor="#72a3b3"
+                                           FontAttributes="Bold"
+                                           Margin="50"
+                                           FontSize="18"
+                                           HorizontalOptions="Fill"
+                                           HorizontalTextAlignment="Center"/>
+                        </StackLayout>
+                    </ContentView>
+                </BindableLayout.EmptyView>
             </FlexLayout>
         </VerticalStackLayout>
 
diff --git a/Sources/Ohara/PageInfoBateau.xaml b/Sources/Ohara/PageInfoBateau.xaml
index 2fe8de0..eacc5b1 100644
--- a/Sources/Ohara/PageInfoBateau.xaml
+++ b/Sources/Ohara/PageInfoBateau.xaml
@@ -3,8 +3,9 @@
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              x:Class="Ohara.PageInfoBateau"
              Title="PageInfoBateau"
-             BackgroundColor="#e2edf1">
-    <ScrollView  Grid.Row="0" Grid.Column="1" Orientation="Vertical" VerticalScrollBarVisibility="Always">
+             BackgroundColor="#e2edf1"
+             >
+    <ScrollView Orientation="Vertical" VerticalScrollBarVisibility="Always">
 
         <VerticalStackLayout Spacing="10" Padding="40">
 
diff --git a/Sources/Ohara/PageInfoEquipage.xaml b/Sources/Ohara/PageInfoEquipage.xaml
index 1967f03..83a1957 100644
--- a/Sources/Ohara/PageInfoEquipage.xaml
+++ b/Sources/Ohara/PageInfoEquipage.xaml
@@ -77,7 +77,7 @@
                     <Label Text="Membre(s) :"  TextColor="#72a3b3" FontSize="20" FontAttributes="Bold"/>
                     <Line X1="0" Y1="0" X2="3000" Y2="0" StrokeThickness="2" Stroke="#72a3b3" />
                     <ScrollView Orientation="Horizontal">
-                        <CollectionView x:Name="listMembre" ItemsLayout="HorizontalList" ItemsSource="{Binding Membre}" EmptyView="Cet équipage n'à pas de membres..."  SelectionMode="Single" SelectionChanged="listMembre_SelectionChanged"    >
+                        <CollectionView x:Name="listMembre" ItemsLayout="HorizontalList" ItemsSource="{Binding Membre}"  SelectionMode="Single" SelectionChanged="listMembre_SelectionChanged"    >
                             <CollectionView.ItemTemplate>
                                 <DataTemplate>
 
@@ -97,10 +97,20 @@
                                 FontAttributes="Bold" />
                                         </StackLayout>
                                     </Frame>
-
-
                                 </DataTemplate>
                             </CollectionView.ItemTemplate>
+                            <CollectionView.EmptyView>
+                                <ContentView>
+                                    <StackLayout HorizontalOptions="CenterAndExpand" VerticalOptions="CenterAndExpand">
+                                        <Label Text="Cet équipage n'à pas (encore) de membre."                                     
+                                           TextColor="#72a3b3"
+                                           FontAttributes="Bold"
+                                           FontSize="18"
+                                           HorizontalOptions="Fill"
+                                           HorizontalTextAlignment="Center"/>                               
+                                    </StackLayout>
+                                </ContentView>
+                            </CollectionView.EmptyView>
                         </CollectionView>
                     </ScrollView>
                 </VerticalStackLayout>
@@ -132,7 +142,7 @@
           
             <Label Text="Allié(s) :"  TextColor="#72a3b3" FontSize="20" FontAttributes="Bold"/>
             <Line X1="0" Y1="0" X2="3000" Y2="0" StrokeThickness="2" Stroke="#72a3b3" />
-            <CollectionView x:Name="listAlle" ItemsLayout="HorizontalList" ItemsSource="{Binding Allie}" EmptyView="Cet équipage n'à pas d'alliés..." SelectionMode="Single" SelectionChanged="listAlle_SelectionChanged">
+            <CollectionView x:Name="listAlle" ItemsLayout="HorizontalList" ItemsSource="{Binding Allie}" SelectionMode="Single" SelectionChanged="listAlle_SelectionChanged">
                 <CollectionView.ItemTemplate>
                     <DataTemplate>
                         <Frame Style="{StaticResource frameEquip}" Margin="5">
@@ -153,6 +163,18 @@
                         </Frame>
                     </DataTemplate>
                 </CollectionView.ItemTemplate>
+                <CollectionView.EmptyView>
+                    <ContentView>
+                        <StackLayout HorizontalOptions="CenterAndExpand" VerticalOptions="CenterAndExpand">
+                            <Label Text="Cet équipage n'à pas (encore) d'allié."                                     
+                                           TextColor="#72a3b3"
+                                           FontAttributes="Bold"
+                                           FontSize="18"
+                                           HorizontalOptions="Fill"
+                                           HorizontalTextAlignment="Center"/>
+                        </StackLayout>
+                    </ContentView>
+                </CollectionView.EmptyView>
             </CollectionView>
             
         </VerticalStackLayout>
diff --git a/Sources/Ohara/PageInfoPersonnage.xaml b/Sources/Ohara/PageInfoPersonnage.xaml
index 9bc32db..d308998 100644
--- a/Sources/Ohara/PageInfoPersonnage.xaml
+++ b/Sources/Ohara/PageInfoPersonnage.xaml
@@ -115,6 +115,18 @@
                         </Frame>
                     </DataTemplate>
                 </CollectionView.ItemTemplate>
+                <CollectionView.EmptyView>
+                    <ContentView>
+                        <StackLayout HorizontalOptions="CenterAndExpand" VerticalOptions="CenterAndExpand">
+                            <Label Text="Ce personnage n'à pas (encore) de fruit du démon."                                     
+                                           TextColor="#72a3b3"
+                                           FontAttributes="Bold"
+                                           FontSize="18"
+                                           HorizontalOptions="Fill"
+                                           HorizontalTextAlignment="Center"/>
+                        </StackLayout>
+                    </ContentView>
+                </CollectionView.EmptyView>
             </CollectionView>
 
         </VerticalStackLayout>
diff --git a/Sources/Ohara/PagePersonnage.xaml b/Sources/Ohara/PagePersonnage.xaml
index 7a0e9e2..1d539fe 100644
--- a/Sources/Ohara/PagePersonnage.xaml
+++ b/Sources/Ohara/PagePersonnage.xaml
@@ -11,11 +11,9 @@
         <Grid ColumnDefinitions="200,*,150" BackgroundColor="#72a3b3" Padding="10">
             <SearchBar x:Name="searchBar" Placeholder="Rechercher..." BackgroundColor="#bfe5ef" Grid.Column="0"/>
             <Button Text="Ajouter" Clicked="ButtonAjouter_Clicked" Style="{StaticResource buttonBarre}" Grid.Column="2"/>
-            
-
         </Grid>
 
-        <ScrollView Orientation="Horizontal" HorizontalScrollBarVisibility="Always">
+        <ScrollView Orientation="Horizontal" HorizontalScrollBarVisibility="Always" >
                 <CollectionView x:Name="listePerso" ItemsSource="{Binding Personnages}" ItemsLayout="HorizontalList" EmptyView="Aucun résultat trouvé." SelectionMode="Single" SelectionChanged="listePerso_SelectionChanged">
                     <CollectionView.ItemTemplate>
                         <DataTemplate>
diff --git a/Sources/Ohara/Resources/Styles/Styles.xaml b/Sources/Ohara/Resources/Styles/Styles.xaml
index bb61fe7..ed0be77 100644
--- a/Sources/Ohara/Resources/Styles/Styles.xaml
+++ b/Sources/Ohara/Resources/Styles/Styles.xaml
@@ -87,7 +87,7 @@
     </Style>
 
     <Style TargetType="Editor">
-        <Setter Property="TextColor" Value="{AppThemeBinding Light={StaticResource Black}, Dark={StaticResource White}}" />
+        <Setter Property="TextColor" Value="White" />
         <Setter Property="BackgroundColor" Value="Transparent" />
         <Setter Property="FontFamily" Value="OpenSansRegular"/>
         <Setter Property="FontSize" Value="14" />
@@ -109,7 +109,7 @@
     </Style>
 
     <Style TargetType="Entry">
-        <Setter Property="TextColor" Value="{AppThemeBinding Light={StaticResource Black}, Dark={StaticResource White}}" />
+        <Setter Property="TextColor" Value="White" />
         <Setter Property="BackgroundColor" Value="Transparent" />
         <Setter Property="FontFamily" Value="OpenSansRegular"/>
         <Setter Property="FontSize" Value="14" />
@@ -158,7 +158,7 @@
     </Style>
 
     <Style TargetType="Label">
-        <Setter Property="TextColor" Value="{AppThemeBinding Light={StaticResource Gray900}, Dark={StaticResource White}}" />
+        <Setter Property="TextColor" Value="White" />
         <Setter Property="BackgroundColor" Value="Transparent" />
         <Setter Property="FontFamily" Value="OpenSansRegular" />
         <Setter Property="FontSize" Value="14" />
@@ -387,6 +387,7 @@
         <Setter Property="Shell.TabBarForegroundColor" Value="{AppThemeBinding Light={StaticResource Primary}, Dark={StaticResource White}}" />
         <Setter Property="Shell.TabBarTitleColor" Value="{AppThemeBinding Light={StaticResource Primary}, Dark={StaticResource White}}" />
         <Setter Property="Shell.TabBarUnselectedColor" Value="{AppThemeBinding Light={StaticResource Gray900}, Dark={StaticResource Gray200}}" />
+        <Setter Property="FlyoutBackgroundColor" Value="White"/>
     </Style>
 
     <Style TargetType="NavigationPage">
diff --git a/Sources/TestConsole/Program.cs b/Sources/TestConsole/Program.cs
index ecdecb2..323dacd 100644
--- a/Sources/TestConsole/Program.cs
+++ b/Sources/TestConsole/Program.cs
@@ -92,7 +92,7 @@ foreach (FruitDuDemon f in manager.FiltrerFDD("Logia"))
 Console.WriteLine("\n");
 Console.WriteLine("\n");
 // Recherche dansune liste de fruit de démon pour afficher les fruits correspondant au mot clé "Nika"
-foreach (FruitDuDemon f in manager.RechercheObjetOhara("Nika", new List<ObjetOhara>(manager.Fruits)))
+foreach (ObjetOhara f in manager.RechercheObjetOhara("Nika", new List<ObjetOhara>(manager.Fruits)) )
 { 
     Console.WriteLine(f.Nom);
 }
diff --git a/Sources/TestProject1/TestEquipage.cs b/Sources/TestProject1/TestEquipage.cs
index 901e797..795d373 100644
--- a/Sources/TestProject1/TestEquipage.cs
+++ b/Sources/TestProject1/TestEquipage.cs
@@ -35,9 +35,26 @@ namespace TestProject1
         {
             Equipage equiapge1= new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
             Equipage equiapge2 = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+            Equipage equiapge3 = new Equipage("Équipage du poux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+            Bateau bateau1 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
             bool resultat = (equiapge1.Equals(equiapge2));
-            Assert.True(resultat, "Les deux equipages devraient etre égaux car ils onts le meme nom");
+            bool resultat2 = (equiapge1.Equals(equiapge3));
+            bool resultat3 = (equiapge1.Equals(bateau1));
+            Assert.True(resultat, "Les deux equipages devraient etre égaux car ils ont le meme nom");
+            Assert.False(resultat2, "Les deux equipages  ne devraient pas etre égaux car ils n'ont pas le meme nom");
+            Assert.False(resultat3, "Les deux equipages  ne devraient pas etre égaux car ils n'ont pas le meme type");
 
         }
+        [Fact]
+        public void GetHashCode_EquipageWithSameProperties_ReturnsSameHashCode()
+        {
+            Equipage obj1 = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+            Equipage obj2 = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+
+            var hashCode1 = obj1.GetHashCode();
+            var hashCode2 = obj2.GetHashCode();
+
+            Assert.True(hashCode1 == hashCode2);
+        }
     }
 }
diff --git a/Sources/TestProject1/TestManager.cs b/Sources/TestProject1/TestManager.cs
index 37606da..d2bf4e1 100644
--- a/Sources/TestProject1/TestManager.cs
+++ b/Sources/TestProject1/TestManager.cs
@@ -96,11 +96,11 @@ namespace TestProject1
 
             // Act
             var result = manager.RechercheObjetOhara(text, liste);
-
+            var result2 = manager.RechercheObjetOhara("", liste);
             // Assert
             Assert.True(1==result.Count);
             Assert.Contains(obj1, result);
- 
+            Assert.True(result2.Count() == liste.Count());
             Assert.True(!result.Contains(obj3));
         }
 
@@ -346,5 +346,190 @@ namespace TestProject1
             Assert.Contains(ancienBateau, manager.Bateaux);
             Assert.DoesNotContain(nouveauBateau, manager.Bateaux);
         }
+
+        [Fact]
+        public void TestAjouterFDD()
+        {
+            var fruit = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "", "", "");
+
+            manager.AjouterFDD(fruit);
+         
+            Assert.Contains(fruit, manager.Fruits);
+        }
+
+        [Fact]
+        public void TestAjouterEquip()
+        {
+            var equipage = new Equipage("Equipage", "", "", 97, 48,true ,"", "");
+
+            manager.AjouterEquip(equipage);
+
+            Assert.Contains(equipage, manager.Equipages);
+        }
+
+        [Fact]
+        public void TestAjouterBest()
+        {
+            var best = new Bestiaire("Bestiaire", "", "", "");
+
+            manager.AjouterBest(best);
+
+            Assert.Contains(best, manager.Bestiaire);
+        }
+
+        [Fact]
+        public void TestAjouterPerso()
+        {
+            var perso = new Personnage("Personnage",0, "",0,0, "", "","");
+
+            manager.AjouterPerso(perso);
+
+            Assert.Contains(perso, manager.Personnages);
+        }
+
+        [Fact]
+        public void TestAjouterIle()
+        {
+            var ile = new Ile("Ile", "","", 0, 0, "", "", "");
+
+            manager.AjouterIle(ile);
+
+            Assert.Contains(ile, manager.Iles);
+        }
+
+        [Fact]
+        public void TestAjouterBateau()
+        {
+            var bateau = new Bateau("Bateau", "", 0, 0, "", "");
+
+            manager.AjouterBateau(bateau);
+
+            Assert.Contains(bateau, manager.Bateaux);        
+        }
+        [Fact]
+        public void TestSupprimerFDD()
+        {
+            var fruit = new FruitDuDemon("Fruit", "", "Logia", 97, 48, "", "", "");
+
+            manager.AjouterFDD(fruit);
+            manager.SupprimerFDD(fruit);
+              
+            Assert.DoesNotContain(fruit, manager.Fruits);
+        }
+
+        [Fact]
+        public void TestSupprimerEquip()
+        {
+            var equipage = new Equipage("Equipage", "", "", 97, 48, true, "", "");
+
+            manager.AjouterEquip(equipage);
+            manager.SupprimerEquip(equipage);
+
+            Assert.DoesNotContain(equipage, manager.Equipages);
+        }
+
+        [Fact]
+        public void TestSupprimerBest()
+        {
+            var best = new Bestiaire("Bestiaire", "", "", "");
+
+            manager.AjouterBest(best);
+            manager.SupprimerBest(best);
+
+            Assert.DoesNotContain(best, manager.Bestiaire);
+        }
+
+        [Fact]
+        public void TestSupprimerPerso()
+        {
+            var perso = new Personnage("Personnage", 0, "", 0, 0, "", "", "");
+
+            manager.AjouterPerso(perso);
+            manager.SupprimerPerso(perso);
+
+
+            Assert.DoesNotContain(perso, manager.Personnages);
+        }
+
+        [Fact]
+        public void TestSupprimerIle()
+        {
+            var ile = new Ile("Ile", "", "", 0, 0, "", "", "");
+
+            manager.AjouterIle(ile);
+            manager.SupprimerIle(ile);
+
+            Assert.DoesNotContain(ile, manager.Iles);
+        }
+
+        [Fact]
+        public void TestSupprimerBateau()
+        {
+            var bateau = new Bateau("Bateau", "", 0, 0, "", "");
+
+            manager.AjouterBateau(bateau);
+            manager.SupprimerBateau(bateau);
+
+            Assert.DoesNotContain(bateau, manager.Bateaux);
+        }
+        [Fact]
+        public void TestGetPersonnages()
+        {
+            List<Personnage> resultat = new List<Personnage>();
+
+            var result = manager.GetPersonnages();
+
+            Assert.NotNull(result);
+        }
+
+        [Fact]
+        public void TestGetFruits()
+        {
+            List<FruitDuDemon> resultat = new List<FruitDuDemon>();
+
+            var result = manager.GetFruits();
+
+            Assert.NotNull(result);
+        }
+
+        [Fact]
+        public void TestGetEquipages()
+        {
+            List<Equipage> resultat = new List<Equipage>();
+
+            var result = manager.GetEquipages();
+
+            Assert.NotNull(result);
+        }
+
+        [Fact]
+        public void TestGetBateaux()
+        {
+            List<Bateau> resultat = new List<Bateau>();
+
+            var result = manager.GetBateaux();
+
+            Assert.NotNull(result);
+        }
+
+        [Fact]
+        public void TestGetBestiaires()
+        {
+            List<Bestiaire> resultat = new List<Bestiaire>();
+
+            var result = manager.GetBestiaires();
+
+            Assert.NotNull(result);
+        }
+
+        [Fact]
+        public void TestGetIles()
+        {
+            List<Ile> resultat = new List<Ile>();
+         
+            var result = manager.GetIles();
+
+            Assert.NotNull(result);
+        }
     }
 }
diff --git a/Sources/TestProject1/TestPersonnage.cs b/Sources/TestProject1/TestPersonnage.cs
index 99586ab..c44aa7a 100644
--- a/Sources/TestProject1/TestPersonnage.cs
+++ b/Sources/TestProject1/TestPersonnage.cs
@@ -43,8 +43,25 @@ namespace TestProject1
         {
             Personnage personnage1 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
             Personnage personnage2 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
+            Personnage personnage3 = new Personnage("Lufty", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
+            Bateau bateau1 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
             bool resultat = (personnage1.Equals(personnage1));
-            Assert.True(resultat, "Les personanges devraient etre égales car ils onts le meme nom");
+            bool resultat2 = (personnage1.Equals(personnage3));
+            bool resultat3 = (personnage1.Equals(bateau1));
+            Assert.True(resultat, "Les personanges devraient etre égaux car ils ont le meme nom");
+            Assert.False(resultat2, "Les personanges ne devraient pas etre égaux car ils n'ont pas le meme nom");
+            Assert.False(resultat3, "Les deux objets ne devraient pas etre égaux car ils n'ont pas le meme type");
+        }
+        [Fact]
+        public void GetHashCode_PersonnageWithSameProperties_ReturnsSameHashCode()
+        {
+            Personnage obj1 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
+            Personnage obj2 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
+
+            var hashCode1 = obj1.GetHashCode();
+            var hashCode2 = obj2.GetHashCode();
+
+            Assert.True(hashCode1 == hashCode2);
         }
     }
 }
diff --git a/Sources/TestProject1/XMLSerializerTests.cs b/Sources/TestProject1/XMLSerializerTests.cs
index f504236..99137a2 100644
--- a/Sources/TestProject1/XMLSerializerTests.cs
+++ b/Sources/TestProject1/XMLSerializerTests.cs
@@ -1,11 +1,13 @@
 using Model.Classes;
 using Model.Serializer;
+using Model.Stub;
 using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Runtime.Serialization;
 using System.Text;
 using System.Threading.Tasks;
+using System.Xml.Serialization;
 
 namespace TestProject1
 {
@@ -13,6 +15,12 @@ namespace TestProject1
     {
         public XML_Serializer serializer = new XML_Serializer();
 
+        [Fact]
+        public void InitialiserFichiers_FileDoesNotExist_SetPersonnageCalled()
+        {
+            
+            Assert.True(File.Exists(Path.Combine(serializer.Chemin, "./personnage.xml")));
+        }
         [Fact]
         public void SetPersonnage_SerializesAndWritesToFile()
         {

From 5ea4f7287a445ce9ea95d9820b930b7342592c4b Mon Sep 17 00:00:00 2001
From: Yoan <yoan2004.yb@gmail.com>
Date: Wed, 14 Jun 2023 16:41:00 +0200
Subject: [PATCH 2/5] Tests Unitaire

---
 Sources/Model/Managers/IDataManager.cs     | 12 ++--
 Sources/Model/Managers/Manager.cs          | 37 +++++++----
 Sources/Model/Serializer/XML_Serializer.cs | 76 +++++++++++-----------
 Sources/Model/Stub/StubManager.cs          | 12 ++--
 Sources/Ohara.sln                          |  1 +
 Sources/Ohara/AppShell.xaml.cs             |  5 +-
 Sources/Ohara/PageBateau.xaml              | 23 ++-----
 Sources/Ohara/PageIle.xaml                 | 12 +---
 Sources/Ohara/PagePersonnage.xaml          | 15 ++---
 Sources/Ohara/Resources/Styles/Styles.xaml |  6 ++
 Sources/TestConsole/Program.cs             |  1 -
 Sources/TestProject1/TestBateau.cs         | 27 ++++++++
 Sources/TestProject1/TestEquipage.cs       | 50 +++++++++++++-
 Sources/TestProject1/TestFDD.cs            | 26 +++++++-
 Sources/TestProject1/TestIle.cs            | 26 ++++++++
 Sources/TestProject1/TestPersonnage.cs     | 39 +++++++++++
 16 files changed, 261 insertions(+), 107 deletions(-)

diff --git a/Sources/Model/Managers/IDataManager.cs b/Sources/Model/Managers/IDataManager.cs
index 564ca42..7d36e67 100644
--- a/Sources/Model/Managers/IDataManager.cs
+++ b/Sources/Model/Managers/IDataManager.cs
@@ -9,12 +9,12 @@ namespace Model.Managers
 {
     public interface IDataManager
     {
-        IEnumerable<Bateau> GetBateaux();
-        IEnumerable<Bestiaire> GetBestiaires();
-        IEnumerable<Equipage> GetEquipages();
-        IEnumerable<FruitDuDemon> GetFruits();
-        IEnumerable<Ile> GetIles();
-        IEnumerable<Personnage> GetPersonnages();
+        IEnumerable<Bateau>? GetBateaux();
+        IEnumerable<Bestiaire>? GetBestiaires();
+        IEnumerable<Equipage>? GetEquipages();
+        IEnumerable<FruitDuDemon>? GetFruits();
+        IEnumerable<Ile>? GetIles();
+        IEnumerable<Personnage>? GetPersonnages();
         void SetBateau(List<Bateau> listeBateaux);
         void SetBestiaire(List<Bestiaire> listeBest);
         void SetEquipage(List<Equipage> listeEquip);
diff --git a/Sources/Model/Managers/Manager.cs b/Sources/Model/Managers/Manager.cs
index 800d77c..6f5e08a 100644
--- a/Sources/Model/Managers/Manager.cs
+++ b/Sources/Model/Managers/Manager.cs
@@ -63,49 +63,62 @@ namespace Model.Managers
         public Manager(IDataManager dataManager)
         {
             DataManager = dataManager;
-            Bateaux = new ObservableCollection<Bateau>(DataManager.GetBateaux());
-            Personnages = new ObservableCollection<Personnage>(DataManager.GetPersonnages());
-            Fruits = new ObservableCollection<FruitDuDemon>(DataManager.GetFruits());
-            Iles = new ObservableCollection<Ile>(DataManager.GetIles());
-            Bestiaire = new ObservableCollection<Bestiaire>(DataManager.GetBestiaires());
-            Equipages = new ObservableCollection<Equipage>(DataManager.GetEquipages());
+            var bateaux = DataManager.GetBateaux();
+            Bateaux = bateaux != null ? new ObservableCollection<Bateau>(bateaux) : new ObservableCollection<Bateau>();
+            var personnages = DataManager.GetPersonnages();
+            Personnages = personnages != null ? new ObservableCollection<Personnage>(personnages) : new ObservableCollection<Personnage>();     
+            var fruits = DataManager.GetFruits();
+            Fruits = fruits != null ? new ObservableCollection<FruitDuDemon>(fruits) : new ObservableCollection<FruitDuDemon>();
+            var iles = DataManager.GetIles();
+            Iles = iles != null ? new ObservableCollection<Ile>(iles) : new ObservableCollection<Ile>();
+            var bests = DataManager.GetBestiaires();
+            Bestiaire = bests != null ? new ObservableCollection<Bestiaire>(bests) : new ObservableCollection<Bestiaire>();
+            var equipages = DataManager.GetEquipages();
+            Equipages = equipages != null ? new ObservableCollection<Equipage>(equipages) : new ObservableCollection<Equipage>();
         }
 
+
         /// \brief Récupère la liste des personnages.
         /// \return La liste des personnages.
         public List<Personnage> GetPersonnages()
         {
-            return DataManager.GetPersonnages().ToList();
+            var personnages = DataManager.GetPersonnages();
+            return personnages != null ? personnages.ToList() : new List<Personnage>();
         }
         /// \brief Récupère la liste des fruits du démon.
         /// \return La liste des fruits du démon.
         public List<FruitDuDemon> GetFruits()
         {
-            return DataManager.GetFruits().ToList();
+            var fruits = DataManager.GetFruits();
+            return fruits != null ? fruits.ToList() : new List<FruitDuDemon>();
         }
         /// \brief Récupère la liste des équipages.
         /// \return La liste des équipages.
         public List<Equipage> GetEquipages()
         {
-            return DataManager.GetEquipages().ToList();
+            var equipages = DataManager.GetEquipages();
+            return equipages != null ? equipages.ToList() : new List<Equipage>();
         }
         /// \brief Récupère la liste des bateaux.
         /// \return La liste des bateaux.
         public List<Bateau> GetBateaux()
         {
-            return DataManager.GetBateaux().ToList();
+            var bateaux = DataManager.GetBateaux();
+            return bateaux != null ? bateaux.ToList() : new List<Bateau>();
         }
         /// \brief Récupère la liste des bestiaires.
         /// \return La liste des bestiaires.
         public List<Bestiaire> GetBestiaires()
         {
-            return DataManager.GetBestiaires().ToList();
+            var bestiaires = DataManager.GetBestiaires();
+            return bestiaires != null ? bestiaires.ToList() : new List<Bestiaire>();
         }
         /// \brief Récupère la liste des îles.
         /// \return La liste des îles.
         public List<Ile> GetIles()
         {
-            return DataManager.GetIles().ToList();
+            var iles = DataManager.GetIles();
+            return iles != null ? iles.ToList() : new List<Ile>();
         }
 
         /// \brief Filtre les fruits du démon par type.
diff --git a/Sources/Model/Serializer/XML_Serializer.cs b/Sources/Model/Serializer/XML_Serializer.cs
index 75d8ffd..69843e8 100644
--- a/Sources/Model/Serializer/XML_Serializer.cs
+++ b/Sources/Model/Serializer/XML_Serializer.cs
@@ -30,27 +30,39 @@ namespace Model.Serializer
         {
             if (!File.Exists(Path.Combine(Chemin, "./personnage.xml")))
             {
-                SetPersonnage(stubManager.GetPersonnages().ToList());
+                var personnages = stubManager.GetPersonnages();
+                if(personnages!=null)
+                    SetPersonnage(personnages.ToList());
             }
             if (!File.Exists(Path.Combine(Chemin, "./bateau.xml")))
             {
-                SetBateau(stubManager.GetBateaux().ToList());
+                var bateaux = stubManager.GetBateaux();
+                if(bateaux !=null)
+                    SetBateau(bateaux.ToList());
             }
             if (!File.Exists(Path.Combine(Chemin, "./fruitdudemon.xml")))
             {
-                SetFDD(stubManager.GetFruits().ToList());
+                var fruits = stubManager.GetFruits();
+                if(fruits!=null)
+                    SetFDD(fruits.ToList());
             }
             if (!File.Exists(Path.Combine(Chemin, "./bestiaire.xml")))
             {
-                SetBestiaire(stubManager.GetBestiaires().ToList());
+                var bestiaires= stubManager.GetBestiaires();
+                if(bestiaires!=null)
+                    SetBestiaire(bestiaires.ToList());
             }
             if (!File.Exists(Path.Combine(Chemin, "./equipage.xml")))
             {
-                SetEquipage(stubManager.GetEquipages().ToList());
+                var equipages= stubManager.GetEquipages();
+                if(equipages!=null)
+                    SetEquipage(equipages.ToList());
             }
             if (!File.Exists(Path.Combine(Chemin, "./ile.xml")))
             {
-                SetIle(stubManager.GetIles().ToList());
+                var iles = stubManager.GetIles();
+                if(iles!=null)
+                    SetIle(iles.ToList());
             }
         }
 
@@ -161,86 +173,74 @@ namespace Model.Serializer
             }
         }
 
-        public IEnumerable<Bateau> GetBateaux()
+        public IEnumerable<Bateau>? GetBateaux()
         {
-            List<Bateau>? listeBateau = new List<Bateau>();
+            
             var serializer = new DataContractSerializer(typeof(List<Bateau>));
             string xmlFile = "bateau.xml";
             Directory.SetCurrentDirectory(Path.Combine(Chemin, "./"));
-
             using (Stream s = File.OpenRead(xmlFile))
             {
-                listeBateau = serializer.ReadObject(s) as List<Bateau>;
-            }
-
-            
-            return listeBateau;
-
+                return serializer.ReadObject(s) as List<Bateau>;
+            }                     
         }
-        public IEnumerable<Bestiaire> GetBestiaires()
+        public IEnumerable<Bestiaire>? GetBestiaires()
         {
-            List<Bestiaire>? listeBest = new List<Bestiaire>();
+          
             var serializer = new DataContractSerializer(typeof(List<Bestiaire>));
             string xmlFile = "bestiaire.xml";
             Directory.SetCurrentDirectory(Path.Combine(Chemin, "./"));
 
             using (Stream s = File.OpenRead(xmlFile))
             {
-                listeBest = serializer.ReadObject(s) as List<Bestiaire>;
+                return  serializer.ReadObject(s) as List<Bestiaire>;
             }
-            return listeBest;
+           
         }
-        public IEnumerable<Equipage> GetEquipages()
-        {
-            List<Equipage>? listeEquip = new List<Equipage>();
+        public IEnumerable<Equipage>? GetEquipages()
+        {    
             var serializer = new DataContractSerializer(typeof(List<Equipage>));
             string xmlFile = "equipage.xml";
             Directory.SetCurrentDirectory(Path.Combine(Chemin, "./"));
 
             using (Stream s = File.OpenRead(xmlFile))
             {
-                listeEquip = serializer.ReadObject(s) as List<Equipage>;
+                return serializer.ReadObject(s) as List<Equipage>;
             }
-            return listeEquip;
+           
         }
-        public IEnumerable<FruitDuDemon> GetFruits()
+        public IEnumerable<FruitDuDemon>? GetFruits()
         {
-            List<FruitDuDemon>? listeFDD = new List<FruitDuDemon>();
             var serializer = new DataContractSerializer(typeof(List<FruitDuDemon>));
             string xmlFile = "fruitdudemon.xml";
             Directory.SetCurrentDirectory(Path.Combine(Chemin, "./"));
 
             using (Stream s = File.OpenRead(xmlFile))
             {
-                listeFDD = serializer.ReadObject(s) as List<FruitDuDemon>;
+                return serializer.ReadObject(s) as List<FruitDuDemon>;
             }
-            return listeFDD;
         }
-        public IEnumerable<Ile> GetIles()
+        public IEnumerable<Ile>? GetIles()
         {
-            List<Ile>? listeIle = new List<Ile>();
             var serializer = new DataContractSerializer(typeof(List<Ile>));
             string xmlFile = "ile.xml";
             Directory.SetCurrentDirectory(Path.Combine(Chemin, "./"));
 
             using (Stream s = File.OpenRead(xmlFile))
             {
-                listeIle = serializer.ReadObject(s) as List<Ile>;
-            }
-            return listeIle;
+                return serializer.ReadObject(s) as List<Ile>;
+            }       
         }
-        public IEnumerable<Personnage> GetPersonnages()
+        public IEnumerable<Personnage>? GetPersonnages()
         {
-            List<Personnage>? listePerso = new List<Personnage>();
             var serializer = new DataContractSerializer(typeof(List<Personnage>));
             string xmlFile = "personnage.xml";
             Directory.SetCurrentDirectory(Path.Combine(Chemin, "./"));
-           
+
             using (Stream s = File.OpenRead(xmlFile))
             {
-                listePerso = serializer.ReadObject(s) as List<Personnage>;
+                return serializer.ReadObject(s) as List<Personnage>;
             }
-            return listePerso;
         }
     }
 }
diff --git a/Sources/Model/Stub/StubManager.cs b/Sources/Model/Stub/StubManager.cs
index 361dba6..135bbd3 100644
--- a/Sources/Model/Stub/StubManager.cs
+++ b/Sources/Model/Stub/StubManager.cs
@@ -30,30 +30,30 @@ namespace Model.Stub
         }
        
 
-        public IEnumerable<Personnage> GetPersonnages()
+        public IEnumerable<Personnage>? GetPersonnages()
         { 
          
             return StubPersonnage.RecupererPersonnage();
         }
 
-        public IEnumerable<FruitDuDemon> GetFruits()
+        public IEnumerable<FruitDuDemon>? GetFruits()
         {
             return StubFruitDuDemon.RecupererFruit();
         }
-        public IEnumerable<Equipage> GetEquipages()
+        public IEnumerable<Equipage>? GetEquipages()
         {
             StubEquipage.ChargerEquipage(StubPersonnage.RecupererPersonnage().ToList());
             return StubEquipage.RecupererEquipage();
         }
-        public IEnumerable<Bateau> GetBateaux()
+        public IEnumerable<Bateau>? GetBateaux()
         {
             return StubBateau.RecupererBateau();
         }
-        public IEnumerable<Bestiaire> GetBestiaires()
+        public IEnumerable<Bestiaire>? GetBestiaires()
         {
             return StubBestiaire.RecupererBestiaire();
         }
-        public IEnumerable<Ile> GetIles()
+        public IEnumerable<Ile>? GetIles()
         {
             return StubIle.RecupererIle();
         }
diff --git a/Sources/Ohara.sln b/Sources/Ohara.sln
index 25fd92f..b473946 100644
--- a/Sources/Ohara.sln
+++ b/Sources/Ohara.sln
@@ -12,6 +12,7 @@ EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "TestProject1", "TestProject1\TestProject1.csproj", "{4AD3B218-1007-4859-BC93-2B3E957632E0}"
 	ProjectSection(ProjectDependencies) = postProject
 		{A0307D66-E621-4BC1-A239-87C9021E7CFD} = {A0307D66-E621-4BC1-A239-87C9021E7CFD}
+		{EEC24A76-0EB3-4C67-A71B-7FF823713645} = {EEC24A76-0EB3-4C67-A71B-7FF823713645}
 	EndProjectSection
 EndProject
 Global
diff --git a/Sources/Ohara/AppShell.xaml.cs b/Sources/Ohara/AppShell.xaml.cs
index 250878a..85d2bde 100644
--- a/Sources/Ohara/AppShell.xaml.cs
+++ b/Sources/Ohara/AppShell.xaml.cs
@@ -4,10 +4,9 @@ using Model;
 
 public partial class AppShell : Shell
 {
-    
     public AppShell()
 	{
 		InitializeComponent();
-		
-	}
+      
+    }
 }
diff --git a/Sources/Ohara/PageBateau.xaml b/Sources/Ohara/PageBateau.xaml
index b947eda..6337576 100644
--- a/Sources/Ohara/PageBateau.xaml
+++ b/Sources/Ohara/PageBateau.xaml
@@ -7,7 +7,7 @@
              BackgroundColor="#e2edf1">
 
 
-    <VerticalStackLayout  Spacing="40">
+    <VerticalStackLayout >
         <Grid ColumnDefinitions="200,*,100,10,150" BackgroundColor="#72a3b3" Padding="10">
             <SearchBar x:Name="searchBar" Placeholder="Rechercher..." BackgroundColor="#bfe5ef" Grid.Column="0"/>
             <Button Text="Ajouter" Clicked="Button_Clicked" Style="{StaticResource buttonBarre}" Grid.Column="4"/>
@@ -17,35 +17,20 @@
             <CollectionView x:Name="listeBateau" ItemsSource="{Binding Bateaux}" ItemsLayout="HorizontalList" EmptyView="Aucun résultat trouvé." SelectionMode="Single" SelectionChanged="listeBateau_SelectionChanged" >
                 <CollectionView.ItemTemplate>
                     <DataTemplate>
-
                         <Grid Padding="50" ColumnSpacing="10">
                             <Grid.ColumnDefinitions>
                                 <ColumnDefinition Width="400"/>
                             </Grid.ColumnDefinitions>
                             <Grid.RowDefinitions>
-                                <RowDefinition Height="750"/>
+                                <RowDefinition Height="80*"/>
                             </Grid.RowDefinitions>
-
-
-                            <Frame
-                                    CornerRadius="25"
-                                    BorderColor="#e2edf1"
-                                    IsClippedToBounds="True"
-                                    Padding="0"
-                                    HeightRequest="800"
-                                    WidthRequest="450">
+                            <Frame Style="{StaticResource frameObjet3}"  HeightRequest="800" WidthRequest="450">
                                 <Image
-                                        Source="{Binding Image}"
-                                        
+                                        Source="{Binding Image}"                                     
                                         Aspect="Fill"
                                         />
                             </Frame>
-
-
-
                             <Frame Style="{StaticResource frameObjet2}">
-
-
                                 <StackLayout Orientation="Vertical" Spacing="25" VerticalOptions="Center">
                                     <Label 
                                                 Text="{Binding Nom}" 
diff --git a/Sources/Ohara/PageIle.xaml b/Sources/Ohara/PageIle.xaml
index e45d266..f2db43d 100644
--- a/Sources/Ohara/PageIle.xaml
+++ b/Sources/Ohara/PageIle.xaml
@@ -5,7 +5,7 @@
              Title="PageIle"
              Appearing="ContentPage_Appearing"  
              BackgroundColor="#e2edf1">
-    <VerticalStackLayout Spacing="40">
+    <VerticalStackLayout>
         <Grid ColumnDefinitions="200,*,100,10,150"  BackgroundColor="#72a3b3" Padding="10">
             <SearchBar x:Name="searchBar" Placeholder="Rechercher..."  Style="{StaticResource searchBarOhara}" Grid.Column="0"/>
             <Button Text="Ajouter" Clicked="Button_Clicked" Style="{StaticResource buttonBarre}" Grid.Column="4"/>
@@ -31,15 +31,9 @@
                                 <ColumnDefinition Width="400"/>
                             </Grid.ColumnDefinitions>
                             <Grid.RowDefinitions>
-                                <RowDefinition Height="750"/>
+                                <RowDefinition Height="80*"/>
                             </Grid.RowDefinitions>
-                            <Frame
-                                    CornerRadius="25"
-                                    BorderColor="#e2edf1"
-                                    IsClippedToBounds="True"
-                                    Padding="0"
-                                    HeightRequest="800"
-                                    WidthRequest="450">
+                            <Frame Style="{StaticResource frameObjet3}" HeightRequest="800" WidthRequest="450">
                                 <Image
                                         Source="{Binding Image}"
                                         
diff --git a/Sources/Ohara/PagePersonnage.xaml b/Sources/Ohara/PagePersonnage.xaml
index 1d539fe..afbd861 100644
--- a/Sources/Ohara/PagePersonnage.xaml
+++ b/Sources/Ohara/PagePersonnage.xaml
@@ -7,7 +7,7 @@
              BackgroundColor="#e2edf1">
 
   
-    <VerticalStackLayout Spacing="10">
+    <VerticalStackLayout>
         <Grid ColumnDefinitions="200,*,150" BackgroundColor="#72a3b3" Padding="10">
             <SearchBar x:Name="searchBar" Placeholder="Rechercher..." BackgroundColor="#bfe5ef" Grid.Column="0"/>
             <Button Text="Ajouter" Clicked="ButtonAjouter_Clicked" Style="{StaticResource buttonBarre}" Grid.Column="2"/>
@@ -26,16 +26,9 @@
                                     <RowDefinition Height="*"/>
                                 </Grid.RowDefinitions>
 
-                                <Frame  CornerRadius="25"
-                                    BorderColor="#e2edf1"
-                                    IsClippedToBounds="True"
-                                    Padding="0"
-                                    HeightRequest="800"
-                                    WidthRequest="450">
-
-                                    <Image
-                                        Source="{Binding Image}"
-                                        
+                            <Frame Style="{StaticResource frameObjet3}" HeightRequest="800" WidthRequest="450">
+                                <Image
+                                        Source="{Binding Image}"                                     
                                         Aspect="Fill"
                                         />
                                 </Frame>
diff --git a/Sources/Ohara/Resources/Styles/Styles.xaml b/Sources/Ohara/Resources/Styles/Styles.xaml
index ed0be77..8d441b1 100644
--- a/Sources/Ohara/Resources/Styles/Styles.xaml
+++ b/Sources/Ohara/Resources/Styles/Styles.xaml
@@ -478,6 +478,12 @@
             </VisualStateGroupList>
         </Setter>
     </Style>
+    <Style x:Key="frameObjet3" TargetType="Frame">
+        <Setter Property="IsClippedToBounds" Value="True"/>
+        <Setter Property="Padding" Value="0"/>
+        <Setter Property="BorderColor" Value="Transparent"/>
+        <Setter Property="CornerRadius" Value="25"/>
+    </Style>
     <Style x:Key="frameEquip" TargetType="Frame">
         <Setter Property="VisualStateManager.VisualStateGroups">
             <VisualStateGroupList>
diff --git a/Sources/TestConsole/Program.cs b/Sources/TestConsole/Program.cs
index 323dacd..c99a4b6 100644
--- a/Sources/TestConsole/Program.cs
+++ b/Sources/TestConsole/Program.cs
@@ -4,7 +4,6 @@ using Model.Serializer;
 using Model.Managers;
 using System.Collections.ObjectModel;
 
-
 // Tests fonctionnels :
 Manager manager = new Manager(new XML_Serializer());
 
diff --git a/Sources/TestProject1/TestBateau.cs b/Sources/TestProject1/TestBateau.cs
index af9698a..1b99ce4 100644
--- a/Sources/TestProject1/TestBateau.cs
+++ b/Sources/TestProject1/TestBateau.cs
@@ -37,9 +37,36 @@ namespace TestProject1
         {
             Bateau bateau1 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
             Bateau bateau2 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
+            Bateau? bateau3 = null;
+            Equipage equipage = new Equipage("Équipage du poux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+
             bool resultat =(bateau1.Equals(bateau2));
+            bool resultat2 =(bateau1.Equals(equipage));
+            bool resultat3 =(bateau1.Equals(bateau3));
+
             Assert.True(resultat, "Les deux bateaux devraient etre égaux car ils onts le meme nom");
+            Assert.False(resultat2);
+            Assert.False(resultat3);
+        }
+        [Fact]
+        public void GetHashCode_BateauWithSameProperties_ReturnsSameHashCode()
+        {
+            Bateau obj1 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
+            Bateau obj2 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
+
+            var hashCode1 = obj1.GetHashCode();
+            var hashCode2 = obj2.GetHashCode();
+
+            Assert.True(hashCode1 == hashCode2);
+        }
+        [Fact]
+        public void ToString_ReturnsCorrectStringRepresentation()
+        {
+            Bateau obj1 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
+
+            var result = obj1.ToString();
 
+            Assert.True("Bateau : Sunny False Sauzando Sani-go  435 321 Le Thousand Sunny est... Ce bateau a pour particularités ...  " == result);
         }
     }
 }
diff --git a/Sources/TestProject1/TestEquipage.cs b/Sources/TestProject1/TestEquipage.cs
index 795d373..911f8bc 100644
--- a/Sources/TestProject1/TestEquipage.cs
+++ b/Sources/TestProject1/TestEquipage.cs
@@ -31,19 +31,58 @@ namespace TestProject1
             Assert.True(resultat, "L'image devrait avoir la valeur : baseimage.png");
         }
         [Fact]
+        public void ParametreCapitaine_GetSetMarcheCorrectement()
+        {
+            Equipage test = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+            Personnage test2 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, 1.74, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !");
+
+            test.Capitaine=test2;
+
+            Assert.True(test.Capitaine==test2);
+        }
+        [Fact]
+        public void RetirerMembreViderMembre_FonctionnentCorrectement()
+        {
+            Personnage test = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, 1.74, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !");
+            FruitDuDemon test2 = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "a", "b", "c", "d");
+
+            test.AjouterFruit(test2);
+            test.RetierFruit(test2);
+
+            Assert.DoesNotContain(test2, test.Fruit);    
+        }
+        [Fact]
+        public void RetirerAllieViderAllie_FonctionnentCorrectement()
+        {
+            Equipage test = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+            Equipage test2 = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+
+
+            test.AjouterAllie(test2);
+            test.RetirerAllie(test2);
+
+            Assert.DoesNotContain(test2, test.Allie);
+
+            test.AjouterAllie(test2);
+            test.ViderAllie();
+
+            Assert.DoesNotContain(test2, test.Allie);
+        }
+        [Fact]
         public void SurchargeEqualsEquipage_Equipage1EgaleEquipage2()
         {
             Equipage equiapge1= new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
             Equipage equiapge2 = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
             Equipage equiapge3 = new Equipage("Équipage du poux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
             Bateau bateau1 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
+
             bool resultat = (equiapge1.Equals(equiapge2));
             bool resultat2 = (equiapge1.Equals(equiapge3));
             bool resultat3 = (equiapge1.Equals(bateau1));
+
             Assert.True(resultat, "Les deux equipages devraient etre égaux car ils ont le meme nom");
             Assert.False(resultat2, "Les deux equipages  ne devraient pas etre égaux car ils n'ont pas le meme nom");
             Assert.False(resultat3, "Les deux equipages  ne devraient pas etre égaux car ils n'ont pas le meme type");
-
         }
         [Fact]
         public void GetHashCode_EquipageWithSameProperties_ReturnsSameHashCode()
@@ -56,5 +95,14 @@ namespace TestProject1
 
             Assert.True(hashCode1 == hashCode2);
         }
+        [Fact]
+        public void ToString_ReturnsCorrectStringRepresentation()
+        {
+            Equipage obj1 = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+
+            var result = obj1.ToString();
+
+            Assert.True("Equipage : Équipage du Roux False Akagami Kalzokudan East Blue 0 0 True L'équipage du Roux ... baseimage.png" == result);
+        }
     }
 }
diff --git a/Sources/TestProject1/TestFDD.cs b/Sources/TestProject1/TestFDD.cs
index 3c92786..b20b792 100644
--- a/Sources/TestProject1/TestFDD.cs
+++ b/Sources/TestProject1/TestFDD.cs
@@ -13,7 +13,7 @@ namespace TestProject1
         public void FDD_PremierChapEtPremierEpSuperieurOuEgalAZero_ReturnTrue()
         {
             FruitDuDemon test = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "", "", "");
-            bool resultat = (test.Image == "baseimage.png");
+            bool resultat = (test.PremierChap >0 && test.PremierEp >0);
             Assert.True(resultat, "Les paramètre PremierChap et PremierEp doivent être supérieur ou égale à 0");
         }
         [Fact]
@@ -36,9 +36,33 @@ namespace TestProject1
         {
             FruitDuDemon fruit1 = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "", "", "", "");
             FruitDuDemon fruit2 = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "", "", "", "");
+            FruitDuDemon? fruit3 =null;
             bool resultat = (fruit1.Equals(fruit2));
+            bool resultat2 = (fruit2.Equals(fruit3));
             Assert.True(resultat, "Les deux fruits du démon devraient etre égaux car ils onts le meme nom");
+            Assert.False(resultat2, "Les deux fruits du démon devraient etre égaux car l'un dentre eux est null");
+
+        }
+        [Fact]
+        public void GetHashCode_FruitDuDemonWithSameProperties_ReturnsSameHashCode()
+        {
+            FruitDuDemon obj1 = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "", "", "", "");
+            FruitDuDemon obj2 = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "", "", "", "");
+
+            var hashCode1 = obj1.GetHashCode();
+            var hashCode2 = obj2.GetHashCode();
+
+            Assert.True(hashCode1 == hashCode2);
+        }
+
+        [Fact]
+        public void ToString_ReturnsCorrectStringRepresentation()
+        {
+            FruitDuDemon obj1 = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "a", "b", "c", "d");
+
+            var result = obj1.ToString();
 
+            Assert.True("FruitDuDemon : Fruit de la fumée False Moku Moku No Mi Logia 97 48 a b c d" == result);
         }
     }
 }
diff --git a/Sources/TestProject1/TestIle.cs b/Sources/TestProject1/TestIle.cs
index 04d241c..62e1d2d 100644
--- a/Sources/TestProject1/TestIle.cs
+++ b/Sources/TestProject1/TestIle.cs
@@ -35,8 +35,34 @@ namespace TestProject1
         {
             Ile ile1 = new Ile("Dawn", "Don-to", "East Blue", 1, 4, "L'île de Dawn est ...", "Cette île est situé dans la mer d'East Blue près de ...", "");
             Ile ile2 = new Ile("Dawn", "Don-to", "East Blue", 1, 4, "L'île de Dawn est ...", "Cette île est situé dans la mer d'East Blue près de ...", "");
+            FruitDuDemon fruit2 = new FruitDuDemon("Fruit de la fumée", "Moku Moku No Mi", "Logia", 97, 48, "", "", "", "");
+            Ile? ile3 = null;
             bool resultat = (ile1.Equals(ile2));
+            bool resultat2 = (ile1.Equals(ile3));
+            bool resultat3 = (ile1.Equals(fruit2));
             Assert.True(resultat, "Les iles devraient etre égales car ils onts le meme nom");
+            Assert.False(resultat2, "Les iles devraient etre égales car l'une d'entre elle est null");
+            Assert.False(resultat3, "Ce test devraient etre false car il comparent une ile et un fruitdudemon");
+        }
+        [Fact]
+        public void GetHashCode_IleWithSameProperties_ReturnsSameHashCode()
+        {
+            Ile obj1 = new Ile("Dawn", "Don-to", "East Blue", 1, 4, "L'île de Dawn est ...", "Cette île est situé dans la mer d'East Blue près de ...", "");
+            Ile obj2 = new Ile("Dawn", "Don-to", "East Blue", 1, 4, "L'île de Dawn est ...", "Cette île est situé dans la mer d'East Blue près de ...", "");
+
+            var hashCode1 = obj1.GetHashCode();
+            var hashCode2 = obj2.GetHashCode();
+
+            Assert.True(hashCode1 == hashCode2);
+        }
+        [Fact]
+        public void ToString_ReturnsCorrectStringRepresentation()
+        {
+            Ile obj1 = new Ile("Dawn", "Don-to", "East Blue", 1, 4, "L'île de Dawn est ...", "Cette île est situé dans la mer d'East Blue près de ...", "");
+
+            var result = obj1.ToString();
+
+            Assert.True("Ile : Dawn Don-to East Blue 1 4 L'île de Dawn est ... Cette île est situé dans la mer d'East Blue près de ... baseimage.png" == result);
         }
     }
 }
diff --git a/Sources/TestProject1/TestPersonnage.cs b/Sources/TestProject1/TestPersonnage.cs
index c44aa7a..c76a13a 100644
--- a/Sources/TestProject1/TestPersonnage.cs
+++ b/Sources/TestProject1/TestPersonnage.cs
@@ -36,18 +36,48 @@ namespace TestProject1
         {
             Personnage test = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
             bool resultat = (test.Taille >= 0);
+
             Assert.True(resultat, "La taille du personnage doit avoir une valeur positive");
         }
         [Fact]
+        public void Personnage_EquipageSetFonctionneCorrectement()
+        {
+            Personnage test = new Personnage("Luffy", -100, "Luffy au Chapeau de Paille", 19, 1.74, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
+            Equipage test2 = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+
+            test.Equipage = test2;
+            bool resultat = (test.Equipage == test2);
+
+            Assert.True(resultat, "La valeur Equipage du personnage devrait etre égal à test2");
+        }
+        [Fact]
+        public void PersonnageRetirerFruit_FonctionneCorrectement()
+        {
+            Equipage test = new Equipage("Équipage du Roux", "Akagami Kalzokudan", "East Blue", -1, 0, true, "L'équipage du Roux ...", "");
+            Personnage test2 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, 1.74, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !");
+
+            test.AjouterMembre(test2);
+            test.RetirerMembre(test2);
+
+            Assert.DoesNotContain(test2, test.Membre);
+
+            test.AjouterMembre(test2);
+            test.ViderMembre();
+
+            Assert.DoesNotContain(test2, test.Membre);
+        }
+        [Fact]
         public void SurchargeEqualsPersonnage_Personnage1EgalePersonnage2()
         {
             Personnage personnage1 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
             Personnage personnage2 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
             Personnage personnage3 = new Personnage("Lufty", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
             Bateau bateau1 = new Bateau("Sunny", "Sauzando Sani-go", 435, 321, "Le Thousand Sunny est...", "Ce bateau a pour particularités ...", " ");
+
             bool resultat = (personnage1.Equals(personnage1));
             bool resultat2 = (personnage1.Equals(personnage3));
             bool resultat3 = (personnage1.Equals(bateau1));
+
             Assert.True(resultat, "Les personanges devraient etre égaux car ils ont le meme nom");
             Assert.False(resultat2, "Les personanges ne devraient pas etre égaux car ils n'ont pas le meme nom");
             Assert.False(resultat3, "Les deux objets ne devraient pas etre égaux car ils n'ont pas le meme type");
@@ -63,5 +93,14 @@ namespace TestProject1
 
             Assert.True(hashCode1 == hashCode2);
         }
+        [Fact]
+        public void ToString_ReturnsCorrectStringRepresentation()
+        {
+            Personnage obj1 = new Personnage("Luffy", 3000000000, "Luffy au Chapeau de Paille", 19, -1, "East Blue", "Monkey D. Luffy est...", "Le Roi des Pirates, ce sera moi !", "luffy.png");
+
+            var result = obj1.ToString();
+
+            Assert.True("Personnage : Luffy False 3000000000 Luffy au Chapeau de Paille 19 East Blue Monkey D. Luffy est... Le Roi des Pirates, ce sera moi !  System.Collections.ObjectModel.ObservableCollection`1[Model.Classes.FruitDuDemon] luffy.png" == result);
+        }
     }
 }

From 6c58c3218f130812c3b5210509b3f341321c0565 Mon Sep 17 00:00:00 2001
From: Yoan <yoan2004.yb@gmail.com>
Date: Wed, 14 Jun 2023 18:10:03 +0200
Subject: [PATCH 3/5] Tests unitaires, Int2Color converter et test coverage

---
 .drone.yml                                    |  2 +-
 Sources/Model/Managers/Manager.cs             | 18 +++---
 .../Ohara/Converters/Int2ColorConverter.cs    | 31 ++++++++++
 Sources/Ohara/ModalBateau.xaml                | 10 +++-
 Sources/Ohara/ModalBateau.xaml.cs             | 12 ++++
 Sources/Ohara/ModalEquipage.xaml              |  8 ++-
 Sources/Ohara/ModalEquipage.xaml.cs           | 10 ++++
 Sources/Ohara/ModalIle.xaml                   |  8 ++-
 Sources/Ohara/ModalIle.xaml.cs                | 10 ++++
 Sources/Ohara/PageFavoris.xaml.cs             |  2 +-
 Sources/TestConsole/Program.cs                |  4 +-
 Sources/TestProject1/TestBestiaire.cs         | 20 +++++++
 Sources/TestProject1/TestManager.cs           |  2 +-
 Sources/TestProject1/XMLSerializerTests.cs    | 59 +++++++++++++++++--
 14 files changed, 172 insertions(+), 24 deletions(-)
 create mode 100644 Sources/Ohara/Converters/Int2ColorConverter.cs

diff --git a/.drone.yml b/.drone.yml
index 1cbf34b..4f6cb47 100644
--- a/.drone.yml
+++ b/.drone.yml
@@ -30,7 +30,7 @@ steps:
       sonar_token:
         from_secret: SECRET_SONAR_LOGIN
       project_key: Ohara
-      coverage_exclusions: "Tests/**"
+      coverage_exclusions: "Tests/**","Ohara/**"
     commands:
       - cd Sources/
       - dotnet restore Ohara.sln
diff --git a/Sources/Model/Managers/Manager.cs b/Sources/Model/Managers/Manager.cs
index 6f5e08a..d28ebfb 100644
--- a/Sources/Model/Managers/Manager.cs
+++ b/Sources/Model/Managers/Manager.cs
@@ -181,15 +181,15 @@ namespace Model.Managers
         }
         /// \brief Récupère la liste des objets Ohara ayant le parametre EstFavori égal a true.
         /// \return La liste des objets Ohara ayant le parametre EstFavori égal a true.
-        public List<ObjetOhara> GetFavoris()
-        {
-            List<ObjetOhara> listeFavoris = new List<ObjetOhara>();
-            listeFavoris.AddRange(Bateaux);
-            listeFavoris.AddRange(Equipages);
-            listeFavoris.AddRange(Bestiaire);
-            listeFavoris.AddRange(Fruits);
-            listeFavoris.AddRange(Iles);
-            listeFavoris.AddRange(Personnages);
+        public HashSet<ObjetOhara> GetFavoris()
+        {
+            HashSet<ObjetOhara> listeFavoris = new HashSet<ObjetOhara>();
+            listeFavoris.UnionWith(Bateaux);
+            listeFavoris.UnionWith(Equipages);
+            listeFavoris.UnionWith(Bestiaire);
+            listeFavoris.UnionWith(Fruits);
+            listeFavoris.UnionWith(Iles);
+            listeFavoris.UnionWith(Personnages);
             foreach(ObjetOhara obj in listeFavoris.ToList())
             {
                 if (obj.EstFavori == false)
diff --git a/Sources/Ohara/Converters/Int2ColorConverter.cs b/Sources/Ohara/Converters/Int2ColorConverter.cs
new file mode 100644
index 0000000..bb3f5b4
--- /dev/null
+++ b/Sources/Ohara/Converters/Int2ColorConverter.cs
@@ -0,0 +1,31 @@
+using System;
+using System.Collections.Generic;
+using System.Globalization;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace Ohara.Converters
+{
+    public class Int2ColorConverter : IValueConverter
+    {
+        public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
+        {
+            if (value is not int) return Colors.Red;
+            switch (value)
+            {
+                case < 0:
+                    return Colors.Orange;
+                case >0:
+                    return Colors.Green;
+                default:
+                    return Colors.White;
+            }
+        }
+
+        public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
+        {
+            throw new NotImplementedException();
+        }
+    }
+}
diff --git a/Sources/Ohara/ModalBateau.xaml b/Sources/Ohara/ModalBateau.xaml
index b60df87..237f75c 100644
--- a/Sources/Ohara/ModalBateau.xaml
+++ b/Sources/Ohara/ModalBateau.xaml
@@ -2,9 +2,13 @@
 <ContentPage xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              x:Class="Ohara.ModalBateau"
+             xmlns:conv="clr-namespace:Ohara.Converters"
              Title="ModalBateau"
-              BackgroundColor="#e2edf1"
+             BackgroundColor="#e2edf1"
              Shell.PresentationMode="ModalAnimated">
+    <ContentPage.Resources>
+        <conv:Int2ColorConverter x:Key="int2ColorConverter"/>
+    </ContentPage.Resources>
     <ScrollView>
         <FlexLayout  AlignItems="End" Wrap="Wrap" Direction="Row"  JustifyContent="Center" HorizontalOptions="Center" VerticalOptions="Center" >
             <VerticalStackLayout Spacing="4" Margin="2">
@@ -24,13 +28,13 @@
                 </Grid>
                 <Frame  Style="{StaticResource frameModif}">
                     <HorizontalStackLayout HorizontalOptions="Center">
-                        <Label Text="Premier Chapitre :" FontAttributes="Bold"/>
+                        <Label Text="Premier Chapitre :" FontAttributes="Bold" TextColor="{Binding PremierChap, Converter={StaticResource int2ColorConverter}}"/>
                         <Entry Text="{Binding PremierChap}"  />
                     </HorizontalStackLayout>
                 </Frame>
                 <Frame  Style="{StaticResource frameModif}">
                     <HorizontalStackLayout HorizontalOptions="Center">
-                        <Label Text="Premier Episode :" FontAttributes="Bold"/>
+                        <Label Text="Premier Episode :" FontAttributes="Bold" TextColor="{Binding PremierEp, Converter={StaticResource int2ColorConverter}}"/>
                         <Entry Text="{Binding PremierEp}" />
                     </HorizontalStackLayout >
                 </Frame>
diff --git a/Sources/Ohara/ModalBateau.xaml.cs b/Sources/Ohara/ModalBateau.xaml.cs
index 6b09d8e..d48bdcd 100644
--- a/Sources/Ohara/ModalBateau.xaml.cs
+++ b/Sources/Ohara/ModalBateau.xaml.cs
@@ -1,3 +1,4 @@
+
 using Model.Classes;
 using Model.Managers;
 
@@ -27,12 +28,23 @@ public partial class ModalBateau : ContentPage
     {
         if (manager.SelectedItem != null)
         {
+            if (nouveauBateau.PremierChap < 0 || nouveauBateau.PremierEp < 0)
+            {
+                await DisplayAlert("Erreur", "Les paramètres PremierChap et PremierEp doivent être superieur à 0.", "OK");
+                return;
+            }
             manager.ModifierBateau(nouveauBateau, ancienNom);
             nouveauBateau = manager.SelectedItem as Bateau;
         }
         else
         {
+            if (nouveauBateau.PremierChap<0 || nouveauBateau.PremierEp < 0)
+            {
+                await DisplayAlert("Erreur", "Les paramètres PremierChap et PremierEp doivent être superieur à 0.", "OK");
+                return;
+            }
             manager.AjouterBateau(nouveauBateau);
+
         }
 
         await Navigation.PopModalAsync();
diff --git a/Sources/Ohara/ModalEquipage.xaml b/Sources/Ohara/ModalEquipage.xaml
index 4866043..ef5b200 100644
--- a/Sources/Ohara/ModalEquipage.xaml
+++ b/Sources/Ohara/ModalEquipage.xaml
@@ -2,9 +2,13 @@
 <ContentPage xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              x:Class="Ohara.ModalEquipage"
+             xmlns:conv="clr-namespace:Ohara.Converters"
              Title="ModalEquipage"
              BackgroundColor="#e2edf1"
              Shell.PresentationMode="ModalAnimated">
+    <ContentPage.Resources>
+        <conv:Int2ColorConverter x:Key="int2ColorConverter"/>
+    </ContentPage.Resources>
     <ScrollView>
         <FlexLayout  AlignItems="End" Wrap="Wrap" Direction="Row"  JustifyContent="Center" HorizontalOptions="Center" VerticalOptions="Center" >
             <VerticalStackLayout Spacing="4" Margin="2">
@@ -36,13 +40,13 @@
                 </Frame>
                 <Frame  Style="{StaticResource frameModif}">
                     <HorizontalStackLayout HorizontalOptions="Center">
-                        <Label Text="Premier Chapitre :" FontAttributes="Bold"/>
+                        <Label Text="Premier Chapitre :" FontAttributes="Bold" TextColor="{Binding PremierChap, Converter={StaticResource int2ColorConverter}}"/>
                         <Entry Text="{Binding PremierChap}"  />
                     </HorizontalStackLayout>
                 </Frame>
                 <Frame  Style="{StaticResource frameModif}">
                     <HorizontalStackLayout HorizontalOptions="Center">
-                        <Label Text="Premier Episode :" FontAttributes="Bold"/>
+                        <Label Text="Premier Episode :" FontAttributes="Bold" TextColor="{Binding PremierEp, Converter={StaticResource int2ColorConverter}}"/>
                         <Entry Text="{Binding PremierEp}" />
                     </HorizontalStackLayout >
                 </Frame>
diff --git a/Sources/Ohara/ModalEquipage.xaml.cs b/Sources/Ohara/ModalEquipage.xaml.cs
index a50c0d1..2e7256e 100644
--- a/Sources/Ohara/ModalEquipage.xaml.cs
+++ b/Sources/Ohara/ModalEquipage.xaml.cs
@@ -30,11 +30,21 @@ public partial class ModalEquipage : ContentPage
     {
         if (manager.SelectedItem != null)
         {
+            if (nouvelEquipage.PremierChap < 0 || nouvelEquipage.PremierEp < 0)
+            {
+                await DisplayAlert("Erreur", "Les paramètres PremierChap et PremierEp doivent être superieur à 0.", "OK");
+                return;
+            }
             manager.ModifierEquipage(nouvelEquipage, ancienNom);
             nouvelEquipage = manager.SelectedItem as Equipage;
         }
         else
         {
+            if (nouvelEquipage.PremierChap < 0 || nouvelEquipage.PremierEp < 0)
+            {
+                await DisplayAlert("Erreur", "Les paramètres PremierChap et PremierEp doivent être superieur à 0.", "OK");
+                return;
+            }
             manager.AjouterEquip(nouvelEquipage);
         }
 
diff --git a/Sources/Ohara/ModalIle.xaml b/Sources/Ohara/ModalIle.xaml
index 91c8a4d..d94a965 100644
--- a/Sources/Ohara/ModalIle.xaml
+++ b/Sources/Ohara/ModalIle.xaml
@@ -2,9 +2,13 @@
 <ContentPage xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              x:Class="Ohara.ModalIle"
+             xmlns:conv="clr-namespace:Ohara.Converters"
              Title="ModalIle"
              BackgroundColor="#e2edf1"
              Shell.PresentationMode="ModalAnimated">
+    <ContentPage.Resources>
+        <conv:Int2ColorConverter x:Key="int2ColorConverter"/>
+    </ContentPage.Resources>
     <ScrollView>
         <FlexLayout  AlignItems="End" Wrap="Wrap" Direction="Row"  JustifyContent="Center" HorizontalOptions="Center" VerticalOptions="Center" >
             <VerticalStackLayout Spacing="4" Margin="2">
@@ -36,13 +40,13 @@
                 </Frame>
                 <Frame  Style="{StaticResource frameModif}">
                     <HorizontalStackLayout HorizontalOptions="Center">
-                        <Label Text="Premier Chapitre :" FontAttributes="Bold"/>
+                        <Label Text="Premier Chapitre :" FontAttributes="Bold" TextColor="{Binding PremierChap, Converter={StaticResource int2ColorConverter}}"/>
                         <Entry Text="{Binding PremierChap}"  />
                     </HorizontalStackLayout>
                 </Frame>
                 <Frame  Style="{StaticResource frameModif}">
                     <HorizontalStackLayout HorizontalOptions="Center">
-                        <Label Text="Premier Episode :" FontAttributes="Bold"/>
+                        <Label Text="Premier Episode :" FontAttributes="Bold" TextColor="{Binding PremierEp, Converter={StaticResource int2ColorConverter}}"/>
                         <Entry Text="{Binding PremierEp}" />
                     </HorizontalStackLayout >
                 </Frame>
diff --git a/Sources/Ohara/ModalIle.xaml.cs b/Sources/Ohara/ModalIle.xaml.cs
index 16292d3..2959dd4 100644
--- a/Sources/Ohara/ModalIle.xaml.cs
+++ b/Sources/Ohara/ModalIle.xaml.cs
@@ -29,11 +29,21 @@ public partial class ModalIle : ContentPage
     {
         if (manager.SelectedItem != null)
         {
+            if (nouvelIle.PremierChap < 0 || nouvelIle.PremierEp < 0)
+            {
+                await DisplayAlert("Erreur", "Les paramètres PremierChap et PremierEp doivent être superieur à 0.", "OK");
+                return;
+            }
             manager.ModifierIle(nouvelIle,ancienNom);
             manager.SelectedItem=nouvelIle;
         }
         else
         {
+            if (nouvelIle.PremierChap < 0 || nouvelIle.PremierEp < 0)
+            {
+                await DisplayAlert("Erreur", "Les paramètres PremierChap et PremierEp doivent être superieur à 0.", "OK");
+                return;
+            }
             manager.AjouterIle(nouvelIle);
         }
 
diff --git a/Sources/Ohara/PageFavoris.xaml.cs b/Sources/Ohara/PageFavoris.xaml.cs
index cd7dec6..e1723ca 100644
--- a/Sources/Ohara/PageFavoris.xaml.cs
+++ b/Sources/Ohara/PageFavoris.xaml.cs
@@ -15,7 +15,7 @@ public partial class PageFavoris : ContentPage
         void OnTextChanged(object sender, EventArgs e)
         {
             SearchBar searchBar = (SearchBar)sender;
-            BindableLayout.SetItemsSource(listeFav, manager.RechercheObjetOhara(searchBar.Text, manager.GetFavoris()));
+            BindableLayout.SetItemsSource(listeFav, manager.RechercheObjetOhara(searchBar.Text, manager.GetFavoris().ToList()));
         }
         searchBar.TextChanged += OnTextChanged;
 
diff --git a/Sources/TestConsole/Program.cs b/Sources/TestConsole/Program.cs
index c99a4b6..329d999 100644
--- a/Sources/TestConsole/Program.cs
+++ b/Sources/TestConsole/Program.cs
@@ -4,6 +4,8 @@ using Model.Serializer;
 using Model.Managers;
 using System.Collections.ObjectModel;
 
+
+
 // Tests fonctionnels :
 Manager manager = new Manager(new XML_Serializer());
 
@@ -105,7 +107,7 @@ foreach (Bateau b in manager.Bateaux)
 
 
 // Recherche dansune liste de objet ohara pour afficher les objets ohara correspondant au mot clé "Sunny"
-foreach (ObjetOhara o in manager.RechercheObjetOhara("Sunny", manager.GetFavoris()))
+foreach (ObjetOhara o in manager.RechercheObjetOhara("Sunny", manager.GetFavoris().ToList()))
 {
     Console.WriteLine(o.Nom);
 }
diff --git a/Sources/TestProject1/TestBestiaire.cs b/Sources/TestProject1/TestBestiaire.cs
index 12ce704..7f0ad5f 100644
--- a/Sources/TestProject1/TestBestiaire.cs
+++ b/Sources/TestProject1/TestBestiaire.cs
@@ -37,5 +37,25 @@ namespace TestProject1
             Assert.False(resultat2);
             Assert.False(resultat3);
         }
+        [Fact]
+        public void GetHashCode_BestiaireWithSameProperties_ReturnsSameHashCode()
+        {
+            Bestiaire obj1 = new Bestiaire("Humains", "??", "Les humains sont ...", "Ils possèdent les caractéristiques suivantes ...", "");
+            Bestiaire obj2 = new Bestiaire("Humains", "??", "Les humains sont ...", "Ils possèdent les caractéristiques suivantes ...", "");
+
+            var hashCode1 = obj1.GetHashCode();
+            var hashCode2 = obj2.GetHashCode();
+
+            Assert.True(hashCode1 == hashCode2);
+        }
+        [Fact]
+        public void ToString_ReturnsCorrectStringRepresentation()
+        {
+            Bestiaire obj1 = new Bestiaire("Humains", "??", "Les humains sont ...", "Ils possèdent les caractéristiques suivantes ...", "");
+
+            var result = obj1.ToString();
+
+            Assert.True("Bestiaire : Humains False ?? Les humains sont ... Ils possèdent les caractéristiques suivantes ... baseimage.png" == result);
+        }
     }
 }
diff --git a/Sources/TestProject1/TestManager.cs b/Sources/TestProject1/TestManager.cs
index d2bf4e1..3eac1ac 100644
--- a/Sources/TestProject1/TestManager.cs
+++ b/Sources/TestProject1/TestManager.cs
@@ -110,7 +110,7 @@ namespace TestProject1
             var obj1 =manager.Fruits.First();
             var obj2 = manager.Equipages.First();
             manager.ModifierFavFDD(obj1, true);
-            var result = manager.GetFavoris();
+            var result = manager.GetFavoris().ToList();
           
             Assert.True(1==result.Count);
             Assert.Contains(obj1, result);
diff --git a/Sources/TestProject1/XMLSerializerTests.cs b/Sources/TestProject1/XMLSerializerTests.cs
index 99137a2..ee5442b 100644
--- a/Sources/TestProject1/XMLSerializerTests.cs
+++ b/Sources/TestProject1/XMLSerializerTests.cs
@@ -14,12 +14,63 @@ namespace TestProject1
     public class XMLSerializerTests
     {
         public XML_Serializer serializer = new XML_Serializer();
-
+        private string TestFolderPath = Directory.GetCurrentDirectory();
         [Fact]
-        public void InitialiserFichiers_FileDoesNotExist_SetPersonnageCalled()
+        public void InitialiserFichiers_FileDoesNotExist_SetPersonnageIfStubManagerReturnsNonNull()
         {
-            
-            Assert.True(File.Exists(Path.Combine(serializer.Chemin, "./personnage.xml")));
+            // Arrange
+            var serializer = new XML_Serializer(TestFolderPath);
+            var stubManager = new StubManager();
+            var expectedPersonnages = stubManager.GetPersonnages();
+            var expectedBateaux = stubManager.GetBateaux();
+            var expectedFruits = stubManager.GetFruits();
+            var expectedBestiaires= stubManager.GetBestiaires();
+            var expectedEquipages=stubManager.GetEquipages();
+            var expectedIles = stubManager.GetIles();
+
+            // Act
+            serializer.InitialiserFichiers(stubManager);
+
+            // Assert
+            if (!File.Exists(Path.Combine(TestFolderPath, "./personnage.xml")))
+            {
+                if (expectedPersonnages != null)
+                {
+                    Assert.True(expectedPersonnages.ToList()== serializer.GetPersonnages());
+                }
+            }
+
+            if (!File.Exists(Path.Combine(TestFolderPath, "./bateau.xml")))
+            {
+                if (expectedBateaux != null)
+                {
+                    Assert.True(expectedBateaux.ToList() == serializer.GetBateaux());
+                }
+            }
+
+            if (!File.Exists(Path.Combine(TestFolderPath, "./fruitdudemon.xml")))
+            {
+                if (expectedFruits != null)
+                {
+                    Assert.True(expectedFruits.ToList() == serializer.GetFruits());
+                }
+            }
+
+            if (!File.Exists(Path.Combine(TestFolderPath, "./equipage.xml")))
+            {
+                if (expectedEquipages != null)
+                {
+                    Assert.True(expectedEquipages.ToList() == serializer.GetEquipages());
+                }
+            }
+
+            if (!File.Exists(Path.Combine(TestFolderPath, "./ile.xml")))
+            {
+                if (expectedIles != null)
+                {
+                    Assert.True(expectedIles.ToList() == serializer.GetIles());
+                }
+            }
         }
         [Fact]
         public void SetPersonnage_SerializesAndWritesToFile()

From 7fdac10b645473882d7de78fa72546024ce462f9 Mon Sep 17 00:00:00 2001
From: Yoan <yoan2004.yb@gmail.com>
Date: Wed, 14 Jun 2023 18:15:17 +0200
Subject: [PATCH 4/5] test coverage

---
 .drone.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.drone.yml b/.drone.yml
index 4f6cb47..6a99bb1 100644
--- a/.drone.yml
+++ b/.drone.yml
@@ -30,7 +30,7 @@ steps:
       sonar_token:
         from_secret: SECRET_SONAR_LOGIN
       project_key: Ohara
-      coverage_exclusions: "Tests/**","Ohara/**"
+      coverage_exclusions: Tests/**
     commands:
       - cd Sources/
       - dotnet restore Ohara.sln

From 1429aef0fc0afe6d035fc55ad38def5ad0e58a22 Mon Sep 17 00:00:00 2001
From: Yoan <yoan2004.yb@gmail.com>
Date: Wed, 14 Jun 2023 18:43:30 +0200
Subject: [PATCH 5/5] tests coverage

---
 .drone.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.drone.yml b/.drone.yml
index 6a99bb1..bfdd95b 100644
--- a/.drone.yml
+++ b/.drone.yml
@@ -30,7 +30,7 @@ steps:
       sonar_token:
         from_secret: SECRET_SONAR_LOGIN
       project_key: Ohara
-      coverage_exclusions: Tests/**
+      coverage_exclusions: Tests/**,Ohara/**
     commands:
       - cd Sources/
       - dotnet restore Ohara.sln