Skip to content

Add standalone unit tests and CI workflow for Unreal GSDK#194

Open
Copilot wants to merge 6 commits intomainfrom
copilot/add-unit-tests-gsdk
Open

Add standalone unit tests and CI workflow for Unreal GSDK#194
Copilot wants to merge 6 commits intomainfrom
copilot/add-unit-tests-gsdk

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026 •
edited
Loading

  • Add standalone C++ unit tests for Unreal GSDK (CMake + Google Test + nlohmann/json)
  • Add CI workflow .github/workflows/unreal-tests.yml for PRs affecting UnrealPlugin/**
  • Add "Plugin files" section to UnrealPlugin/README.md clarifying which files are needed for integration
  • Update UnrealPlugin/ThirdPersonMPGSDKSetup.md to list specific files to copy
  • Rebase onto latest origin/main

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Add unit tests for Unreal GSDK Add standalone unit tests and CI workflow for Unreal GSDK Mar 3, 2026
@dgkanatsios
Copy link
Contributor

dgkanatsios commented Mar 3, 2026

@copilot can you amend the Unreal README to mention that these files are not needed, when someone uses the Unreal GSDK?

Copy link
Contributor Author

Copilot AI commented Mar 3, 2026

@copilot can you amend the Unreal README to mention that these files are not needed, when someone uses the Unreal GSDK?

Added a "Directory structure" section to UnrealPlugin/README.md in 9eb3025 that lists the three files/directories needed for integration (Source/, Resources/, PlayFabGSDK.uplugin) and clarifies that Tests/, TestingProject/, Documentation/, docs, and license files are not needed for game projects.

@dgkanatsios
Copy link
Contributor

dgkanatsios commented Mar 3, 2026

@copilot can you try again?

Copy link
Contributor Author

Copilot AI commented Mar 3, 2026

@copilot can you try again?

Revised the README section in f18d845 — renamed to "Plugin files" with a clearer note callout, and also updated ThirdPersonMPGSDKSetup.md to list the specific files to copy (Source/, Resources/, PlayFabGSDK.uplugin) instead of "drag all files."

@dgkanatsios dgkanatsios marked this pull request as ready for review March 5, 2026 01:03
@dgkanatsios
Copy link
Contributor

dgkanatsios commented Mar 19, 2026

@copilot can you rebase?

Copilot AI requested a review from dgkanatsios March 19, 2026 15:39
Copilot stopped work on behalf of dgkanatsios due to an error March 19, 2026 15:39
The session was cancelled by the user.
@dgkanatsios
Copy link
Contributor

dgkanatsios commented Mar 19, 2026

@copilot can you rebase?

Copy link
Contributor Author

Copilot AI commented Mar 19, 2026

@copilot can you rebase?

Rebased onto latest origin/main (ba9b01f) in 97b0ef1.

Copilot AI and others added 6 commits March 19, 2026 10:27
Initial plan
39ff6d0
@dgkanatsios
Add standalone C++ unit tests and CI workflow for Unreal GSDK
de191e7 
Co-authored-by: dgkanatsios <8256138+dgkanatsios@users.noreply.github.com>
@dgkanatsios
Add directory structure section to Unreal README clarifying which fil…
5852dcc 
…es are needed

Co-authored-by: dgkanatsios <8256138+dgkanatsios@users.noreply.github.com>
@dgkanatsios
Improve README plugin files section and update setup guide to list sp…
7c7a554 
…ecific files to copy

Co-authored-by: dgkanatsios <8256138+dgkanatsios@users.noreply.github.com>
@dgkanatsios
Add Java GSDK unit tests and CI workflow (#195)
2e2b832 
* Initial plan

* Initial plan for Java GSDK unit tests

Co-authored-by: dgkanatsios <8256138+dgkanatsios@users.noreply.github.com>

* Add Java GSDK unit tests and CI workflow

Add 84 unit tests across 8 test classes covering:
- ConnectedPlayer data class
- SessionHostHeartbeatInfo getters/setters and JSON deserialization
- SessionConfig including ToMapAllStrings
- Operation enum Gson serialization with @SerializedName casings
- MaintenanceSchedule/MaintenanceEvent deserialization
- JsonFileConfiguration config file parsing and validation
- GameHostHealth/SessionHostStatus enum values
- GameserverSDK public API constant keys

Add .github/workflows/java-tests.yml CI workflow to run tests on PRs.

Co-authored-by: dgkanatsios <8256138+dgkanatsios@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: dgkanatsios <8256138+dgkanatsios@users.noreply.github.com>
Initial plan
4fae61a
@dgkanatsios dgkanatsios force-pushed the copilot/add-unit-tests-gsdk branch from 97b0ef1 to 4fae61a Compare March 19, 2026 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dgkanatsios dgkanatsios Awaiting requested review from dgkanatsios

At least 1 approving review is required to merge this pull request.

Assignees

@dgkanatsios dgkanatsios

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dgkanatsios