#215 Prevent mapping of inaccessible fields to record

[marcustalbot-ov]

• Fixed bug where inaccessible fields (usually private fields without accessor) would be mapped to a target record, disregarding mapping rules.

[marcustalbot-ov]

The Codacy analysis only flags the 'unused' private fields in the test-classes. As such, I recommend ignoring those issues.

[JoelDuurkoop]

1 test failure

[marcustalbot-ov]

Fixing the pipeline is not something I can do easily. Steps to fix the problem can be found here: https://rieckpil.de/fix-sonatype-oss-index-errors-for-owasp-maven-plugin/

Unless setting the ossindexAnalyzerEnabled-flag to false is acceptable. In that case, please let me know.

Or, if you were talking about a test-failure in the library itself, could you tell me what is failing? It runs on my machine, with Java 21.

[JoelDuurkoop]

Library tests run successfully for java 21, but when running with the recently added java 25 this results in an IllegalArgumentException for testMapEntityWithMapToEntityResultWithMap.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[marcustalbot-ov]

Library tests run successfully for java 21, but when running with the recently added java 25 this results in an IllegalArgumentException for testMapEntityWithMapToEntityResultWithMap.

That's a known issue, which was fixed in the latest merged PR. I have now rebased this branch to include the fix.

[codecov]

Codecov Report

❌ Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 86.24%. Comparing base (20d1fde) to head (c26bf49).
⚠️ Report is 90 commits behind head on master.

Files with missing lines	Patch %	Lines
...va/io/beanmapper/strategy/MapToRecordStrategy.java	88.88%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #217      +/-   ##
============================================
- Coverage     87.61%   86.24%   -1.38%     
- Complexity      998     1169     +171     
============================================
  Files           110      129      +19     
  Lines          2205     2603     +398     
  Branches        246      298      +52     
============================================
+ Hits           1932     2245     +313     
- Misses          183      230      +47     
- Partials         90      128      +38     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[marcustalbot-ov]

While the OWASP Dependency Check succeeds, the logs contain the following warning:

Warning: Disabling OSS Index analyzer due to missing user/password credentials. Authentication is now required: https://ossindex.sonatype.org/doc/auth-required

Ideally, authentication is arranged for the OSS Index Analyzer.