Synchronize counts for link and unlink (#242)

Closes #239.
This commit is contained in:
Paolo G. Giarrusso 2016-05-15 13:07:58 +02:00 committed by Mike McQuaid
parent aae447996c
commit 2f20a42292
3 changed files with 9 additions and 4 deletions

View File

@ -529,6 +529,11 @@ module ObserverPathnameExtension
ObserverPathnameExtension.n += 1
end
def mkpath
super
puts "mkdir -p #{self}" if ObserverPathnameExtension.verbose?
end
def rmdir
super
puts "rmdir #{self}" if ObserverPathnameExtension.verbose?

View File

@ -234,7 +234,7 @@ class Keg
dirs.reverse_each(&:rmdir_if_possible)
end
ObserverPathnameExtension.total
ObserverPathnameExtension.n
end
def lock
@ -374,7 +374,7 @@ class Keg
unlink
raise
else
ObserverPathnameExtension.total
ObserverPathnameExtension.n
end
def remove_oldname_opt_record

View File

@ -56,7 +56,7 @@ class LinkTests < Homebrew::TestCase
def test_unlinking_keg
@keg.link
assert_predicate @dst, :symlink?
assert_equal 4, @keg.unlink
assert_equal 3, @keg.unlink
refute_predicate @dst, :symlink?
end
@ -239,7 +239,7 @@ class LinkTests < Homebrew::TestCase
def test_unlink_ignores_nonexistent_file
@keg.link
@dst.delete
assert_equal 3, @keg.unlink
assert_equal 2, @keg.unlink
end
def test_pkgconfig_is_mkpathed