summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorRyan Winograd <[email protected]>2020-08-18 18:24:10 -0500
committerRyan Winograd <[email protected]>2020-08-19 14:56:27 -0500
commit33a53474a90f1825ce20c66dab481fdcfa1106bf (patch)
tree935ad003a991e41c950e9c6faffdadeb4b58072d
parent40efd8444ee6b1b0509a5eb766d1dbf518b1dfe9 (diff)
downloadcaxlsx-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.rb14
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