rspreadsheet 0.2.15 → 0.3

data/spec/image_spec.rb

@@ -0,0 +1,104 @@
+require 'spec_helper'
+ 
+describe Rspreadsheet::Image do
+  before do
+    @testfile_filename = './spec/testfile2-images.ods'
+    @tmp_testfile_filename = '/tmp/testfile2.ods'
+    File.delete(@tmp_testfile_filename) if File.exists?(@tmp_testfile_filename) # delete temp file
+
+    @testimage_filename  = './spec/test-image-blue.png'
+    @testimage2_filename = './spec/test-image.png'
+    @workbook = Rspreadsheet.new(@testfile_filename)
+    @sheet  = @workbook.worksheets(1)
+    @sheet2 = @workbook.worksheets(2)
+  end
+  it 'is accesible when included in spreadsheet', :xpending do
+    @sheet.images_count.should == 1
+    @image = @sheet.images(1)
+    
+    @image.name.should == 'Obr-a'
+    @sheet.insert_image(@testimage_filename)  ## should it be named this way?
+    
+    @sheet.images.count.should == 2
+    @image = @sheet.images(2)
+    @image.original_filename.should == @testimage_filename ## should it be named this way? - investigate File object
+    @image.name = 'name1'
+    @image.name.should == 'name1'
+    @image.name = 'name2'
+    @image.name.should_not == 'name1'
+    
+    @image.width = '30mm'
+    @image.width.should == '30mm'
+    @image.height = '31mm'
+    @image.height.should == '31mm'
+    @image.x = '32mm'
+    @image.x.should == '32mm'
+    @image.y = '34mm'
+    @image.y.should == '34mm'
+  end
+  it 'can be inserted in sheet without any pictures' do
+    @sheet2.insert_image_to('10mm','10mm',@testimage_filename)
+  end
+  
+  it 'can be inserted to a sheet and moved around' do
+    x,y = '15mm', '17mm'
+    @image = @sheet.insert_image_to(x,y,@testimage_filename)
+    # moving image
+    @image.x = '21mm'
+    @image.x.should == '21mm'
+    @image.move_to('51mm','52mm')
+    @image.y.should == '52mm'
+    # resizing image
+    @image.width = '30mm'
+    @image.height = '30mm'
+    @image.width.should == '30mm'
+    # copying image
+    @sheet.images_count.should == 2
+    @sheet2.images_count.should == 0
+    @image.copy_to(x,y,@sheet2)
+    @sheet.images_count.should == 2
+    @sheet2.images_count.should == 1
+  end
+  
+  it 'can be inserted into file and is saved correctly to it' do
+    tmp_test_image = '/tmp/test-image.png'
+    
+    # create new file, insert image into it and save it
+    book = Rspreadsheet.new
+    @sheet = book.add_worksheet
+    @sheet.images_count.should == 0
+    book.worksheets(1).insert_image_to('10mm','10mm',@testimage2_filename)
+    @sheet.images_count.should == 1
+    book.save(@tmp_testfile_filename)
+
+    # reopen it and check the contents
+    book2 = Rspreadsheet.new(@tmp_testfile_filename)
+    @sheet2 = book2.worksheets(1)
+    @sheet2.images_count.should == 1
+    @image = @sheet2.images(1)
+    
+    File.delete(tmp_test_image) if File.exists?(tmp_test_image)
+    Zip::File.open(@tmp_testfile_filename) do |zip|  ## TODO: this is UGLY - it should not be extracting contents here
+      zip.extract(@image.internal_filename,tmp_test_image)
+    end
+    File.binread(tmp_test_image).unpack("H*").should == File.binread(@testimage2_filename).unpack("H*")
+  end
+  
+  it 'generates internal_filename on save randomly and they are different' do
+    i1 = @sheet.insert_image(@testimage_filename)
+    i2 = @sheet.insert_image(@testimage2_filename)
+    i3 = @sheet.insert_image(@testimage2_filename)
+    i1.move_to('40mm','40mm')
+    i2.move_to('40mm','40mm')
+    i3.move_to('40mm','40mm')
+    @workbook.save(@tmp_testfile_filename)
+    
+    i1.internal_filename.should_not == i2.internal_filename
+    i1.internal_filename.should_not == i3.internal_filename
+    i2.internal_filename.should_not == i3.internal_filename
+  end
+  
+#   it 'has dimensions defaulting to size of the image once it is inserted' do
+#   end
+  
+end

data/spec/included_image_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+ 
+describe Rspreadsheet::Worksheet, :focus do
+  before do
+  end
+  it 'name of test' do
+  end
+end

data/spec/io_spec.rb

@@ -1,38 +1,50 @@
 require 'spec_helper'
