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.

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

tests(clustering/rpc): rpc mock/hook #14030

Merged
merged 1 commit into from
Dec 27, 2024
Merged

tests(clustering/rpc): rpc mock/hook #14030

merged 1 commit into from
Dec 27, 2024

Conversation

StarlightIbuki
Copy link
Contributor

@StarlightIbuki StarlightIbuki commented Dec 17, 2024

Summary

RPC needs mock tests

doc: https://github.com/Kong/kong/blob/rpc-mock/spec/helpers/rpc_mock/readme.md

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md

Issue reference

Fix KAG-5551

@github-actions github-actions bot added the cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee label Dec 17, 2024
@chronolaw chronolaw self-requested a review December 18, 2024 07:30
@chronolaw chronolaw changed the title tests(rpc): rpc mock/hook tests(clustering/rpc): rpc mock/hook Dec 18, 2024
@StarlightIbuki StarlightIbuki force-pushed the rpc-mock branch 2 times, most recently from c79e2be to 5438bcc Compare December 18, 2024 07:57
spec/02-integration/01-helpers/05-rpc-mock_spec.lua Outdated Show resolved Hide resolved
spec/01-unit/19-hybrid/04-rpc_spec.lua Outdated Show resolved Hide resolved
spec/01-unit/19-hybrid/04-rpc_spec.lua Outdated Show resolved Hide resolved
spec/02-integration/01-helpers/05-rpc-mock_spec.lua Outdated Show resolved Hide resolved
spec/01-unit/19-hybrid/04-rpc_spec.lua Outdated Show resolved Hide resolved
spec/helpers/rpc_mock/client.lua Outdated Show resolved Hide resolved
spec/helpers/rpc_mock/server.lua Outdated Show resolved Hide resolved
spec/helpers/rpc_mock/server.lua Outdated Show resolved Hide resolved
@StarlightIbuki StarlightIbuki force-pushed the rpc-mock branch 8 times, most recently from 0174ea2 to 419e823 Compare December 24, 2024 06:16
spec/internal/misc.lua Outdated Show resolved Hide resolved
@StarlightIbuki StarlightIbuki force-pushed the rpc-mock branch 2 times, most recently from c618289 to f238be2 Compare December 24, 2024 09:36
spec/01-unit/09-balancer/03-consistent_hashing_spec.lua Outdated Show resolved Hide resolved
spec/02-integration/01-helpers/05-rpc-mock_spec.lua Outdated Show resolved Hide resolved
spec/02-integration/01-helpers/05-rpc-mock_spec.lua Outdated Show resolved Hide resolved
spec/helpers.lua Outdated Show resolved Hide resolved
spec/helpers/rpc_mock/client.lua Outdated Show resolved Hide resolved
spec/helpers/rpc_mock/client.lua Outdated Show resolved Hide resolved
spec/helpers/rpc_mock/client.lua Outdated Show resolved Hide resolved
spec/02-integration/01-helpers/05-rpc-mock_spec.lua Outdated Show resolved Hide resolved
spec/helpers.lua Outdated Show resolved Hide resolved
@ADD-SP ADD-SP merged commit d303ae6 into master Dec 27, 2024
25 checks passed
@ADD-SP ADD-SP deleted the rpc-mock branch December 27, 2024 07:47
@team-gateway-bot
Copy link
Collaborator

Successfully created cherry-pick PR for master:

local get_node_id = misc.get_node_id
local DP_PREFIX = "servroot_dp"

describe("rpc mock/hooc", function()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rpc mock/hook?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants