Skip to content

acceptance: modify framework to make dynamic fixture passing easier#1879

Merged
github-actions[bot] merged 2 commits into
quay:mainfrom
crozzy:framework-changes
May 18, 2026
Merged

acceptance: modify framework to make dynamic fixture passing easier#1879
github-actions[bot] merged 2 commits into
quay:mainfrom
crozzy:framework-changes

Conversation

@crozzy
Copy link
Copy Markdown
Contributor

@crozzy crozzy commented May 14, 2026

For some test profiles it is nice to have the ability to dynamically fetch VEX data as opposed to them being static. These kind of tests help have live signal.

@crozzy crozzy requested a review from a team as a code owner May 14, 2026 22:27
@crozzy crozzy requested review from hdonnay and removed request for a team May 14, 2026 22:27
@crozzy crozzy force-pushed the framework-changes branch 7 times, most recently from ec03890 to 49862c2 Compare May 18, 2026 18:09
hdonnay
hdonnay previously approved these changes May 18, 2026
Copy link
Copy Markdown
Member

@hdonnay hdonnay left a comment

Choose a reason for hiding this comment

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

LGTM

crozzy added 2 commits May 18, 2026 13:33
For some test profiles it is nice to have the ability to dynamically
fetch VEX data as opposed to them being static. These kind of tests help
have live signal.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
Add documentation explaining how to use dynamic VEX files as fixtures to
acceptance tests to get signal on current live data.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
@crozzy crozzy force-pushed the framework-changes branch from f829e78 to 4fc46df Compare May 18, 2026 20:33
@crozzy crozzy requested a review from hdonnay May 18, 2026 20:48
@crozzy
Copy link
Copy Markdown
Contributor Author

crozzy commented May 18, 2026

/fast-forward

@github-actions github-actions Bot merged commit 4fc46df into quay:main May 18, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants