Fix DispatcherBase; modify argument order and defaults for APTrustDis…

…patcher, BagCourier, and uses
mlibrary · Aug 6, 2024 · 8fad97b · 8fad97b
1 parent 7f0cc30
commit 8fad97b
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 12 deletions.
diff --git a/lib/bag_courier.rb b/lib/bag_courier.rb
@@ -34,13 +34,13 @@ def initialize(
       tags:,
       data_transfer:,
       validator:,
+      detect_hidden:,
       target_client:,
       status_event_repo:,
       working_dir:,
       export_dir:,
       dry_run:,
-      remove_export:,
-      detect_hidden:
+      remove_export:
     )
       @bag_id = bag_id
       @bag_info = bag_info

diff --git a/lib/dispatcher.rb b/lib/dispatcher.rb
@@ -9,9 +9,7 @@ class DispatcherBase
     def dispatch(
       object_metadata:,
       data_transfer:,
-      context: nil,
-      validator: nil,
-      extra_bag_info_data: nil
+      validator: nil
     )
       raise NotImplementedError
     end
@@ -26,9 +24,9 @@ def initialize(
       target_client:,
       context: nil,
       extra_bag_info_data: nil,
+      detect_hidden: true,
       status_event_repo: StatusEventRepository::StatusEventInMemoryRepository.new,
-      bag_repo: BagRepository::BagInMemoryRepository.new,
-      detect_hidden:
+      bag_repo: BagRepository::BagInMemoryRepository.new
     )
       @settings = settings
       @repository = repository
@@ -76,13 +74,13 @@ def dispatch(
         tags: tags,
         data_transfer: data_transfer,
         validator: validator,
+        detect_hidden: @detect_hidden,
         target_client: @target_client,
         status_event_repo: @status_event_repo,
         working_dir: @settings.working_dir,
         export_dir: @settings.export_dir,
         dry_run: @settings.dry_run,
-        remove_export: @settings.remove_export,
-        detect_hidden: @detect_hidden
+        remove_export: @settings.remove_export
       )
     end
   end

diff --git a/run_dark_blue.rb b/run_dark_blue.rb
@@ -48,11 +48,11 @@ def create_dispatcher(context:, extra_bag_info_data:)
         type: @aptrust_config.remote.type,
         settings: @aptrust_config.remote.settings
       ),
-      status_event_repo: @status_event_repo,
-      bag_repo: @bag_repo,
       context: context,
       extra_bag_info_data: extra_bag_info_data,
-      detect_hidden: @detect_hidden
+      detect_hidden: @detect_hidden,
+      status_event_repo: @status_event_repo,
+      bag_repo: @bag_repo
     )
   end