fast_excel 0.2.1 → 0.3.0

data/test/reopen_test.rb

@@ -0,0 +1,22 @@
+require_relative 'test_helper'
+
+describe "FastExcel::Worksheet" do
+
+  it "should get_worksheet_by_name" do
+    workbook = FastExcel.open(constant_memory: false)
+    workbook.add_worksheet("Page1")
+
+    ws = workbook.get_worksheet_by_name("Page1")
+    ws.write_row(0, ['1'])
+
+    workbook.close
+
+    error = assert_raises do
+      FastExcel.open(workbook.filename, constant_memory: false)
+    end
+
+    assert_equal(error.class, ArgumentError)
+    assert_equal(error.message, "File '#{workbook.filename}' already exists. FastExcel can not open existing files, only create new files")
+  end
+
+end

data/test/test_helper.rb

@@ -1,6 +1,7 @@
 require 'bundler/setup'
 require 'minitest/autorun'
 require "minitest/reporters"
+require 'pp'
 
 ##Encoding.default_external = Encoding::UTF_8
 ##Encoding.default_internal = Encoding::UTF_8
@@ -20,9 +21,13 @@ ensure
   File.delete(file_path)
 end
 
+def get_arrays(workbook)
+  workbook.close
+  parse_xlsx_as_matrix(workbook.filename)
+end
+
 def parse_xlsx_as_array(file_path)
-  excel = parse_xlsx(file_path)
-  data = excel.to_matrix.to_a
+  data = parse_xlsx_as_matrix(file_path)
   headers = data.shift
 
   data.map do |row|
@@ -32,5 +37,19 @@ end
 
 def parse_xlsx_as_matrix(file_path)
   excel = parse_xlsx(file_path)
-  excel.to_matrix.to_a
-end
+
+  sheet = excel.sheet(0)
+
+  rows = []
+  if sheet.last_row
+    1.upto(sheet.last_row) do |row_number|
+      row = 1.upto(sheet.last_column).map do |col|
+        sheet.cell(row_number, col)
+      end
+
+      rows << row
+    end
+  end
+
+  return rows
+end

data/test/text_width_test.rb

@@ -0,0 +1,80 @@
+require_relative 'test_helper'
+
+
+describe "FastExcel Arial text_width" do
+
+  before do
+    skip "Use basic width calc for now"
+  end
+
+  it "should calculate width for character" do
+    assert_in_delta(FastExcel.arial_text_width('a'), 55.61, 0.1)
+  end
+
+  it "should calculate width with kerning" do
+    assert_in_delta(FastExcel.arial_text_width('A'), 66.69, 0.1)
+    assert_in_delta(FastExcel.arial_text_width('V'), 66.69, 0.1)
+
+    assert_in_delta(FastExcel.arial_text_width('AV'), 125.97, 0.1)
+
+    assert_in_delta(FastExcel.arial_text_width('11'), 103.80, 0.1)
+  end
+
+  it "should skip system characters" do
+    assert_in_delta(FastExcel.arial_text_width(10.chr), 0.0)
+  end
+
+  it "should handle multiline text" do
+    assert_in_delta(FastExcel.arial_text_width("More\nThen\nOne Line"), 405.85, 0.1)
+  end
+end
+
+describe "FastExcel Calibri text_width" do
+
+  before do
+    skip "Use basic width calc for now"
+  end
+
+  it "should calculate width for character" do
+    assert_in_delta(FastExcel.calibri_text_width('a'), 47.90, 0.1)
+  end
+
+  it "should calculate width with kerning" do
+    assert_in_delta(FastExcel.calibri_text_width('A'), 57.86, 0.1)
+    assert_in_delta(FastExcel.calibri_text_width('V'), 56.73, 0.1)
+
+    assert_in_delta(FastExcel.calibri_text_width('AV'), 110.25, 0.1)
+
+    assert_in_delta(FastExcel.calibri_text_width('11'), 101.36, 0.1)
+  end
+
+  it "should skip system characters" do
+    assert_in_delta(FastExcel.calibri_text_width(10.chr), 0.0)
+  end
+
+end
+
+describe "FastExcel Times New Roman text_width" do
+
+  before do
+    skip "Use basic width calc for now"
+  end
+
+  it "should calculate width for character" do
+    assert_in_delta(FastExcel.times_new_roman_text_width('a'), 44.38, 0.1)
+  end
+
+  it "should calculate width with kerning" do
+    assert_in_delta(FastExcel.times_new_roman_text_width('A'), 72.21, 0.1)
+    assert_in_delta(FastExcel.times_new_roman_text_width('V'), 72.21, 0.1)
+
+    assert_in_delta(FastExcel.times_new_roman_text_width('AV'), 131.54, 0.1)
+
+    assert_in_delta(FastExcel.times_new_roman_text_width('11'), 96.28, 0.1)
+  end
+
+  it "should skip system characters" do
+    assert_in_delta(FastExcel.times_new_roman_text_width(10.chr), 0.0)
+  end
+
+end

