Revert "Refactor incomplete specs handling"

This reverts commit 69580f8b72f41c58cae57d1ada4db909922b3891.

lib/bundler.rb

@@ -62,7 +62,6 @@ module Bundler
   autoload :GemHelpers,             File.expand_path("bundler/gem_helpers", __dir__)
   autoload :GemVersionPromoter,     File.expand_path("bundler/gem_version_promoter", __dir__)
   autoload :Graph,                  File.expand_path("bundler/graph", __dir__)
-  autoload :IncompleteSpecification, File.expand_path("bundler/incomplete_specification", __dir__)
   autoload :Index,                  File.expand_path("bundler/index", __dir__)
   autoload :Injector,               File.expand_path("bundler/injector", __dir__)
   autoload :Installer,              File.expand_path("bundler/installer", __dir__)

lib/bundler/incomplete_specification.rb

@@ -1,24 +0,0 @@
-# frozen_string_literal: true
-
-module Bundler
-  #
-  # Represents a package name that was found to be incomplete when trying to
-  # materialize a fresh resolution or the lockfile.
-  #
-  # Holds the actual partially complete set of specifications for the name.
-  # These are used so that they can be unlocked in a future resolution, and fix
-  # the situation.
-  #
-  class IncompleteSpecification
-    attr_reader :name, :partially_complete_specs
-
-    def initialize(name, partially_complete_specs = [])
-      @name = name
-      @partially_complete_specs = partially_complete_specs
-    end
-
-    def ==(other)
-      partially_complete_specs == other.partially_complete_specs
-    end
-  end
-end

lib/bundler/resolver/base.rb

@@ -34,11 +34,9 @@ module Bundler
         @base[name]
       end
 
-      def delete(incomplete_specs)
-        incomplete_specs.each do |incomplete_spec|
-          incomplete_spec.partially_complete_specs.each do |spec|
-            @base.delete(spec)
-          end
+      def delete(specs)
+        specs.each do |spec|
+          @base.delete(spec)
         end
       end
 

lib/bundler/spec_set.rb

@@ -7,8 +7,11 @@ module Bundler
     include Enumerable
     include TSort
 
-    def initialize(specs)
+    attr_reader :incomplete_specs
+
+    def initialize(specs, incomplete_specs = [])
       @specs = specs
+      @incomplete_specs = incomplete_specs
     end
 
     def for(dependencies, check = false, platforms = [nil])
@@ -42,7 +45,7 @@ module Bundler
         end
 
         if incomplete && check
-          specs << IncompleteSpecification.new(name, lookup[name])
+          @incomplete_specs += lookup[name].any? ? lookup[name] : [LazySpecification.new(name, nil, nil)]
         end
       end
 
@@ -81,7 +84,7 @@ module Bundler
     def materialize(deps)
       materialized = self.for(deps, true)
 
-      SpecSet.new(materialized)
+      SpecSet.new(materialized, incomplete_specs)
     end
 
     # Materialize for all the specs in the spec set, regardless of what platform they're for
@@ -100,19 +103,17 @@ module Bundler
     def incomplete_ruby_specs?(deps)
       return false if @specs.empty?
 
-      materialized = self.for(deps, true, [Gem::Platform::RUBY])
+      @incomplete_specs = []
 
-      SpecSet.new(materialized).incomplete_specs.any?
+      self.for(deps, true, [Gem::Platform::RUBY])
+
+      @incomplete_specs.any?
     end
 
     def missing_specs
       @specs.select {|s| s.is_a?(LazySpecification) }
     end
 
-    def incomplete_specs
-      @specs.select {|s| s.is_a?(IncompleteSpecification) }
-    end
-
     def merge(set)
       arr = sorted.dup
       set.each do |set_spec|