Refactor specs after code review

Author: estolfo

spec/unit/outputs/elasticsearch_spec.rb

@@ -1499,9 +1499,9 @@
     end
   end
 
-  context 'drop_error_types config option' do
+  describe 'drop_error_types' do
 
-    let(:error_type) { 'role_restriction_exception' }
+    let(:error_type) { 'index_closed_exception' }
 
     let(:options) { super().merge('drop_error_types' => [error_type]) }
 
@@ -1511,6 +1511,8 @@
 
     let(:error_code) { 403 }
 
+    let(:event_action_tuples) { subject.map_events(events) }
+
     let(:bulk_response) do
       {
         "took"=>1, "ingest_took"=>11, "errors"=>true, "items"=>
@@ -1528,22 +1530,24 @@
 
     context 'DLQ is enabled' do
 
-      it 'does not write the event to the DLQ' do
+      before(:each) do
         allow(subject).to receive(:dlq_enabled?).and_return(true)
-        expect(dlq_writer).not_to receive(:write)
+      end
 
-        event_action_tuples = subject.map_events(events)
+      it 'does not write the event to the DLQ' do
+        expect(dlq_writer).not_to receive(:write)
         subject.send(:submit, event_action_tuples)
       end
     end
 
     context 'DLQ is not enabled' do
 
-      it 'does not write the event to the DLQ' do
+      before(:each) do
         allow(subject).to receive(:dlq_enabled?).and_return(false)
-        expect(dlq_writer).not_to receive(:write)
+      end
 
-        event_action_tuples = subject.map_events(events)
+      it 'does not write the event to the DLQ' do
+        expect(dlq_writer).not_to receive(:write)
         subject.send(:submit, event_action_tuples)
       end
     end