data/test/tmpfile_test.rb

@@ -17,6 +17,7 @@ describe "FastExcel" do
     workbook.read_string
 
     refute(File.exist?(workbook.filename))
+    refute(File.exist?(File.dirname(workbook.filename)))
     refute(workbook.is_open)
   end
 

data/test/validations_test.rb

@@ -0,0 +1,47 @@
+require_relative 'test_helper'
+
+describe "FastExcel validations" do
+
+  it "should raise error when duplicate sheet name" do
+    workbook = FastExcel.open(constant_memory: true)
+
+    workbook.add_worksheet("Payments Report")
+    error = assert_raises do
+      workbook.add_worksheet("Payments Report")
+    end
+
+    assert_equal(ArgumentError, error.class)
+    assert_equal("Worksheet name 'Payments Report' is already in use", error.message)
+  end
+
+  it "should not raise error when worksheet name is null" do
+    workbook = FastExcel.open(constant_memory: true)
+
+    ws1 = workbook.add_worksheet()
+    ws2 = workbook.add_worksheet()
+    ws2.write_value(0, 2, "aaa")
+
+    assert_equal("Sheet1", ws1[:name])
+    assert_equal("Sheet2", ws2[:name])
+  end
+
+  it "should raise error when the sheet name exceeds maximum length" do
+    workbook = FastExcel.open(constant_memory: true)
+
+    error = assert_raises do
+      workbook.add_worksheet("ABCDEFGHIJKLMNOPQRSTUVWXYZ012345")
+    end
+
+    assert_equal(ArgumentError, error.class)
+    assert_equal("Worksheet name 'ABCDEFGHIJKLMNOPQRSTUVWXYZ012345' exceeds Excel's limit of 31 characters", error.message)
+  end
+
+  it "should not raise error when the sheet name is at maximum length" do
+    workbook = FastExcel.open(constant_memory: true)
+
+    worksheet = workbook.add_worksheet("ABCDEFGHIJKLMNOPQRSTUVWXYZ01234")
+    worksheet.append_row(["aaa", "bbb", "ccc"])
+
+    assert_equal("ABCDEFGHIJKLMNOPQRSTUVWXYZ01234", worksheet[:name])
+  end
+end

data/test/worksheet_test.rb

