Added test & formal changes

Author: rabanti-github

NanoXLSX.Core/Interfaces/Reader/IPluginBaseReader.cs

@@ -6,7 +6,6 @@
  */
 
 using System;
-using System.CodeDom.Compiler;
 using System.IO;
 
 namespace NanoXLSX.Interfaces.Reader

NanoXLSX.Core/Interfaces/Reader/IPluginPackageReader.cs

@@ -8,7 +8,7 @@
 namespace NanoXLSX.Interfaces.Reader
 {
     /// <summary>
-    /// Interface, used by ipackage reader plug-ins
+    /// Interface, used by package reader plug-ins
     /// </summary>
     internal interface IPluginPackageReader : IPluginQueueReader
     {

NanoXLSX.Core/Interfaces/Reader/IPluginReader.cs

@@ -5,9 +5,6 @@
  * You find a copy of the license in project folder or on: http://opensource.org/licenses/MIT
  */
 
-using System;
-using System.IO;
-
 namespace NanoXLSX.Interfaces.Reader
 {
     /// <summary>

NanoXLSX.Core/Worksheet.cs

@@ -989,9 +989,13 @@ public void AddCellRange(IReadOnlyList<object> values, Range cellRange, Style st
         /// <param name="style">Style to apply on the all cells of the range</param>
         /// \remark <remarks>The data types in the passed list can be mixed. Recognized are the following data types: Cell (prepared object), string, int, double, float, long, DateTime, TimeSpan, bool. 
         /// All other types will be cast into a string using the default ToString() method</remarks>
-        /// <exception cref="RangeException">Throws a RangeException if the number of cells differs from the number of passed values</exception>
+        /// <exception cref="RangeException">Throws a RangeException if the passes list is null or the number of cells differs from the number of passed values</exception>
         private void AddCellRangeInternal<T>(IReadOnlyList<T> values, Address startAddress, Address endAddress, Style style)
         {
+            if (values == null)
+            {
+                throw new RangeException("The passed value list cannot be null");
+            }
             List<Address> addresses = Cell.GetCellRange(startAddress, endAddress) as List<Address>;
             if (values.Count != addresses.Count)
             {

NanoXlsx.Core.Test/Worksheets/AddCellRangeTest.cs

@@ -229,6 +229,14 @@ public void AddCellRangeFailingTest2(string givenRange, string falseRange, Range
             Assert.Throws<RangeException>(() => worksheet.AddCellRange(data.Values, falseRange));
         }
 
+        [Fact(DisplayName = "Test of the failing AddCellRange function with a null as passed list")]
+        public void AddCellRangeFailingTest3()
+        {
+            NanoXLSX.Range cellRange = "A1:A1";
+            Worksheet worksheet = new Worksheet();
+            Assert.Empty(worksheet.Cells);
+            Assert.Throws<RangeException>(() => worksheet.AddCellRange(null, cellRange));
+        }
 
         private void AssertRange(Worksheet worksheet, ListTuple expectedData)
         {