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

[tqlotel] Fix assertion in the limit function tests. #12990

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

kovrus
Copy link
Member

@kovrus kovrus commented Aug 6, 2022

Description:

Test_limit was asserting expected value against expected. This
change fixes the assertion.

Link to tracking Issue:

Testing:
Modifying expected test values and see whether tests fail.

Documentation:

@kovrus kovrus requested a review from a team August 6, 2022 20:48
@kovrus kovrus changed the title [processor/transform] Fix assertion in limit test function. [tqlotel] Fix assertion in limit test function. Aug 6, 2022
@kovrus kovrus changed the title [tqlotel] Fix assertion in limit test function. [tqlotel] Fix assertion in the limit function tests. Aug 6, 2022
@TylerHelmuth TylerHelmuth added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 6, 2022
@bogdandrutu bogdandrutu merged commit 2afb28a into open-telemetry:main Aug 8, 2022
@kovrus kovrus deleted the tql/tests branch August 9, 2022 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants