Mirrored_Repos/DarkflameServer
1
0
Fork 0
mirror of https://github.com/DarkflameUniverse/DarkflameServer.git synced 2026-10-02 10:53:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: clear the game globals the test fixture deletes

Browse Source 
GameDependenciesTest deleted the logger, config and managers but left the
pointers set, so a later test without the fixture (packet capture) logged
through a freed logger and crashed when the whole suite ran.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Aaron Kimbrell
2026-09-29 22:25:28 -05:00
parent 9043f97622
commit 0b58b3ec97
1 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
tests/dGameTests/GameDependencies.h
View File

@@ -73,13 +73,18 @@ protected:
void TearDownDependencies() {
if (Game::server) delete Game::server;
if (Game::entityManager) delete Game::entityManager;
if (Game::zoneManager) delete Game::zoneManager;
// Cleared too: tests without these fixtures (packet capture) run after them and check them
delete Game::entityManager;
Game::entityManager = nullptr;
delete Game::zoneManager;
Game::zoneManager = nullptr;
if (Game::logger) {
Game::logger->Flush();
delete Game::logger;
Game::logger = nullptr;
}
if (Game::config) delete Game::config;
delete Game::config;
Game::config = nullptr;
}
EntityInfo info{};