Remove class method mock (Net::HTTP.any_instance), affects other tests

lib/theme_check/remote_asset_file.rb

@@ -31,13 +31,7 @@ def content
       return if @uri.nil?
       return @content unless @content.nil?
 
-      res = Net::HTTP.start(@uri.hostname, @uri.port, use_ssl: @uri.scheme == 'https') do |http|
-        req = Net::HTTP::Get.new(@uri)
-        req['Accept-Encoding'] = 'gzip, deflate, br'
-        http.request(req)
-      end
-
-      @content = res.body
+      @content = request(@uri)
 
     rescue OpenSSL::SSL::SSLError, Zlib::StreamError, *NET_HTTP_EXCEPTIONS
       @contents = ''
@@ -47,5 +41,17 @@ def gzipped_size
       return if @uri.nil?
       @gzipped_size ||= content.bytesize
     end
+
+    private
+
+    def request(uri)
+      res = Net::HTTP.start(uri.hostname, uri.port, use_ssl: uri.scheme == 'https') do |http|
+        req = Net::HTTP::Get.new(uri)
+        req['Accept-Encoding'] = 'gzip, deflate, br'
+        http.request(req)
+      end
+
+      res.body
+    end
   end
 end

test/remote_asset_file_test.rb

@@ -3,8 +3,6 @@
 
 module ThemeCheck
   class RemoteAssetFileTest < Minitest::Test
-    FakeResponse = Struct.new(:body)
-
     def setup
       @src = 'https://cdnjs.cloudflare.com/ajax/libs/jquery/3.6.0/jquery.min.js'
       @asset = RemoteAssetFile.from_src(@src)
@@ -16,10 +14,8 @@ def test_instance_caching
     end
 
     def test_network_request
-      Net::HTTP.any_instance
-        .expects(:request)
-        .with { |req| req['Accept-Encoding'] == 'gzip, deflate, br' }
-        .returns(FakeResponse.new("..."))
+      @asset.expects(:request).with(uri).returns("...")
+
       assert_equal("...", @asset.content)
       assert_equal(3, @asset.gzipped_size)
     end
@@ -35,5 +31,11 @@ def test_handles_eaddr_not_avail_errors
       assert(asset.gzipped_size == 0)
       assert(asset.content.empty?)
     end
+
+    private
+
+    def uri
+      RemoteAssetFile.uri(@src)
+    end
   end
 end