-
Notifications
You must be signed in to change notification settings - Fork 16
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
Test DNSPolicy health check additional headers #310
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall too many changes for one additional test only due to a refactor. I would prefer to do a separate PR for the rename or keep the name for now
testsuite/tests/mgc/dnspolicy/health_check/test_additional_headers.py
Outdated
Show resolved
Hide resolved
35a79b4
to
58e24e3
Compare
58e24e3
to
b8d7ae5
Compare
7f1de0c
to
7ffb530
Compare
7ffb530
to
4bb7e6c
Compare
rebase^ |
8d7a0d3
to
06e4642
Compare
rebase^ |
I tried running this myself but ran into some issues with |
06e4642
to
6c05aed
Compare
rebase^ |
Reran after rebase, working as expected now. |
Test case
DNSPolicy health check additional headers
Changes
closes #252