Added even more better test

src/UnitTests/UnitTestDataContractJson.cs

@@ -12,16 +12,16 @@ namespace UnitTests
         /// <summary>
         /// Vérifie que la propriété FileName a la valeur par défaut "SauvegardeDonnees.json".
         /// </summary>
-        public class DataContractXmlTests
+        public class DataContractJsonTests
         {
             [Fact]
             public void FileName_Should_HaveDefaultValue()
             {
                 // Arrange
-                var dataContractXml = new DataContractJson();
+                var dataContractJson = new DataContractJson();
 
                 // Assert
-                Assert.Equal("SauvegardeDonnees.json", dataContractXml.FileName);
+                Assert.Equal("SauvegardeDonnees.json", dataContractJson.FileName);
             }
 
             /// <summary>
@@ -31,10 +31,28 @@ namespace UnitTests
             public void FilePath_Should_HaveDefaultValue()
             {
                 // Arrange
-                var dataContractXml = new DataContractJson();
+                var dataContractJson = new DataContractJson();
 
                 // Assert
-                Assert.Equal(Path.Combine(AppDomain.CurrentDomain.BaseDirectory), dataContractXml.FilePath);
+                Assert.Equal(Path.Combine(AppDomain.CurrentDomain.BaseDirectory), dataContractJson.FilePath);
+            }
+
+            /// <summary>
+            /// Vérifie que la propriété FilePath peut être assignée avec une autre valeur et que la valeur assignée est correcte.
+            /// </summary>
+            [Theory]
+            [InlineData("C:\\Data\\")]
+            [InlineData("D:\\Backup\\")]
+            public void FilePath_Should_BeAssignable(string filePath)
+            {
+                // Arrange
+                var dataContractJson = new DataContractJson();
+
+                // Act
+                dataContractJson.FilePath = filePath;
+
+                // Assert
+                Assert.Equal(filePath, dataContractJson.FilePath);
             }
         }
     }

src/UnitTests/UnitTestDataContractXml.cs

@@ -41,6 +41,24 @@ namespace UnitTests
                 // Assert
                 Assert.Equal(Path.Combine(AppDomain.CurrentDomain.BaseDirectory), dataContractXml.FilePath);
             }
+
+            /// <summary>
+            /// Vérifie que la propriété FilePath peut être assignée avec une autre valeur et que la valeur assignée est correcte.
+            /// </summary>
+            [Theory]
+            [InlineData("C:\\Data\\")]
+            [InlineData("D:\\Backup\\")]
+            public void FilePath_Should_BeAssignable(string filePath)
+            {
+                // Arrange
+                var dataContractXml = new DataContractXml();
+
+                // Act
+                dataContractXml.FilePath = filePath;
+
+                // Assert
+                Assert.Equal(filePath, dataContractXml.FilePath);
+            }
         }
     }
 }

src/UnitTests/UnitTestDataToPersist.cs

@@ -12,7 +12,7 @@ namespace UnitTests
     public class UnitTestDataToPersist
     {
         /// <summary>
-        ///  Vérifie que la propriété Oeuvres est initialisée et est une instance de ObservableCollection<Oeuvre>.
+        /// Vérifie que la propriété Oeuvres est initialisée et est une instance de ObservableCollection<Oeuvre>.
         /// </summary>
         [Fact]
         public void Oeuvres_Should_BeInitialized()
@@ -42,16 +42,20 @@ namespace UnitTests
         /// <summary>
         /// Vérifie que la propriété Oeuvres peut être assignée avec une autre collection d'œuvres et que la valeur assignée est correcte.
         /// </summary>
-        [Fact]
-        public void Oeuvres_Should_BeAssignable()
+        [Theory]
+        [InlineData(1)]
+        [InlineData(2)]
+        public void Oeuvres_Should_BeAssignable(int oeuvresCount)
         {
             // Arrange
             var dataToPersist = new DataToPersist();
-            var oeuvres = new ObservableCollection<Oeuvre>
+            var oeuvres = new ObservableCollection<Oeuvre>();
+
+            // Add oeuvresCount number of oeuvres
+            for (int i = 0; i < oeuvresCount; i++)
             {
-                new Oeuvre("Evangelion", new List<string> { "Action", "Future" }, "TV", "C'est une bonne série", 4, 150, "evangelion.jpg"),
-                new Oeuvre("[Oshi No Ko]", new List<string> { "Action", "Future" }, "DVD", "A la fin il meurt", 2, 24, "oshinoko.png"),
-            };
+                oeuvres.Add(new Oeuvre($"Oeuvre{i}", new List<string>(), "Type", "Description", 0, 0, "Affiche"));
+            }
 
             // Act
             dataToPersist.Oeuvres = oeuvres;
@@ -61,14 +65,22 @@ namespace UnitTests
         }
 
         /// <summary>
-        /// vérifie que la propriété Utilisateurs peut être assignée avec une autre liste d'utilisateurs et que la valeur assignée est correcte.
+        /// Vérifie que la propriété Utilisateurs peut être assignée avec une autre liste d'utilisateurs et que la valeur assignée est correcte.
         /// </summary>
-        [Fact]
-        public void Utilisateurs_Should_BeAssignable()
+        [Theory]
+        [InlineData(1)]
+        [InlineData(2)]
+        public void Utilisateurs_Should_BeAssignable(int utilisateursCount)
         {
             // Arrange
             var dataToPersist = new DataToPersist();
-            var utilisateurs = new List<Utilisateur> { new Utilisateur(), new Utilisateur() };
+            var utilisateurs = new List<Utilisateur>();
+
+            // Add utilisateursCount number of utilisateurs
+            for (int i = 0; i < utilisateursCount; i++)
+            {
+                utilisateurs.Add(new Utilisateur());
+            }
 
             // Act
             dataToPersist.Utilisateurs = utilisateurs;

src/UnitTests/UnitTestIPersistanceManager.cs

@@ -49,7 +49,8 @@ namespace UnitTests
         }
 
         [Fact]
-        public void Sauvegarder_CallsConsoleWriteLine()
+        public void Sauvegarder_CallsConsoleWriteLine() //pas besoin d'utiliser les attributs [Theory] et [InlineData]
+                                                        //car il ne teste pas différentes variations de données.
         {
             // Arrange
             var stub = new Stub.Stub();