Skip to content

Fix and run unit tests#1

Draft
erizhang wants to merge 2 commits intomasterfrom
cursor/fix-and-run-unit-tests-4e29
Draft

Fix and run unit tests#1
erizhang wants to merge 2 commits intomasterfrom
cursor/fix-and-run-unit-tests-4e29

Conversation

@erizhang
Copy link
Copy Markdown
Owner

@erizhang erizhang commented Sep 9, 2025

Fix unit test compilation by resolving a global variable multiple definition error and update cpputest submodule.

The shutdown_server function pointer was defined directly in src/html_ui/html_game.h, causing multiple definition linker errors when included in multiple compilation units. This PR corrects it by declaring shutdown_server as extern in the header and providing its definition in src/html_ui/html_game.c.


Open in Cursor Open in Web

cursoragent and others added 2 commits September 9, 2025 11:31
Co-authored-by: zhang.lyuan <zhang.lyuan@gmail.com>
Co-authored-by: zhang.lyuan <zhang.lyuan@gmail.com>
@cursor
Copy link
Copy Markdown

cursor bot commented Sep 9, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants