Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

tests: add e2e test validating regex route headers #305

Merged
merged 2 commits into from
Jul 19, 2022

Conversation

EJKinger
Copy link
Contributor

@EJKinger EJKinger commented Jul 18, 2022

This adds an e2e test validating that route header regex expressions are working as implemented in Kong/kong#6079.

@EJKinger EJKinger requested a review from a team as a code owner July 18, 2022 16:53
Copy link
Member

@hbagdi hbagdi left a comment

Choose a reason for hiding this comment

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

Integration tests are failing.

internal/test/e2e/basic_test.go Outdated Show resolved Hide resolved
internal/test/e2e/basic_test.go Outdated Show resolved Hide resolved
@EJKinger EJKinger force-pushed the tests/header-regex-e2e branch 2 times, most recently from 6903dbb to 958c011 Compare July 18, 2022 20:17
@EJKinger EJKinger requested a review from hbagdi July 18, 2022 20:18
@EJKinger
Copy link
Contributor Author

Integration tests are failing.

Fixed this, the regex test now only runs for v2.8.0+

@hbagdi hbagdi merged commit 0c359a9 into main Jul 19, 2022
@hbagdi hbagdi deleted the tests/header-regex-e2e branch July 19, 2022 18:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants