jas88 / TypeGuesser

89%

main: 89%

Build Type

Pull #13

github

Committed by jas88

Commit Message

Fix 4 additional CodeQL code quality issues

Test code improvements:
- Added using statement for Guesser in multiple dispose test (line 60)
- Reverted DecimalTypeDecider to not use 'using' (not IDisposable)
- Converted foreach with variable mapping to LINQ All() in TypeDeciderFactoryTests
- Converted foreach with break to LINQ Where().FirstOrDefault() pattern
- Added System.Linq using directive to TypeDeciderFactoryTests

All changes improve code quality and follow CodeQL recommendations.
Tests: 377/377 passing. Build: 0 warnings, 0 errors.

Pull Request Pull Request #13: Feature/test coverage improvements

Run Details

440 of 530 branches covered (83.02%)

Branch coverage included in aggregate %.

156 of 184 new or added lines in 12 files covered. (84.78%)

3 existing lines in 2 files now uncovered.

1262 of 1379 relevant lines covered (91.52%)

195827.64 hits per line