leehaut opened a new pull request, #999:
URL: https://github.com/apache/fesod/pull/999
## Purpose of the pull request
Improve unit test coverage for individual scalar converter implementations.
`ConverterTest` previously only exercised `FloatNumberConverter` write
conversion;
this change turns it into a focused `@Tag(unit)` suite covering the common
Boolean / Number / String converters.
## What's changed?
- Rewrite `ConverterTest` as pure unit tests (no Excel I/O / round-trip).
- Add parameterized checks for `supportJavaTypeKey()` /
`supportExcelTypeKey()` across 31 converters.
- Cover read/write conversion for:
- `Boolean`, `Integer`, `Long`, `Short`, `Byte`, `Float`, `Double`
- `BigDecimal`, `BigInteger`
- `String` (`STRING` / `BOOLEAN` / `NUMBER` / `ERROR`)
- Keep factory method naming separate from the test method
(`supportKeysProvider`) to avoid JUnit `@MethodSource` name clashes.
No functional changes are included in this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
