From 0f54f7f072e4a53a303ba5621b37b4a8f7eef9fd Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Mon, 7 Jul 2014 17:34:30 -0500 Subject: [PATCH] Don't need mocks here --- Library/Homebrew/test/test_build_environment.rb | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/Library/Homebrew/test/test_build_environment.rb b/Library/Homebrew/test/test_build_environment.rb index 9e148a7bc5..d560f29152 100644 --- a/Library/Homebrew/test/test_build_environment.rb +++ b/Library/Homebrew/test/test_build_environment.rb @@ -35,17 +35,15 @@ class BuildEnvironmentTests < Homebrew::TestCase end def test_env_block - foo = mock("foo") - @env.proc = Proc.new { foo.some_message } - foo.expects(:some_message) - @env.modify_build_environment(self) + error = Class.new(StandardError) + @env.proc = Proc.new { raise error } + assert_raises(error) { @env.modify_build_environment(self) } end def test_env_block_with_argument - foo = mock("foo") - @env.proc = Proc.new { |x| x.some_message } - foo.expects(:some_message) - @env.modify_build_environment(foo) + error = Class.new(StandardError) + @env.proc = Proc.new { |x| raise x } + assert_raises(error) { @env.modify_build_environment(error) } end end