Skip to content

Commit

Permalink
Merge pull request #1317 from Homebrew/tap_dumper_installed
Browse files Browse the repository at this point in the history
bundle/tap_dumper: check for installed taps.
  • Loading branch information
MikeMcQuaid committed Mar 4, 2024
2 parents ffd2348 + 594eb81 commit f1f8663
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/bundle/tap_dumper.rb
Expand Up @@ -34,7 +34,7 @@ def tap_names
def taps
@taps ||= begin
require "tap"
Tap.each.to_a
Tap.select(&:installed?).to_a
end
end
private_class_method :taps
Expand Down
2 changes: 1 addition & 1 deletion spec/bundle/tap_dumper_spec.rb
Expand Up @@ -33,7 +33,7 @@
private_tap = instance_double(Tap, name: "privatebrew/private", custom_remote?: true,
remote: "https://#{ENV.fetch("HOMEBREW_GITHUB_API_TOKEN")}@github.com/privatebrew/homebrew-private")

allow(Tap).to receive(:each).and_return [bar, baz, foo, private_tap]
allow(Tap).to receive(:select).and_return [bar, baz, foo, private_tap]
end

after do
Expand Down
2 changes: 1 addition & 1 deletion spec/stub/tap.rb
@@ -1,7 +1,7 @@
# frozen_string_literal: true

class Tap
def self.each
def self.select(*)
[]
end

Expand Down

0 comments on commit f1f8663

Please sign in to comment.