fast_excel 0.4.1 → 0.6.0

data/test/tmpdir_test.rb

@@ -0,0 +1,32 @@
+require_relative 'test_helper'
+require 'tmpdir'
+
+describe "FastExcel tmpdir option" do
+  it "uses system Dir.tmpdir by default without creating extra temp directories" do
+    temp_file = File.join(Dir.tmpdir, "test_default_tmpdir_#{Time.now.to_i}.xlsx")
+    begin
+      wb = FastExcel.open(temp_file, constant_memory: true)
+      ws = wb.add_worksheet
+      ws.append_row(["test", 123])
+      wb.close
+
+      assert File.exist?(temp_file)
+      assert File.size(temp_file) > 0
+    ensure
+      File.delete(temp_file) if File.exist?(temp_file)
+    end
+  end
+
+  it "accepts custom tmpdir option" do
+    Dir.mktmpdir do |custom_tmp|
+      temp_file = File.join(custom_tmp, "test_custom_tmpdir.xlsx")
+      wb = FastExcel.open(temp_file, constant_memory: true, tmpdir: custom_tmp)
+      ws = wb.add_worksheet
+      ws.append_row(["hello", "world"])
+      wb.close
+
+      assert File.exist?(temp_file)
+      assert File.size(temp_file) > 0
+    end
+  end
+end

data/test/validations_test.rb

@@ -2,23 +2,23 @@ require_relative 'test_helper'
 
 describe "FastExcel validations" do
 
-  it "should raise error when duplicate sheet name" do
-    workbook = FastExcel.open(constant_memory: true)
+  before do
+    @workbook = FastExcel.open(constant_memory: true)
+  end
 
-    workbook.add_worksheet("Payments Report")
+  it "should raise error when duplicate sheet name" do
+    @workbook.add_worksheet("Payments Report")
     error = assert_raises do
-      workbook.add_worksheet("Payments Report")
+      @workbook.add_worksheet("Payments Report")
     end
 
     assert_equal(ArgumentError, error.class)
