[Fetcher] Fixes potentially incorrect test failure

rubygems · Mar 26, 2015 · d148ac5 · d148ac5
1 parent 3bb2e93
commit d148ac5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/lib/bundler/fetcher.rb b/lib/bundler/fetcher.rb
@@ -141,14 +141,12 @@ def specs(gem_names, source)
 
       specs = {}
       fetchers.dup.each do |f|
-        next if f.api_fetcher? && !gem_names
-        if specs = f.specs(gem_names)
-          break
-        else
-          fetchers.delete(f)
+        unless f.api_fetcher? && !gem_names
+          break if specs = f.specs(gem_names)
         end
-        @use_api = false if fetchers.none?(&:api_fetcher?)
+        fetchers.delete(f)
       end
+      @use_api = false if fetchers.none?(&:api_fetcher?)
 
       specs[remote_uri].each do |name, version, platform, dependencies|
         next if name == 'bundler'

diff --git a/spec/install/gems/dependency_api_spec.rb b/spec/install/gems/dependency_api_spec.rb
@@ -347,8 +347,6 @@ def require(*args)
     G
 
     bundle :install, :artifice => "endpoint_extra_missing"
-    puts out
-    puts err
     should_be_installed "back_deps 1.0"
   end