Add DictionaryTrait

[RiscadoA]

Closes #484, #541 and #542.

Description

Adds the DictionaryTrait and defines reflection for both std::map<K, V> and std::unordered_map<K, V>.

Checklist

• Self-review changes.
• Evaluate impact on the documentation.
• Ensure test coverage.
• Write new samples.

[github-actions]

PR Preview Action v1.4.4
🚀 Deployed preview to https://GameDevTecnico.github.io/cubos/pr-preview/pr-573/
on branch gh-pages at 2023-09-28 10:01 UTC

[codecov]

Codecov Report

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

Comparison is base (272177a) 31.80% compared to head (20be5a0) 33.92%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #573      +/-   ##
==========================================
+ Coverage   31.80%   33.92%   +2.11%     
==========================================
  Files          92       95       +3     
  Lines        6357     6594     +237     
==========================================
+ Hits         2022     2237     +215     
- Misses       4335     4357      +22     

Files	Coverage Δ
...ore/include/cubos/core/reflection/external/map.hpp	90.00% <90.00%> (ø)
...e/cubos/core/reflection/external/unordered_map.hpp	90.00% <90.00%> (ø)
...re/src/cubos/core/reflection/traits/dictionary.cpp	91.24% <91.24%> (ø)

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

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (1/7)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (2/7)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (3/7)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (4/7)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (5/7)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (6/7)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (7/7)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (1/2)

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (2/2)

[RiscadoA]

The coverage isn't so great on this PR, but I'm not sure if we should worry too much about it, at least for now.
What do you think? @luishfonseca

Already resolved issues

[luishfonseca]

Should we add .view(...)? It would allow the following syntax

for (auto [k, v] : dictionaryTrait.view(myDict))
{
...
}

[luishfonseca]

The coverage isn't so great on this PR, but I'm not sure if we should worry too much about it, at least for now. What do you think? @luishfonseca

It's fine, it will get covered as it gets used

No clang-tidy warnings found so I assume my comments were addressed

[github-actions]

⚠️ clang-tidy found issue(s) with the introduced code (1/1)

No clang-tidy warnings found so I assume my comments were addressed