Skip to content

feat: add fuzz testing for Excel reading and improve error handling in XlsxSaxAnalyser#549

Closed
alaahong wants to merge 7 commits intoapache:mainfrom
alaahong:feature/fuzz_xlsx_sax_analuser_read_exception
Closed

feat: add fuzz testing for Excel reading and improve error handling in XlsxSaxAnalyser#549
alaahong wants to merge 7 commits intoapache:mainfrom
alaahong:feature/fuzz_xlsx_sax_analuser_read_exception

Conversation

@alaahong
Copy link
Copy Markdown
Member

@alaahong alaahong commented Aug 25, 2025

Purpose of the pull request

add fuzz testing for Excel reading and improve error handling in XlsxSaxAnalyser
Related: #521 #548

What's changed?

Before
image

After
image

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

@alaahong alaahong requested a review from delei August 25, 2025 17:04
alaahong and others added 6 commits August 26, 2025 01:14
* feature: add an afterSheetDispose method to the SheetWriteHandler

* feature: add an afterSheetDispose method to the SheetWriteHandler

* refactor(fastexcel): Adjust the code according to Spotless

* refactor: Optimize class imports (#401)

* Update Excel writing method in test code

Replace EasyExcel.write with FastExcel.write

Modify two files related to template filling and regular writing tests

---------

Co-authored-by: wangmeng <wangmeng@gigacloudtech.com>
Co-authored-by: DeleiGuo <deleiguo@163.com>
* chore: reset test location and exclude spring-boot test

* refactor: adjust indentation for improved readability in FillTest.java

* chore: update Mockito version to 4.11.0 for jdk 8
# Conflicts:
#	fastexcel/src/test/java/cn/idev/excel/exception/XlsxSaxAnalyserReadOpcPackageTest.java
#	pom.xml
@alaahong alaahong closed this Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants