-
Notifications
You must be signed in to change notification settings - Fork 801
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
Move Allocator unit tests to better test module #2441
Move Allocator unit tests to better test module #2441
Conversation
Refactoring and cleanup I made while investigating googleforgames#2282. Lots of the allocator tests where tied into the GameServerAllocation controller tests, when there was no actual need for that to be the case. This refactor de-couples those tests from the controller to have better isolation in the testing, make it easier to create new tests, and make test discovery for the Allocator system easier. Also added a few more tests for extra coverage, and cleaned up a few pieces here and there with the migration.
Build Succeeded 👏 Build Id: c39bb42e-9ec4-4e89-bbd0-2844ded80c5d 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:
|
New changes are detected. LGTM label has been removed. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: markmandel, roberthbailey The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Build Succeeded 👏 Build Id: b7f1bece-cc46-4636-aa8b-27c8c5698f15 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:
|
What type of PR is this?
/kind cleanup
What this PR does / Why we need it:
Lots of the allocator tests where tied into the GameServerAllocation controller tests, when there was no actual need for that to be the case.
This refactor de-couples those tests from the controller to have better isolation in the testing, make it easier to create new tests, and make test discovery for the Allocator system easier.
Also added a few more tests for extra coverage, and cleaned up a few pieces here and there with the migration.
Which issue(s) this PR fixes:
Refactoring and cleanup I made while investigating #2282.
Special notes for your reviewer:
N/A