-    assert_equal("Invalid worksheet name 'Payments Report': (16 - error_sheetname_already_used) Worksheet name is already in use.", error.message)
+    assert_equal("Invalid worksheet name 'Payments Report': (17 - error_sheetname_already_used) Worksheet name 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()
+    ws1 = @workbook.add_worksheet()
+    ws2 = @workbook.add_worksheet()
     ws2.write_value(0, 2, "aaa")
 
     assert_equal("Sheet1", ws1[:name])
@@ -26,33 +26,39 @@ describe "FastExcel validations" do
   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")
+      @workbook.add_worksheet("ABCDEFGHIJKLMNOPQRSTUVWXYZ012345")
     end
 
     assert_equal(ArgumentError, error.class)
-    assert_equal("Invalid worksheet name 'ABCDEFGHIJKLMNOPQRSTUVWXYZ012345': (13 - error_sheetname_length_exceeded) Worksheet name exceeds Excel's limit of 31 characters.", error.message)
+    assert_equal("Invalid worksheet name 'ABCDEFGHIJKLMNOPQRSTUVWXYZ012345': (14 - error_sheetname_length_exceeded) Worksheet name 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 = @workbook.add_worksheet("ABCDEFGHIJKLMNOPQRSTUVWXYZ01234")
     worksheet.append_row(["aaa", "bbb", "ccc"])
 
     assert_equal("ABCDEFGHIJKLMNOPQRSTUVWXYZ01234", worksheet[:name])
   end
 
   it "should validate using Libxlsxwriter validation" do
-    workbook = FastExcel.open(constant_memory: true)
     error = assert_raises do
-      worksheet = workbook.add_worksheet('a?')
+      worksheet = @workbook.add_worksheet('a?')
       worksheet.write_value(1, 1, 'a') # without the validation, this method will crash the process
     end
 
     assert_equal(ArgumentError, error.class)
-    assert_equal("Invalid worksheet name 'a?': (14 - error_invalid_sheetname_character) Worksheet name cannot contain invalid characters: '[ ] : * ? / \\'", error.message)
+    assert_equal("Invalid worksheet name 'a?': (15 - error_invalid_sheetname_character) Worksheet name cannot contain invalid characters: '[ ] : * ? / \\'", error.message)
+  end
+
+  it "should raise error when reopening workbook" do
+    @workbook.close
+
+    error = assert_raises do
+      FastExcel.open(@workbook.filename, constant_memory: true)
+    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/worksheet_test.rb

@@ -96,8 +96,10 @@ describe "FastExcel::WorksheetExt append_row" do
   it "should map fields correctly" do
     workbook = FastExcel.open(constant_memory: true)
 
-    ws = workbook.add_worksheet
+    ws = workbook.add_worksheet("1 should map fields correctly")
+
     ws.set_right_to_left
+
     assert_equal(ws[:right_to_left], 1)
 
     ws = workbook.add_worksheet
@@ -117,6 +119,7 @@ describe "FastExcel::WorksheetExt append_row" do
     assert_equal(ws[:margin_top], 3.5)
     assert_equal(ws[:margin_bottom], 4.5)
 
+    assert_equal(ws[:vbreaks_count], 0)
 
     breaks = [20, 40, 60, 20, 0]
     FFI::MemoryPointer.new(:uint16, breaks.size) do |buffer|
@@ -124,6 +127,8 @@ describe "FastExcel::WorksheetExt append_row" do
       ws.set_v_pagebreaks(buffer)
     end
 
+    # FastExcel.print_ffi_obj(ws)
+
     assert_equal(ws[:vbreaks_count], 4)
   end
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: fast_excel
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.6.0
 platform: ruby
 authors:
 - Pavel Evstigneev
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-01-12 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -16,7 +15,7 @@ dependencies:
     requirements:
     - - ">"
       - !ruby/object:Gem::Version
-        version: '1.9'
+        version: '1.17'
     - - "<"
       - !ruby/object:Gem::Version
         version: '2'
@@ -26,7 +25,27 @@ dependencies:
     requirements:
     - - ">"
       - !ruby/object:Gem::Version
-        version: '1.9'
+        version: '1.17'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2'
+- !ruby/object:Gem::Dependency
+  name: base64
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.2'
     - - "<"
       - !ruby/object:Gem::Version
         version: '2'
@@ -41,6 +60,7 @@ files:
 - ".dockerignore"
 - ".github/workflows/test.yml"
 - ".gitignore"
+- ".octocov.yml"
 - CHANGELOG.md
 - Dockerfile.test
 - Gemfile
@@ -55,11 +75,13 @@ files:
 - benchmarks/memory.rb
 - benchmarks/profiler.rb
 - benchmarks/write_value.rb
+- examples/comment_options.rb
 - examples/example.rb
 - examples/example_align.rb
 - examples/example_auto_width.rb
 - examples/example_chart.rb
 - examples/example_colors.rb
+- examples/example_date_time.rb
 - examples/example_filters.rb
 - examples/example_formula.rb
 - examples/example_hyperlink.rb
@@ -77,15 +99,27 @@ files:
 - lib/fast_excel/binding/format.rb
 - lib/fast_excel/binding/workbook.rb
 - lib/fast_excel/binding/worksheet.rb
+- libxlsxwriter/.github/FUNDING.yml
+- libxlsxwriter/.github/ISSUE_TEMPLATE.md
+- libxlsxwriter/.github/PULL_REQUEST_TEMPLATE.md
+- libxlsxwriter/.github/workflows/cmake_actions.yml
+- libxlsxwriter/.github/workflows/code_style.yml
+- libxlsxwriter/.github/workflows/coverity.yml
+- libxlsxwriter/.github/workflows/make_actions.yml
+- libxlsxwriter/.github/workflows/valgrind.yml
+- libxlsxwriter/.github/workflows/windows_build.yml
+- libxlsxwriter/.github/workflows/zig_build.yml
 - libxlsxwriter/.gitignore
 - libxlsxwriter/.indent.pro
-- libxlsxwriter/.travis.yml
 - libxlsxwriter/CMakeLists.txt
 - libxlsxwriter/CONTRIBUTING.md
 - libxlsxwriter/Changes.txt
 - libxlsxwriter/LICENSE.txt
 - libxlsxwriter/Makefile
+- libxlsxwriter/Package.swift
 - libxlsxwriter/Readme.md
+- libxlsxwriter/build.zig
+- libxlsxwriter/build.zig.zon
 - libxlsxwriter/cmake/FindMINIZIP.cmake
 - libxlsxwriter/cmake/FindPackage.cmake
 - libxlsxwriter/cmake/i686-toolchain.cmake
@@ -95,6 +129,7 @@ files:
 - libxlsxwriter/include/xlsxwriter/app.h
 - libxlsxwriter/include/xlsxwriter/chart.h
 - libxlsxwriter/include/xlsxwriter/chartsheet.h
+- libxlsxwriter/include/xlsxwriter/comment.h
 - libxlsxwriter/include/xlsxwriter/common.h
 - libxlsxwriter/include/xlsxwriter/content_types.h
 - libxlsxwriter/include/xlsxwriter/core.h
@@ -102,17 +137,22 @@ files:
 - libxlsxwriter/include/xlsxwriter/drawing.h
 - libxlsxwriter/include/xlsxwriter/format.h
 - libxlsxwriter/include/xlsxwriter/hash_table.h
+- libxlsxwriter/include/xlsxwriter/metadata.h
 - libxlsxwriter/include/xlsxwriter/packager.h
 - libxlsxwriter/include/xlsxwriter/relationships.h
 - libxlsxwriter/include/xlsxwriter/shared_strings.h
 - libxlsxwriter/include/xlsxwriter/styles.h
+- libxlsxwriter/include/xlsxwriter/table.h
 - libxlsxwriter/include/xlsxwriter/theme.h
+- libxlsxwriter/include/xlsxwriter/third_party/emyg_dtoa.h
 - libxlsxwriter/include/xlsxwriter/third_party/ioapi.h
+- libxlsxwriter/include/xlsxwriter/third_party/md5.h
 - libxlsxwriter/include/xlsxwriter/third_party/queue.h
 - libxlsxwriter/include/xlsxwriter/third_party/tmpfileplus.h
 - libxlsxwriter/include/xlsxwriter/third_party/tree.h
 - libxlsxwriter/include/xlsxwriter/third_party/zip.h
 - libxlsxwriter/include/xlsxwriter/utility.h
+- libxlsxwriter/include/xlsxwriter/vml.h
 - libxlsxwriter/include/xlsxwriter/workbook.h
 - libxlsxwriter/include/xlsxwriter/worksheet.h
 - libxlsxwriter/include/xlsxwriter/xmlwriter.h
@@ -122,21 +162,31 @@ files:
 - libxlsxwriter/src/app.c
 - libxlsxwriter/src/chart.c
 - libxlsxwriter/src/chartsheet.c
+- libxlsxwriter/src/comment.c
 - libxlsxwriter/src/content_types.c
 - libxlsxwriter/src/core.c
 - libxlsxwriter/src/custom.c
 - libxlsxwriter/src/drawing.c
 - libxlsxwriter/src/format.c
 - libxlsxwriter/src/hash_table.c
+- libxlsxwriter/src/metadata.c
 - libxlsxwriter/src/packager.c
 - libxlsxwriter/src/relationships.c
 - libxlsxwriter/src/shared_strings.c
 - libxlsxwriter/src/styles.c
+- libxlsxwriter/src/table.c
 - libxlsxwriter/src/theme.c
 - libxlsxwriter/src/utility.c
+- libxlsxwriter/src/vml.c
 - libxlsxwriter/src/workbook.c
 - libxlsxwriter/src/worksheet.c
 - libxlsxwriter/src/xmlwriter.c
+- libxlsxwriter/third_party/dtoa/Makefile
+- libxlsxwriter/third_party/dtoa/emyg_dtoa.c
+- libxlsxwriter/third_party/dtoa/emyg_dtoa.h
+- libxlsxwriter/third_party/md5/Makefile
+- libxlsxwriter/third_party/md5/md5.c
+- libxlsxwriter/third_party/md5/md5.h
 - libxlsxwriter/third_party/minizip/Makefile
 - libxlsxwriter/third_party/minizip/Makefile.am
 - libxlsxwriter/third_party/minizip/Makefile.orig
@@ -165,12 +215,16 @@ files:
 - libxlsxwriter/third_party/tmpfileplus/tmpfileplus.c
 - libxlsxwriter/third_party/tmpfileplus/tmpfileplus.h
 - libxlsxwriter/version.txt
+- sonar-project.properties
 - test/auto_width_test.rb
+- test/color_to_hex_test.rb
+- test/comment_opt_test.rb
 - test/date_test.rb
 - test/default_format_test.rb
 - test/format_test.rb
-- test/reopen_test.rb
+- test/get_worksheet_test.rb
 - test/test_helper.rb
+- test/tmpdir_test.rb
 - test/tmpfile_test.rb
 - test/validations_test.rb
 - test/worksheet_test.rb
@@ -178,7 +232,6 @@ homepage: https://github.com/paxa/fast_excel
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -187,17 +240,13 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ">="
     - !ruby/object:Gem::Version
       version: '2.0'
-  - - "<"
-    - !ruby/object:Gem::Version
-      version: '4.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.3
-signing_key:
+rubygems_version: 4.0.16
 specification_version: 4
 summary: Ultra Fast Excel Writer
 test_files: []

data/libxlsxwriter/.travis.yml

@@ -1,37 +0,0 @@
-language: c
-dist: trusty
-
-compiler:
-  - clang
-  - gcc
-
-env:
-  -                                      CFLAGS='-Werror'
-  - NO_VALGRIND=1 USE_STANDARD_TMPFILE=1 CFLAGS='-Werror'
-  - NO_VALGRIND=1                        CFLAGS='-Werror -m32'
-  - NO_VALGRIND=1 USE_SYSTEM_MINIZIP=1   CFLAGS='-Werror'
-  - NO_VALGRIND=1 USE_DOUBLE_FUNCTION=1  CFLAGS='-Werror'
-
-install:
-  - sudo pip install pytest
-  - sudo apt-get update
-  - sudo apt-get remove -y oracle-java9-installer
-  - sudo apt-get install -y gcc-multilib
-  - sudo apt-get install -y zlib1g-dev
-  - sudo apt-get install -y zlib1g-dev:i386
-  - sudo apt-get install -y gcc
-  - sudo apt-get install -y valgrind
-
-  - wget http://mirrors.kernel.org/ubuntu/pool/universe/m/minizip/libminizip-dev_1.1-8_amd64.deb
-  - wget http://mirrors.kernel.org/ubuntu/pool/universe/m/minizip/libminizip1_1.1-8_amd64.deb
-  - sudo dpkg -i libminizip1_1.1-8_amd64.deb
-  - sudo dpkg -i libminizip-dev_1.1-8_amd64.deb
-
-script:
-  - make V=1
-  - make test_functional V=1
-  - make test_unit V=1
-  - make examples V=1
-  - make test_valgrind V=1
-  - sudo make install V=1
-  - sudo make uninstall V=1

data/test/reopen_test.rb

@@ -1,22 +0,0 @@
-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