rails-excel-reporter 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 20959c647dec1e3909f74634cea69215623e3728486448a849d94eed9b730645
-  data.tar.gz: 149463d5536535f0000d4753bc5fd50d75c8ec7a96ca8474c230db8ee05b011a
+  metadata.gz: d5f69c565c35735198074df4b774f284045b4193650cf09c5bb098eee945b0a4
+  data.tar.gz: cbb2aa4f5cb6dd736c3c6cb9d8e8ce83f8ae52d9047881b3dbffbe6d6877e56c
 SHA512:
-  metadata.gz: 8b43a9d68db9834e4f2acc863579fc9c7c24947a6c36d7a7eb394833c8f99c4c699bd315db2b7dad4ce76203be985a83be87fa46838072feef7d23b2e81e264c
-  data.tar.gz: 850816793615b0870b0fd9118bbc65dcaf1a5e114039c7892be80137f16e43c0555d62593006270ecd791bf801cc6d6a830c0a56c69f9a655cd88b1bb1e08565
+  metadata.gz: '090ada371d1359ef1a8a8642fda57b6d4f12fde107ce6f672982762a4d0db64ae8d2aad3ff31c4ec634efeb59d90b493c37c9abf46ae1d4f70f0f79d9b49128c'
+  data.tar.gz: 55c8659faa49e50f6cbf99fade6dec23c9e5bf4135d62051f5251ece5cfe0acf60685e84b62bf92a66c197333ddac044db932fce969581cbdc188300660d79a9

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rails-excel-reporter (1.0.1)
+    rails-excel-reporter (1.0.2)
       activesupport (>= 8.0)
       caxlsx (~> 4.0)
       rails (>= 8.0)
@@ -125,7 +125,6 @@ GEM
     marcel (1.0.4)
     method_source (1.1.0)
     mini_mime (1.1.5)
-    mini_portile2 (2.8.9)
     minitest (5.25.5)
     net-imap (0.5.9)
       date
@@ -261,8 +260,6 @@ GEM
       simplecov_json_formatter (~> 0.1)
     simplecov-html (0.13.1)
     simplecov_json_formatter (0.1.4)
-    sqlite3 (1.7.3)
-      mini_portile2 (~> 2.8.0)
     sqlite3 (1.7.3-aarch64-linux)
     sqlite3 (1.7.3-arm-linux)
     sqlite3 (1.7.3-arm64-darwin)

data/lib/rails_excel_reporter/streaming.rb

@@ -65,8 +65,22 @@ module RailsExcelReporter
 
     def stream_large_dataset(&block)
       if @collection.respond_to? :find_each
-        @collection.find_each(batch_size: 1000, &block)
-      elsif @collection.respond_to? :each
+        stream_with_find_each(&block)
+      else
+        stream_with_fallback(&block)
+      end
+    end
+
+    private
+
+    def stream_with_find_each(&block)
+      @collection.find_each(batch_size: 1000, &block)
+    rescue ArgumentError
+      stream_with_fallback(&block)
+    end
+
+    def stream_with_fallback(&block)
+      if @collection.respond_to? :each
         @collection.each(&block)
       else
         @collection.to_a.each(&block)

data/lib/rails_excel_reporter/version.rb

@@ -1,3 +1,3 @@
 module RailsExcelReporter
-  VERSION = '1.0.1'
+  VERSION = '1.0.2'
 end

data/spec/rails_excel_reporter/base_spec.rb

@@ -125,7 +125,6 @@ RSpec.describe RailsExcelReporter::Base do
 
     describe 'custom methods' do
       it 'calls custom methods for attribute values' do
-        # Since we can't easily inspect binary xlsx data, test the underlying data
         report_data = report.to_h[:data]
         expect(report_data[0][1]).to eq('John Doe (Custom)')
         expect(report_data[1][1]).to eq('Jane Smith (Custom)')

data/spec/rails_excel_reporter/streaming_spec.rb

