logstash-input-beats 2.1.2 → 2.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3394e299d7c7ecf132134ebed27de22b49d2bdc2
-  data.tar.gz: 3e92f0bdb6b00a5d28955673a647099dfe905d4e
+  metadata.gz: 6b827272694308645c0341eae3bd6ee1caa6429a
+  data.tar.gz: 1128ffe70105c3c226da35bdf9b73615000c5feb
 SHA512:
-  metadata.gz: 3e2456b1703cf75ab2724fa61014d8dc0e1b56bdaf05045ab06bef319f0511a99de7627bb581c39e2296d9468c7f2399a770c2c59f115de6a7353b0cfca46605
-  data.tar.gz: 3ce48912d957e5539ba445b0580ef7e312f0d57b961a40247146fbf63d72778a2035bbb1ed25ceb0f3904b5ff4c06d7cb20373961a926d3a36161fcf7f503289
+  metadata.gz: 18e3f4fea0b3324bcabd32f54d0ee34fcc0c11547f1f9b26a807f5e6854f9888b9f9fafb91283572f1abadd3ef136fab60764d8566bfa418b90cc5937abe0d44
+  data.tar.gz: 0f3821b6baad697b89183418141d82134f208e4b38865433f40ac2b104f23df2f2e717bf34f90ff496ee4dbfdfde0bd14de4df86e0303b0181034779a229a27f

data/CHANGELOG.md

@@ -1,3 +1,5 @@
+# 2.1.3
+  - Make sure we stop all the threads after running the tests #48
 # 2.1.2
   - Catch the `java.lang.InterruptedException` in the events broker
   - Give a bit more time to the Thread to be started in the test #42

data/logstash-input-beats.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name            = "logstash-input-beats"
-  s.version         = "2.1.2"
+  s.version         = "2.1.3"
   s.licenses        = ["Apache License (2.0)"]
   s.summary         = "Receive events using the lumberjack protocol."
   s.description     = "This gem is a logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/plugin install gemname. This gem is not a stand-alone program"

data/spec/inputs/beats_spec.rb

@@ -89,27 +89,33 @@ describe LogStash::Inputs::Beats do
       # Event if we dont mock the actual socket work
       # we have to call run because it will correctly setup the queues
       # instances variables
-      t = Thread.new do
+      @t = Thread.new do
         plugin.run(pipeline_queue)
       end
 
-      # Give a bit of time to the server to correctly 
+      # Give a bit of time to the server to correctly
       # start the thread.
-      sleep(1) 
+      sleep(1)
     end
 
     after :each do
       plugin.stop
+      # I am forcing a kill on the thread since we are using a mock connection and we have not implemented
+      # all the  blocking IO, joining the thread and waiting wont work..
+      # I want to remove all the blockings in a new version real soon.
+      # If we dont do this the thread can still be present in other tests causing this kind of issue:
+      # https://github.com/logstash-plugins/logstash-input-beats/issues/48
+      @t.kill rescue nil
     end
 
     context "when an exception occur" do
       let(:connection_handler) { LogStash::Inputs::BeatsSupport::ConnectionHandler.new(connection, plugin, buffer_queue) }
-      before do 
+      before do
         expect(LogStash::Inputs::BeatsSupport::ConnectionHandler).to receive(:new).with(any_args).and_return(connection_handler)
       end
 
       it "calls flush on the handler and tag the events" do
-        # try multiples times to make sure the 
+        # try multiples times to make sure the
         # thread containing the `#run` is correctly started.
         try do
           expect(connection_handler).to receive(:accept) { raise LogStash::Inputs::Beats::InsertingToQueueTakeTooLong }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-input-beats
 version: !ruby/object:Gem::Version
-  version: 2.1.2
+  version: 2.1.3
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2016-01-12 00:00:00.000000000 Z
+date: 2016-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logstash-core