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.

Sign up for GitHub

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

Jump to bottom

Enable response tests on Kuadrant #159

Merged
merged 1 commit into from
Jan 26, 2023
Merged

Enable response tests on Kuadrant #159

merged 1 commit into from
Jan 26, 2023

Conversation

pehala
Copy link
Contributor

@pehala pehala commented Jan 6, 2023

Requires #158. Adds authorization_name fixture, which enables parametrization of Authorization without recreating it.

@pehala pehala requested a review from averevki January 6, 2023 11:53
@pehala
Add authorization_name fixture
d7ba7cf 
- Can be used for ensuring new Authorization is created when parametrization is used
@pehala pehala force-pushed the fix_kuadrant_response branch from 83e9ca2 to d7ba7cf Compare January 24, 2023 11:57
@pehala pehala marked this pull request as ready for review January 24, 2023 11:58
@pehala pehala requested a review from jsmolar January 24, 2023 11:58
@pehala pehala merged commit 0d72438 into Kuadrant:main Jan 26, 2023
@averevki averevki mentioned this pull request Mar 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@averevki averevki averevki approved these changes

@jsmolar jsmolar jsmolar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pehala @jsmolar @averevki