path: root/spec/bundler/bundler/compact_index_client
diff options
authorDavid Rodríguez <>2021-11-29 13:43:09 +0100
committergit <>2021-12-04 05:18:11 +0900
commitc38c1d82b14237cd9d4268b88fef28e9379280f9 (patch)
tree320a2c8f1d0df537dae2c38840982d132dd1f466 /spec/bundler/bundler/compact_index_client
parent54ca530dbe39f389da2b5972a574460a3665eeff (diff)
[rubygems/rubygems] Let original EACCES error be raised
This block of code already wraps file operations with `SharedHelpers.filesystem_access`, which rescues and re-raises more friendly errors. Also, I'm not fully sure creating a temporary directory can end up raising an `Errno::EACCES` error from reading `tmpdir` sources. Finally, this rescue block apparently leads to some false positives when firewall is blocking the ruby executable on Windows, or at least that's what we've got reported. In any case, I think it's best to let the original error be raised.
Diffstat (limited to 'spec/bundler/bundler/compact_index_client')
1 files changed, 0 insertions, 10 deletions
diff --git a/spec/bundler/bundler/compact_index_client/updater_spec.rb b/spec/bundler/bundler/compact_index_client/updater_spec.rb
index 4acd7dbc63..fe417e3920 100644
--- a/spec/bundler/bundler/compact_index_client/updater_spec.rb
+++ b/spec/bundler/bundler/compact_index_client/updater_spec.rb
@@ -36,16 +36,6 @@ RSpec.describe Bundler::CompactIndexClient::Updater do
- context "when bundler doesn't have permissions on Dir.tmpdir" do
- it "Errno::EACCES is raised" do
- allow(Bundler::Dir).to receive(:mktmpdir) { raise Errno::EACCES }
- expect do
- updater.update(local_path, remote_path)
- raise_error(Bundler::PermissionError)
- end
- end
context "when receiving non UTF-8 data and default internal encoding set to ASCII" do
let(:response) { double(:response, :body => "\x8B".b) }