reflect csv row count when skipped rows

app/models/eligible_fe_providers_importer.rb

@@ -19,7 +19,7 @@ def initialize(file, academic_year)
   end
 
   def results_message
-    "Replaced #{deleted_row_count} existing providers with #{rows.count} new providers"
+    "Replaced #{deleted_row_count} existing providers with #{rows_with_data_count} new providers"
   end
 
   private

lib/csv_importer/base.rb

@@ -4,12 +4,13 @@ module CsvImporter
   class Base
     include CsvImporter::Config
 
-    attr_reader :errors, :rows, :deleted_row_count
+    attr_reader :errors, :rows, :deleted_row_count, :skipped_row_count
 
     def initialize(file)
       @errors = []
       @rows = parse_csv_file(file)
       @deleted_row_count = 0
+      @skipped_row_count = 0
 
       check_headers if rows && with_headers?
     end
@@ -21,8 +22,10 @@ def run
         Rails.logger.info "Processing #{target_data_model.to_s.titleize} batch #{i}"
 
         record_hashes = batch_rows.map do |row|
-          next if empty_row?(row)
-          next if valid_skip_row_conditions?(row)
+          if empty_row?(row) || valid_skip_row_conditions?(row)
+            @skipped_row_count += 1
+            next
+          end
 
           convert_row_to_hash(row)
         end.compact
@@ -31,6 +34,10 @@ def run
       end
     end
 
+    def rows_with_data_count
+      rows.count - skipped_row_count
+    end
+
     private
 
     def empty_row?(row)

spec/models/eligible_fe_providers_importer_spec.rb

@@ -80,6 +80,12 @@ def to_row(hash)
       it "does not raise errors" do
         expect { subject.run }.not_to raise_error
       end
+
+      it "returns correct rows_with_data_count" do
+        subject.run
+
+        expect(subject.rows_with_data_count).to eql(3)
+      end
     end
 
     context "with valid data" do