logstash-output-elasticsearch 10.8.6-java → 11.0.3-java

data/spec/integration/outputs/index_spec.rb

@@ -80,8 +80,11 @@ describe "indexing" do
       response = http_client.get("#{index_url}/_search?q=*&size=1000")
       result = LogStash::Json.load(response.body)
       result["hits"]["hits"].each do |doc|
-        expect(doc["_type"]).to eq(type) if ESHelper.es_version_satisfies?(">= 6", "< 8")
-        expect(doc).not_to include("_type") if ESHelper.es_version_satisfies?(">= 8")
+        if ESHelper.es_version_satisfies?("< 8")
+          expect(doc["_type"]).to eq(type)
+        else
+          expect(doc).not_to include("_type")
+        end
         expect(doc["_index"]).to eq(index)
       end
     end

data/spec/integration/outputs/index_version_spec.rb

@@ -1,101 +1,97 @@
 require_relative "../../../spec/es_spec_helper"
 require "logstash/outputs/elasticsearch"
 
-if ESHelper.es_version_satisfies?(">= 2")
-  describe "Versioned indexing", :integration => true do
-    require "logstash/outputs/elasticsearch"
+describe "Versioned indexing", :integration => true do
+  require "logstash/outputs/elasticsearch"
 
-    let(:es) { get_client }
+  let(:es) { get_client }
 
-    before :each do
-      # Delete all templates first.
-      # Clean ES of data before we start.
-      es.indices.delete_template(:name => "*")
-      # This can fail if there are no indexes, ignore failure.
-      es.indices.delete(:index => "*") rescue nil
-      es.indices.refresh
-    end
+  before :each do
+    # Delete all templates first.
+    # Clean ES of data before we start.
+    es.indices.delete_template(:name => "*")
+    # This can fail if there are no indexes, ignore failure.
+    es.indices.delete(:index => "*") rescue nil
+    es.indices.refresh
+  end
 
-    context "when index only" do
-      subject { LogStash::Outputs::ElasticSearch.new(settings) }
+  context "when index only" do
+    subject { LogStash::Outputs::ElasticSearch.new(settings) }
+
+    before do
+      subject.register
+    end
 
-      before do
-        subject.register
+    describe "unversioned output" do
+      let(:settings) do
+        {
+          "manage_template" => true,
+          "index" => "logstash-index",
+          "template_overwrite" => true,
+          "hosts" => get_host_port(),
+          "action" => "index",
+          "document_id" => "%{my_id}"
+        }
       end
 
-      describe "unversioned output" do
-        let(:settings) do
-          {
-            "manage_template" => true,
-            "index" => "logstash-index",
-            "template_overwrite" => true,
-            "hosts" => get_host_port(),
-            "action" => "index",
-            "script_lang" => "groovy",
-            "document_id" => "%{my_id}"
-          }
-        end
+      it "should default to ES version" do
+        subject.multi_receive([LogStash::Event.new("my_id" => "123", "message" => "foo")])
+        r = es.get(:index => 'logstash-index', :type => doc_type, :id => "123", :refresh => true)
+        expect(r["_version"]).to eq(1)
+        expect(r["_source"]["message"]).to eq('foo')
+        subject.multi_receive([LogStash::Event.new("my_id" => "123", "message" => "foobar")])
+        r2 = es.get(:index => 'logstash-index', :type => doc_type, :id => "123", :refresh => true)
+        expect(r2["_version"]).to eq(2)
+        expect(r2["_source"]["message"]).to eq('foobar')
+      end  
+    end
 
-        it "should default to ES version" do
-          subject.multi_receive([LogStash::Event.new("my_id" => "123", "message" => "foo")])
-          r = es.get(:index => 'logstash-index', :type => doc_type, :id => "123", :refresh => true)
-          expect(r["_version"]).to eq(1)
-          expect(r["_source"]["message"]).to eq('foo')
-          subject.multi_receive([LogStash::Event.new("my_id" => "123", "message" => "foobar")])
-          r2 = es.get(:index => 'logstash-index', :type => doc_type, :id => "123", :refresh => true)
-          expect(r2["_version"]).to eq(2)
-          expect(r2["_source"]["message"]).to eq('foobar')
-        end  
+    describe "versioned output" do
+      let(:settings) do 
+        {
+          "manage_template" => true,
+          "index" => "logstash-index",
+          "template_overwrite" => true,
+          "hosts" => get_host_port(),
+          "action" => "index",
+          "document_id" => "%{my_id}",
+          "version" => "%{my_version}",
+          "version_type" => "external",
+        }
       end
 
