lib/dry/rails/container.rb in dry-rails-0.3.0 vs lib/dry/rails/container.rb in dry-rails-0.4.0

@@ -25,123 +25,50 @@
       #
       #   @param features [Array<Symbol>]
       #
       #   @api public
       # @!scope class
-      setting :features, %i[application_contract safe_params controller_helpers], reader: true
+      setting :features,
+              default: %i[application_contract safe_params controller_helpers],
+              reader: true
 
-      # @overload config.auto_register_paths=(paths)
-      #   Set an array of path/block pairs for auto-registration
+      # @overload config.auto_inject_constant=(auto_inject_constant)
+      #   Set a custom import constant name
       #
-      #   This is a low-level setting that typically should not be set explicitly,
-      #   use `auto_register!` instead.
+      #   @param auto_inject_constant [String]
       #
-      #   @param paths [Array<Array>]
-      #
       #   @api public
       # @!scope class
-      setting :auto_register_paths, [].freeze, reader: true
+      setting :auto_inject_constant,
+              default: "Deps",
+              reader: true
 
-      # @overload config.auto_inject_constant=(auto_inject_constant)
-      #   Set a custom import constant name
+      # @overload config.container_constant=(container_constant)
+      #   Set a custom container constant
       #
-      #   @param auto_inject_constant [String]
+      #   @param container_constant [String]
       #
       #   @api public
       # @!scope class
-      setting :auto_inject_constant, "Deps", reader: true
+      setting :container_constant,
+              default: "Container",
+              reader: true
 
       # @!endgroup
 
       # The railtie has a rails-specific auto-registrar which is app-dir aware
       config.auto_registrar = Rails::AutoRegistrars::App
 
       class << self
-        # Set up auto-registration paths and optional a configuration block
-        #
-        # @example set up a single path
-        #   Dry::Rails.container do
-        #     auto_register!("app/operations")
-        #   end
-        #
-        # @example set up a single path with a configuration block
-        #   Dry::Rails.container do
-        #     auto_register!("app/operations") do |config|
-        #       config.exclude do |component|
-        #         component.path.start_with?("concerns")
-        #       end
-        #     end
-        #   end
-        #
-        # @example set up multiple paths
-        #   Dry::Rails.container do
-        #     auto_register!("lib", "app/operations")
-        #   end
-        #
-        # @example set up multiple paths with a configuration block
-        #   Dry::Rails.container do
-        #     # in this case the config block will be applied to all paths
-        #     auto_register!("lib", "app/operations") do |config|
-        #       config.exclude do |component|
-        #         component.path.start_with?("concerns")
-        #       end
-        #     end
-        #   end
-        #
-        # @param paths [Array<String>] One or more paths relative to the root
-        # @param set_load_paths [Boolean] Whether the paths should be added to $LOAD_PATH
-        # @param load_files [Boolean] Whether files should be `required`-ed already
-        #
-        # @return [self]
-        #
-        # @api public
-        #
-        # TODO: this should be moved to dry-system
-        def auto_register!(*paths, set_load_paths: true, load_files: false, &block)
-          load_paths!(*paths) if set_load_paths
-
-          if load_files
-            paths.each { |path| super(path, &block) }
-          else
-            config.auto_register_paths.concat(paths.product([block]))
-          end
-
-          self
-        end
-
-        # Finalize the container
-        #
-        # This is called automatically via the railtie, so typically you won't be using this method
-        # directly
-        #
-        # @param freeze [Boolean] Whether the container should be frozen upon finalization
-        #
-        # @return [self]
-        #
-        # @api public
-        #
-        # TODO: just like auto_register!, this should be moved to dry-system
-        def finalize!(freeze: false, &block)
-          features.each do |feature|
-            start(feature)
-          end
-
-          auto_register_paths.each do |(path, path_block)|
-            auto_register!(path, set_load_paths: false, load_files: true, &path_block)
-          end
-
-          super
-        end
-
         # Return if a given component was booted
         #
         # @return [Boolean]
         #
         # @api private
         #
         # TODO: this should be moved to dry-system
         def booted?(name)
-          booter.booted.map(&:identifier).include?(name)
+          booter.booted.map(&:name).include?(name)
         end
 
         # TODO: confirm that this is really needed
         if ::Rails.version.start_with?("5")
           # @api private