Temporary just to get code coverage

Author: voltechs

.github/workflows/ci.yml

@@ -2,9 +2,9 @@ name: CI
 
 on:
   push:
-    branches: [master, develop]
+    branches: [main, develop]
   pull_request:
-    branches: [master, develop]
+    branches: [main, develop]
 
 permissions:
   actions: write

lib/sudo/proxy.rb

@@ -17,8 +17,12 @@ def proxy(object, method=:itself, *args, &blk)
     end
 
     def loaded_specs
-      # Something's weird with this method when called outside
-      Gem.loaded_specs.to_a.to_h
+      # Return only the keys (gem names) to avoid marshaling StubSpecification objects
+      # which can fail in newer Bundler versions
+      Gem.loaded_specs.keys
+    rescue => e
+      warn "Warning: Could not get loaded gem specs (#{e.class}: #{e.message}). Returning empty list."
+      []
     end
 
     def load_path

lib/sudo/wrapper.rb

@@ -31,7 +31,7 @@ def run(ruby_opts: '', load_gems: true) # :yields: sudo
       rescue Exception => e # Bubble all exceptions...
         raise e
       ensure # and ensure sudo stops
-        sudo.stop!
+        sudo&.stop!
       end
 
       # Do the actual resources clean-up.
@@ -130,7 +130,12 @@ def load_gems?
     end
 
     def prospective_gems
-      (Gem.loaded_specs.keys - @proxy.loaded_specs.keys)
+      proxy_loaded_specs = @proxy.loaded_specs
+      local_loaded_specs = Gem.loaded_specs.keys
+      (local_loaded_specs - proxy_loaded_specs)
+    rescue => e
+      warn "Warning: Could not compare loaded gems (#{e.class}: #{e.message}). Skipping gem loading."
+      []
     end
 
     # Load needed libraries in the DRb server. Usually you don't need

spec/lib/sudo/proxy_spec.rb

@@ -8,8 +8,10 @@
 
   context '#loaded_specs' do
 
-    it 'returns a hash' do
-      expect(subject.loaded_specs).to be_a(Hash)
+    it 'returns an array of gem names' do
+      expect(subject.loaded_specs).to be_a(Array)
+      expect(subject.loaded_specs).to_not be_empty
+      expect(subject.loaded_specs).to all(be_a(String))
     end
 
   end

spec/lib/sudo/system_spec.rb

@@ -13,8 +13,8 @@
     end
 
     it 'raises exception if unable to delete file' do
-      allow_any_instance_of(Kernel).to receive(:system).and_return(false)
-      allow(File).to receive(:exists?).and_return(true)
+      allow(File).to receive(:exist?).and_return(true)
+      allow(described_class).to receive(:system).and_return(false)
       expect{described_class.unlink('/tmp/bar')}.to raise_exception(Sudo::System::FileStillExists)
     end