+using ClassExtensions if RUBY_VERSION > '2.1'
 
 describe Rspreadsheet do
+  before do
+    @tmp_filename = '/tmp/testfile.ods'        
+    File.delete(@tmp_filename) if File.exists?(@tmp_filename)  # delete temp file
+  end
+  
+  
   it 'can open spreadsheet and save it to file, resulting file has same content as original' do
     spreadsheet = Rspreadsheet.new($test_filename)                 # open a file
-    
-    # save it to temp file
-    tmp_filename = '/tmp/testfile1.ods'        
-    File.delete(tmp_filename) if File.exists?(tmp_filename)  # first delete temp file
-    spreadsheet.save(tmp_filename)                                  # and save spreadsheet as temp file
+    spreadsheet.save(@tmp_filename)                                # and save spreadsheet as temp file
     
     # now compare content saved file to original
-    contents_of_files_are_identical($test_filename,tmp_filename)
+    contents_of_files_are_identical?($test_filename,@tmp_filename).should == true
   end
   
-#   it 'can open spreadsheet and store it to IO object', :xpending => 'Under development' do
-#     spreadsheet = Rspreadsheet.new($test_filename)                 # open a file
-#     
-#     stringio = StringIO.new
-#     spreadsheet.save(stringio)
-#     raise stringio.read
-#   
-#   end
-end
-
-def contents_of_files_are_identical(filename1,filename2)
-    @content_xml1 = Zip::File.open(filename1) do |zip|
-      LibXML::XML::Document.io zip.get_input_stream('content.xml')
-    end
-    @content_xml2 = Zip::File.open(filename2) do |zip|
-      LibXML::XML::Document.io zip.get_input_stream('content.xml')
-    end
+  it 'can open spreadsheet and store it to IO object', :pending => 'Under development' do
+    spreadsheet = Rspreadsheet.new($test_filename)                 # open a file
     
-    @content_xml2.root.first_diff(@content_xml1.root).should be_nil
-    @content_xml1.root.first_diff(@content_xml2.root).should be_nil
+    stringio = StringIO.new
+    spreadsheet.save_to_io(stringio)
+    stringio.size.should > 300000
     
-    @content_xml1.root.equals?(@content_xml2.root).should == true
-end
+    # save it to temp file
+    File.open(@tmp_filename, "w") do |f|
+      f.write stringio.read                                  
+    end
+     
+    contents_of_files_are_identical?($test_filename,@tmp_filename).should == true
+  end
+end
+
+def contents_of_files_are_identical?(filename1,filename2)
+  @content_xml1 = Zip::File.open(filename1) do |zip|
+    LibXML::XML::Document.io zip.get_input_stream('content.xml')
+  end
+  @content_xml2 = Zip::File.open(filename2) do |zip|
+    LibXML::XML::Document.io zip.get_input_stream('content.xml')
+  end
+  return contents_are_identical?(@content_xml1,@content_xml2)
+end
+
+def contents_are_identical?(content_xml1,content_xml2)
+  content_xml2.root.first_diff(content_xml1.root).should be_nil
+  content_xml1.root.first_diff(content_xml2.root).should be_nil
+  
+  return (content_xml1.root == content_xml2.root)
+end

data/spec/row_spec.rb

@@ -3,7 +3,8 @@ require 'spec_helper'
 describe Rspreadsheet::Row do
   before do 
     @sheet1 = Rspreadsheet.new.create_worksheet
-    @sheet2 = Rspreadsheet.new($test_filename).worksheets(1)
+    @book2 = Rspreadsheet.new($test_filename)
+    @sheet2 = @book2.worksheets(1)
   end
   it 'allows access to cells in a row' do
     @row = @sheet2.rows(1)
@@ -199,6 +200,16 @@ describe Rspreadsheet::Row do
     @sheet2.add_row_above(1)
     @sheet2.rows(1).should be_kind_of(Rspreadsheet::Row)
   end
+  it 'inserted is empty even is surrounded by nonempty rows' do
+    @sheet2.row(4).cells.size.should > 1
+    @sheet2.row(5).cells.size.should > 1
+    @row5 = @sheet2.row(5)
+    @sheet2.add_row_above(5)
+    @sheet2.row(4).cells.size.should > 1
+    @sheet2.row(5).cells.size.should == 0
+    @sheet2.row(6).should == @row5
+  end
+  
   it 'can be deleted' do
     @sheet1[15,4]='data'
     @row = @sheet1.rows(15)
@@ -255,6 +266,10 @@ describe Rspreadsheet::Row do
     @row.truncate
     @row.size.should == 0
   end
+  it 'remembers its parent correctly' do
+    @row = @sheet1.rows(5)
+    @row.worksheet.should == @sheet1
+  end
 end
 
  

