From 6aed54d13f1684a991cdc160379053189a189d07 Mon Sep 17 00:00:00 2001
From: Nicolas BLONDEAU <niblondeau>
Date: Tue, 16 May 2023 15:24:23 +0200
Subject: [PATCH] Tests modifications Stub, test fonctions persistance et
 moidification bug xml -> ajout de l'ordre des attributs des classes User et
 Monstre

---
 Sources/Modèle/Monstre.cs          | 70 +++++++++++++++++------------
 Sources/Modèle/User.cs             | 13 ++++--
 Sources/Persistance/LoaderStub.cs   |  7 ++-
 Sources/Persistance/saves/users.xml | 48 +++++++++++++++-----
 4 files changed, 94 insertions(+), 44 deletions(-)

diff --git a/Sources/Modèle/Monstre.cs b/Sources/Modèle/Monstre.cs
index aafc51b..1c087aa 100644
--- a/Sources/Modèle/Monstre.cs
+++ b/Sources/Modèle/Monstre.cs
@@ -1,41 +1,53 @@
 using System.ComponentModel;
 using System.Reflection.Metadata;
+using System.Runtime.Serialization;
 
-namespace Model;
-
-public class Monstre
+namespace Model
 {
-    public int Id { get; set; }
-    public string Name { get; set; }
-    public string Dangerosite { get; private init; }
+
+    [DataContract]
+    public class Monstre
+    {
+        [DataMember(Order = 1)]
+        public int Id { get; set; }
+
+        [DataMember(Order = 2)]
+        public string Name { get; set; }
+
+        [DataMember(Order = 3)]
+        public string Dangerosite { get; private init; }
         //EN FAIT IL FAUDRAIT FAIRE UN ENUM DU TYPE DE DANGEROSITÉ, pour rajouter lors de
         //l'affichage de la liste des monstres une couleur selon ça,
         //genre rouge dangereux, violet hyper dangereux, et vert passif
-    public string Description { get; set; }
-    
-    
-    public List<string> CharacteristicsList
-    {
-        get; init;
-    }
 
-    
-    public List<string> AppearanceList
-    {
-        get; init;
-    }
+        [DataMember(Order = 4)] 
+        public string Description { get; set; }
 
-    public Monstre(int id, string name, string danger, string desc, List<string> characList, List<string> appearList)
-    {
-        Id = id;
-        Name = name;
-        Dangerosite = danger;
-        Description = desc;
-        CharacteristicsList = characList;
-        AppearanceList = appearList;
-        if (string.IsNullOrWhiteSpace(Name) || string.IsNullOrWhiteSpace(Description) || string.IsNullOrWhiteSpace(danger))
+        [DataMember(Order = 5)]
+        public List<string> CharacteristicsList
+        {
+            get; init;
+        }
+
+        [DataMember(Order = 6)]
+        public List<string> AppearanceList
         {
-            throw new ArgumentException("Un monstre doit avoir un nom, une description et une dangerosité!");
+            get; init;
+        }
+
+        public Monstre(int id, string name, string danger, string desc, List<string> characList, List<string> appearList)
+        {
+            Id = id;
+            Name = name;
+            Dangerosite = danger;
+            Description = desc;
+            CharacteristicsList = characList;
+            AppearanceList = appearList;
+            if (string.IsNullOrWhiteSpace(Name) || string.IsNullOrWhiteSpace(Description) || string.IsNullOrWhiteSpace(danger))
+            {
+                throw new ArgumentException("Un monstre doit avoir un nom, une description et une dangerosité!");
+            }
         }
     }
-}
\ No newline at end of file
+}
+
diff --git a/Sources/Modèle/User.cs b/Sources/Modèle/User.cs
index 85977e6..590aab8 100644
--- a/Sources/Modèle/User.cs
+++ b/Sources/Modèle/User.cs
@@ -18,14 +18,19 @@ namespace Model
     [DataContract]
     public class User
     {
-        [DataMember]
+        [DataMember(Order = 3)]
         public string Pseudo { get; private set; }
-        [DataMember]
+
+        [DataMember(Order = 1)]
         public string Nom { get; private set; }
-        [DataMember]
+
+        [DataMember(Order = 2)]
         public string Prenom { get; private set; }
-        [DataMember]
+
+        [DataMember(Order = 4)]
         private string Mdp { get; set; }
+
+
         [DataMember]
         public List<Monstre>? monstresDejaVu { get; private set; }
 
diff --git a/Sources/Persistance/LoaderStub.cs b/Sources/Persistance/LoaderStub.cs
index 259ebf1..878cd0b 100644
--- a/Sources/Persistance/LoaderStub.cs
+++ b/Sources/Persistance/LoaderStub.cs
@@ -20,7 +20,12 @@ namespace Persistance
             List<User> lu = new List<User>();
             lu.Add(new User("DedeDu42", "dede", "dodo", "mdp", new List<Monstre> { }));
             lu.Add(new User("Moi", "Monchanin", "Liam", "feur", new List<Monstre> { }));
-            lu.Add(new User("Nikoala", "Blondeau", "Nicolas", "niblondeau", new List<Monstre> { }));
+            lu.Add(new User("Nikoala", "Blondeau", "Nicolas", "niblondeau", new List<Monstre> { 
+                                                                                    new Monstre(2, 
+                                                                                    "Mouton", "passif", "Je suis présent un peu partout, sauf dnas le desert.", 
+                                                                                    new List<string> { "Avec une cisaille il est possible de raser la laine d'un mouton, il se retrouvera sans laine.","Pour faire repousser la laine d'un mouton, il faut qu'il ait de l'herbe sous ses pattes pour qu'il puisse manger. Une fois l'herbe mangée, la laine repousse instantanément !" }, 
+                                                                                    new List<string> { "Apparence1", "App2", "App3" }) }));
+
             lu.Add(new User("Yadoumir", "Doumir", "Yannis", "mdp", new List<Monstre> { }));
             lu.Add(new User("osuplayer123", "Bonetti", "Martin", "oSu!727", new List<Monstre> { }));
             return lu;
diff --git a/Sources/Persistance/saves/users.xml b/Sources/Persistance/saves/users.xml
index 8536d7a..a9575d3 100644
--- a/Sources/Persistance/saves/users.xml
+++ b/Sources/Persistance/saves/users.xml
@@ -1,38 +1,66 @@
 <?xml version="1.0" encoding="utf-8"?>
 <ArrayOfUser xmlns:i="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://schemas.datacontract.org/2004/07/Model">
   <User>
-    <Mdp>mdp</Mdp>
+    <monstresDejaVu i:nil="true" />
     <Nom>dede</Nom>
     <Prenom>dodo</Prenom>
     <Pseudo>DedeDu42</Pseudo>
-    <monstresDejaVu />
+    <Mdp>mdp</Mdp>
   </User>
   <User>
-    <Mdp>feur</Mdp>
+    <monstresDejaVu i:nil="true" />
     <Nom>Monchanin</Nom>
     <Prenom>Liam</Prenom>
     <Pseudo>Moi</Pseudo>
-    <monstresDejaVu />
+    <Mdp>feur</Mdp>
   </User>
   <User>
-    <Mdp>niblondeau</Mdp>
+    <monstresDejaVu i:nil="true" />
     <Nom>Blondeau</Nom>
     <Prenom>Nicolas</Prenom>
     <Pseudo>Nikoala</Pseudo>
-    <monstresDejaVu />
+    <Mdp>niblondeau</Mdp>
   </User>
   <User>
-    <Mdp>mdp</Mdp>
+    <monstresDejaVu i:nil="true" />
     <Nom>Doumir</Nom>
     <Prenom>Yannis</Prenom>
     <Pseudo>Yadoumir</Pseudo>
-    <monstresDejaVu />
+    <Mdp>mdp</Mdp>
   </User>
   <User>
-    <Mdp>oSu!727</Mdp>
+    <monstresDejaVu i:nil="true" />
     <Nom>Bonetti</Nom>
     <Prenom>Martin</Prenom>
     <Pseudo>osuplayer123</Pseudo>
-    <monstresDejaVu />
+    <Mdp>oSu!727</Mdp>
+  </User>
+  <User>
+    <monstresDejaVu i:nil="true" />
+    <Nom>PasMonchanin</Nom>
+    <Prenom>PasMoi</Prenom>
+    <Pseudo>PasLiam</Pseudo>
+    <Mdp>pasfeur</Mdp>
+  </User>
+  <User>
+    <monstresDejaVu i:nil="true" />
+    <Nom>PasMonchanin</Nom>
+    <Prenom>PasLiam</Prenom>
+    <Pseudo>PasMoi</Pseudo>
+    <Mdp>pasfeur</Mdp>
+  </User>
+  <User>
+    <monstresDejaVu i:nil="true" />
+    <Nom>Aled</Nom>
+    <Prenom>Jémal</Prenom>
+    <Pseudo>osKour</Pseudo>
+    <Mdp>aidezmwa</Mdp>
+  </User>
+  <User>
+    <monstresDejaVu i:nil="true" />
+    <Nom>TestNom</Nom>
+    <Prenom>TestPrenom</Prenom>
+    <Pseudo>TestPseudo</Pseudo>
+    <Mdp>TestMdp</Mdp>
   </User>
 </ArrayOfUser>
\ No newline at end of file