Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DEBUG] Adding debug dump events / correcting IO methods. #109

Merged
merged 8 commits into from
Dec 26, 2024

Conversation

Neloreck
Copy link
Member

@Neloreck Neloreck commented Dec 25, 2024

Adding debug dump event on debug panel click, possible extension with keybind.
Collects large JSON file with state of the game.

Changes

  • Reformat code, remove not used imports
  • Update types
  • Update file save-load methods
  • Add debug manager and dumping methods/event
  • Handle debug dump in managers
  • Add more tests
  • New database manager for debug dump
  • toJSON -> produce valid json

Checklist

  • tests are included
  • documentation is changed or added

Signed-off-by: Neloreck <Neloreck@gmail.com>
@Neloreck Neloreck self-assigned this Dec 25, 2024
@github-actions github-actions bot added cli Changes are affecting XRF cli tools test Changes are updating or introducing unit tests managers Changes are affecting game managers ui Changes are affecting game UI / forms scripts dependencies labels Dec 25, 2024
Signed-off-by: Neloreck <Neloreck@gmail.com>
Signed-off-by: Neloreck <Neloreck@gmail.com>
Signed-off-by: Neloreck <Neloreck@gmail.com>
@github-actions github-actions bot added the schemes Changes are affecting game logics schemes label Dec 26, 2024
Copy link

codecov bot commented Dec 26, 2024

Codecov Report

Attention: Patch coverage is 90.78212% with 33 lines in your changes missing coverage. Please review.

Project coverage is 72.96%. Comparing base (d892bf5) to head (d2e7f83).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/engine/core/managers/weather/WeatherManager.ts 56.60% 23 Missing ⚠️
...gine/core/ui/debug/sections/DebugGeneralSection.ts 50.00% 4 Missing ⚠️
...gine/core/managers/statistics/StatisticsManager.ts 94.91% 3 Missing ⚠️
...c/engine/core/managers/database/DatabaseManager.ts 92.85% 2 Missing ⚠️
src/engine/core/utils/fs/fs_io.ts 95.23% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #109      +/-   ##
==========================================
+ Coverage   72.81%   72.96%   +0.14%     
==========================================
  Files        1304     1308       +4     
  Lines       36752    36951     +199     
  Branches     6254     6260       +6     
==========================================
+ Hits        26762    26962     +200     
+ Misses       9990     9989       -1     

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

Signed-off-by: Neloreck <Neloreck@gmail.com>
Signed-off-by: Neloreck <Neloreck@gmail.com>
Signed-off-by: Neloreck <Neloreck@gmail.com>
… / logs.

Signed-off-by: Neloreck <Neloreck@gmail.com>
@Neloreck Neloreck marked this pull request as ready for review December 26, 2024 02:09
@Neloreck Neloreck merged commit e60b5a9 into main Dec 26, 2024
13 checks passed
@Neloreck Neloreck deleted the dumb-debug-info branch December 26, 2024 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Changes are affecting XRF cli tools dependencies managers Changes are affecting game managers schemes Changes are affecting game logics schemes scripts test Changes are updating or introducing unit tests ui Changes are affecting game UI / forms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant