Refactor IP filtering module structure and update references in helpers and specs

Author: GrantBirki

lib/hooks/app/api.rb

@@ -4,7 +4,7 @@
 require "json"
 require "securerandom"
 require_relative "helpers"
-require_relative "network/ip_filtering"
+require_relative "../core/network/ip_filtering"
 require_relative "auth/auth"
 require_relative "rack_env_builder"
 require_relative "../plugins/handlers/base"

lib/hooks/app/helpers.rb

@@ -3,7 +3,7 @@
 require "securerandom"
 require_relative "../security"
 require_relative "../core/plugin_loader"
-require_relative "network/ip_filtering"
+require_relative "../core/network/ip_filtering"
 
 module Hooks
   module App
@@ -108,7 +108,7 @@ def load_handler(handler_class_name)
       # @return [void]
       # @note This method will halt execution with an error if IP filtering rules fail.
       def ip_filtering!(headers, endpoint_config, global_config, request_context, env)
-        Network::IpFiltering.ip_filtering!(headers, endpoint_config, global_config, request_context, env)
+        Hooks::Core::Network::IpFiltering.ip_filtering!(headers, endpoint_config, global_config, request_context, env)
       end
 
       private

lib/hooks/app/network/ip_filtering.rb renamed to lib/hooks/core/network/ip_filtering.rb

@@ -4,7 +4,7 @@
 require_relative "../../plugins/handlers/error"
 
 module Hooks
-  module App
+  module Core
     module Network
       # Application-level IP filtering functionality for HTTP requests.
       #

spec/unit/app/network/ip_filtering_spec.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
 require_relative "../../../unit/spec_helper"
-require_relative "../../../../lib/hooks/app/network/ip_filtering"
+require_relative "../../../../lib/hooks/core/network/ip_filtering"
 
-RSpec.describe Hooks::App::Network::IpFiltering do
+describe Hooks::Core::Network::IpFiltering do
   describe ".ip_filtering!" do
     let(:headers) { { "X-Forwarded-For" => "192.168.1.100" } }
     let(:endpoint_config) { {} }

spec/unit/lib/hooks/app/helpers_spec.rb

@@ -354,7 +354,7 @@ def error!(message, code)
     let(:env) { {} }
 
     it "delegates to Network::IpFiltering.ip_filtering!" do
-      expect(Hooks::App::Network::IpFiltering).to receive(:ip_filtering!).with(
+      expect(Hooks::Core::Network::IpFiltering).to receive(:ip_filtering!).with(
         headers,
         endpoint_config,
         global_config,