@@ -0,0 +1,129 @@
+require_relative 'test_helper'
+
+describe "FastExcel::WorksheetExt append_row" do
+
+  before do
+    @workbook = FastExcel.open(constant_memory: false)
+    @worksheet = @workbook.add_worksheet
+  end
+
+  it "should have last_row_number = -1" do
+    assert_equal(-1, @worksheet.last_row_number)
+  end
+
+  it "should write_value and update last_row_number" do
+    @worksheet.write_value(0, 2, "aaa")
+    assert_equal(0, @worksheet.last_row_number)
+    assert_equal([[nil, nil, "aaa"]], get_arrays(@workbook))
+  end
+
+  it "should append row and update last_row_number" do
+    @worksheet.append_row(["aaa", "bbb", "ccc"])
+    @worksheet.append_row(["ddd", "eee", "fff"])
+
+    assert_equal(1, @worksheet.last_row_number)
+    assert_equal([["aaa", "bbb", "ccc"], ["ddd", "eee", "fff"]], get_arrays(@workbook))
+  end
+
+  it "should write_row then append and update last_row_number" do
+    @worksheet.write_row(3, ["aaa", "bbb", "ccc"])
+    @worksheet.append_row(["ddd", "eee", "fff"])
+
+    assert_equal(4, @worksheet.last_row_number)
+    assert_equal(
+      [
+        [nil, nil, nil],
+        [nil, nil, nil],
+        [nil, nil, nil],
+        ["aaa", "bbb", "ccc"],
+        ["ddd", "eee", "fff"]
+      ],
+      get_arrays(@workbook)
+    )
+  end
+
+  it "should not reduce last_row_number" do
+    @worksheet.append_row(["aaa", "bbb", "ccc"])
+    @worksheet.append_row(["ddd", "eee", "fff"])
+    @worksheet.write_value(0, 4, "foo")
+    @worksheet.append_row(["111", "222", "333"])
+
+    assert_equal(2, @worksheet.last_row_number)
+    assert_equal(
+      [
+        ["aaa", "bbb", "ccc", nil, "foo"],
+        ["ddd", "eee", "fff", nil, nil],
+        ["111", "222", "333", nil, nil]
+      ],
+      get_arrays(@workbook)
+    )
+  end
+
+  it "should not allow to write rows that already saved" do
+    @workbook = FastExcel.open(constant_memory: true)
+    @worksheet = @workbook.add_worksheet
+
+    @worksheet.append_row(["aaa", "bbb", "ccc"])
+    @worksheet.append_row(["ddd", "eee", "fff"])
+
+    error = assert_raises(ArgumentError) do
+      @worksheet.write_value(0, 4, "foo")
+    end
+
+    assert_equal("Can not write to saved row in constant_memory mode (attempted row: 0, last saved row: 1)", error.message)
+  end
+
+  it "should write bigdecimal as a number" do
+    require 'bigdecimal'
+
+    @workbook = FastExcel.open(constant_memory: true)
+    @worksheet = @workbook.add_worksheet
+
+    @worksheet.append_row([BigDecimal("0.1234")])
+
+    assert_equal([[0.1234]], get_arrays(@workbook))
+  end
+
+  it "should set name correctly" do
+    workbook = FastExcel.open(constant_memory: true)
+    ws1 = workbook.add_worksheet("foo")
+    ws2 = workbook.add_worksheet("")
+
+    assert_equal("foo", ws1[:name])
+    assert_equal("", ws2[:name])
+  end
+
+  it "should map fields correctly" do
+    workbook = FastExcel.open(constant_memory: true)
+
+    ws = workbook.add_worksheet
+    ws.set_right_to_left
+    assert_equal(ws[:right_to_left], 1)
+
+    ws = workbook.add_worksheet
+    ws.center_vertically
+    assert_equal(ws[:print_options_changed], 1)
+    assert_equal(ws[:vcenter], 1)
+
+    ws = workbook.add_worksheet
+    ws.print_row_col_headers
+    assert_equal(ws[:print_headers], 1)
+    assert_equal(ws[:print_options_changed], 1)
+
+    ws = workbook.add_worksheet
+    ws.set_margins(1.5, 2.5, 3.5, 4.5)
+    assert_equal(ws[:margin_left], 1.5)
+    assert_equal(ws[:margin_right], 2.5)
+    assert_equal(ws[:margin_top], 3.5)
+    assert_equal(ws[:margin_bottom], 4.5)
+
+
+    breaks = [20, 40, 60, 20, 0]
+    FFI::MemoryPointer.new(:uint16, breaks.size) do |buffer|
+      buffer.write_array_of_uint16(breaks)
+      ws.set_v_pagebreaks(buffer)
+    end
+
+    assert_equal(ws[:vbreaks_count], 4)
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fast_excel
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Pavel Evstigneev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-20 00:00:00.000000000 Z
+date: 2020-07-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -38,23 +38,41 @@ extensions:
 - extconf.rb
 extra_rdoc_files: []
 files:
