@@ -1,3 +1,3 @@ coverage .DS_Store -pkg \ No newline at end of file +pkg .gitignore @@ -23,10 +23,9 @@ namespace :extlib do t.spec_opts << '--loadby' << 'random' t.spec_files = Pathname.glob(ENV['FILES'] || 'spec/**/*_spec.rb') t.rcov = ENV['NO_RCOV'] != 'false' - t.rcov_opts << '--exclude' << 'spec,environment.rb' + t.rcov_opts << '--exclude' << 'spec' t.rcov_opts << '--text-summary' t.rcov_opts << '--sort' << 'coverage' << '--sort-reverse' - t.rcov_opts << '--only-uncovered' end end Rakefile @@ -3,7 +3,7 @@ class LazyArray # borrowed partially from StrokeDB # these methods should return self or nil RETURN_SELF = [ :<<, :clear, :concat, :collect!, :each, :each_index, :each_with_index, :insert, :map!, :push, :reject!, :reverse!, - :reverse_each, :replace, :sort!, :unshift ] + :reverse_each, :sort!, :unshift ] # these methods should return an instance of this class when an Array # would normally be returned @@ -52,6 +52,12 @@ class LazyArray # borrowed partially from StrokeDB [ wrap(true_results), wrap(false_results) ] end + def replace(other) + @load_with_proc = nil + @array.replace(other) + self + end + def eql?(other) @array.eql?(other.entries) end lib/extlib/lazy_array.rb @@ -452,6 +452,27 @@ describe LazyArray do end end + it 'should provide #replace' do + @lazy_array.should respond_to(:replace) + end + + describe '#replace' do + it 'should return self' do + @lazy_array.replace(@other).object_id.should == @lazy_array.object_id + end + + it 'should replace itself with the other object' do + replaced = @lazy_array.replace(@other) + replaced.length.should == 1 + replaced.should == @other + end + + it 'should be loaded' do + replaced = @lazy_array.replace(@other) + replaced.should be_loaded + end + end + it 'should provide #reverse' do @lazy_array.should respond_to(:reverse) end spec/lazy_array_spec.rb 7867646f7ad854cac9e6c50b7b4830fac619b23a Dan Kubb dan.kubb@autopilotmarketing.com http://github.com/sam/extlib/commit/cb59455c917c730e57bf4e7b7d476008170d264a cb59455c917c730e57bf4e7b7d476008170d264a 2008-06-05T09:43:34-07:00 2008-06-05T09:43:04-07:00 Updated LazyArray#replace to not be a kicker method * Since internal array is being replaced it makes no sense to lazy-load the array first and then replace it. * Updated coverage report to show all libraries * Stripped whitespace with "sake strip" 8954327e4c592f8c989bc76194e8231fff63c71e Dan Kubb dan.kubb@autopilotmarketing.com