rspreadsheet 0.0.5 → 0.0.6

data/spec/row_spec.rb

@@ -0,0 +1,127 @@
+require 'spec_helper'
+
+describe Rspreadsheet::Row do
+  before do 
+    @sheet1 = Rspreadsheet.new.create_worksheet
+    @sheet2 = Rspreadsheet.new($test_filename).worksheets[1]
+  end
+  it 'allows access to cells in a row' do
+    @row = @sheet2.rows(1)
+    @c = @row.cells(1)
+    @c.value = 3
+    @c.value.should == 3
+  end
+  it 'creates minimal row on demand, which contains correctly namespaced attributes only'do
+    @sheet1.rows(1).cells(2).value = 'nejakydata'
+    @row = @sheet1.rows(1)
+    @xmlnode = @row.xmlnode
+    @xmlnode.namespaces.to_a.size.should >5
+    @xmlnode.namespaces.namespace.should_not be_nil
+    @xmlnode.namespaces.namespace.prefix.should == 'table'
+  end
+  it 'cells in row are settable through sheet' do
+    @sheet1.rows(9).add_cell
+    @sheet1.rows(9).cells(1).value = 2
+    @sheet1.rows(9).cells(1).value.should == 2
+    
+    @sheet1.rows(7).add_cell
+    @sheet1.rows(7).cells(1).value = 7
+    @sheet1.rows(7).cells(1).value.should == 7
+    
+    @sheet1.rows(5).cells(1).value = 5
+    @sheet1.rows(5).cells(1).value.should == 5
+
+    (2..5).each { |i| @sheet1.rows(3).cells(i).value = i }
+    (2..5).each { |i| 
+      a = @sheet1.rows(3)
+      c = a.cells(i)
+      c.value.should == i 
+    } 
+  end
+  it 'is found even in empty sheets' do
+    @sheet1.rows(5).should be_kind_of(Rspreadsheet::Row)
+    @sheet1.rows(25).should be_kind_of(Rspreadsheet::Row)
+    @sheet1.cells(10,10).value = 'ahoj'
+    @sheet1.rows(9).should be_kind_of(Rspreadsheet::Row)
+    @sheet1.rows(10).should be_kind_of(Rspreadsheet::Row)
+    @sheet1.rows(11).should be_kind_of(Rspreadsheet::Row)
+  end
+  it 'detachment works as expected' do
+    @sheet1.rows(5).detach
+    @sheet1.rows(5).repeated?.should == false
+    @sheet1.rows(5).repeated.should == 1
+    @sheet1.rows(5).xmlnode.should_not == nil
+    @sheet1.rows(3).repeated?.should == true
+    @sheet1.rows(3).repeated.should == 4
+    @sheet1.rows(3).xmlnode.should_not == nil
+    @table_ns_href = "urn:oasis:names:tc:opendocument:xmlns:table:1.0"
+    @sheet1.rows(3).xmlnode.attributes.get_attribute_ns(@table_ns_href,'number-rows-repeated').value.should == '4'
+  end
+  it 'by assigning value, the repeated row is automatically detached' do
+    row = @sheet1.rows(5)
+    row.detach
+    @table_ns_href = "urn:oasis:names:tc:opendocument:xmlns:table:1.0"
+    
+    @sheet1.rows(5).style_name = 'newstylename'
+    @sheet1.rows(5).xmlnode.attributes.get_attribute_ns(@table_ns_href,'style-name').value.should == 'newstylename'
+
+    @sheet1.rows(17).style_name = 'newstylename2'
+    @sheet1.rows(17).xmlnode.attributes.get_attribute_ns(@table_ns_href,'style-name').value.should == 'newstylename2'
+  end
+  it 'ignores negative any zero row indexes' do
+    @sheet1.rows(0).should be_nil
+    @sheet1.rows(-78).should be_nil
+  end
+  it 'has correct rowindex' do
+    @sheet1.rows(5).detach
+    (4..6).each do |i|
+      @sheet1.rows(i).row.should == i
+    end
+  end
+  it 'can open ods testfile and read its content' do
+    book = Rspreadsheet.new($test_filename)
+    s = book.worksheets[1]
+    (1..10).each do |i|
+      s.rows(i).should be_kind_of(Rspreadsheet::Row)
+      s.rows(i).repeated.should == 1
+      s.rows(i).used_range.size>0
+    end
+    s[1,2].should === 'text'
+    s[2,2].should === Date.new(2014,1,1)
+  end
+  it 'normalizes to itself if single line' do
+    @sheet1.rows(5).detach
+    @sheet1.rows(5).cells(4).value='test'
+    @sheet1.rows(5).normalize
+    @sheet1.rows(5).cells(4).value.should == 'test'
+  end
+  it 'cell manipulation does not contain attributes without namespace nor doubled attributes' do # inspired by a real bug regarding namespaces manipulation
+    @sheet2.rows(1).xmlnode.attributes.each { |attr| attr.ns.should_not be_nil}
+    @sheet2.rows(1).cells(1).value.should_not == 'xyzxyz'
+    @sheet2.rows(1).cells(1).value ='xyzxyz'
+    @sheet2.rows(1).cells(1).value.should == 'xyzxyz'
+
+    ## attributes have namespaces
+    @sheet2.rows(1).xmlnode.attributes.each { |attr| attr.ns.should_not be_nil}
+    
+    ## attributes are not douubled
+    xmlattrs = @sheet2.rows(1).xmlnode.attributes.to_a.collect{ |a| [a.ns.andand.prefix.to_s,a.name].reject{|x| x.empty?}.join(':')}
+    duplication_hash = xmlattrs.inject(Hash.new(0)){ |h,e| h[e] += 1; h }
+    duplication_hash.each { |k,v| v.should_not >1 }  # should not contain duplicates
+  end
+  it 'out of bount automagically generated row pick up values when created' do
+    @row1 = @sheet1.rows(23)
+    @row2 = @sheet1.rows(23)
+    @row2.cells(5).value = 'hojala'
+    @row1.cells(5).value.should == 'hojala'
+  end
+  it 'nonempty cells work properly' do
+    nec = @sheet2.rows(1).nonemptycells
+    nec.collect{ |c| [c.row,c.col]}.should == [[1,1],[1,2]]
+    
+    nec = @sheet2.rows(19).nonemptycells
+    nec.collect{ |c| [c.row,c.col]}.should == [[19,6]]
+  end
+end
+
+ 

