Skip to content

Commit 301581b

Browse files
committed
Add events method to QueueBatchDelegator
After elastic/logstash#11737 was committed, plugin unit tests using `sample` targeted against branches with this change merged were failing due to the lack of `events` method on the QueueBatchDelegator. This commit adds an `events` method to fix the failing tests.
1 parent ccaaa22 commit 301581b

File tree

3 files changed

+12
-1
lines changed

3 files changed

+12
-1
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 2.0.3
2+
- Fix: add missing `events` method to QueuedBatchDelegator, which was causing test failures
3+
after https://github.com/elastic/logstash/pull/11737 was committed.
4+
15
## 2.0.2
26
- Fix: add plain codec as runtime dependency for TestPipeline helper
37

lib/logstash/test_pipeline.rb

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,13 @@ def filteredSize
151151
@delegate.to_java.filtered_size
152152
end
153153

154+
def events
155+
@delegate.events.tap do |events|
156+
# filters out rogue (cancelled) events
157+
@event_tracker.filtered_events events
158+
end
159+
end
160+
154161
# @override void merge(IRubyObject event);
155162
def merge(event)
156163
@delegate.merge(event)

logstash-devutils.gemspec

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ Gem::Specification.new do |spec|
44
files = %x{git ls-files}.split("\n")
55

66
spec.name = "logstash-devutils"
7-
spec.version = "2.0.2"
7+
spec.version = "2.0.3"
88
spec.license = "Apache-2.0"
99
spec.authors = ["Elastic"]
1010
spec.email = "info@elastic.co"

0 commit comments

Comments
 (0)