From 452e98f7db2f881acf43493a8702e54db0d95922 Mon Sep 17 00:00:00 2001
From: Rebecca Tolmach <10993987+rmtolmach@users.noreply.github.com>
Date: Mon, 23 Sep 2024 16:08:18 -0400
Subject: [PATCH] add and change logging for flipper initializer

---
 config/initializers/flipper.rb | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/config/initializers/flipper.rb b/config/initializers/flipper.rb
index 42da224ff86..a92b39d119d 100644
--- a/config/initializers/flipper.rb
+++ b/config/initializers/flipper.rb
@@ -42,10 +42,12 @@
 
   Flipper::UI.configuration.feature_creation_enabled = false
   # Make sure that each feature we reference in code is present in the UI, as long as we have a Database already
+  added_flippers = []
   begin
     FLIPPER_FEATURE_CONFIG['features'].each do |feature, feature_config|
       unless Flipper.exist?(feature)
         Flipper.add(feature)
+        added_flippers.push(feature)
 
         # Default features to enabled for test and those explicitly set for development
         if Rails.env.test? || (Rails.env.development? && feature_config['enable_in_development'])
@@ -57,9 +59,13 @@
       Flipper.enable(feature) if Settings.vsp_environment == 'development' && feature_config['enable_in_development']
     end
 
+    Rails.logger.info "The following feature flippers were added: #{added_flippers}" unless added_flippers.empty?
     removed_features = Flipper.features.collect(&:name) - FLIPPER_FEATURE_CONFIG['features'].keys
-    Rails.logger.warn "Consider removing features no longer in config/features.yml: #{removed_features.join(', ')}"
-  rescue
+    unless removed_features.empty?
+      Rails.logger.warn "Consider removing features no longer in config/features.yml: #{removed_features.join(', ')}"
+    end
+  rescue => e
+    Rails.logger.error "Error processing Flipper features: #{e.message}"
     # make sure we can still run rake tasks before table has been created
     nil
   end