[rubygems/rubygems] Refactor `Gem::RemoteFetcher::FetchError` initializer to `build` method

The `initialize` method is already doing a lot and by adding the `Gem::PrintableUri` to redact sensitive information, things are getting complicated and hard to read here. For the start, I have refactored the `initialize` method into a class method called `build`.

https://github.com/rubygems/rubygems/commit/4312e8fdf5

1 files changed, 2 insertions, 2 deletions

diff --git a/test/rubygems/utilities.rb b/test/rubygems/utilities.rb
index 20416fe70b..36513dcd31 100644
--- a/test/rubygems/utilities.rb
+++ b/test/rubygems/utilities.rb
@@ -51,7 +51,7 @@ class Gem::FakeFetcher
     raise ArgumentError, 'need full URI' unless path.start_with?("https://", "http://")
 
     unless @data.key? path
-      raise Gem::RemoteFetcher::FetchError.new("no data for #{path}", path)
+      raise Gem::RemoteFetcher::FetchError.build("no data for #{path}", path)
     end
 
     if @data[path].kind_of?(Array) && @data[path].first.kind_of?(Array)
@@ -124,7 +124,7 @@ class Gem::FakeFetcher
     raise ArgumentError, 'need full URI' unless path =~ %r{^http://}
 
     unless @data.key? path
-      raise Gem::RemoteFetcher::FetchError.new("no data for #{path}", path)
+      raise Gem::RemoteFetcher::FetchError.build("no data for #{path}", path)
     end
 
     data = @data[path]