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

Fix broken mtls test #293

Merged
merged 1 commit into from
Dec 6, 2023
Merged

Fix broken mtls test #293

merged 1 commit into from
Dec 6, 2023

Conversation

averevki
Copy link
Contributor

Fix broken mtls test after #259

  • Slightly refactor Result class to support more errors
  • Refactor tls tests to reflect changes in the httpx errors handling
  • Fix mtls test

@averevki averevki added bug Something isn't working enhancement Improvement to existing test labels Nov 24, 2023
@averevki averevki requested a review from pehala November 24, 2023 16:06
@averevki averevki self-assigned this Nov 24, 2023
testsuite/httpx/__init__.py Outdated Show resolved Hide resolved
testsuite/httpx/__init__.py Outdated Show resolved Hide resolved
testsuite/httpx/__init__.py Outdated Show resolved Hide resolved
testsuite/httpx/__init__.py Outdated Show resolved Hide resolved
pehala
pehala previously approved these changes Nov 27, 2023
@averevki averevki requested review from azgabur and jsmolar November 27, 2023 13:05
@averevki
Copy link
Contributor Author

averevki commented Dec 4, 2023

^rebase

@pehala pehala merged commit 86d8879 into Kuadrant:main Dec 6, 2023
2 checks passed
@averevki averevki deleted the fix-mtls-test branch November 11, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement Improvement to existing test
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants