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

[cherry-pick master] tests(plugins): add old version plugin compatibility test (#9077) #13459

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

catbro666
Copy link
Contributor

@catbro666 catbro666 commented Aug 6, 2024

Summary

Cherry-picking https://github.com/Kong/kong-ee/pull/9077.

Test the old version plugin against the latest core code. We only focus on whether there is any runtime error in the happy path. Because logic changes from version to version, it is not feasible to test the behavior of the plugin.

This test is not guaranteed to capture all interface changes, because it may not be able to cover all the dependent interfaces, and even if the interface is covered, it can't detect interface semantic changes or internal logic changes. These kinds of things still rely on the tests of the interfaces themselves to cover.

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
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

https://konghq.atlassian.net/browse/FTI-5923

Test the old version plugin against the latest core code. We only focus on
whether there is any runtime error in the happy path. Because logic
changes from version to version, it is not feasible to test the behavior
of the plugin.

This test is not guaranteed to capture all interface changes, because it may not be able to cover all the dependent interfaces, and even if the interface is covered, it can't detect interface semantic changes or internal logic changes. These kinds of things still rely on the tests of the interfaces themselves to cover.

https://konghq.atlassian.net/browse/FTI-5923
@github-actions github-actions bot added chore Not part of the core functionality of kong, but still needed cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Aug 6, 2024
@ms2008 ms2008 merged commit c2860b1 into master Aug 13, 2024
39 checks passed
@ms2008 ms2008 deleted the cherry-9077-plugin-compatibility-test branch August 13, 2024 06:54
@github-actions github-actions bot added the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Aug 13, 2024
@catbro666 catbro666 removed the cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee label Aug 13, 2024
@Kong Kong deleted a comment from team-gateway-bot Aug 13, 2024
@catbro666 catbro666 removed the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Not part of the core functionality of kong, but still needed size/L skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants