diff options
| author | Ryan Winograd <[email protected]> | 2020-08-18 18:24:10 -0500 |
|---|---|---|
| committer | Ryan Winograd <[email protected]> | 2020-08-19 14:56:27 -0500 |
| commit | 33a53474a90f1825ce20c66dab481fdcfa1106bf (patch) | |
| tree | 935ad003a991e41c950e9c6faffdadeb4b58072d | |
| parent | 40efd8444ee6b1b0509a5eb766d1dbf518b1dfe9 (diff) | |
| download | caxlsx-33a53474a90f1825ce20c66dab481fdcfa1106bf.tar.gz caxlsx-33a53474a90f1825ce20c66dab481fdcfa1106bf.zip | |
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.
| -rw-r--r-- | test/tc_package.rb | 14 |
1 files 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 |