data/spec/rspreadsheet_spec.rb

@@ -1,9 +1,7 @@
 require 'spec_helper'
 
-$test_filename = './spec/testfile1.ods'
-
 describe Rspreadsheet::Tools do
-  it '=Converts correctly cell adresses' do
+  it 'converts correctly cell adresses' do
     Rspreadsheet::Tools.convert_cell_address('A1') [0].should == 1
     Rspreadsheet::Tools.convert_cell_address('A1')[1].should == 1
     Rspreadsheet::Tools.convert_cell_address('C5')[0].should == 5
@@ -11,24 +9,10 @@ describe Rspreadsheet::Tools do
   end
 end
 
-describe Rspreadsheet::Tools::SparseRepeatedArray do
-  before do
-#     @a = Rspreadsheet::Tools::SparseRepeatedArray.new()
-  end
-#   Float::INFINITY
-  its 'set values can be read back unchanged' do
-    skip
-    @a[3..5]='test'
-    @a[4].should == 'test'
-  end
-end
-
 describe Rspreadsheet do
   it 'can open ods testfile and reads its content correctly' do
     book = Rspreadsheet.new($test_filename)
-    book.worksheets[0].should_not == nil
-    book.worksheets[0].class.should == Rspreadsheet::Worksheet
-    s = book.worksheets[0]
+    s = book.worksheets[1]
     (1..10).each do |i|
       s[i,1].should === i
     end
@@ -43,8 +27,8 @@ describe Rspreadsheet do
     
     book1 = Rspreadsheet.new($test_filename)   # now open both again
     book2 = Rspreadsheet.new(tmp_filename)
-    @sheet1 = book1.worksheets[0]
-    @sheet2 = book2.worksheets[0]
+    @sheet1 = book1.worksheets[1]
+    @sheet2 = book2.worksheets[1]
     
     @sheet1.nonemptycells.each do |cell|       # and test identity
       @sheet2[cell.row,cell.col].should == cell.value
@@ -63,15 +47,23 @@ describe Rspreadsheet do
     @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.equals?(@content_xml2.root).should == true
   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.worksheets[0][1,1].should_not == 'xyzxyz'
