test: don't use deprecated way of capturing logs #7
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Background
The next td-agent will embed Ruby 3.2.
I was checking each embedded plugin and noticed that this plugin has no GitHub CI.
I added CI to my forked repository, but I found the tests are unstable on Windows and macOS.
So, I created this PR first.
Following this, I would like to create a PR to add GitHub CI.
About this fix
We should use
Driver::logs
instead ofcapture_log
to fix unstable tests.capture_log
is defined by this plugin here.fluent-plugin-flowcounter-simple/test/helper.rb
Lines 12 to 28 in 5081e88
However, this is for older versions as well as the following, and this makes tests unstable.
Reproduce failing tests
Windows seems to have the highest probability of failure.