Maintenance / Minor changes

[bcoconni]

This PR brings a number of minor changes and fixes:

• In the CI workflow, MinGW32 has been renamed to MinGW as MinGW is no longer limited to 32 bits as stated on their website.
• Fix the codecov job which was broken since PR Bump codecov/codecov-action from 3 to 4 #1032: the coverage data were no longer uploaded to codecov.io.
• Extend the usage of CMake target_include_directories, following the lead of PR fix: target include directories #1022 (there are still a few places that use include_directories and this will be addressed in a later PR).
• Make the unit test test_inherit_with_extra_args more robust. Indeed there are some subtleties between __cinit__ and __init__:
  • The test now checks that the parameters root_dir and pm_root are properly handled.
  • Since the class FGFDMExec has no constructor __init__, there is no need to call it.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.37%. Comparing base (6f0c4d8) to head (ce8bd80).
Report is 11 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #1050       +/-   ##
===========================================
+ Coverage   24.87%   41.37%   +16.49%     
===========================================
  Files         168      198       +30     
  Lines       18908    24071     +5163     
  Branches        0     3170     +3170     
===========================================
+ Hits         4704     9960     +5256     
+ Misses      14204    14062      -142     
- Partials        0       49       +49     

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

[bcoconni]

The codecov report is back, confirming that this PR fixes the coverage job in cpp-python-build.yml 😄