@@ -128,7 +128,6 @@ RSpec.describe RailsExcelReporter::Streaming do
       report = report_class.new(small_data, &callback)
 
       report.with_progress_tracking do |item, progress|
-        # Just iterate
       end
 
       expect(callback_calls.size).to eq(10)
@@ -136,4 +135,106 @@ RSpec.describe RailsExcelReporter::Streaming do
       expect(callback_calls.last.current).to eq(10)
     end
   end
+
+  describe 'handling very large datasets' do
+    let :very_large_data do
+      (1..1500).map { |i| OpenStruct.new id: i, name: "User #{i}", email: "user#{i}@example.com" }
+    end
+
+    let :huge_data do
+      (1..2500).map { |i| OpenStruct.new id: i, name: "User #{i}", email: "user#{i}@example.com" }
+    end
+
+    it 'handles 1500+ records efficiently' do
+      report = report_class.new very_large_data
+
+      expect(report.should_stream?).to be true
+      expect(report.collection_size).to eq(1500)
+
+      yielded_items = []
+      report.stream_data do |item|
+        yielded_items << item
+      end
+
+      expect(yielded_items.size).to eq(1500)
+      expect(yielded_items.first.id).to eq(1)
+      expect(yielded_items.last.id).to eq(1500)
+    end
+
+    it 'tracks progress correctly for 1500+ records' do
+      report = report_class.new very_large_data
+      progress_updates = []
+
+      report.with_progress_tracking do |item, progress|
+        progress_updates << progress if progress_updates.size < 5 || progress.current == 1500
+      end
+
+      expect(progress_updates.size).to be >= 5
+      expect(progress_updates.first.current).to eq(1)
+      expect(progress_updates.first.total).to eq(1500)
+      expect(progress_updates.first.percentage).to eq(0.07)
+      expect(progress_updates.last.current).to eq(1500)
+      expect(progress_updates.last.percentage).to eq(100.0)
+    end
+
+    it 'handles extremely large datasets (2500+ records)' do
+      report = report_class.new huge_data
+
+      expect(report.should_stream?).to be true
+      expect(report.collection_size).to eq(2500)
+
+      item_count = 0
+      report.stream_data do |item|
+        item_count += 1
+      end
+
+      expect(item_count).to eq(2500)
+    end
+
+    it 'handles ActiveRecord-like collections with large datasets' do
+      mock_collection = double 'Large ActiveRecord Collection'
+      allow(mock_collection).to receive(:respond_to?).with(:count).and_return(true)
+      allow(mock_collection).to receive(:count).and_return(1500)
+      allow(mock_collection).to receive(:respond_to?).with(:find_each).and_return(true)
+
+      yielded_items = []
+      allow(mock_collection).to receive(:find_each).with(batch_size: 1000) do |&block|
+        very_large_data.each(&block)
+      end
+
+      report = report_class.new mock_collection
+
+      report.stream_data do |item|
+        yielded_items << item
+      end
+
+      expect(yielded_items.size).to eq(1500)
+    end
+
+    it 'falls back gracefully when find_each fails on large datasets' do
+      mock_collection = double 'Problematic ActiveRecord Collection'
+      allow(mock_collection).to receive(:respond_to?).with(:count).and_return(true)
+      allow(mock_collection).to receive(:count).and_return(1500)
+      allow(mock_collection).to receive(:respond_to?).with(:find_each).and_return(true)
+      allow(mock_collection).to receive(:respond_to?).with(:each).and_return(true)
+
+      allow(mock_collection).to receive(:find_each).with(batch_size: 1000).and_raise(ArgumentError,
+'Invalid batch_size')
+
+      yielded_items = []
+      allow(mock_collection).to receive(:each) do |&block|
+        very_large_data.each(&block)
+      end
+
+      report = report_class.new mock_collection
+
+      expect {
+        report.stream_data do |item|
+          yielded_items << item
+        end
+      }.not_to raise_error
+
+      expect(yielded_items.size).to eq(1500)
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-excel-reporter
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Elí Sebastian Herrera Aguilar