data/spec/rspreadsheet_spec.rb

@@ -1,6 +1,15 @@
 require 'spec_helper'
+using ClassExtensions if RUBY_VERSION > '2.1'
 
 describe Rspreadsheet do
+  before do
+    @tmp_filename = '/tmp/testfile.ods'        # delete temp file before tests
+    File.delete(@tmp_filename) if File.exists?(@tmp_filename)
+  end
+  after do
+    File.delete(@tmp_filename) if File.exists?(@tmp_filename) # delete temp file after tests
+  end
+  
   it 'can open ods testfile and reads its content correctly' do
     book = Rspreadsheet.new($test_filename)
     s = book.worksheets(1)
@@ -11,13 +20,12 @@ describe Rspreadsheet do
     s[2,2].should === Date.new(2014,1,1)
   end
   it 'can open and save file, and saved file has same cells as original' do
-    tmp_filename = '/tmp/testfile1.ods'        # first delete temp file
-    File.delete(tmp_filename) if File.exists?(tmp_filename)
-    book = Rspreadsheet.new($test_filename)    # than open test file
-    book.save(tmp_filename)                    # and save it as temp file
+    
+    book = Rspreadsheet.new($test_filename)     # open test file
+    book.save(@tmp_filename)                    # and save it as temp file
     
     book1 = Rspreadsheet.new($test_filename)   # now open both again
-    book2 = Rspreadsheet.new(tmp_filename)
+    book2 = Rspreadsheet.new(@tmp_filename)
     @sheet1 = book1.worksheets(1)
     @sheet2 = book2.worksheets(1)
     
@@ -25,21 +33,38 @@ describe Rspreadsheet do
       @sheet2[cell.rowi,cell.coli].should == cell.value
     end
   end
+
+  it 'can open and save file, and saved file is exactly same as original' do
+    book = Rspreadsheet.new($test_filename)    # open test file
+    book.save(@tmp_filename)                    # and save it as temp file
+    
+    # now compare them
+    @content_xml1 = Zip::File.open($test_filename) do |zip|
+      LibXML::XML::Document.io zip.get_input_stream('content.xml')
+    end
+    @content_xml2 = Zip::File.open(@tmp_filename) do |zip|
+      LibXML::XML::Document.io zip.get_input_stream('content.xml')
+    end
+    
+    @content_xml2.root.first_diff(@content_xml1.root).should be_nil
+    @content_xml1.root.first_diff(@content_xml2.root).should be_nil
+    
+    @content_xml1.root.should == @content_xml2.root
+  end
+
   it 'when open and save file modified, than the file is different' do
-    tmp_filename = '/tmp/testfile1.ods'        # first delete temp file
-    File.delete(tmp_filename) if File.exists?(tmp_filename)
-    book = Rspreadsheet.new($test_filename)    # than open test file
+    book = Rspreadsheet.new($test_filename)    # open test file
     book.worksheets(1).rows(1).cells(1).value.should_not == 'xyzxyz'
     book.worksheets(1).rows(1).cells(1).value ='xyzxyz'
     book.worksheets(1).rows(1).cells(1).value.should == 'xyzxyz'
     
-    book.save(tmp_filename)                    # and save it as temp file
+    book.save(@tmp_filename)                    # and save it as temp file
     
     # now compare them
     @content_doc1 = Zip::File.open($test_filename) do |zip|
       LibXML::XML::Document.io zip.get_input_stream('content.xml')
     end
-    @content_doc2 = Zip::File.open(tmp_filename) do |zip|
+    @content_doc2 = Zip::File.open(@tmp_filename) do |zip|
       LibXML::XML::Document.io zip.get_input_stream('content.xml')
     end
     @content_doc1.eql?(@content_doc2).should == false
@@ -52,39 +77,57 @@ describe Rspreadsheet do
     book.create_worksheet
   end
   it 'examples from README file are working' do
-    book = Rspreadsheet.open($test_filename)
-    sheet = book.worksheets(1)
-    sheet.B5 = 'cell value'
-    
-    sheet.B5.should eq 'cell value'
-    sheet[5,2].should eq 'cell value'
-    sheet.rows(5).cells(2).value.should eq 'cell value'
-    
-    expect {
+    Rspreadsheet.open($test_filename).save(@tmp_filename)
+    @testimage_filename  = './spec/test-image-blue.png'
+    def puts(*par); end # supress puts in the example
+    expect do
+      book = Rspreadsheet.open(@tmp_filename)
+      sheet = book.worksheets(1)
+
+      # get value of a cell B5 (there are more ways to do this)
+      sheet.B5                       # => 'cell value'
+      sheet[5,2]                     # => 'cell value'
+      sheet.row(5).cell(2).value   # => 'cell value'
+
+      # set value of a cell B5
       sheet.F5 = 'text'
       sheet[5,2] = 7
-      sheet.cells(5,2).value = 1.78
-      
-      sheet.cells(5,2).format.bold = true
-      sheet.cells(5,2).format.background_color = '#FF0000'
-    }.not_to raise_error
-    
-    sheet.rows(4).cellvalues.sum{|val| val.to_f}.should eq 4+7*4
-    sheet.rows(4).cells.sum{ |cell| cell.value.to_f }.should eq 4+7*4
+      sheet.cell(5,2).value = 1.78
 
