Fix code smells and add tests

[qdraw]

PR Details

Description / Motivation and Context

Related Issue

How Has This Been Tested

• C# Unit tests
• Typescript Unit tests
• Other Unit tests
• Manual tests
• Automatic End2end tests (starsky-tools/end2end)

Types of changes

• Added for new features
• Breaking change fix or feature that would cause existing functionality to change
• Changed for non-breaking changes in existing functionality for example docs change / refactoring / dependency upgrades
• Deprecated for soon-to-be removed features
• Removed for now removed features
• Fixed for any bug fixes
• Security in case of vulnerabilities

Checklist

• My code follows the code style of this project.
• I have updated the documentation accordingly (update when needed)
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.
• I have updated the ./history.md document

[codecov]

Codecov Report

Attention: 20 lines in your changes are missing coverage. Please review.

Comparison is base (74f2de6) 94.60% compared to head (82ffc3d) 94.68%.

❗ Current head 82ffc3d differs from pull request most recent head 666c272. Consider uploading reports for the commit 666c272 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1278      +/-   ##
==========================================
+ Coverage   94.60%   94.68%   +0.07%     
==========================================
  Files         572      573       +1     
  Lines       20244    20285      +41     
  Branches     3187     3169      -18     
==========================================
+ Hits        19152    19206      +54     
+ Misses        723      711      -12     
+ Partials      369      368       -1     

Flag	Coverage Δ
all	94.68% <94.85%> (+0.07%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...foundation.readmeta/ReadMetaHelpers/ReadMetaXmp.cs	97.38% <100.00%> (+0.26%)	⬆️
...mbnailmeta/Services/OffsetDataMetaExifThumbnail.cs	100.00% <100.00%> (ø)
...components/organisms/menu-archive/menu-archive.tsx	92.04% <100.00%> (+2.35%)	⬆️
...organisms/menu-archive/shared/upload-menu-item.tsx	100.00% <100.00%> (ø)
...ure.health/UpdateCheck/Services/CheckForUpdates.cs	94.28% <0.00%> (ø)
.../starsky.feature.metaupdate/Services/DeleteItem.cs	98.57% <96.55%> (-1.43%)	⬇️
...ndation.database/Helpers/FileIndexCompareHelper.cs	89.23% <75.00%> (-0.62%)	⬇️
...ebar/archive-sidebar-label-edit-search-replace.tsx	92.77% <90.47%> (+0.17%)	⬆️
...oundation.readmeta/ReadMetaHelpers/ReadMetaExif.cs	94.36% <95.52%> (+3.03%)	⬆️
...ation.platform/Helpers/AppSettingsCompareHelper.cs	92.24% <92.24%> (+0.65%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

96.8% Coverage
0.1% Duplication