Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: use invokePrivate for test #49290

Merged
merged 1 commit into from
Nov 14, 2024
Merged

fix: use invokePrivate for test #49290

merged 1 commit into from
Nov 14, 2024

Conversation

SebastianKrupinski
Copy link
Contributor

@SebastianKrupinski SebastianKrupinski commented Nov 14, 2024 •
edited
Loading

Summary

Use built in invokePrivate method to test protected function

Checklist

@SebastianKrupinski
fix: use invokePrivate for test
fbd8afb 
Signed-off-by: SebastianKrupinski <[email protected]>
@SebastianKrupinski SebastianKrupinski added the 3. to review Waiting for reviews label Nov 14, 2024
@SebastianKrupinski SebastianKrupinski self-assigned this Nov 14, 2024
@SebastianKrupinski
Copy link
Contributor Author

/backport to stable30

@SebastianKrupinski SebastianKrupinski merged commit 8cf41ee into master Nov 14, 2024
185 checks passed
@SebastianKrupinski SebastianKrupinski deleted the fix/use-invokeprivate-for-test branch November 14, 2024 16:37
@backportbot backportbot bot mentioned this pull request Nov 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kesselb kesselb kesselb approved these changes

@st3iny st3iny st3iny approved these changes

@tcitworld tcitworld Awaiting requested review from tcitworld tcitworld is a code owner

Assignees

@SebastianKrupinski SebastianKrupinski

Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SebastianKrupinski @st3iny @kesselb