jas88 / TypeGuesser / 19211625466

89%

main: 90%

Build Type

Pull #13

github

Committed by jas88

Commit Message

Fix all 14 CodeQL alerts (1 error + 13 warnings)

Critical fix:
- #19 ERROR: Implemented IEquatable<DecimalSize> and fixed unsafe equality pattern
  Changed Equals(object) to use pattern matching instead of unsafe cast

Code quality improvements in tests:
- #22,#21: Fixed comparison of identical values (use separate references)
- #12,#11: Fixed null argument to Equals (use explicit null cast or Is.Null)
- #20: Added null-forgiving operator after null assertion
- #26-#23: Removed useless object upcasts (declare array as object[] instead)
- #10,#18,#17,#16: Wrapped IDisposable usage in using statements for guaranteed cleanup

All changes in test files only. Production code quality improved via IEquatable.
Build: 0 warnings, 0 errors. Tests: 377/377 passing.

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

Run Details

439 of 530 branches covered (82.83%)

Branch coverage included in aggregate %.

155 of 184 new or added lines in 12 files covered. (84.24%)

3 existing lines in 2 files now uncovered.

1261 of 1379 relevant lines covered (91.44%)

195827.6 hits per line