-      describe "versioned output" do
-        let(:settings) do 
-          {
-            "manage_template" => true,
-            "index" => "logstash-index",
-            "template_overwrite" => true,
-            "hosts" => get_host_port(),
-            "action" => "index",
-            "script_lang" => "groovy",
-            "document_id" => "%{my_id}",
-            "version" => "%{my_version}",
-            "version_type" => "external",
-          }
-        end
-
-        it "should respect the external version" do
-          id = "ev1"
-          subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "99", "message" => "foo")])
-          r = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
-          expect(r["_version"]).to eq(99)
-          expect(r["_source"]["message"]).to eq('foo')
-        end
+      it "should respect the external version" do
+        id = "ev1"
+        subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "99", "message" => "foo")])
+        r = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
+        expect(r["_version"]).to eq(99)
+        expect(r["_source"]["message"]).to eq('foo')
+      end
 
-        it "should ignore non-monotonic external version updates" do
-          id = "ev2"
-          subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "99", "message" => "foo")])
-          r = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
-          expect(r["_version"]).to eq(99)
-          expect(r["_source"]["message"]).to eq('foo')
+      it "should ignore non-monotonic external version updates" do
+        id = "ev2"
+        subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "99", "message" => "foo")])
+        r = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
+        expect(r["_version"]).to eq(99)
+        expect(r["_source"]["message"]).to eq('foo')
 
-          subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "98", "message" => "foo")])
-          r2 = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
-          expect(r2["_version"]).to eq(99)
-          expect(r2["_source"]["message"]).to eq('foo')
-        end
+        subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "98", "message" => "foo")])
+        r2 = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
+        expect(r2["_version"]).to eq(99)
+        expect(r2["_source"]["message"]).to eq('foo')
+      end
 
-        it "should commit monotonic external version updates" do
-          id = "ev3"
-          subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "99", "message" => "foo")])
-          r = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
-          expect(r["_version"]).to eq(99)
-          expect(r["_source"]["message"]).to eq('foo')
+      it "should commit monotonic external version updates" do
+        id = "ev3"
+        subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "99", "message" => "foo")])
+        r = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
+        expect(r["_version"]).to eq(99)
+        expect(r["_source"]["message"]).to eq('foo')
 
-          subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "100", "message" => "foo")])
-          r2 = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
-          expect(r2["_version"]).to eq(100)
-          expect(r2["_source"]["message"]).to eq('foo')
-        end
+        subject.multi_receive([LogStash::Event.new("my_id" => id, "my_version" => "100", "message" => "foo")])
+        r2 = es.get(:index => 'logstash-index', :type => doc_type, :id => id, :refresh => true)
+        expect(r2["_version"]).to eq(100)
+        expect(r2["_source"]["message"]).to eq('foo')
       end
     end
   end

data/spec/integration/outputs/ingest_pipeline_spec.rb

@@ -1,74 +1,74 @@
 require_relative "../../../spec/es_spec_helper"
 
-if ESHelper.es_version_satisfies?(">= 5")
-  describe "Ingest pipeline execution behavior", :integration => true do
-    subject! do
-      require "logstash/outputs/elasticsearch"
-      settings = {
-        "hosts" => "#{get_host_port()}",
-        "pipeline" => "apache-logs"
-      }
-      next LogStash::Outputs::ElasticSearch.new(settings)
-    end
+describe "Ingest pipeline execution behavior", :integration => true do
+  subject! do
+    require "logstash/outputs/elasticsearch"
+    settings = {
+      "hosts" => "#{get_host_port()}",
+      "pipeline" => "apache-logs",
+      "data_stream" => 'false'
+    }
+    next LogStash::Outputs::ElasticSearch.new(settings)
+  end
 
-    let(:http_client) { Manticore::Client.new }
-    let(:ingest_url) { "http://#{get_host_port()}/_ingest/pipeline/apache-logs" }
-    let(:apache_logs_pipeline) { '
-    {
-      "description" : "Pipeline to parse Apache logs",
-      "processors" : [
-        {
-          "grok": {
-            "field": "message",
-            "patterns": ["%{COMBINEDAPACHELOG}"]
-          }
+  let(:http_client) { Manticore::Client.new }
+  let(:ingest_url) { "http://#{get_host_port()}/_ingest/pipeline/apache-logs" }
+  let(:apache_logs_pipeline) { '
+  {
+    "description" : "Pipeline to parse Apache logs",
+    "processors" : [
+      {
+        "grok": {
+          "field": "message",
+          "patterns": ["%{COMBINEDAPACHELOG}"]
         }
-      ]
-    }'
-    }
+      }
+    ]
+  }'
+  }
 
-    before :each do
-      # Delete all templates first.
-      require "elasticsearch"
+  before :each do
+    # Delete all templates first.
+    require "elasticsearch"
 
-      # Clean ES of data before we start.
-      @es = get_client
-      @es.indices.delete_template(:name => "*")
+    # Clean ES of data before we start.
+    @es = get_client
+    @es.indices.delete_template(:name => "*")
 
-      # This can fail if there are no indexes, ignore failure.
-      @es.indices.delete(:index => "*") rescue nil
+    # This can fail if there are no indexes, ignore failure.
+    @es.indices.delete(:index => "*") rescue nil
 
-      # delete existing ingest pipeline
-      http_client.delete(ingest_url).call
+    # delete existing ingest pipeline
+    http_client.delete(ingest_url).call
 
-      # register pipeline
-      http_client.put(ingest_url, :body => apache_logs_pipeline, :headers => {"Content-Type" => "application/json" }).call
+    # register pipeline
+    http_client.put(ingest_url, :body => apache_logs_pipeline, :headers => {"Content-Type" => "application/json" }).call
 
-      #TODO: Use esclient
-      #@es.ingest.put_pipeline :id => 'apache_pipeline', :body => pipeline_defintion
+    #TODO: Use esclient
+    #@es.ingest.put_pipeline :id => 'apache_pipeline', :body => pipeline_defintion
 
-      subject.register
-      subject.multi_receive([LogStash::Event.new("message" => '183.60.215.50 - - [01/Jun/2015:18:00:00 +0000] "GET /scripts/netcat-webserver HTTP/1.1" 200 182 "-" "Mozilla/5.0 (compatible; EasouSpider; +http://www.easou.com/search/spider.html)"')])
-      @es.indices.refresh
+    subject.register
+    subject.multi_receive([LogStash::Event.new("message" => '183.60.215.50 - - [01/Jun/2015:18:00:00 +0000] "GET /scripts/netcat-webserver HTTP/1.1" 200 182 "-" "Mozilla/5.0 (compatible; EasouSpider; +http://www.easou.com/search/spider.html)"')])
+    @es.indices.refresh
 
-      #Wait or fail until everything's indexed.
-      Stud::try(20.times) do
-        r = @es.search(index: 'logstash-*')
-        expect(r).to have_hits(1)
-      end
+    #Wait or fail until everything's indexed.
+    Stud::try(10.times) do
+      r = @es.search(index: 'logstash-*')
+      expect(r).to have_hits(1)
+      sleep(0.1)
     end
+  end
 
-    it "indexes using the proper pipeline" do
-      results = @es.search(:index => 'logstash-*', :q => "message:\"netcat\"")
-      expect(results).to have_hits(1)
-      expect(results["hits"]["hits"][0]["_source"]["response"]).to eq("200")
-      expect(results["hits"]["hits"][0]["_source"]["bytes"]).to eq("182")
-      expect(results["hits"]["hits"][0]["_source"]["verb"]).to eq("GET")
-      expect(results["hits"]["hits"][0]["_source"]["request"]).to eq("/scripts/netcat-webserver")
-      expect(results["hits"]["hits"][0]["_source"]["auth"]).to eq("-")
-      expect(results["hits"]["hits"][0]["_source"]["ident"]).to eq("-")
-      expect(results["hits"]["hits"][0]["_source"]["clientip"]).to eq("183.60.215.50")
-      expect(results["hits"]["hits"][0]["_source"]["junkfieldaaaa"]).to eq(nil)
-    end
+  it "indexes using the proper pipeline" do
+    results = @es.search(:index => 'logstash-*', :q => "message:\"netcat\"")
+    expect(results).to have_hits(1)
+    expect(results["hits"]["hits"][0]["_source"]["response"]).to eq("200")
+    expect(results["hits"]["hits"][0]["_source"]["bytes"]).to eq("182")
+    expect(results["hits"]["hits"][0]["_source"]["verb"]).to eq("GET")
+    expect(results["hits"]["hits"][0]["_source"]["request"]).to eq("/scripts/netcat-webserver")
+    expect(results["hits"]["hits"][0]["_source"]["auth"]).to eq("-")
+    expect(results["hits"]["hits"][0]["_source"]["ident"]).to eq("-")
+    expect(results["hits"]["hits"][0]["_source"]["clientip"]).to eq("183.60.215.50")
+    expect(results["hits"]["hits"][0]["_source"]["junkfieldaaaa"]).to eq(nil)
   end
 end

data/spec/integration/outputs/painless_update_spec.rb

@@ -1,188 +1,98 @@
 require_relative "../../../spec/es_spec_helper"
 
-if ESHelper.es_version_satisfies?(">= 5")
-  describe "Update actions using painless scripts", :integration => true, :update_tests => 'painless' do
-    require "logstash/outputs/elasticsearch"
-
-    def get_es_output( options={} )
-      settings = {
-        "manage_template" => true,
-        "index" => "logstash-update",
-        "template_overwrite" => true,
-        "hosts" => get_host_port(),
-        "action" => "update"
-      }
-      if ESHelper.es_version_satisfies?('<6')
-        settings.merge!({"script_lang" => "painless"})
-      end
-      LogStash::Outputs::ElasticSearch.new(settings.merge!(options))
-    end
-
-    before :each do
-      @es = get_client
-      # Delete all templates first.
-      # Clean ES of data before we start.
-      @es.indices.delete_template(:name => "*")
-      # This can fail if there are no indexes, ignore failure.
-      @es.indices.delete(:index => "*") rescue nil
-      @es.index(
-        :index => 'logstash-update',
-        :type => doc_type,
-        :id => "123",
-        :body => { :message => 'Test', :counter => 1 }
-      )
-      @es.indices.refresh
-    end
+describe "Update actions using painless scripts", :integration => true, :update_tests => 'painless' do
+  require "logstash/outputs/elasticsearch"
+
+  def get_es_output( options={} )
+    settings = {
+      "manage_template" => true,
+      "index" => "logstash-update",
+      "template_overwrite" => true,
+      "hosts" => get_host_port(),
+      "action" => "update"
+    }
+    LogStash::Outputs::ElasticSearch.new(settings.merge!(options))
+  end
 
-    context "scripted updates" do
-      if ESHelper.es_version_satisfies?('<6')
-        context 'with file based scripts' do
-          it "should increment a counter with event/doc 'count' variable" do
-            subject = get_es_output({ 'document_id' => "123", 'script' => 'scripted_update', 'script_type' => 'file' })
-            subject.register
-            subject.multi_receive([LogStash::Event.new("count" => 2)])
-            r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "123", :refresh => true)
-            expect(r["_source"]["counter"]).to eq(3)
-          end
+  before :each do
+    @es = get_client
+    # Delete all templates first.
+    # Clean ES of data before we start.
+    @es.indices.delete_template(:name => "*")
+    # This can fail if there are no indexes, ignore failure.
+    @es.indices.delete(:index => "*") rescue nil
+    @es.index(
+      :index => 'logstash-update',
+      :type => doc_type,
+      :id => "123",
+      :body => { :message => 'Test', :counter => 1 }
+    )
+    @es.indices.refresh
+  end
 
-          it "should increment a counter with event/doc '[data][count]' nested variable" do
-            subject = get_es_output({ 'document_id' => "123", 'script' => 'scripted_update_nested', 'script_type' => 'file' })
-            subject.register
-            subject.multi_receive([LogStash::Event.new("data" => { "count" => 3 })])
-            r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "123", :refresh => true)
-            expect(r["_source"]["counter"]).to eq(4)
-          end
-        end
-      end
+  context "scripted updates" do
+    context 'with an indexed script' do
+      it "should increment a counter with event/doc 'count' variable with indexed script" do
+        @es.perform_request(:put, "_scripts/indexed_update", {}, {"script" => {"source" => "ctx._source.counter += params.event.count", "lang" => "painless"}})
 
-      it "should increment a counter with event/doc 'count' variable with inline script" do
-        subject = get_es_output({
+        plugin_parameters = {
           'document_id' => "123",
-          'script' => 'ctx._source.counter += params.event.counter',
-          'script_type' => 'inline'
-        })
-        subject.register
-        subject.multi_receive([LogStash::Event.new("counter" => 3 )])
-        r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "123", :refresh => true)
-        expect(r["_source"]["counter"]).to eq(4)
-      end
+          'script' => 'indexed_update',
+          'script_type' => 'indexed'
+        }
 