+- ".dockerignore"
 - ".gitignore"
+- ".travis.yml"
 - CHANGELOG.md
+- Dockerfile.test
 - Gemfile
 - Gemfile.lock
+- LICENSE
 - Makefile
 - README.md
 - Rakefile
+- appveyor.yml
 - benchmarks/1k_rows.rb
 - benchmarks/20k_rows.rb
+- benchmarks/auto_width.rb
 - benchmarks/init.rb
 - benchmarks/memory.rb
+- benchmarks/profiler.rb
+- benchmarks/write_value.rb
 - examples/example.rb
+- examples/example_align.rb
+- examples/example_auto_width.rb
 - examples/example_chart.rb
+- examples/example_colors.rb
+- examples/example_filters.rb
 - examples/example_formula.rb
+- examples/example_hyperlink.rb
 - examples/example_image.rb
+- examples/example_styles.rb
+- examples/logo.png
+- ext/fast_excel/extconf.rb
+- ext/fast_excel/text_width_ext.c
 - extconf.rb
 - fast_excel.gemspec
+- letters.html
 - lib/fast_excel.rb
 - lib/fast_excel/binding.rb
 - lib/fast_excel/binding/chart.rb
@@ -64,16 +82,21 @@ files:
 - libxlsxwriter/.gitignore
 - libxlsxwriter/.indent.pro
 - libxlsxwriter/.travis.yml
+- libxlsxwriter/CMakeLists.txt
 - libxlsxwriter/CONTRIBUTING.md
 - libxlsxwriter/Changes.txt
 - libxlsxwriter/LICENSE.txt
 - libxlsxwriter/Makefile
 - libxlsxwriter/Readme.md
+- libxlsxwriter/cmake/FindMINIZIP.cmake
+- libxlsxwriter/cmake/FindPackage.cmake
+- libxlsxwriter/cmake/i686-toolchain.cmake
 - libxlsxwriter/cocoapods/libxlsxwriter-umbrella.h
 - libxlsxwriter/cocoapods/libxlsxwriter.modulemap
 - libxlsxwriter/include/xlsxwriter.h
 - libxlsxwriter/include/xlsxwriter/app.h
 - libxlsxwriter/include/xlsxwriter/chart.h
+- libxlsxwriter/include/xlsxwriter/chartsheet.h
 - libxlsxwriter/include/xlsxwriter/common.h
 - libxlsxwriter/include/xlsxwriter/content_types.h
 - libxlsxwriter/include/xlsxwriter/core.h
@@ -100,6 +123,7 @@ files:
 - libxlsxwriter/src/Makefile
 - libxlsxwriter/src/app.c
 - libxlsxwriter/src/chart.c
+- libxlsxwriter/src/chartsheet.c
 - libxlsxwriter/src/content_types.c
 - libxlsxwriter/src/core.c
 - libxlsxwriter/src/custom.c
@@ -143,10 +167,16 @@ files:
 - libxlsxwriter/third_party/tmpfileplus/tmpfileplus.c
 - libxlsxwriter/third_party/tmpfileplus/tmpfileplus.h
 - libxlsxwriter/version.txt
+- test/auto_width_test.rb
 - test/date_test.rb
 - test/default_format_test.rb
+- test/format_test.rb
+- test/reopen_test.rb
 - test/test_helper.rb
+- test/text_width_test.rb
 - test/tmpfile_test.rb
+- test/validations_test.rb
+- test/worksheet_test.rb
 homepage: https://github.com/paxa/fast_excel
 licenses:
 - MIT
@@ -166,9 +196,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
-summary: Ultra Fast Excel Writter
+summary: Ultra Fast Excel Writer
 test_files: []