-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
test(testutil): Fix value comparison #15886
test(testutil): Fix value comparison #15886
Conversation
d6f47d8
to
f62e5c0
Compare
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 👍 This pull request doesn't change the Telegraf binary size 📦 Click here to get additional PR build artifactsArtifact URLs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @LarsStegman! I wonder why this went unnoticed for so long...
Yeah I was really surprised to see it's been in there since 2019 😂 |
(cherry picked from commit 65999bd)
Summary
One of my tests failed, but it was because the comparison function for test cmp opts didn't work properly
Checklist
Related issues
resolves #15885