-      it "should increment a counter with event/doc 'count' variable with event/doc as upsert and inline script" do
-        subject = get_es_output({
-          'document_id' => "123",
-          'doc_as_upsert' => true,
-          'script' => 'if( ctx._source.containsKey("counter") ){ ctx._source.counter += params.event.counter; } else { ctx._source.counter = params.event.counter; }',
-          'script_type' => 'inline'
-        })
-        subject.register
-        subject.multi_receive([LogStash::Event.new("counter" => 3 )])
-        r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "123", :refresh => true)
-        expect(r["_source"]["counter"]).to eq(4)
-      end
+        plugin_parameters.merge!('script_lang' => '')
 
-      it "should, with new doc, set a counter with event/doc 'count' variable with event/doc as upsert and inline script" do
-        subject = get_es_output({
-          'document_id' => "456",
-          'doc_as_upsert' => true,
-          'script' => 'if( ctx._source.containsKey("counter") ){ ctx._source.counter += params.event.counter; } else { ctx._source.counter = params.event.counter; }',
-          'script_type' => 'inline'
-        })
+        subject = get_es_output(plugin_parameters)
         subject.register
-        subject.multi_receive([LogStash::Event.new("counter" => 3 )])
-        r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
-        expect(r["_source"]["counter"]).to eq(3)
+        subject.multi_receive([LogStash::Event.new("count" => 4 )])
+        r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "123", :refresh => true)
+        expect(r["_source"]["counter"]).to eq(5)
       end
+    end
+   end
+
+  context "when update with upsert" do
+    it "should create new documents with provided upsert" do
+      subject = get_es_output({ 'document_id' => "456", 'upsert' => '{"message": "upsert message"}' })
+      subject.register
+      subject.multi_receive([LogStash::Event.new("message" => "sample message here")])
+      r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
+      expect(r["_source"]["message"]).to eq('upsert message')
+    end
 
-      context 'with an indexed script' do
-        it "should increment a counter with event/doc 'count' variable with indexed script" do
-          if ESHelper.es_version_satisfies?('<6')
-            @es.perform_request(:put, "_scripts/painless/indexed_update", {}, {"script" => "ctx._source.counter += params.event.count" })
-          else
-            @es.perform_request(:put, "_scripts/indexed_update", {}, {"script" => {"source" => "ctx._source.counter += params.event.count", "lang" => "painless"}})
-          end
-
-          plugin_parameters = {
-            'document_id' => "123",
-            'script' => 'indexed_update',
-            'script_type' => 'indexed'
-          }
-
-          if ESHelper.es_version_satisfies?('>= 6.0.0')
-            plugin_parameters.merge!('script_lang' => '')
-          end
+    it "should create new documents with event/doc as upsert" do
+      subject = get_es_output({ 'document_id' => "456", 'doc_as_upsert' => true })
+      subject.register
+      subject.multi_receive([LogStash::Event.new("message" => "sample message here")])
+      r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
+      expect(r["_source"]["message"]).to eq('sample message here')
+    end
 
