From f4af812a28b03508c96853739aea53f7a6714abf Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 13 Nov 2012 20:22:13 +0000 Subject: tests: remove assert_nothing_raised (part 2) assert_nothing_raised ends up hiding errors and backtraces, making things harder to debug. Since Test::Unit already fails on uncaught exceptions, there is no need to assert on the lack of exceptions for a successful test run. This is a followup to commit 5acf5522295c947d3118926d1a1077007f615de9 --- test/unit/test_upload.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'test/unit/test_upload.rb') diff --git a/test/unit/test_upload.rb b/test/unit/test_upload.rb index 7777e00..bcce4bc 100644 --- a/test/unit/test_upload.rb +++ b/test/unit/test_upload.rb @@ -163,7 +163,7 @@ class UploadTest < Test::Unit::TestCase assert_raise(Errno::ECONNRESET, Errno::EPIPE) do ::Unicorn::Const::CHUNK_SIZE.times { sock.syswrite(buf) } end - assert_nothing_raised { sock.gets } + sock.gets tmp.rewind assert_equal length, tmp.read.to_i end -- cgit v1.2.3-24-ge0c7