-    book.worksheets[0][1,1]='xyzxyz'
-    book.worksheets[0][1,1].should == 'xyzxyz'
+    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.worksheets[1][1,1].should_not == 'xyzxyz'
+#     book.worksheets[1][1,1]='xyzxyz'
+#     book.worksheets[1][1,1].should == 'xyzxyz'
     book.save(tmp_filename)                    # and save it as temp file
     
     # now compare them
@@ -92,91 +84,30 @@ describe Rspreadsheet do
   end
 end
 
-describe Rspreadsheet::Cell do
-  before do 
-    book1 = Rspreadsheet.new
-    @sheet1 = book1.create_worksheet
-    @sheet1[0,0] = 'text'
-    book2 = Rspreadsheet.new($test_filename)
-    @sheet2 = book2.worksheets[0]
-  end
-  it 'contains good row and col coordinates' do
-    @cell = @sheet1.cells(1,3)
-    @cell.row.should == 1
-    @cell.col.should == 3
-    @cell.coordinates.should == [1,3]
-    
-    @cell = @sheet2.cells(7,2)
-    @cell.row.should == 7
-    @cell.col.should == 2
-    @cell.coordinates.should == [7,2]
-  end
-  it 'can be referenced by more vars and both are synchronized' do
-    @cell = @sheet1.cells(1,1)
-    @sheet1[1,1] = 'novinka'
-    @cell.value.should == 'novinka'
-  end
-  it 'can be modified by more ways and all are identical' do
-    @cell = @sheet1.cells(2,2)
-    @sheet1[2,2] = 'zaprve'
-    @cell.value.should == 'zaprve'
-    @sheet1.cells(2,2).value = 'zadruhe'
-    @cell.value.should == 'zadruhe'
-    @sheet1.B2 = 'zatreti'
-    @cell.value.should == 'zatreti'
-  end
-  it 'can include links' do
-    @sheet2.A12.should == '[http://example.org/]'
-  end
-  it 'contains good row and col coordinates even after table:number-columns-repeated cells' do
-    skip
-    @cell = @sheet2.cells(13,5)
-    @cell.value.should == 'afterrepeated'
-    @cell.row.should == 13
-    @cell.col.should == 5
-  end
-end
 
-describe Rspreadsheet::Worksheet do
-  before do
-    book = Rspreadsheet.new
-    @sheet = book.create_worksheet
-  end
-  it 'remembers the value stored to A1 cell' do
-    @sheet[1,1].should == nil
-    @sheet[1,1] = 'test text'
-    @sheet[1,1].class.should == String
-    @sheet[1,1].should == 'test text'
-  end
-  it 'value stored to A1 is accesible using different syntax' do
-    @sheet[1,1] = 'test text'
-    @sheet[1,1].should == 'test text'
-    @sheet.cells(1,1).value.should == 'test text'
-  end
-  it 'makes Cell object accessible' do
-    @sheet.cells(1,1).value = 'test text'
-    @sheet.cells(1,1).class.should == Rspreadsheet::Cell
-  end
-  it 'has name, which can be changed and is remembered' do
-    @sheet.name.should be(nil)
-    @sheet.name = 'Icecream'
-    @sheet.name.should == 'Icecream'
-    @sheet.name = 'Cofee'
-    @sheet.name.should == 'Cofee'    
-  end
-end
 
-describe Rspreadsheet::Row do
-  before do 
-    book1 = Rspreadsheet.new
-    @sheet1 = book1.create_worksheet
-  end
-  it 'allows access to cells in a row' do
-    (2..5).each { |i| @sheet1[7,i] = i }
-    (2..5).each { |i| 
-      a = @sheet1.rows(7)
-      c = a.cells(i)
-      c.value.should == i 
-    }
-  end
-end
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+

data/spec/spec_helper.rb

@@ -1,4 +1,11 @@
+RSpec.configure do |c|
+  c.fail_fast = true
+#   c.warnings = true
+end
+
 require 'coveralls'
 Coveralls.wear!
 
+$test_filename = './spec/testfile1.ods'
+
 require 'rspreadsheet'

