Able to skip on write, #2

ExcelORM/ExcelORM/ExcelWriter.cs

@@ -18,6 +18,8 @@ public class ExcelWriter
         var properties = typeof(T).GetProperties();
         foreach (var property in properties)
         {
+            if (property.GetCustomAttributes(typeof(SkipAttribute), false).FirstOrDefault() != null) continue;
+
             var columnAttribute = property.GetCustomAttributes(typeof(ColumnAttribute), false).FirstOrDefault() as ColumnAttribute;
             worksheet.Cell(rowIndex, cellIndex).Value = columnAttribute is { Names.Length: > 0 } ? columnAttribute.Names.First() : property.Name;
             cellIndex++;
@@ -42,6 +44,8 @@ public class ExcelWriter
             var properties = typeof(T).GetProperties();
             foreach (var property in properties)
             {
+                if (property.GetCustomAttributes(typeof(SkipAttribute), false).FirstOrDefault() != null) continue;
+
                 cellIndex++;
                 var valueToSet = property.GetValue(value);
                 if (valueToSet == null) continue;

ExcelORM/ExcelORMTests/TestSkip.cs

@@ -7,5 +7,6 @@ namespace ExcelORMTests
         [Skip]
         public string? Text { get; set; }
         public DateTime? Date { get; set; }
+        public double? Int { get; set; }
     }
 }

ExcelORM/ExcelORMTests/WriterTests.cs

@@ -17,6 +17,12 @@ public class WriterTests
         new Test { Name = "Donald", Surname = "Trump", Job = "Bankrupt"},
     };
 
+    private readonly TestSkip[] arrayWithSkip =
+    {
+        new() {Text = "Lorem", Date = DateTime.Now.AddHours(1), Int = 1},
+        new() {Text = "Ipsum", Date = null, Int = 2},
+    };
+
     [Fact]
     public void WriteWithAppend()
     {
@@ -54,7 +60,7 @@ public class WriterTests
         writer.SaveAs(testFile);
 
         var reader = new ExcelReader(testFile);
-        Assert.Equal(3, reader.Read<Test>().Count());
+        Assert.Equal(arrayOfThree.Length, reader.Read<Test>().Count());
 
         writer.Write(listOfTwo, append: true);
         writer.SaveAs(testFile);
@@ -97,4 +103,28 @@ public class WriterTests
 
         File.Delete(testFile);
     }
+
+    [Fact]
+    public void WriteWithSkip()
+    {
+        var testFile = Path.GetRandomFileName();
+        testFile = Path.ChangeExtension(testFile, "xlsx");
+
+        const string worksheetName = "Test";
+        var writer = new ExcelWriter(testFile);
+        writer.Write(arrayWithSkip, worksheetName);
+        writer.SaveAs(testFile);
+
+        var reader = new ExcelReader(testFile);
+        var readArray = reader.Read<TestSkip>(worksheetName).ToArray();
+        Assert.Equal(arrayWithSkip.Length, readArray.Length);
+
+        for (int i = 0; i < readArray.Length; i++)
+        {
+            Assert.Equal(arrayWithSkip[i].Date.ToString(), readArray[i].Date.ToString());
+            Assert.Equal(arrayWithSkip[i].Int, readArray[i].Int);
+        }
+
+        File.Delete(testFile);
+    }
 }