Fix Keg tests
* unreverse expected/actuals * don't assume linked file order is guaranteed - it wasn't!
This commit is contained in:
parent
e99ca2d59f
commit
18f5bf1092
@ -19,26 +19,25 @@ class LinkTests < Test::Unit::TestCase
|
|||||||
end
|
end
|
||||||
|
|
||||||
def test_linking_keg
|
def test_linking_keg
|
||||||
assert_equal @keg.link, 3
|
assert_equal 3, @keg.link
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_unlinking_keg
|
def test_unlinking_keg
|
||||||
@keg.link
|
@keg.link
|
||||||
assert_equal @keg.unlink, 3
|
assert_equal 3, @keg.unlink
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_link_dry_run
|
def test_link_dry_run
|
||||||
mode = OpenStruct.new
|
mode = OpenStruct.new
|
||||||
mode.dry_run = true
|
mode.dry_run = true
|
||||||
|
|
||||||
assert_equal @keg.link(mode), 0
|
assert_equal 0, @keg.link(mode)
|
||||||
assert !@keg.linked?
|
assert !@keg.linked?
|
||||||
|
|
||||||
assert_equal $stdout.string, <<-EOS.undent
|
['hiworld', 'helloworld', 'goodbye_cruel_world'].each do |file|
|
||||||
/private/tmp/testbrew/prefix/bin/hiworld
|
assert_match "/private/tmp/testbrew/prefix/bin/#{file}", $stdout.string
|
||||||
/private/tmp/testbrew/prefix/bin/helloworld
|
end
|
||||||
/private/tmp/testbrew/prefix/bin/goodbye_cruel_world
|
assert_equal 3, $stdout.string.lines.count
|
||||||
EOS
|
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_linking_fails_when_already_linked
|
def test_linking_fails_when_already_linked
|
||||||
@ -59,7 +58,7 @@ class LinkTests < Test::Unit::TestCase
|
|||||||
FileUtils.touch HOMEBREW_PREFIX/"bin/helloworld"
|
FileUtils.touch HOMEBREW_PREFIX/"bin/helloworld"
|
||||||
mode = OpenStruct.new
|
mode = OpenStruct.new
|
||||||
mode.overwrite = true
|
mode.overwrite = true
|
||||||
assert_equal @keg.link(mode), 3
|
assert_equal 3, @keg.link(mode)
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_link_overwrite_dryrun
|
def test_link_overwrite_dryrun
|
||||||
@ -68,10 +67,10 @@ class LinkTests < Test::Unit::TestCase
|
|||||||
mode.overwrite = true
|
mode.overwrite = true
|
||||||
mode.dry_run = true
|
mode.dry_run = true
|
||||||
|
|
||||||
assert_equal @keg.link(mode), 0
|
assert_equal 0, @keg.link(mode)
|
||||||
assert !@keg.linked?
|
assert !@keg.linked?
|
||||||
|
|
||||||
assert_equal $stdout.string, "/private/tmp/testbrew/prefix/bin/helloworld\n"
|
assert_equal "/private/tmp/testbrew/prefix/bin/helloworld\n", $stdout.string
|
||||||
end
|
end
|
||||||
|
|
||||||
def teardown
|
def teardown
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user