Merge pull request #290 from ixti/fix-appenders-close

CHANGELOG.md

@@ -5,6 +5,8 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [unreleased]
 
+- Fix regression in `SemanticLogger::Appenders#close`
+
 ## [4.16.0]
 
 - Add appender for Honeybadger Insights using the events API

lib/semantic_logger/appenders.rb

@@ -49,9 +49,9 @@ def close
       closed_appenders = []
       each do |appender|
         logger.trace "Closing appender: #{appender.name}"
-        appenders << appender
         appender.flush
         appender.close
+        closed_appenders << appender
       rescue Exception => e
         logger.error "Failed to close appender: #{appender.name}", e
       end

test/appenders_test.rb

@@ -100,5 +100,16 @@ class AppendersTest < Minitest::Test
       #   assert_instance_of SemanticLogger::Appender::Async, appender
       # end
     end
+
+    describe "#close" do
+      it "closes appenders" do
+        appender = appenders.add(file_name: "test.log")
+
+        appenders.close
+
+        assert_equal 0, capture_logger.events.count { |it| it.message.match?(/failed/i) }
+        assert_equal 0, appenders.size
+      end
+    end
   end
 end