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(tools): add tests against kong.tools.http.parse_directive_header() #13482

Merged
merged 2 commits into from
Aug 12, 2024

Conversation

fffonion
Copy link
Contributor

@fffonion fffonion commented Aug 9, 2024

Summary

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

From https://github.com/Kong/kong-ee/pull/9910

@github-actions github-actions bot added the cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee label Aug 9, 2024
@ADD-SP ADD-SP changed the title chore(tests): port tools.http tests from EE tests(tools): add tests against kong.tools.http.parse_directive_header() Aug 12, 2024
@ADD-SP ADD-SP force-pushed the port-http-parse-header-tests branch from 67d2c5b to 4fce3b3 Compare August 12, 2024 09:27
@ADD-SP
Copy link
Contributor

ADD-SP commented Aug 12, 2024

Rebased on master and added an extra test

@fffonion fffonion merged commit 97b1680 into master Aug 12, 2024
24 checks passed
@fffonion fffonion deleted the port-http-parse-header-tests branch August 12, 2024 18:25
@team-gateway-bot
Copy link
Collaborator

Successfully created cherry-pick PR for master:

ProBrian pushed a commit that referenced this pull request Aug 13, 2024
curiositycasualty pushed a commit that referenced this pull request Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants