log cleanup

Author: NSHkr

config/test.exs

@@ -3,7 +3,7 @@ import Config
 # Test environment configuration
 config :elixir_scope,
   test_mode: true,  # Enable test mode for test supervisor
-  debug_registry: true,
+  debug_registry: false,
   ecto_repos: [],
   ai: [
     provider: :mock,

lib/elixir_scope/foundation/graceful_degradation.ex

@@ -33,27 +33,17 @@ defmodule ElixirScope.Foundation.Config.GracefulDegradation do
   """
   def cleanup_fallback_system do
     try do
-      :ets.delete(@fallback_table)
-
-      # Only log if not in test mode or if debug is enabled
-      test_mode = Application.get_env(:elixir_scope, :test_mode, false)
-      debug_enabled = Application.get_env(:elixir_scope, :debug_registry, false)
+      case :ets.info(@fallback_table) do
+        :undefined ->
+          :ok
 
-      if not test_mode or debug_enabled do
-        Logger.debug("Fallback system cleaned up")
+        _ ->
+          :ets.delete(@fallback_table)
+          :ok
       end
-
-      :ok
-    catch
-      :error, :badarg ->
-        # Only log if not in test mode or if debug is enabled
-        test_mode = Application.get_env(:elixir_scope, :test_mode, false)
-        debug_enabled = Application.get_env(:elixir_scope, :debug_registry, false)
-
-        if not test_mode or debug_enabled do
-          Logger.debug("Fallback table already cleaned up")
-        end
-
+    rescue
+      ArgumentError ->
+        # Table doesn't exist or already deleted
         :ok
     end
   end

lib/elixir_scope/foundation/process_registry.ex

@@ -503,8 +503,8 @@ defmodule ElixirScope.Foundation.ProcessRegistry do
     # Log cleanup summary
     require Logger
 
-    # Only log if there were actually services to clean up or if not in test mode
-    if cleanup_count > 0 or not Application.get_env(:elixir_scope, :test_mode, false) do
+    # Only log if there were actually services to clean up
+    if cleanup_count > 0 do
       Logger.debug("Cleaned up #{cleanup_count} services from test namespace #{inspect(test_ref)}")
     end
 

lib/elixir_scope/foundation/service_registry.ex

@@ -349,9 +349,8 @@ defmodule ElixirScope.Foundation.ServiceRegistry do
           healthy_services: non_neg_integer()
         }
   def get_service_info(namespace) do
-    # Only log debug info if not in test mode or if debug_registry is enabled
-    if not Application.get_env(:elixir_scope, :test_mode, false) or
-         Application.get_env(:elixir_scope, :debug_registry, false) do
+    # Only log debug info if debug_registry is explicitly enabled
+    if Application.get_env(:elixir_scope, :debug_registry, false) do
       Logger.debug("Getting service info for namespace #{inspect(namespace)}")
     end
 
@@ -413,11 +412,6 @@ defmodule ElixirScope.Foundation.ServiceRegistry do
       unless test_mode do
         Logger.info("Cleanup completed for test namespace #{inspect(namespace)}")
       end
-    else
-      # Only log in debug mode or if not in test mode
-      if not test_mode or Application.get_env(:elixir_scope, :debug_registry, false) do
-        Logger.debug("No services to cleanup in test namespace #{inspect(namespace)}")
-      end
     end
 
     :ok