data/spec/workbook_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe Rspreadsheet::Workbook do
+  it 'has correct number of sheets' do
+    book = Rspreadsheet.new($test_filename)
+    book.worksheets_count.should == 1
+    book.worksheets[0].should be_nil
+    book.worksheets[1].should be_kind_of(Rspreadsheet::Worksheet)
+    book.worksheets[2].should be_nil
+    book.worksheets[nil].should be_nil
+  end
+  it 'freshly created has correctly namespaced xmlnode' do
+    @xmlnode = Rspreadsheet.new.xmlnode
+    @xmlnode.namespaces.to_a.size.should >5
+    @xmlnode.namespaces.find_by_prefix('office').should_not be_nil
+    @xmlnode.namespaces.find_by_prefix('table').should_not be_nil
+    @xmlnode.namespaces.namespace.prefix.should == 'office'
+  end
+end

data/spec/worksheet_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe Rspreadsheet::Worksheet do
+  before do 
+    @sheet = Rspreadsheet.new($test_filename).worksheets[1]
+  end
+  it 'contains nonempty xml in rows for testfile' do
+    @sheet.rows(1).xmlnode.elements.size.should be >1
+  end
+  it 'freshly created has correctly namespaced xmlnode' do
+    @spreadsheet = Rspreadsheet.new
+    @spreadsheet.create_worksheet
+    @xmlnode = @spreadsheet.worksheets[1].xmlnode
+    @xmlnode.namespaces.to_a.size.should >5
+    @xmlnode.namespaces.find_by_prefix('office').should_not be_nil
+    @xmlnode.namespaces.find_by_prefix('table').should_not be_nil
+    @xmlnode.namespaces.namespace.should_not be_nil
+    @xmlnode.namespaces.namespace.prefix.should == 'table'
+  end
+ 
+end
+
+describe Rspreadsheet::Worksheet do
+  before do
+    book = Rspreadsheet.new
+    @sheet = book.create_worksheet
+  end
+  it 'remembers the value stored to A1 cell' do
+    @sheet[1,1].should == nil
+    @sheet[1,1] = 'test text'
+    @sheet[1,1].class.should == String
+    @sheet[1,1].should == 'test text'
+  end
+  it 'value stored to A1 is accesible using different syntax' do
+    @sheet[1,1] = 'test text'
+    @sheet[1,1].should == 'test text'
+    @sheet.cells(1,1).value.should == 'test text'
+  end
+  it 'makes Cell object accessible' do
+    @sheet.cells(1,1).value = 'test text'
+    @sheet.cells(1,1).class.should == Rspreadsheet::Cell
+  end
+  it 'has name, which can be changed and is remembered' do
+    @sheet.name.should be(nil)
+    @sheet.name = 'Icecream'
+    @sheet.name.should == 'Icecream'
+    @sheet.name = 'Cofee'
+    @sheet.name.should == 'Cofee'    
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspreadsheet
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Jakub A.Těšínský
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-26 00:00:00.000000000 Z
+date: 2014-08-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: libxml-ruby
@@ -95,33 +95,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2'
 - !ruby/object:Gem::Dependency
-  name: coveralls
+  name: pry-nav
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: test_notifier
+  name: coveralls
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: guard
   requirement: !ruby/object:Gem::Requirement
@@ -169,19 +169,25 @@ files:
 - LICENSE.md
 - README.md
 - Rakefile
+- investigate.rb
 - lib/class_extensions.rb
 - lib/rspreadsheet.rb
 - lib/rspreadsheet/cell.rb
 - lib/rspreadsheet/empty_file_template.ods
+- lib/rspreadsheet/empty_file_template.old.ods
 - lib/rspreadsheet/row.rb
 - lib/rspreadsheet/tools.rb
 - lib/rspreadsheet/version.rb
 - lib/rspreadsheet/workbook.rb
 - lib/rspreadsheet/worksheet.rb
 - rspreadsheet.gemspec
+- spec/cell_spec.rb
+- spec/row_spec.rb
 - spec/rspreadsheet_spec.rb
 - spec/spec_helper.rb
 - spec/testfile1.ods
+- spec/workbook_spec.rb
+- spec/worksheet_spec.rb
 homepage: https://github.com/gorn/rspreadsheet
 licenses:
 - GPL
@@ -208,6 +214,10 @@ specification_version: 4
 summary: Manipulating spreadsheets with Ruby (read / create / modify OpenDocument
   Spreadsheet).
 test_files:
+- spec/cell_spec.rb
+- spec/row_spec.rb
 - spec/rspreadsheet_spec.rb
 - spec/spec_helper.rb
 - spec/testfile1.ods
+- spec/workbook_spec.rb
+- spec/worksheet_spec.rb