Skip to content

PlayerTracking Removal: Delete Unit Tests - #4719

Open
markmandel wants to merge 1 commit into
agones-dev:mainfrom
markmandel:playertracking/remove-unit-tests
Open

PlayerTracking Removal: Delete Unit Tests#4719
markmandel wants to merge 1 commit into
agones-dev:mainfrom
markmandel:playertracking/remove-unit-tests

Conversation

@markmandel

Copy link
Copy Markdown
Member

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug

/kind cleanup

/kind documentation
/kind feature
/kind hotfix
/kind release

What this PR does / Why we need it:

Removal of the unit tests for PlayerTracking as part of removing this functionality from Agones.

Which issue(s) this PR fixes:

Work on #4491

Did you use AI tools in preparing this PR?:

N

Special notes for your reviewer:

Removal of the unit tests for PlayerTracking as part of removing this
functionality from Agones.

Work on agones-dev#4491

Signed-off-by: Mark Mandel <mark@compoundtheory.com>
@markmandel
markmandel requested a lite review from Copilot September 5, 2026 03:01
@markmandel markmandel added the area/tests Unit tests, e2e tests, anything to make sure things don't break label Sep 5, 2026
@github-actions github-actions Bot added kind/cleanup Refactoring code, fixing up documentation, etc size/L labels Sep 5, 2026

This comment was marked as resolved.

@markmandel

Copy link
Copy Markdown
Member Author

Going to hide the copilot review - it doesn't love I'm deleting the unit tests before the functionality - but since the docs for all this functionality are gone, this felt like a safe section of code to clear out in a chunk.

@agones-bot

Copy link
Copy Markdown
Collaborator

Build Succeeded 🥳

Build Id: 08ebfaec-d767-4093-b652-f7073e44bc35

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4719/head:pr_4719 && git checkout pr_4719
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.61.0-dev-901f0cb

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

Labels

area/tests Unit tests, e2e tests, anything to make sure things don't break kind/cleanup Refactoring code, fixing up documentation, etc size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants