Merge pull request #1033 from technicalpickles/frozen-string-literal

Frozen string literal
bblimke · Aug 19, 2023 · 9305ab5 · 9305ab5
2 parents d4afbd9 + d028c49
commit 9305ab5
Show file tree

Hide file tree

Showing 59 changed files with 119 additions and 1 deletion.
diff --git a/lib/webmock.rb b/lib/webmock.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'singleton'
 
 require 'addressable/uri'

diff --git a/lib/webmock/api.rb b/lib/webmock/api.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   module API
     extend self

diff --git a/lib/webmock/assertion_failure.rb b/lib/webmock/assertion_failure.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class AssertionFailure
     @error_class = RuntimeError

diff --git a/lib/webmock/callback_registry.rb b/lib/webmock/callback_registry.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class CallbackRegistry
     @@callbacks = []

diff --git a/lib/webmock/config.rb b/lib/webmock/config.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class Config
     include Singleton

diff --git a/lib/webmock/cucumber.rb b/lib/webmock/cucumber.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'webmock'
 require 'webmock/rspec/matchers'
 

diff --git a/lib/webmock/deprecation.rb b/lib/webmock/deprecation.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class Deprecation
     class << self

diff --git a/lib/webmock/errors.rb b/lib/webmock/errors.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   class NetConnectNotAllowedError < Exception

diff --git a/lib/webmock/http_lib_adapters/async_http_client_adapter.rb b/lib/webmock/http_lib_adapters/async_http_client_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'async'
   require 'async/http'

diff --git a/lib/webmock/http_lib_adapters/curb_adapter.rb b/lib/webmock/http_lib_adapters/curb_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'curb'
 rescue LoadError

diff --git a/lib/webmock/http_lib_adapters/em_http_request_adapter.rb b/lib/webmock/http_lib_adapters/em_http_request_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'em-http-request'
 rescue LoadError

diff --git a/lib/webmock/http_lib_adapters/excon_adapter.rb b/lib/webmock/http_lib_adapters/excon_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'excon'
 rescue LoadError

diff --git a/lib/webmock/http_lib_adapters/http_lib_adapter.rb b/lib/webmock/http_lib_adapters/http_lib_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class HttpLibAdapter
     def self.adapter_for(lib)

diff --git a/lib/webmock/http_lib_adapters/http_lib_adapter_registry.rb b/lib/webmock/http_lib_adapters/http_lib_adapter_registry.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class HttpLibAdapterRegistry
     include Singleton

diff --git a/lib/webmock/http_lib_adapters/http_rb/client.rb b/lib/webmock/http_lib_adapters/http_rb/client.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module HTTP
   class Client
     alias_method :__perform__, :perform

diff --git a/lib/webmock/http_lib_adapters/http_rb/request.rb b/lib/webmock/http_lib_adapters/http_rb/request.rb
@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module HTTP
   class Request
     def webmock_signature
       request_body = if defined?(HTTP::Request::Body)
-                       ''.tap { |string| body.each { |part| string << part } }
+                       String.new.tap { |string| body.each { |part| string << part } }
                      else
                        body
                      end

diff --git a/lib/webmock/http_lib_adapters/http_rb/response.rb b/lib/webmock/http_lib_adapters/http_rb/response.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module HTTP
   class Response
     def to_webmock

diff --git a/lib/webmock/http_lib_adapters/http_rb/streamer.rb b/lib/webmock/http_lib_adapters/http_rb/streamer.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module HTTP
   class Response
     class Streamer

diff --git a/lib/webmock/http_lib_adapters/http_rb/webmock.rb b/lib/webmock/http_lib_adapters/http_rb/webmock.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module HTTP
   class WebMockPerform
     def initialize(request, options, &perform)

diff --git a/lib/webmock/http_lib_adapters/http_rb_adapter.rb b/lib/webmock/http_lib_adapters/http_rb_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require "http"
 rescue LoadError

diff --git a/lib/webmock/http_lib_adapters/httpclient_adapter.rb b/lib/webmock/http_lib_adapters/httpclient_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'httpclient'
   require 'jsonclient' # defined in 'httpclient' gem as well

diff --git a/lib/webmock/http_lib_adapters/manticore_adapter.rb b/lib/webmock/http_lib_adapters/manticore_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'manticore'
 rescue LoadError

diff --git a/lib/webmock/http_lib_adapters/net_http.rb b/lib/webmock/http_lib_adapters/net_http.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'net/http'
 require 'net/https'
 require 'stringio'

diff --git a/lib/webmock/http_lib_adapters/net_http_response.rb b/lib/webmock/http_lib_adapters/net_http_response.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This code is entierly copied from VCR (http://github.com/myronmarston/vcr) by courtesy of Myron Marston
 
 # A Net::HTTP response that has already been read raises an IOError when #read_body

diff --git a/lib/webmock/http_lib_adapters/patron_adapter.rb b/lib/webmock/http_lib_adapters/patron_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'patron'
 rescue LoadError

diff --git a/lib/webmock/http_lib_adapters/typhoeus_hydra_adapter.rb b/lib/webmock/http_lib_adapters/typhoeus_hydra_adapter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'typhoeus'
 rescue LoadError

diff --git a/lib/webmock/matchers/any_arg_matcher.rb b/lib/webmock/matchers/any_arg_matcher.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   module Matchers
     # this is a based on RSpec::Mocks::ArgumentMatchers::AnyArgMatcher

diff --git a/lib/webmock/matchers/hash_argument_matcher.rb b/lib/webmock/matchers/hash_argument_matcher.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   module Matchers
     # Base class for Hash matchers

diff --git a/lib/webmock/matchers/hash_excluding_matcher.rb b/lib/webmock/matchers/hash_excluding_matcher.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   module Matchers
     # this is a based on RSpec::Mocks::ArgumentMatchers::HashExcludingMatcher

diff --git a/lib/webmock/matchers/hash_including_matcher.rb b/lib/webmock/matchers/hash_including_matcher.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   module Matchers
     # this is a based on RSpec::Mocks::ArgumentMatchers::HashIncludingMatcher

diff --git a/lib/webmock/minitest.rb b/lib/webmock/minitest.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'minitest/test'
   test_class = Minitest::Test

diff --git a/lib/webmock/rack_response.rb b/lib/webmock/rack_response.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class RackResponse < Response
     def initialize(app)

diff --git a/lib/webmock/request_body_diff.rb b/lib/webmock/request_body_diff.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "hashdiff"
 require "json"
 

diff --git a/lib/webmock/request_execution_verifier.rb b/lib/webmock/request_execution_verifier.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class RequestExecutionVerifier
 

diff --git a/lib/webmock/request_pattern.rb b/lib/webmock/request_pattern.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   module RSpecMatcherDetector

diff --git a/lib/webmock/request_registry.rb b/lib/webmock/request_registry.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   class RequestRegistry

diff --git a/lib/webmock/request_signature.rb b/lib/webmock/request_signature.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   class RequestSignature

diff --git a/lib/webmock/request_signature_snippet.rb b/lib/webmock/request_signature_snippet.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pp"
 
 module WebMock

diff --git a/lib/webmock/request_stub.rb b/lib/webmock/request_stub.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class RequestStub
 

diff --git a/lib/webmock/response.rb b/lib/webmock/response.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pathname"
 
 module WebMock

diff --git a/lib/webmock/responses_sequence.rb b/lib/webmock/responses_sequence.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   class ResponsesSequence

diff --git a/lib/webmock/rspec.rb b/lib/webmock/rspec.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'webmock'
 
 # RSpec 1.x and 2.x compatibility

diff --git a/lib/webmock/rspec/matchers.rb b/lib/webmock/rspec/matchers.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'webmock'
 require 'webmock/rspec/matchers/request_pattern_matcher'
 require 'webmock/rspec/matchers/webmock_matcher'

diff --git a/lib/webmock/rspec/matchers/request_pattern_matcher.rb b/lib/webmock/rspec/matchers/request_pattern_matcher.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class RequestPatternMatcher
 

diff --git a/lib/webmock/rspec/matchers/webmock_matcher.rb b/lib/webmock/rspec/matchers/webmock_matcher.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class WebMockMatcher
 

diff --git a/lib/webmock/stub_registry.rb b/lib/webmock/stub_registry.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   class StubRegistry

diff --git a/lib/webmock/stub_request_snippet.rb b/lib/webmock/stub_request_snippet.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class StubRequestSnippet
     def initialize(request_stub)

diff --git a/lib/webmock/test_unit.rb b/lib/webmock/test_unit.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test/unit'
 require 'webmock'
 

diff --git a/lib/webmock/util/hash_counter.rb b/lib/webmock/util/hash_counter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'thread'
 
 module WebMock

diff --git a/lib/webmock/util/hash_keys_stringifier.rb b/lib/webmock/util/hash_keys_stringifier.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   module Util
     class HashKeysStringifier

diff --git a/lib/webmock/util/hash_validator.rb b/lib/webmock/util/hash_validator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   class HashValidator
     def initialize(hash)

diff --git a/lib/webmock/util/headers.rb b/lib/webmock/util/headers.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'base64'
 
 module WebMock

diff --git a/lib/webmock/util/json.rb b/lib/webmock/util/json.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This is a copy of https://github.com/jnunemaker/crack/blob/master/lib/crack/json.rb
 # with date parsing removed
 # Copyright (c) 2004-2008 David Heinemeier Hansson

diff --git a/lib/webmock/util/query_mapper.rb b/lib/webmock/util/query_mapper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock::Util
   class QueryMapper
     class << self

diff --git a/lib/webmock/util/uri.rb b/lib/webmock/util/uri.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   module Util

diff --git a/lib/webmock/util/values_stringifier.rb b/lib/webmock/util/values_stringifier.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class WebMock::Util::ValuesStringifier
   def self.stringify_values(value)
     case value

diff --git a/lib/webmock/util/version_checker.rb b/lib/webmock/util/version_checker.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This code was created based on https://github.com/myronmarston/vcr/blob/master/lib/vcr/util/version_checker.rb
 # Thanks to @myronmarston
 

diff --git a/lib/webmock/version.rb b/lib/webmock/version.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
   VERSION = '3.18.1' unless defined?(::WebMock::VERSION)
 end
diff --git a/lib/webmock/webmock.rb b/lib/webmock/webmock.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module WebMock
 
   def self.included(clazz)