spec/outputs/s3_spec.rb in logstash-output-s3-4.3.4 vs spec/outputs/s3_spec.rb in logstash-output-s3-4.3.5

@@ -138,11 +138,11 @@
       end
     end
 
     describe "temporary directory" do
       let(:temporary_directory) { Stud::Temporary.pathname }
-      let(:options) { super.merge({ "temporary_directory" => temporary_directory }) }
+      let(:options) { super().merge({ "temporary_directory" => temporary_directory }) }
 
       it "creates the directory when it doesn't exist" do
         expect(Dir.exist?(temporary_directory)).to be_falsey
         subject.register
         expect(Dir.exist?(temporary_directory)).to be_truthy
@@ -158,16 +158,18 @@
       s3 = described_class.new(options.merge({ "prefix" => "`no\><^" }))
       expect { s3.register }.to raise_error(LogStash::ConfigurationError)
     end
 
     describe "additional_settings" do
-      context "when enabling force_path_style" do
+      context "supported settings" do
         let(:additional_settings) do
-          { "additional_settings" => { "force_path_style" => true } }
+          { "additional_settings" => { "force_path_style" => 'true', "ssl_verify_peer" => 'false', "profile" => 'logstash' } }
         end
 
         it "validates the prefix" do
-          expect(Aws::S3::Bucket).to receive(:new).twice.with(anything, hash_including(:force_path_style => true)).and_call_original
+          expect(Aws::S3::Bucket).to receive(:new).twice.
+              with(anything, hash_including(:force_path_style => true, :ssl_verify_peer => false, :profile => 'logstash')).
+              and_call_original
           described_class.new(options.merge(additional_settings)).register
         end
       end
       context "when using a non existing setting" do
         let(:additional_settings) do