Skip to content

Commit

Permalink
Remove an old test from merge.
Browse files Browse the repository at this point in the history
  • Loading branch information
rlburkes committed Dec 31, 2014
1 parent 387fe1a commit ab1b32e
Showing 1 changed file with 0 additions and 47 deletions.
47 changes: 0 additions & 47 deletions spec/lib/roo/excelx_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -360,51 +360,4 @@
end
end
end

describe '.new with streaming' do
subject {
Roo::Excelx.new('test/files/numbers1withnull.xlsx', :minimal_load => true)
}

it 'has pad_cell set to true by default' do
# last row should have two nil values
# on in the first col and one in the fourth col

rows = []

subject.each_row_streaming do |row|
rows << row.map do |c|
# can't call .value on a padded value (it's nil)
c.value rescue nil
end
end

# were padding empty cells (should be 2 total)
expect(rows.first[1]).to be_nil
expect(rows.first[3]).to be_nil
expect(rows.first[2]).not_to be_nil

end

it 'has pad_cell set to true by default' do
# last row should have two nil values
# on in the first col and one in the fourth col

rows = []

subject.each_row_streaming(:pad_cells=>false) do |row|
rows << row.map do |c|
# can't call .value on a padded value (it's nil)
c.value rescue nil
end
end

# we are not padding empty cells
expect(rows.first[1]).not_to be_nil

end


end

end

0 comments on commit ab1b32e

Please sign in to comment.