From 33a53474a90f1825ce20c66dab481fdcfa1106bf Mon Sep 17 00:00:00 2001 From: Ryan Winograd Date: Tue, 18 Aug 2020 18:24:10 -0500 Subject: Remove unnecessary rescue in tests Back in `1e5388ce`, a rescue block was added to `#test_serialization` to prevent the test from breaking on boxes where the file system is not writable. This extra protection doesn't seem necessary anymore, so we are removing the protection in favor of letting the test error in this case. --- test/tc_package.rb | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/test/tc_package.rb b/test/tc_package.rb index 1e5ce322..23078862 100644 --- a/test/tc_package.rb +++ b/test/tc_package.rb @@ -127,16 +127,10 @@ class TestPackage < Test::Unit::TestCase end def test_serialization - assert_nothing_raised do - begin - @package.serialize(@fname) - zf = Zip::File.open(@fname) - @package.send(:parts).each{ |part| zf.get_entry(part[:entry]) } - File.delete(@fname) - rescue Errno::EACCES - puts "WARNING:: test_serialization requires write access." - end - end + @package.serialize(@fname) + zf = Zip::File.open(@fname) + @package.send(:parts).each{ |part| zf.get_entry(part[:entry]) } + File.delete(@fname) end # See comment for Package#zip_entry_for_part -- cgit v1.2.3