Refactor code to remove analyzer warnings

[rjmurillo]

Code clean up. Remove most instances of AV1535

Changes

• Remove cases where a type was marked as nullable when it wasn't actually nullable
• Convert Helpers class to various extension methods
• Add code comments to aid in clarity for future maintainers
• Remove suppressions for AV1535

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
✅ -0.37% (target: -1.00%)	❌ 93.75% (target: 95.00%)

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (ba89475)	406	378	93.10%
Head commit (11c4c4b)	413 (+7)	383 (+5)	92.74% (-0.37%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#113)	80	75	93.75%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

---

_{🚀 Don’t miss a bit, follow what’s new on Codacy.}

_{Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more}