-          subject = get_es_output(plugin_parameters)
-          subject.register
-          subject.multi_receive([LogStash::Event.new("count" => 4 )])
-          r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "123", :refresh => true)
-          expect(r["_source"]["counter"]).to eq(5)
-        end
-      end
-     end
+    it "should fail on documents with event/doc as upsert at external version" do
+      subject = get_es_output({ 'document_id' => "456", 'doc_as_upsert' => true, 'version' => 999, "version_type" => "external" })
+      expect { subject.register }.to raise_error(LogStash::ConfigurationError)
+    end
+  end
 
-    context "when update with upsert" do
-      it "should create new documents with provided upsert" do
-        subject = get_es_output({ 'document_id' => "456", 'upsert' => '{"message": "upsert message"}' })
+  context "updates with scripted upsert" do
+    context 'with an inline script' do
+      it "should create new documents with upsert content" do
+        subject = get_es_output({ 'document_id' => "456", 'script' => 'ctx._source.counter = params.event.counter', 'upsert' => '{"message": "upsert message"}', 'script_type' => 'inline' })
         subject.register
+
         subject.multi_receive([LogStash::Event.new("message" => "sample message here")])
         r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
         expect(r["_source"]["message"]).to eq('upsert message')
       end
 
-      it "should create new documents with event/doc as upsert" do
-        subject = get_es_output({ 'document_id' => "456", 'doc_as_upsert' => true })
+      it "should create new documents with event/doc as script params" do
+        subject = get_es_output({ 'document_id' => "456", 'script' => 'ctx._source.counter = params.event.counter', 'scripted_upsert' => true, 'script_type' => 'inline' })
         subject.register
-        subject.multi_receive([LogStash::Event.new("message" => "sample message here")])
+        subject.multi_receive([LogStash::Event.new("counter" => 1)])
+        @es.indices.refresh
         r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
-        expect(r["_source"]["message"]).to eq('sample message here')
-      end
-
-      it "should fail on documents with event/doc as upsert at external version" do
-        subject = get_es_output({ 'document_id' => "456", 'doc_as_upsert' => true, 'version' => 999, "version_type" => "external" })
-        expect { subject.register }.to raise_error(LogStash::ConfigurationError)
-      end
-    end
-
-    context "updates with scripted upsert" do
-      if ESHelper.es_version_satisfies?('<6')
-        context 'with file based scripts' do
-          it "should create new documents with upsert content" do
-            subject = get_es_output({ 'document_id' => "456", 'script' => 'scripted_update', 'upsert' => '{"message": "upsert message"}', 'script_type' => 'file' })
-            subject.register
-            subject.multi_receive([LogStash::Event.new("message" => "sample message here")])
-            r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
-            expect(r["_source"]["message"]).to eq('upsert message')
-          end
-
-          it "should create new documents with event/doc as script params" do
-            subject = get_es_output({ 'document_id' => "456", 'script' => 'scripted_upsert', 'scripted_upsert' => true, 'script_type' => 'file' })
-            subject.register
-            subject.multi_receive([LogStash::Event.new("counter" => 1)])
-            @es.indices.refresh
-            r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
-            expect(r["_source"]["counter"]).to eq(1)
-          end
-        end
-      end
-
-      context 'with an inline script' do
-        it "should create new documents with upsert content" do
-          subject = get_es_output({ 'document_id' => "456", 'script' => 'ctx._source.counter = params.event.counter', 'upsert' => '{"message": "upsert message"}', 'script_type' => 'inline' })
-          subject.register
-
-          subject.multi_receive([LogStash::Event.new("message" => "sample message here")])
-          r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
-          expect(r["_source"]["message"]).to eq('upsert message')
-        end
-
-        it "should create new documents with event/doc as script params" do
-          subject = get_es_output({ 'document_id' => "456", 'script' => 'ctx._source.counter = params.event.counter', 'scripted_upsert' => true, 'script_type' => 'inline' })
-          subject.register
-          subject.multi_receive([LogStash::Event.new("counter" => 1)])
-          @es.indices.refresh
-          r = @es.get(:index => 'logstash-update', :type => doc_type, :id => "456", :refresh => true)
-          expect(r["_source"]["counter"]).to eq(1)
-        end
+        expect(r["_source"]["counter"]).to eq(1)
       end
     end
   end