Test refactorings for improved code coverage#976
Conversation
Added .runsettings file for configuring code coverage, separated OpenXmlImporter and OpenXmlExporter tests, added new tests for retrieving sheet informations, sheet dimensions and column names, added new queryrange tests, added new addpicture test, removed obsolete FileHelper test utility class
There was a problem hiding this comment.
Code Review
This pull request introduces several enhancements and test refactorings, including adding a .runsettings file for code coverage, making NextResult abstract in MiniExcelDataReaderBase, adding .xlsm format validation, and splitting OpenXml importer and exporter tests into dedicated files. Feedback on the changes highlights a redundant stream in FillTemplateAsync, potential file locking issues in template tests due to stream scope, and compiler warnings from synchronous test methods incorrectly marked as async Task. Additionally, it is recommended to use StringComparison.OrdinalIgnoreCase for file extension comparisons.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
The following changes have been implemented to improve test coverage:
closes #864