-    total = 0
-    sheet.rows.each do |row|
-      expect {"Sponsor #{row[1]} with email #{row[2]} has donated #{row[3]} USD." }.not_to raise_error
-      total += row[1].to_f
-    end
-    total.should eq 55
+      # working with cell format
+      sheet.cell(5,2).format.bold = true
+      sheet.cell(5,2).format.background_color = '#FF0000'
+
+      # calculate sum of cells in row
+      sheet.row(5).cellvalues.sum
+      sheet.row(5).cells.sum{ |cell| cell.value.to_f }
+
+      # or set formula to a cell
+      sheet.cell('A1').formula='=SUM(A2:A9)'
+
+      # insert company logo to the file
+      sheet.insert_image_to('10mm','15mm',@testimage_filename)
+      
+      # iterating over list of people and displaying the data
+      total = 0
+      sheet.rows.each do |row|
+        puts "Sponsor #{row[1]} with email #{row[2]} has donated #{row[3]} USD."
+        total += row[3].to_f
+      end
+      puts "Totally fundraised #{total} USD"
+
+      # saving file
+      book.save
+      book.save('/tmp/different_filename.ods')
+    end.not_to raise_error
+    File.delete('/tmp/different_filename.ods') if File.exists?('/tmp/different_filename.ods') # delete after tests
   end
   it 'examples from advanced syntax GUIDE are working' do
     def p(*par); end # supress p in the example
     expect do 
       book = Rspreadsheet::Workbook.new
       sheet = book.create_worksheet 'Top icecreams'
-      
+
       sheet[1,1] = 'My top 5'
       p sheet[1,1].class # => String
       p sheet[1,1] # => "My top 5"
@@ -121,9 +164,25 @@ describe Rspreadsheet do
 #       sheet.cells[2,1..5] = ['Vanilla', 'Pistacia', 'Chocolate', 'Annanas', 'Strawbery']
 #       sheet.columns(1).cells(1).format.color = :red
       
-      book.save('testfile.ods')
+      book.save('/tmp/testfile.ods')
     end.not_to raise_error
   end
+  it 'can save file to io stream and the content is the same as when saving to file', :skip do
+    book = Rspreadsheet.new($test_filename)    # open test file
+    
+    file = File.open(@tmp_filename, 'w')        # and save the stream to @tmp_filename
+    file.write(book.save_to_io)
+    file.close
+    
+    book1 = Rspreadsheet.new($test_filename)   # now open both again
+    book2 = Rspreadsheet.new(@tmp_filename)
+    @sheet1 = book1.worksheets(1)
+    @sheet2 = book2.worksheets(1)
+    
+    @sheet1.nonemptycells.each do |cell|       # and test if they are identical
+      @sheet2[cell.rowi,cell.coli].should == cell.value
+    end
+  end
 end
 
 

data/spec/spec_helper.rb

@@ -1,4 +1,3 @@
-
 # This tells Bundler only load gems in side gemspec (not the locally installed ones). See http://stackoverflow.com/questions/4398262/setup-rspec-to-test-a-gem-not-rails
 require 'bundler/setup'
 Bundler.setup

data/spec/workbook_spec.rb

@@ -56,4 +56,31 @@ describe Rspreadsheet::Workbook do
     book.sheet(1).should == sheet
     book.sheets(2).should_not == sheet
   end
+  it 'can access sheet using negative indexes and returns the same object' do
+    book = Rspreadsheet::Workbook.new
+    book.create_worksheet('test')
+    book.create_worksheet('test2')
+    sheet1 = book.worksheets(1)
+    sheet2 = book.worksheets(2)
+    book.worksheet(-1).should == sheet2
+    book.sheet(-2).should == sheet1
+    book[-2].should == sheet1
+  end
+  it 'raises error when attemting to use nonsence index' do
+    book = Rspreadsheet::Workbook.new
+    expect { book.worksheet(Array.new()) }.to raise_error
+  end
 end 
+
+
+
+
+
+
+
+
+
+
+
+
+