Refactor InvalidLogsHelper module structure

lib/helpers/invalid_logs_helper.rb

@@ -1,26 +1,27 @@
-module InvalidLogsHelper
+module Helpers
-  module_function
+  module InvalidLogsHelper
+    module_function
 
-  def count_and_display_invalid_logs(model, log_type, year)
+    def count_and_display_invalid_logs(model, log_type, year)
-    invalid_logs = fetch_invalid_logs(model, year)
+      invalid_logs = fetch_invalid_logs(model, year)
-    puts "Number of invalid #{log_type} for year #{year}: #{invalid_logs.size}"
+      puts "Number of invalid #{log_type} for year #{year}: #{invalid_logs.size}"
-    puts "Invalid #{log_type} IDs: #{invalid_logs.map(&:id).join(', ')}"
+      puts "Invalid #{log_type} IDs: #{invalid_logs.map(&:id).join(', ')}"
-  end
+    end
 
-  def surface_invalid_logs(model, log_type, year)
+    def surface_invalid_logs(model, log_type, year)
-    invalid_logs = fetch_invalid_logs(model, year)
+      invalid_logs = fetch_invalid_logs(model, year)
-    if invalid_logs.any?
+      if invalid_logs.any?
-      invalid_logs.each do |log|
+        invalid_logs.each do |log|
-        puts "#{log_type} ID: #{log.id}"
+          puts "#{log_type} ID: #{log.id}"
-        log.errors.full_messages.each { |message| puts "  - #{message}" }
+          log.errors.full_messages.each { |message| puts "  - #{message}" }
+        end
+      else
+        puts "No invalid #{log_type} found for year #{year}."
       end
-    else
-      puts "No invalid #{log_type} found for year #{year}."
     end
-  end
 
-  def fetch_invalid_logs(model, year)
+    def fetch_invalid_logs(model, year)
-    model.filter_by_year(year).reject(&:valid?)
+      model.filter_by_year(year).reject(&:valid?)
+    end
   end
 end
-