https://bz.apache.org/bugzilla/show_bug.cgi?id=65046

--- Comment #1 from Dominik Stadler <dominik.stad...@gmx.at> ---
Thanks for the suggestion, yes, sound like a good idea to clean up the
integration-testing a bit and also the speedup sounds very good as it is
becoming a bit long-running already.

I will review the impact on the mass-testing, the following items are used
currently: 
* reuse the excludes via TestAllFiles.SCAN_EXCLUDES
* a way to lookup matching handlers based on file-extension via POIFileScanner
* use the interface FileHandler for executing the test for each file
* use BaseIntegrationTest.test() for executing tests for each file

None of the logic in TestAllFiles itself is needed, so refactoring it should be
possible as long as the basic FileHandler implementations are kept as-is.

Removing POIFileScanner would need some other way of getting a list of matching
documents. Keeping this in one place would be nice to not duplicate this logic
in both projects. 

Not sure how much impact the changed Exception-handling will have, this will
need a closer look.

But let's keep this until after the 5.0.0 release so we do not introduce more
changes right now.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Reply via email to