LensModel to loop and avoid returning default values & LensModel add tests for sony raw files to avoid default values

[qdraw]

PR Details

Description / Motivation and Context

• (Fixed) Back-end LensModel to loop and avoid returning default values. (PR LensModel to loop and avoid returning default values & LensModel add tests for sony raw files to avoid default values #1807)
• (Fixed) Back-end LensModel add tests for sony raw files to avoid default values (PR LensModel to loop and avoid returning default values & LensModel add tests for sony raw files to avoid default values #1807)

This pull request includes multiple changes to improve the codebase, enhance functionality, and add new tests. The most important changes include namespace adjustments, method reordering, and new test additions.

Codebase Improvements:

• Changed namespace declarations to file-scoped namespaces in multiple files (IReadMeta.cs, ReadMetaHelpers/ReadMetaExif.cs, Services/ReadMeta.cs). [1] [2] [3]
• Reordered and grouped methods logically within ReadMeta.cs to improve readability and maintainability. [1] [2] [3] [4] [5]

Functional Enhancements:

• Added logic to handle Sony lens models more accurately in ReadMetaExif.cs, including ignoring generic E-mount and T-mount lenses. [1] [2]
• Improved the handling of Make values by trimming trailing spaces in SetArrayBasedItemsMakeModel method.
• Enhanced GetMakeLensModel to iterate over directories and avoid returning placeholder values.

New Tests:

• Added a new test class SonyLensIdConverterTest to verify the functionality of Sony lens ID conversion methods.
• Expanded ReadMetaExifTest with a new test method to validate parsing of EXIF directories with example data.
• Updated ReadMetaSubPathStorageTest to include a summary comment and improve readability.

New Utility Classes:

• Created utility classes CreateAnImageA330Raw and CreateAnImageA6600Raw to facilitate testing with raw image data. [1] [2]

Documentation:

• Added README files to document the process of creating test image data. [1] [2]

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

[sonarqubecloud]

Quality Gate failed

Failed conditions
22.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud