diff options
| author | Jack Nagel | 2012-09-27 17:45:06 -0500 |
|---|---|---|
| committer | Jack Nagel | 2012-09-27 17:50:13 -0500 |
| commit | 7e6c274f8129b65763a70d7628af8fac0cf40ad0 (patch) | |
| tree | c9b4eadd1773340a3e325e54f7eee6ee1bc99fc9 | |
| parent | dd4302ae9bd75f05be2b9a1a3b44f8925d1ea700 (diff) | |
| download | brew-7e6c274f8129b65763a70d7628af8fac0cf40ad0.tar.bz2 | |
test_bucket: don't try to clean a non-existent keg
It is a no-op under Ruby 1.8 but Pathname#find raises ENOENT if the
directory doesn't exist under 1.9. But we really shouldn't try to clean
a non-existent keg anyway, and FormulaInstaller will have bailed out
long before the clean step if the keg doesn't exist, so lets just not
even test this condition; the cleaner has its own tests anyway.
Alternatively we can adjust the cleaner to handle this case, but I don't
see much value in that.
n.b. with this, the tests now pass under 1.9.3 (for me), which is nice
because I often run them on that version inadvertently.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
| -rw-r--r-- | Library/Homebrew/test/test_bucket.rb | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/Library/Homebrew/test/test_bucket.rb b/Library/Homebrew/test/test_bucket.rb index 2963a5532..1279d2ac2 100644 --- a/Library/Homebrew/test/test_bucket.rb +++ b/Library/Homebrew/test/test_bucket.rb @@ -84,7 +84,6 @@ class BeerTasting < Test::Unit::TestCase assert_nothing_raised do f=TestBallWithRealPath.new Homebrew.info_formula f - Cleaner.new f Homebrew.prune #TODO test diy function too end |