config: adds GrpcMux tests with xds config tracker and resources delegate#45930
Open
birenroy wants to merge 3 commits into
Open
config: adds GrpcMux tests with xds config tracker and resources delegate#45930birenroy wants to merge 3 commits into
birenroy wants to merge 3 commits into
Conversation
…legate Add integration tests for XdsConfigTracker and XdsResourcesDelegate in both SotW (GrpcMuxImpl) and Delta (NewGrpcMuxImpl) implementations. Verify that: - onConfigAccepted/onConfigRejected are called on XdsConfigTracker. - getResources/onConfigUpdated/onResourceLoadFailed are called on XdsResourcesDelegate (SotW only). TAG=agy CONV=b942214e-2f01-4571-9478-b3eda0877a4c Signed-off-by: Biren Roy <birenroy@google.com>
…d mocks Move MockXdsConfigTracker and MockXdsResourcesDelegate from local test files to test/mocks/config/mocks.h to allow reuse and avoid duplication. Update BUILD dependencies accordingly. TAG=agy CONV=b942214e-2f01-4571-9478-b3eda0877a4c Signed-off-by: Biren Roy <birenroy@google.com>
…tests Signed-off-by: Biren Roy <birenroy@google.com>
birenroy
changed the title
Contributor
Author
|
/assign @adisuissa I'm adding some test coverage in advance of #45898. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds unit tests that integrate a
XdsConfigTrackerandXdsResourcesDelegatein both SotW (GrpcMuxImpl) and Delta (NewGrpcMuxImpl) implementations.Verifies that:
onConfigAccepted/onConfigRejectedare called onXdsConfigTracker.getResources/onConfigUpdated/onResourceLoadFailedare called onXdsResourcesDelegate(SotW only).I used the Gemini AI model to write these test cases.
Commit Message:
Additional Description:
Risk Level: none
Testing:
Docs Changes:
Release Notes:
Platform Specific Features: