Skip to content

Update trlp tests#759

Merged
averevki merged 1 commit intoKuadrant:mainfrom
emmaaroche:trlp-tests-updates
Sep 16, 2025
Merged

Update trlp tests#759
averevki merged 1 commit intoKuadrant:mainfrom
emmaaroche:trlp-tests-updates

Conversation

@emmaaroche
Copy link
Copy Markdown
Contributor

@emmaaroche emmaaroche commented Sep 15, 2025

Description:
Adjustments to TRLP tests to address inconsistent failures and improve overall test quality.

Changes made:

  • Set test_multiple_trlp_iterations to use a 50-token limit with a 20-second window
  • Introduced FREE_USER_LIMIT and PAID_USER_LIMIT constants to avoid hardcoding limit values
  • Improved assertion messages to include iteration count and token usage

@emmaaroche emmaaroche self-assigned this Sep 15, 2025
@emmaaroche emmaaroche added the enhancement Improvement to existing test label Sep 15, 2025
@emmaaroche emmaaroche requested a review from averevki September 15, 2025 11:21
@emmaaroche emmaaroche force-pushed the trlp-tests-updates branch 2 times, most recently from e40104b to ebb34db Compare September 15, 2025 16:18
Signed-off-by: emmaaroche <eroche@redhat.com>
@averevki averevki merged commit aadfb5d into Kuadrant:main Sep 16, 2025
3 checks passed
@emmaaroche emmaaroche deleted the trlp-tests-updates branch September 16, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improvement to existing test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants