#2115 error when importing empty row

src/OSPSuite.Infrastructure.Import/Core/DataSheet.cs

@@ -173,7 +173,7 @@ public void RemoveEmptyColumns()
          }
       }
 
-      public void RemoveEmptyRows()
+      public void RemoveEmptyRowsAtTheEnd()
       {
          for (var i = _rawDataTable.Count - 1; i >= 0; i--)
          {
@@ -184,6 +184,15 @@ public void RemoveEmptyRows()
          }
       }
 
+      public void RemoveEmptyRows()
+      {
+         for (var i = _rawDataTable.Count - 1; i >= 0; i--)
+         {
+            if (_rawDataTable[i].All(x => x.IsNullOrEmpty()))
+               _rawDataTable.RemoveAt(i);
+         }
+      }
+
       public string SheetName { get; set; }
    }
 }

src/OSPSuite.Infrastructure.Import/Core/DataSourceFileReaders/CsvDataSourceFile.cs

@@ -59,7 +59,7 @@ protected override void LoadFromFile(string path)
                }
 
                dataSheet.RemoveEmptyColumns();
-               dataSheet.RemoveEmptyRows();
+               dataSheet.RemoveEmptyRowsAtTheEnd();
 
                DataSheets.AddSheet(dataSheet);
             }

src/OSPSuite.Infrastructure.Import/Core/DataSourceFileReaders/ExcelDataSourceFile.cs

@@ -51,7 +51,7 @@ protected override void LoadFromFile(string path)
                }
 
                rawSheetData.RemoveEmptyColumns();
-               rawSheetData.RemoveEmptyRows();
+               rawSheetData.RemoveEmptyRowsAtTheEnd();
 
                DataSheets.AddSheet(rawSheetData);
             }

src/OSPSuite.Presentation/Presenters/Importer/ImporterPresenter.cs

@@ -218,6 +218,8 @@ private void loadSheets(IDataSourceFile dataSourceFile, IReadOnlyList<string> sh
          }
 
          var sheets = dataSourceFile.DataSheets.GetDataSheetsByName(sheetNames);
+         sheets.Each(sheet => sheet.RemoveEmptyRows());
+
          var dataMappings = dataSourceFile.Format.GetParameters<MetaDataFormatParameter>().Where(p => p.ColumnName != null).Select(md =>
             new MetaDataMappingConverter()
             {

tests/OSPSuite.Presentation.Tests/Importer/Presenters/ImporterPresenterSpecs.cs

@@ -208,6 +208,81 @@ public void sets_column_mapping_presenter_settings()
       }
    }
 
+   public class When_setting_data_source_with_empty_rows : concern_for_ImporterPresenter
+   {
+      private RandomDataSourceFile _dataSourceFile;
+      protected Cache<string, DataSheet> _sheets;
+
+      protected override void Context()
+      {
+         base.Context();
+         _sheets = new Cache<string, DataSheet>();
+         _sheets.Add("Sheet1", A.Fake<DataSheet>());
+
+         _dataSourceFile = new RandomDataSourceFile(A.Fake<IImportLogger>());
+         _dataSourceFile.Format = A.Fake<IDataFormat>();
+         _dataSourceFile.Path = "FakePath";
+         A.CallTo(() => _importerDataPresenter.SetDataSource(A<string>.Ignored)).Returns(_dataSourceFile);
+         _importerDataPresenter.OnImportSheets += Raise.With(new ImportSheetsEventArgs()
+            { Filter = "", DataSourceFile = _dataSourceFile, SheetNames = _sheets.Keys.ToList() });
+      }
+
+      [Observation]
+      public void should_not_throw_an_ImporterParsingException()
+      {
+         sut.LoadConfiguration(_importerConfiguration, "");
+      }
+
+      //This class needs to be added to create a random data source file
+      //With the empty rows that we need to test
+      public class RandomDataSourceFile : DataSourceFile
+      {
+         public RandomDataSourceFile(IImportLogger logger) : base(logger)
+         {
+         }
+
+         protected override void LoadFromFile(string path)
+         {
+            Console.WriteLine($"Loading data from file: {path} (simulated random data)");
+
+            var random = new Random();
+            int numberOfSheets = 1;
+
+            for (int i = 0; i < numberOfSheets; i++)
+            {
+               var dataSheet = new DataSheet
+               {
+                  SheetName = $"Sheet{i + 1}"
+               };
+
+               int numberOfRows = 5;
+               int numberOfColumns = 3;
+
+               for (int c = 0; c < numberOfColumns; c++)
+               {
+                  dataSheet.AddColumn($"Column{c + 1}", c);
+               }
+
+               var emptyRow = new List<string>(Enumerable.Repeat(string.Empty, numberOfColumns));
+               dataSheet.AddRow(emptyRow);
+
+               for (int r = 0; r < numberOfRows; r++)
+               {
+                  var row = new List<string>();
+                  for (int c = 0; c < numberOfColumns; c++)
+                  {
+                     row.Add($"Data_{i + 1}_{r + 1}_{c + 1}");
+                  }
+
+                  dataSheet.AddRow(row);
+               }
+
+               DataSheets.AddSheet(dataSheet);
+            }
+         }
+      }
+   }
+
    public class When_import_data : concern_for_ImporterPresenter
    {
       protected bool triggered = false;