diff --git a/.rubocop.yml b/.rubocop.yml index 8a81174d..5b45db0b 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -74,3 +74,198 @@ Style/TrailingCommaInArrayLiteral: Style/TrailingCommaInHashLiteral: EnforcedStyleForMultiline: consistent_comma + +Gemspec/DeprecatedAttributeAssignment: # new in 1.30 + Enabled: true +Gemspec/DevelopmentDependencies: # new in 1.44 + Enabled: true +Gemspec/RequireMFA: # new in 1.23 + Enabled: true +Layout/LineContinuationLeadingSpace: # new in 1.31 + Enabled: true +Layout/LineContinuationSpacing: # new in 1.31 + Enabled: true +Layout/LineEndStringConcatenationIndentation: # new in 1.18 + Enabled: true +Layout/SpaceBeforeBrackets: # new in 1.7 + Enabled: true +Lint/AmbiguousAssignment: # new in 1.7 + Enabled: true +Lint/AmbiguousOperatorPrecedence: # new in 1.21 + Enabled: true +Lint/AmbiguousRange: # new in 1.19 + Enabled: true +Lint/ConstantOverwrittenInRescue: # new in 1.31 + Enabled: true +Lint/DeprecatedConstants: # new in 1.8 + Enabled: true +Lint/DuplicateBranch: # new in 1.3 + Enabled: true +Lint/DuplicateMagicComment: # new in 1.37 + Enabled: true +Lint/DuplicateMatchPattern: # new in 1.50 + Enabled: true +Lint/DuplicateRegexpCharacterClassElement: # new in 1.1 + Enabled: true +Lint/EmptyBlock: # new in 1.1 + Enabled: true +Lint/EmptyClass: # new in 1.3 + Enabled: true +Lint/EmptyInPattern: # new in 1.16 + Enabled: true +Lint/IncompatibleIoSelectWithFiberScheduler: # new in 1.21 + Enabled: true +Lint/LambdaWithoutLiteralBlock: # new in 1.8 + Enabled: true +Lint/MixedCaseRange: # new in 1.53 + Enabled: true +Lint/NoReturnInBeginEndBlocks: # new in 1.2 + Enabled: true +Lint/NonAtomicFileOperation: # new in 1.31 + Enabled: true +Lint/NumberedParameterAssignment: # new in 1.9 + Enabled: true +Lint/OrAssignmentToConstant: # new in 1.9 + Enabled: true +Lint/RedundantDirGlobSort: # new in 1.8 + Enabled: true +Lint/RedundantRegexpQuantifiers: # new in 1.53 + Enabled: true +Lint/RefinementImportMethods: # new in 1.27 + Enabled: true +Lint/RequireRangeParentheses: # new in 1.32 + Enabled: true +Lint/RequireRelativeSelfPath: # new in 1.22 + Enabled: true +Lint/SymbolConversion: # new in 1.9 + Enabled: true +Lint/ToEnumArguments: # new in 1.1 + Enabled: true +Lint/TripleQuotes: # new in 1.9 + Enabled: true +Lint/UnexpectedBlockArity: # new in 1.5 + Enabled: true +Lint/UnmodifiedReduceAccumulator: # new in 1.1 + Enabled: true +Lint/UselessRescue: # new in 1.43 + Enabled: true +Lint/UselessRuby2Keywords: # new in 1.23 + Enabled: true +Metrics/CollectionLiteralLength: # new in 1.47 + Enabled: true +Naming/BlockForwarding: # new in 1.24 + Enabled: true +Security/CompoundHash: # new in 1.28 + Enabled: true +Security/IoMethods: # new in 1.22 + Enabled: true +Style/ArgumentsForwarding: # new in 1.1 + Enabled: true +Style/ArrayIntersect: # new in 1.40 + Enabled: true +Style/CollectionCompact: # new in 1.2 + Enabled: true +Style/ComparableClamp: # new in 1.44 + Enabled: true +Style/ConcatArrayLiterals: # new in 1.41 + Enabled: true +Style/DataInheritance: # new in 1.49 + Enabled: true +Style/DirEmpty: # new in 1.48 + Enabled: true +Style/DocumentDynamicEvalDefinition: # new in 1.1 + Enabled: true +Style/EmptyHeredoc: # new in 1.32 + Enabled: true +Style/EndlessMethod: # new in 1.8 + Enabled: true +Style/EnvHome: # new in 1.29 + Enabled: true +Style/ExactRegexpMatch: # new in 1.51 + Enabled: true +Style/FetchEnvVar: # new in 1.28 + Enabled: true +Style/FileEmpty: # new in 1.48 + Enabled: true +Style/FileRead: # new in 1.24 + Enabled: true +Style/FileWrite: # new in 1.24 + Enabled: true +Style/HashConversion: # new in 1.10 + Enabled: true +Style/HashExcept: # new in 1.7 + Enabled: true +Style/IfWithBooleanLiteralBranches: # new in 1.9 + Enabled: true +Style/InPatternThen: # new in 1.16 + Enabled: true +Style/MagicCommentFormat: # new in 1.35 + Enabled: true +Style/MapCompactWithConditionalBlock: # new in 1.30 + Enabled: true +Style/MapToHash: # new in 1.24 + Enabled: true +Style/MapToSet: # new in 1.42 + Enabled: true +Style/MinMaxComparison: # new in 1.42 + Enabled: true +Style/MultilineInPatternThen: # new in 1.16 + Enabled: true +Style/NegatedIfElseCondition: # new in 1.2 + Enabled: true +Style/NestedFileDirname: # new in 1.26 + Enabled: true +Style/NilLambda: # new in 1.3 + Enabled: true +Style/NumberedParameters: # new in 1.22 + Enabled: true +Style/NumberedParametersLimit: # new in 1.22 + Enabled: true +Style/ObjectThen: # new in 1.28 + Enabled: true +Style/OpenStructUse: # new in 1.23 + Enabled: true +Style/OperatorMethodCall: # new in 1.37 + Enabled: true +Style/QuotedSymbols: # new in 1.16 + Enabled: true +Style/RedundantArgument: # new in 1.4 + Enabled: true +Style/RedundantArrayConstructor: # new in 1.52 + Enabled: true +Style/RedundantConstantBase: # new in 1.40 + Enabled: true +Style/RedundantCurrentDirectoryInPath: # new in 1.53 + Enabled: true +Style/RedundantDoubleSplatHashBraces: # new in 1.41 + Enabled: true +Style/RedundantEach: # new in 1.38 + Enabled: true +Style/RedundantFilterChain: # new in 1.52 + Enabled: true +Style/RedundantHeredocDelimiterQuotes: # new in 1.45 + Enabled: true +Style/RedundantInitialize: # new in 1.27 + Enabled: true +Style/RedundantLineContinuation: # new in 1.49 + Enabled: true +Style/RedundantRegexpArgument: # new in 1.53 + Enabled: true +Style/RedundantRegexpConstructor: # new in 1.52 + Enabled: true +Style/RedundantSelfAssignmentBranch: # new in 1.19 + Enabled: true +Style/RedundantStringEscape: # new in 1.37 + Enabled: true +Style/ReturnNilInPredicateMethodDefinition: # new in 1.53 + Enabled: true +Style/SelectByRegexp: # new in 1.22 + Enabled: true +Style/SingleLineDoEndBlock: # new in 1.57 + Enabled: true +Style/StringChars: # new in 1.12 + Enabled: true +Style/SwapValues: # new in 1.1 + Enabled: true +Style/YAMLFileRead: # new in 1.53 + Enabled: true diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index 008fd20d..0ca1972b 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -1,33 +1,82 @@ # This configuration was generated by # `rubocop --auto-gen-config` -# on 2019-07-30 09:56:31 +0800 using RuboCop version 0.73.0. +# on 2023-11-29 18:39:02 UTC using RuboCop version 1.57.2. # The point is for the user to remove these configuration records # one by one as the offenses are removed from the code base. # Note that changes in the inspected code, or installation of new # versions of RuboCop, may require this file to be generated again. -# Offense count: 23 +# Offense count: 2 +Lint/HashCompareByIdentity: + Exclude: + - 'lib/stripe/stripe_client.rb' + +# Offense count: 27 +# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes. Metrics/AbcSize: - Max: 51 + Max: 55 -# Offense count: 12 -# Configuration parameters: CountComments. +# Offense count: 10 +# Configuration parameters: CountComments, CountAsOne. Metrics/ClassLength: - Max: 694 + Max: 618 # Offense count: 12 +# Configuration parameters: AllowedMethods, AllowedPatterns. Metrics/CyclomaticComplexity: - Max: 15 + Max: 12 -# Offense count: 6 -# Configuration parameters: CountKeywordArgs. -Metrics/ParameterLists: - Max: 7 - -# Offense count: 8 +# Offense count: 9 +# Configuration parameters: AllowedMethods, AllowedPatterns. Metrics/PerceivedComplexity: - Max: 17 + Max: 13 -# Offense count: 86 +# Offense count: 1 +# This cop supports unsafe autocorrection (--autocorrect-all). +# Configuration parameters: MinBranchesCount. +Style/CaseLikeIf: + Exclude: + - 'lib/stripe/stripe_configuration.rb' + +# Offense count: 1 +# This cop supports unsafe autocorrection (--autocorrect-all). +Style/CombinableLoops: + Exclude: + - 'lib/stripe/stripe_client.rb' + +# Offense count: 44 +# Configuration parameters: AllowedConstants. Style/Documentation: Enabled: false + +# Offense count: 1 +# This cop supports unsafe autocorrection (--autocorrect-all). +# Configuration parameters: AllowSplatArgument. +Style/HashConversion: + Exclude: + - 'lib/stripe/stripe_client.rb' + +# Offense count: 3 +# This cop supports unsafe autocorrection (--autocorrect-all). +# Configuration parameters: ConvertCodeThatCanStartToReturnNil, AllowedMethods, MaxChainLength. +# AllowedMethods: present?, blank?, presence, try, try! +Style/SafeNavigation: + Exclude: + - 'lib/stripe/instrumentation.rb' + - 'lib/stripe/stripe_response.rb' + +# Offense count: 15 +# This cop supports unsafe autocorrection (--autocorrect-all). +# Configuration parameters: Mode. +Style/StringConcatenation: + Exclude: + - 'lib/stripe.rb' + - 'lib/stripe/connection_manager.rb' + - 'lib/stripe/multipart_encoder.rb' + - 'lib/stripe/oauth.rb' + - 'lib/stripe/resources/bank_account.rb' + - 'lib/stripe/resources/source.rb' + - 'lib/stripe/stripe_client.rb' + - 'test/stripe/api_resource_test.rb' + - 'test/stripe/stripe_client_test.rb' + - 'test/stripe/webhook_test.rb' diff --git a/Gemfile b/Gemfile index cbb5d797..c25a22f5 100644 --- a/Gemfile +++ b/Gemfile @@ -23,10 +23,13 @@ group :development do # `Gemfile.lock` checked in, so to prevent good builds from suddenly going # bad, pin to a specific version number here. Try to keep this relatively # up-to-date, but it's not the end of the world if it's not. - gem "rubocop", "0.80" + # + # The latest version of rubocop is only compatible with Ruby 2.7+ + gem "rubocop", "1.57.2" if RUBY_VERSION >= "2.7" # jaro_winkler 1.5.5 installation fails for jruby - gem "jaro_winkler", "1.5.4" + # don't install on truffleruby + gem "jaro_winkler", "1.5.4" unless RUBY_ENGINE == "truffleruby" platforms :mri do gem "byebug" diff --git a/OPENAPI_VERSION b/OPENAPI_VERSION index 39c58a74..0a7c2e8e 100644 --- a/OPENAPI_VERSION +++ b/OPENAPI_VERSION @@ -1 +1 @@ -v645 \ No newline at end of file +v645 diff --git a/Rakefile b/Rakefile index 4b3c61cf..44ae7a6c 100644 --- a/Rakefile +++ b/Rakefile @@ -1,7 +1,6 @@ # frozen_string_literal: true require "rake/testtask" -require "rubocop/rake_task" task default: %i[test rubocop] @@ -9,7 +8,10 @@ Rake::TestTask.new do |t| t.pattern = "./test/**/*_test.rb" end -RuboCop::RakeTask.new +if RUBY_VERSION >= "2.7.0" + require "rubocop/rake_task" + RuboCop::RakeTask.new +end desc "Update bundled certs" task :update_certs do @@ -17,7 +19,7 @@ task :update_certs do require "uri" fetch_file "https://curl.haxx.se/ca/cacert.pem", - ::File.expand_path("../lib/data/ca-certificates.crt", __FILE__) + File.expand_path("lib/data/ca-certificates.crt", __dir__) end # @@ -25,11 +27,11 @@ end # def fetch_file(uri, dest) - ::File.open(dest, "w") do |file| + File.open(dest, "w") do |file| resp = Net::HTTP.get_response(URI.parse(uri)) unless resp.code.to_i == 200 abort("bad response when fetching: #{uri}\n" \ - "Status #{resp.code}: #{resp.body}") + "Status #{resp.code}: #{resp.body}") end file.write(resp.body) puts "Successfully fetched: #{uri}" diff --git a/bin/stripe-console b/bin/stripe-console index f3262bce..ecf27d2f 100755 --- a/bin/stripe-console +++ b/bin/stripe-console @@ -5,7 +5,7 @@ require "irb" require "irb/completion" -require "#{::File.dirname(__FILE__)}/../lib/stripe" +require "#{File.dirname(__FILE__)}/../lib/stripe" # Config IRB to enable --simple-prompt and auto indent IRB.conf[:PROMPT_MODE] = :SIMPLE diff --git a/lib/stripe/api_operations/request.rb b/lib/stripe/api_operations/request.rb index 2458aa89..6e559de2 100644 --- a/lib/stripe/api_operations/request.rb +++ b/lib/stripe/api_operations/request.rb @@ -89,7 +89,7 @@ module Stripe raise ArgumentError, "request option '#{opt}' should be a string value " \ - "(was a #{val.class})" + "(was a #{val.class})" end end @@ -98,14 +98,12 @@ module Stripe raise ArgumentError, "request params should be either a Hash or nil " \ - "(was a #{params.class})" + "(was a #{params.class})" end private def warn_on_opts_in_params(params) Util::OPTS_USER_SPECIFIED.each do |opt| - if params.key?(opt) - warn("WARNING: '#{opt}' should be in opts instead of params.") - end + warn("WARNING: '#{opt}' should be in opts instead of params.") if params.key?(opt) end end end diff --git a/lib/stripe/api_operations/save.rb b/lib/stripe/api_operations/save.rb index 1e1d252f..af7d3ada 100644 --- a/lib/stripe/api_operations/save.rb +++ b/lib/stripe/api_operations/save.rb @@ -19,9 +19,7 @@ module Stripe # {APIOperations::Request.execute_resource_request}. def update(id, params = {}, opts = {}) params.each_key do |k| - if protected_fields.include?(k) - raise ArgumentError, "Cannot update protected field: #{k}" - end + raise ArgumentError, "Cannot update protected field: #{k}" if protected_fields.include?(k) end request_stripe_object( @@ -64,7 +62,7 @@ module Stripe values = serialize_params(self).merge(params) - # note that id gets removed here our call to #url above has already + # Please note that id gets removed here our call to #url above has already # generated a uri for this object with an identifier baked in values.delete(:id) @@ -72,9 +70,9 @@ module Stripe initialize_from(resp.data, opts) end extend Gem::Deprecate - deprecate :save, "the `update` class method (for examples"\ - " see https://github.com/stripe/stripe-ruby"\ - "/wiki/Migration-guide-for-v8)", 2022, 11 + deprecate :save, "the `update` class method (for examples " \ + "see https://github.com/stripe/stripe-ruby" \ + "/wiki/Migration-guide-for-v8)", 2022, 11 def self.included(base) # Set `metadata` as additive so that when it's set directly we remember diff --git a/lib/stripe/api_resource_test_helpers.rb b/lib/stripe/api_resource_test_helpers.rb index 2309cd59..28bdcee7 100644 --- a/lib/stripe/api_resource_test_helpers.rb +++ b/lib/stripe/api_resource_test_helpers.rb @@ -31,8 +31,8 @@ module Stripe end def self.resource_url - "/v1/test_helpers/"\ - "#{self::RESOURCE_CLASS::OBJECT_NAME.downcase.tr('.', '/')}s" + "/v1/test_helpers/" \ + "#{self::RESOURCE_CLASS::OBJECT_NAME.downcase.tr('.', '/')}s" end def resource_url diff --git a/lib/stripe/connection_manager.rb b/lib/stripe/connection_manager.rb index af8b25f8..0c0dd23a 100644 --- a/lib/stripe/connection_manager.rb +++ b/lib/stripe/connection_manager.rb @@ -157,9 +157,7 @@ module Stripe connection.open_timeout = config.open_timeout connection.read_timeout = config.read_timeout - if connection.respond_to?(:write_timeout=) - connection.write_timeout = config.write_timeout - end + connection.write_timeout = config.write_timeout if connection.respond_to?(:write_timeout=) connection.use_ssl = uri.scheme == "https" @@ -192,9 +190,9 @@ module Stripe @verify_ssl_warned = true warn("WARNING: Running without SSL cert verification. " \ - "You should never do this in production. " \ - "Execute `Stripe.verify_ssl_certs = true` to enable " \ - "verification.") + "You should never do this in production. " \ + "Execute `Stripe.verify_ssl_certs = true` to enable " \ + "verification.") end end end diff --git a/lib/stripe/errors.rb b/lib/stripe/errors.rb index 6397acf0..3945e146 100644 --- a/lib/stripe/errors.rb +++ b/lib/stripe/errors.rb @@ -4,22 +4,14 @@ module Stripe # StripeError is the base error from which all other more specific Stripe # errors derive. class StripeError < StandardError - attr_reader :message + attr_reader :message, :code, :error, :http_body, :http_headers, :http_status, :json_body, :request_id # Response contains a StripeResponse object that has some basic information # about the response that conveyed the error. - attr_accessor :response - - attr_reader :code - attr_reader :error - attr_reader :http_body - attr_reader :http_headers - attr_reader :http_status - attr_reader :json_body # equivalent to #data - attr_reader :request_id + attr_accessor :response # equivalent to #data # Initializes a StripeError. - def initialize(message = nil, http_status: nil, http_body: nil, + def initialize(message = nil, http_status: nil, http_body: nil, # rubocop:todo Lint/MissingSuper json_body: nil, http_headers: nil, code: nil) @message = message @http_status = http_status diff --git a/lib/stripe/instrumentation.rb b/lib/stripe/instrumentation.rb index a30e4c77..25835540 100644 --- a/lib/stripe/instrumentation.rb +++ b/lib/stripe/instrumentation.rb @@ -4,8 +4,7 @@ module Stripe class Instrumentation # Event emitted on `request_begin` callback. class RequestBeginEvent - attr_reader :method - attr_reader :path + attr_reader :method, :path # Arbitrary user-provided data in the form of a Ruby hash that's passed # from subscribers on `request_begin` to subscribers on `request_end`. @@ -27,16 +26,8 @@ module Stripe # Event emitted on `request_end` callback. class RequestEndEvent - attr_reader :duration - attr_reader :http_status - attr_reader :method - attr_reader :num_retries - attr_reader :path - attr_reader :request_id - attr_reader :response_header - attr_reader :response_body - attr_reader :request_header - attr_reader :request_body + attr_reader :duration, :http_status, :method, :num_retries, :path, :request_id, :response_header, :response_body, + :request_header, :request_body # Arbitrary user-provided data in the form of a Ruby hash that's passed # from subscribers on `request_begin` to subscribers on `request_end`. @@ -62,12 +53,7 @@ module Stripe end class RequestContext - attr_reader :duration - attr_reader :method - attr_reader :path - attr_reader :request_id - attr_reader :body - attr_reader :header + attr_reader :duration, :method, :path, :request_id, :body, :header def initialize(duration:, context:, header:) @duration = duration @@ -80,9 +66,7 @@ module Stripe end class ResponseContext - attr_reader :http_status - attr_reader :body - attr_reader :header + attr_reader :http_status, :body, :header def initialize(http_status:, response:) @http_status = http_status diff --git a/lib/stripe/multipart_encoder.rb b/lib/stripe/multipart_encoder.rb index 0df93460..a3a036df 100644 --- a/lib/stripe/multipart_encoder.rb +++ b/lib/stripe/multipart_encoder.rb @@ -106,22 +106,22 @@ module Stripe end private def write_field(name, data, filename:) - if !@first_field - @body << "\r\n" - else + if @first_field @first_field = false + else + @body << "\r\n" end @body << "--#{@boundary}\r\n" if filename - @body << %(Content-Disposition: form-data) + + @body << (%(Content-Disposition: form-data) + %(; name="#{escape(name.to_s)}") + - %(; filename="#{escape(filename)}"\r\n) + %(; filename="#{escape(filename)}"\r\n)) @body << %(Content-Type: application/octet-stream\r\n) else - @body << %(Content-Disposition: form-data) + - %(; name="#{escape(name.to_s)}"\r\n) + @body << (%(Content-Disposition: form-data) + + %(; name="#{escape(name.to_s)}"\r\n)) end @body << "\r\n" diff --git a/lib/stripe/oauth.rb b/lib/stripe/oauth.rb index 580b424c..0ac1d856 100644 --- a/lib/stripe/oauth.rb +++ b/lib/stripe/oauth.rb @@ -18,12 +18,12 @@ module Stripe client_id = params[:client_id] || Stripe.client_id unless client_id raise AuthenticationError, "No client_id provided. " \ - 'Set your client_id using "Stripe.client_id = ". ' \ - "You can find your client_ids in your Stripe dashboard at " \ - "https://dashboard.stripe.com/account/applications/settings, " \ - "after registering your account as a platform. See " \ - "https://stripe.com/docs/connect/standalone-accounts for details, " \ - "or email support@stripe.com if you have any questions." + 'Set your client_id using "Stripe.client_id = ". ' \ + "You can find your client_ids in your Stripe dashboard at " \ + "https://dashboard.stripe.com/account/applications/settings, " \ + "after registering your account as a platform. See " \ + "https://stripe.com/docs/connect/standalone-accounts for details, " \ + "or email support@stripe.com if you have any questions." end client_id end diff --git a/lib/stripe/request_signing_authenticator.rb b/lib/stripe/request_signing_authenticator.rb index f57969d6..f0fcb530 100644 --- a/lib/stripe/request_signing_authenticator.rb +++ b/lib/stripe/request_signing_authenticator.rb @@ -13,12 +13,8 @@ module Stripe attr_reader :auth_token, :sign_lambda def initialize(auth_token, sign_lambda) - unless auth_token.is_a?(String) - raise ArgumentError, "auth_token must be a string" - end - unless sign_lambda.is_a?(Proc) - raise ArgumentError, "sign_lambda must be a lambda" - end + raise ArgumentError, "auth_token must be a string" unless auth_token.is_a?(String) + raise ArgumentError, "sign_lambda must be a lambda" unless sign_lambda.is_a?(Proc) @auth_token = auth_token @sign_lambda = sign_lambda @@ -68,8 +64,8 @@ module Stripe private def encoded_signature(signature_base) Base64.strict_encode64(sign(signature_base)) rescue StandardError - raise AuthenticationError, "Encountered '#{e.message} (#{e.class})' "\ - "when calculating request signature." + raise AuthenticationError, "Encountered '#{e.message} (#{e.class})' " \ + "when calculating request signature." end private def content_digest(content) diff --git a/lib/stripe/resources/account.rb b/lib/stripe/resources/account.rb index 88811d67..135ebd87 100644 --- a/lib/stripe/resources/account.rb +++ b/lib/stripe/resources/account.rb @@ -123,17 +123,13 @@ module Stripe end def serialize_params_account(_obj, update_hash, options = {}) - if (entity = @values[:legal_entity]) - if (owners = entity[:additional_owners]) - entity_update = update_hash[:legal_entity] ||= {} - entity_update[:additional_owners] = - serialize_additional_owners(entity, owners) - end + if (entity = @values[:legal_entity]) && (owners = entity[:additional_owners]) + entity_update = update_hash[:legal_entity] ||= {} + entity_update[:additional_owners] = + serialize_additional_owners(entity, owners) end - if (individual = @values[:individual]) - if individual.is_a?(Person) && !update_hash.key?(:individual) - update_hash[:individual] = individual.serialize_params(options) - end + if (individual = @values[:individual]) && (individual.is_a?(Person) && !update_hash.key?(:individual)) + update_hash[:individual] = individual.serialize_params(options) end update_hash end diff --git a/lib/stripe/resources/alipay_account.rb b/lib/stripe/resources/alipay_account.rb index 785c184c..22b3588a 100644 --- a/lib/stripe/resources/alipay_account.rb +++ b/lib/stripe/resources/alipay_account.rb @@ -14,7 +14,7 @@ module Stripe end "#{Customer.resource_url}/#{CGI.escape(customer)}/sources" \ - "/#{CGI.escape(id)}" + "/#{CGI.escape(id)}" end def self.update(_id, _params = nil, _opts = nil) diff --git a/lib/stripe/resources/application_fee_refund.rb b/lib/stripe/resources/application_fee_refund.rb index f2052d8b..635494d4 100644 --- a/lib/stripe/resources/application_fee_refund.rb +++ b/lib/stripe/resources/application_fee_refund.rb @@ -15,7 +15,7 @@ module Stripe def resource_url "#{ApplicationFee.resource_url}/#{CGI.escape(fee)}/refunds" \ - "/#{CGI.escape(id)}" + "/#{CGI.escape(id)}" end def self.update(_id, _params = nil, _opts = nil) diff --git a/lib/stripe/resources/bank_account.rb b/lib/stripe/resources/bank_account.rb index db943149..75feba12 100644 --- a/lib/stripe/resources/bank_account.rb +++ b/lib/stripe/resources/bank_account.rb @@ -31,8 +31,8 @@ module Stripe def self.update(_id, _params = nil, _opts = nil) raise NotImplementedError, - "Bank accounts cannot be updated without a customer ID or an " \ - " account ID. Update a bank account using " \ + "Bank accounts cannot be updated without a customer ID or an " \ + "account ID. Update a bank account using " \ "`Customer.update_source('customer_id', 'bank_account_id', " \ "update_params)` or `Account.update_external_account(" \ "'account_id', 'bank_account_id', update_params)`" diff --git a/lib/stripe/resources/capability.rb b/lib/stripe/resources/capability.rb index 17835b08..cf65c1c6 100644 --- a/lib/stripe/resources/capability.rb +++ b/lib/stripe/resources/capability.rb @@ -17,7 +17,7 @@ module Stripe "Capabilities cannot be accessed without an account ID." end "#{Account.resource_url}/#{CGI.escape(account)}/capabilities" \ - "/#{CGI.escape(id)}" + "/#{CGI.escape(id)}" end def self.retrieve(_id, _opts = {}) diff --git a/lib/stripe/resources/event.rb b/lib/stripe/resources/event.rb index d89ed826..0657a690 100644 --- a/lib/stripe/resources/event.rb +++ b/lib/stripe/resources/event.rb @@ -23,8 +23,8 @@ module Stripe # `Event` objects directly to an endpoint on your server. You can manage # webhooks in your # [account settings](https://dashboard.stripe.com/account/webhooks). Learn how - # to [listen for events] - # (/docs/webhooks) so that your integration can automatically trigger reactions. + # to [listen for events](https://stripe.com/docs/webhooks) + # so that your integration can automatically trigger reactions. # # When using [Connect](https://stripe.com/docs/connect), you can also receive event notifications # that occur in connected accounts. For these events, there's an diff --git a/lib/stripe/resources/file.rb b/lib/stripe/resources/file.rb index cfc2572a..c389adf4 100644 --- a/lib/stripe/resources/file.rb +++ b/lib/stripe/resources/file.rb @@ -26,10 +26,8 @@ module Stripe end def self.create(params = {}, opts = {}) - if params[:file] && !params[:file].is_a?(String) - unless params[:file].respond_to?(:read) - raise ArgumentError, "file must respond to `#read`" - end + if params[:file] && !params[:file].is_a?(String) && !params[:file].respond_to?(:read) + raise ArgumentError, "file must respond to `#read`" end config = opts[:client]&.config || Stripe.config diff --git a/lib/stripe/resources/reversal.rb b/lib/stripe/resources/reversal.rb index e459a246..db67f1c9 100644 --- a/lib/stripe/resources/reversal.rb +++ b/lib/stripe/resources/reversal.rb @@ -23,7 +23,7 @@ module Stripe def resource_url "#{Transfer.resource_url}/#{CGI.escape(transfer)}/reversals" \ - "/#{CGI.escape(id)}" + "/#{CGI.escape(id)}" end def self.update(_id, _params = nil, _opts = nil) diff --git a/lib/stripe/resources/tax_id.rb b/lib/stripe/resources/tax_id.rb index 07fecfeb..d81d3e9b 100644 --- a/lib/stripe/resources/tax_id.rb +++ b/lib/stripe/resources/tax_id.rb @@ -18,7 +18,7 @@ module Stripe "Tax IDs cannot be accessed without a customer ID." end "#{Customer.resource_url}/#{CGI.escape(customer)}/tax_ids" \ - "/#{CGI.escape(id)}" + "/#{CGI.escape(id)}" end def self.retrieve(_id, _opts = {}) diff --git a/lib/stripe/stripe_client.rb b/lib/stripe/stripe_client.rb index 4ab434f0..ca7c2f00 100644 --- a/lib/stripe/stripe_client.rb +++ b/lib/stripe/stripe_client.rb @@ -36,8 +36,7 @@ module Stripe end end - attr_reader :config - attr_reader :options + attr_reader :config, :options # Gets a currently active `StripeClient`. Set for the current thread when # `StripeClient#request` is being run so that API operations being executed @@ -80,9 +79,7 @@ module Stripe end end - if thread_context.default_connection_managers.empty? - pruned_contexts << thread_context - end + pruned_contexts << thread_context if thread_context.default_connection_managers.empty? end @thread_contexts_with_connection_managers.subtract(pruned_contexts) @@ -120,7 +117,7 @@ module Stripe when Net::OpenTimeout, Net::ReadTimeout # Retry on timeout-related problems (either on open or read). true - when EOFError, Errno::ECONNREFUSED, Errno::ECONNRESET, + when EOFError, Errno::ECONNREFUSED, Errno::ECONNRESET, # rubocop:todo Lint/DuplicateBranch Errno::EHOSTUNREACH, Errno::ETIMEDOUT, SocketError # Destination refused the connection, the connection was reset, or a # variety of other connection failures. This could occur from a single @@ -290,21 +287,21 @@ module Stripe ERROR_MESSAGE_CONNECTION = "Unexpected error communicating when trying to connect to " \ - "Stripe (%s). You may be seeing this message because your DNS is not " \ - "working or you don't have an internet connection. To check, try " \ - "running `host stripe.com` from the command line." + "Stripe (%s). You may be seeing this message because your DNS is not " \ + "working or you don't have an internet connection. To check, try " \ + "running `host stripe.com` from the command line." ERROR_MESSAGE_SSL = "Could not establish a secure connection to Stripe (%s), you " \ - "may need to upgrade your OpenSSL version. To check, try running " \ - "`openssl s_client -connect api.stripe.com:443` from the command " \ - "line." + "may need to upgrade your OpenSSL version. To check, try running " \ + "`openssl s_client -connect api.stripe.com:443` from the command " \ + "line." # Common error suffix sared by both connect and read timeout messages. ERROR_MESSAGE_TIMEOUT_SUFFIX = "Please check your internet connection and try again. " \ - "If this problem persists, you should check Stripe's service " \ - "status at https://status.stripe.com, or let us know at " \ - "support@stripe.com." + "If this problem persists, you should check Stripe's service " \ + "status at https://status.stripe.com, or let us know at " \ + "support@stripe.com." ERROR_MESSAGE_TIMEOUT_CONNECT = ( "Timed out connecting to Stripe (%s). " + @@ -496,9 +493,7 @@ module Stripe response_block = if block_given? lambda do |response| - unless should_handle_as_error(response.code.to_i) - response.read_body(&read_body_chunk_block) - end + response.read_body(&read_body_chunk_block) unless should_handle_as_error(response.code.to_i) end end @@ -523,26 +518,26 @@ module Stripe private def check_keys!(api_key, authenticator) if api_key && authenticator raise AuthenticationError, "Can't specify both API key " \ - "and authenticator. Either set your API key" \ - 'using "Stripe.api_key = ", or set your authenticator ' \ - 'using "Stripe.authenticator = "' \ + "and authenticator. Either set your API key" \ + 'using "Stripe.api_key = ", or set your authenticator ' \ + 'using "Stripe.authenticator = "' \ end unless api_key || authenticator # Default to missing API key error message for general users. raise AuthenticationError, "No API key provided. " \ - 'Set your API key using "Stripe.api_key = ". ' \ - "You can generate API keys from the Stripe web interface. " \ - "See https://stripe.com/api for details, or email " \ - "support@stripe.com if you have any questions." + 'Set your API key using "Stripe.api_key = ". ' \ + "You can generate API keys from the Stripe web interface. " \ + "See https://stripe.com/api for details, or email " \ + "support@stripe.com if you have any questions." end return unless api_key =~ /\s/ raise AuthenticationError, "Your API key is invalid, as it contains " \ - "whitespace. (HINT: You can double-check your API key from the " \ - "Stripe web interface. See https://stripe.com/api for details, or " \ - "email support@stripe.com if you have any questions.)" + "whitespace. (HINT: You can double-check your API key from the " \ + "Stripe web interface. See https://stripe.com/api for details, or " \ + "email support@stripe.com if you have any questions.)" end # Encodes a set of body parameters using multipart if `Content-Type` is set @@ -571,15 +566,15 @@ module Stripe body = Util.encode_parameters(body_params) end - if api_mode == :preview - body_log = body - else - # We don't use `Util.encode_parameters` partly as an optimization (to - # not redo work we've already done), and partly because the encoded - # forms of certain characters introduce a lot of visual noise and it's - # nice to have a clearer format for logs. - body_log = flattened_params.map { |k, v| "#{k}=#{v}" }.join("&") - end + body_log = if api_mode == :preview + body + else + # We don't use `Util.encode_parameters` partly as an optimization (to + # not redo work we've already done), and partly because the encoded + # forms of certain characters introduce a lot of visual noise and it's + # nice to have a clearer format for logs. + flattened_params.map { |k, v| "#{k}=#{v}" }.join("&") + end [body, body_log] end @@ -605,9 +600,7 @@ module Stripe http_status = resp.code.to_i context = context.dup_from_response_headers(resp) - if should_handle_as_error(http_status) - handle_error_response(resp, context) - end + handle_error_response(resp, context) if should_handle_as_error(http_status) log_response(context, request_start, http_status, resp.body, resp) notify_request_end(context, request_duration, http_status, @@ -657,7 +650,7 @@ module Stripe # Only handle errors when we know we can do so, and re-raise otherwise. # This should be pretty infrequent. - else + else # rubocop:todo Lint/DuplicateBranch raise end end @@ -868,7 +861,7 @@ module Stripe if errors.nil? message = "Unexpected error #{error.class.name} communicating " \ - "with Stripe. Please let us know at support@stripe.com." + "with Stripe. Please let us know at support@stripe.com." end api_base ||= config.api_base @@ -882,9 +875,7 @@ module Stripe private def request_headers(api_key, method, api_mode) user_agent = "Stripe/v1 RubyBindings/#{Stripe::VERSION}" - unless Stripe.app_info.nil? - user_agent += " " + format_app_info(Stripe.app_info) - end + user_agent += " " + format_app_info(Stripe.app_info) unless Stripe.app_info.nil? headers = { "User-Agent" => user_agent, @@ -996,16 +987,8 @@ module Stripe # that we can log certain information. It's useful because it means that we # don't have to pass around as many parameters. class RequestLogContext - attr_accessor :body - attr_accessor :account - attr_accessor :api_key - attr_accessor :authenticator - attr_accessor :api_version - attr_accessor :idempotency_key - attr_accessor :method - attr_accessor :path - attr_accessor :query - attr_accessor :request_id + attr_accessor :body, :account, :api_key, :authenticator, :api_version, :idempotency_key, :method, :path, :query, + :request_id # The idea with this method is that we might want to update some of # context information because a response that we've received from the API diff --git a/lib/stripe/stripe_configuration.rb b/lib/stripe/stripe_configuration.rb index e3467d9c..5e389255 100644 --- a/lib/stripe/stripe_configuration.rb +++ b/lib/stripe/stripe_configuration.rb @@ -25,27 +25,12 @@ module Stripe # If `.logger` is set, the value of `.log_level` is ignored. The decision on # what levels to print is entirely deferred to the logger. class StripeConfiguration - attr_accessor :api_key - attr_accessor :api_version - attr_accessor :authenticator - attr_accessor :client_id - attr_accessor :enable_telemetry - attr_accessor :logger - attr_accessor :stripe_account + attr_accessor :api_key, :api_version, :authenticator, :client_id, :enable_telemetry, :logger, :stripe_account - attr_reader :api_base - attr_reader :uploads_base - attr_reader :connect_base - attr_reader :ca_bundle_path - attr_reader :log_level - attr_reader :initial_network_retry_delay - attr_reader :max_network_retries - attr_reader :max_network_retry_delay - attr_reader :open_timeout - attr_reader :read_timeout - attr_reader :write_timeout - attr_reader :proxy - attr_reader :verify_ssl_certs + attr_reader :api_base, :uploads_base, :connect_base, :ca_bundle_path, :log_level, :initial_network_retry_delay, + # rubocop:todo Layout/LineLength + :max_network_retries, :max_network_retry_delay, :open_timeout, :read_timeout, :write_timeout, :proxy, :verify_ssl_certs + # rubocop:enable Layout/LineLength def self.setup new.tap do |instance| @@ -97,8 +82,8 @@ module Stripe if !val.nil? && !levels.include?(val) raise ArgumentError, - "log_level should only be set to `nil`, `debug`, `info`," \ - " or `error`" + "log_level should only be set to `nil`, `debug`, `info`, " \ + "or `error`" end @log_level = val end @@ -126,9 +111,7 @@ module Stripe end def write_timeout=(write_timeout) - unless Net::HTTP.instance_methods.include?(:write_timeout=) - raise NotImplementedError - end + raise NotImplementedError unless Net::HTTP.instance_methods.include?(:write_timeout=) @write_timeout = write_timeout StripeClient.clear_all_connection_managers(config: self) @@ -179,10 +162,10 @@ module Stripe # and is itself not thread safe. def ca_store @ca_store ||= begin - store = OpenSSL::X509::Store.new - store.add_file(ca_bundle_path) - store - end + store = OpenSSL::X509::Store.new + store.add_file(ca_bundle_path) + store + end end def enable_telemetry? diff --git a/lib/stripe/stripe_object.rb b/lib/stripe/stripe_object.rb index e565c746..1df9c59b 100644 --- a/lib/stripe/stripe_object.rb +++ b/lib/stripe/stripe_object.rb @@ -320,8 +320,8 @@ module Stripe # Here we swallow that error and issue a warning so at least # the program doesn't crash. warn("WARNING: Unable to remove method `#{method_name}`; " \ - "if custom, please consider renaming to a name that doesn't " \ - "collide with an API property name.") + "if custom, please consider renaming to a name that doesn't " \ + "collide with an API property name.") end end end @@ -351,17 +351,15 @@ module Stripe define_method(:"#{k}=") do |v| if v == "" raise ArgumentError, "You cannot set #{k} to an empty string. " \ - "We interpret empty strings as nil in requests. " \ - "You may set (object).#{k} = nil to delete the property." + "We interpret empty strings as nil in requests. " \ + "You may set (object).#{k} = nil to delete the property." end @values[k] = Util.convert_to_stripe_object(v, @opts) dirty_value!(@values[k]) @unsaved_values.add(k) end - if [FalseClass, TrueClass].include?(values[k].class) - define_method(:"#{k}?") { @values[k] } - end + define_method(:"#{k}?") { @values[k] } if [FalseClass, TrueClass].include?(values[k].class) end end end @@ -369,7 +367,6 @@ module Stripe # Disabling the cop because it's confused by the fact that the methods are # protected, but we do define `#respond_to_missing?` just below. Hopefully # this is fixed in more recent Rubocop versions. - # rubocop:disable Style/MissingRespondToMissing protected def method_missing(name, *args) # TODO: only allow setting in updateable classes. if name.to_s.end_with?("=") @@ -405,16 +402,14 @@ module Stripe raise NoMethodError, e.message + ". HINT: The '#{name}' attribute was set in the " \ - "past, however. It was then wiped when refreshing the object " \ - "with the result returned by Stripe's API, probably as a " \ - "result of a save(). The attributes currently available on " \ - "this object are: #{@values.keys.join(', ')}" + "past, however. It was then wiped when refreshing the object " \ + "with the result returned by Stripe's API, probably as a " \ + "result of a save(). The attributes currently available on " \ + "this object are: #{@values.keys.join(', ')}" end end - # rubocop:enable Style/MissingRespondToMissing - protected def respond_to_missing?(symbol, include_private = false) - @values && @values.key?(symbol) || super + (@values && @values.key?(symbol)) || super end # Re-initializes the object based on a hash of values (usually one that's @@ -460,7 +455,8 @@ module Stripe self end - protected def serialize_params_value(value, original, unsaved, force, + # rubocop:todo Metrics/PerceivedComplexity + protected def serialize_params_value(value, original, unsaved, force, # rubocop:todo Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity key: nil) if value.nil? "" @@ -494,8 +490,8 @@ module Stripe value else raise ArgumentError, "Cannot save property `#{key}` containing " \ - "an API resource. It doesn't appear to be persisted and is " \ - "not marked as `save_with_parent`." + "an API resource. It doesn't appear to be persisted and is " \ + "not marked as `save_with_parent`." end elsif value.is_a?(Array) @@ -535,6 +531,7 @@ module Stripe value end end + # rubocop:enable Metrics/PerceivedComplexity # Produces a deep copy of the given object including support for arrays, # hashes, and StripeObjects. diff --git a/lib/stripe/stripe_response.rb b/lib/stripe/stripe_response.rb index 8c841310..cb5ea2ce 100644 --- a/lib/stripe/stripe_response.rb +++ b/lib/stripe/stripe_response.rb @@ -41,9 +41,7 @@ module Stripe def [](name) values = @hash[name.downcase] - if values && values.count > 1 - warn("Duplicate header values for `#{name}`; returning only first") - end + warn("Duplicate header values for `#{name}`; returning only first") if values && values.count > 1 values ? values.first : nil end end diff --git a/lib/stripe/util.rb b/lib/stripe/util.rb index a0c29640..46016c79 100644 --- a/lib/stripe/util.rb +++ b/lib/stripe/util.rb @@ -80,8 +80,8 @@ module Stripe "id should be a string representing the ID of an API resource" end - url = "#{target.resource_url}/"\ - "#{CGI.escape(id)}/"\ + url = "#{target.resource_url}/" \ + "#{CGI.escape(id)}/" \ "#{CGI.escape(http_path)}" resp, opts = resource.execute_resource_request( @@ -140,9 +140,7 @@ module Stripe # set filters so that we can fetch the same limit, expansions, and # predicates when accessing the next and previous pages - if obj && (obj.is_a?(SearchResultObject) || obj.is_a?(ListObject)) - obj.filters = params.dup - end + obj.filters = params.dup if obj && (obj.is_a?(SearchResultObject) || obj.is_a?(ListObject)) obj else @@ -154,7 +152,7 @@ module Stripe config = data.delete(:config) || Stripe.config logger = config.logger || Stripe.logger if !logger.nil? || - !config.log_level.nil? && config.log_level <= Stripe::LEVEL_ERROR + (!config.log_level.nil? && config.log_level <= Stripe::LEVEL_ERROR) log_internal(message, data, color: :cyan, level: Stripe::LEVEL_ERROR, logger: Stripe.logger, out: $stderr) end @@ -164,7 +162,7 @@ module Stripe config = data.delete(:config) || Stripe.config logger = config.logger || Stripe.logger if !logger.nil? || - !config.log_level.nil? && config.log_level <= Stripe::LEVEL_INFO + (!config.log_level.nil? && config.log_level <= Stripe::LEVEL_INFO) log_internal(message, data, color: :cyan, level: Stripe::LEVEL_INFO, logger: Stripe.logger, out: $stdout) end @@ -174,7 +172,7 @@ module Stripe config = data.delete(:config) || Stripe.config logger = config.logger || Stripe.logger if !logger.nil? || - !config.log_level.nil? && config.log_level <= Stripe::LEVEL_DEBUG + (!config.log_level.nil? && config.log_level <= Stripe::LEVEL_DEBUG) log_internal(message, data, color: :blue, level: Stripe::LEVEL_DEBUG, logger: Stripe.logger, out: $stdout) end @@ -186,10 +184,10 @@ module Stripe new_hash = {} object.each do |key, value| key = (begin - key.to_sym - rescue StandardError - key - end) || key + key.to_sym + rescue StandardError + key + end) || key new_hash[key] = symbolize_names(value) end new_hash diff --git a/stripe.gemspec b/stripe.gemspec index 7d3edd5a..b627279a 100644 --- a/stripe.gemspec +++ b/stripe.gemspec @@ -1,6 +1,6 @@ # frozen_string_literal: true -$LOAD_PATH.unshift(::File.join(::File.dirname(__FILE__), "lib")) +$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), "lib")) require "stripe/version" @@ -24,6 +24,7 @@ Gem::Specification.new do |s| "github_repo" => "ssh://github.com/stripe/stripe-ruby", "homepage_uri" => "https://stripe.com/docs/api?lang=ruby", "source_code_uri" => "https://github.com/stripe/stripe-ruby", + "rubygems_mfa_required" => "true", } ignored = Regexp.union( @@ -36,6 +37,6 @@ Gem::Specification.new do |s| ) s.files = `git ls-files`.split("\n").reject { |f| ignored.match(f) } s.executables = `git ls-files -- bin/*`.split("\n") - .map { |f| ::File.basename(f) } + .map { |f| File.basename(f) } s.require_paths = ["lib"] end diff --git a/test/stripe/account_link_test.rb b/test/stripe/account_link_test.rb index 097b7e42..579cca9b 100644 --- a/test/stripe/account_link_test.rb +++ b/test/stripe/account_link_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class AccountLinkTest < Test::Unit::TestCase diff --git a/test/stripe/account_test.rb b/test/stripe/account_test.rb index e4a135e3..7ac373fb 100644 --- a/test/stripe/account_test.rb +++ b/test/stripe/account_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class AccountTest < Test::Unit::TestCase @@ -116,8 +116,7 @@ module Stripe should "serialize a new additional_owners" do obj = Stripe::Util.convert_to_stripe_object({ object: "account", - legal_entity: Stripe::StripeObject.construct_from({ - }), + legal_entity: Stripe::StripeObject.construct_from({}), }, {}) obj.legal_entity.additional_owners = [ { first_name: "Joe" }, diff --git a/test/stripe/api_operations_test.rb b/test/stripe/api_operations_test.rb index acaa81cb..e171ee92 100644 --- a/test/stripe/api_operations_test.rb +++ b/test/stripe/api_operations_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ApiOperationsTest < Test::Unit::TestCase @@ -40,7 +40,7 @@ module Stripe end context ".nested_resource_class_methods" do - class MainResource < APIResource + class MainResource < APIResource # rubocop:todo Lint/ConstantDefinitionInBlock extend Stripe::APIOperations::NestedResource OBJECT_NAME = "mainresource" nested_resource_class_methods :nested, diff --git a/test/stripe/api_resource_test.rb b/test/stripe/api_resource_test.rb index a41fc6c3..1f2597a7 100644 --- a/test/stripe/api_resource_test.rb +++ b/test/stripe/api_resource_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ApiResourceTest < Test::Unit::TestCase @@ -279,7 +279,7 @@ module Stripe Stripe::Charge.create({ amount: 100, currency: "usd" }, idempotency_key: :foo) end assert_equal "request option 'idempotency_key' should be a string value " \ - "(was a Symbol)", + "(was a Symbol)", e.message end @@ -591,7 +591,7 @@ module Stripe end context "#request_stripe_object" do - class HelloTestAPIResource < APIResource + class HelloTestAPIResource < APIResource # rubocop:todo Lint/ConstantDefinitionInBlock OBJECT_NAME = "hello" def say_hello(params = {}, opts = {}) request_stripe_object( @@ -668,7 +668,7 @@ module Stripe end context "#request_stream" do - class StreamTestAPIResource < APIResource + class StreamTestAPIResource < APIResource # rubocop:todo Lint/ConstantDefinitionInBlock OBJECT_NAME = "stream" def read_stream(params = {}, opts = {}, &read_body_chunk_block) request_stream( @@ -720,7 +720,7 @@ module Stripe end context "test helpers" do - class TestHelperAPIResource < APIResource + class TestHelperAPIResource < APIResource # rubocop:todo Lint/ConstantDefinitionInBlock OBJECT_NAME = "hello" def test_helpers diff --git a/test/stripe/apple_pay_domain_test.rb b/test/stripe/apple_pay_domain_test.rb index 088d94c7..280c9f50 100644 --- a/test/stripe/apple_pay_domain_test.rb +++ b/test/stripe/apple_pay_domain_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ApplePayDomainTest < Test::Unit::TestCase @@ -23,7 +23,7 @@ module Stripe assert domain.is_a?(Stripe::ApplePayDomain) end - should "be deletable" do + should "be deletable" do # rubocop:todo Lint/EmptyBlock end context "#delete" do diff --git a/test/stripe/application_fee_refund_test.rb b/test/stripe/application_fee_refund_test.rb index c63594fa..6817090d 100644 --- a/test/stripe/application_fee_refund_test.rb +++ b/test/stripe/application_fee_refund_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ApplicationFeeRefundTest < Test::Unit::TestCase diff --git a/test/stripe/application_fee_test.rb b/test/stripe/application_fee_test.rb index f937cf2f..8a431dbf 100644 --- a/test/stripe/application_fee_test.rb +++ b/test/stripe/application_fee_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ApplicationFeeTest < Test::Unit::TestCase diff --git a/test/stripe/balance_test.rb b/test/stripe/balance_test.rb index 56e1b7b4..4ec5061a 100644 --- a/test/stripe/balance_test.rb +++ b/test/stripe/balance_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class BalanceTest < Test::Unit::TestCase diff --git a/test/stripe/balance_transaction_test.rb b/test/stripe/balance_transaction_test.rb index 532b636a..85e6217c 100644 --- a/test/stripe/balance_transaction_test.rb +++ b/test/stripe/balance_transaction_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class BalanceTransactionTest < Test::Unit::TestCase diff --git a/test/stripe/bank_account_test.rb b/test/stripe/bank_account_test.rb index 92b95461..afa6f424 100644 --- a/test/stripe/bank_account_test.rb +++ b/test/stripe/bank_account_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class BankAccountTest < Test::Unit::TestCase diff --git a/test/stripe/billing_portal/configuration_test.rb b/test/stripe/billing_portal/configuration_test.rb index bc0ce652..e56faab3 100644 --- a/test/stripe/billing_portal/configuration_test.rb +++ b/test/stripe/billing_portal/configuration_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module BillingPortal diff --git a/test/stripe/billing_portal/session_test.rb b/test/stripe/billing_portal/session_test.rb index 440279a5..8a5368c2 100644 --- a/test/stripe/billing_portal/session_test.rb +++ b/test/stripe/billing_portal/session_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module BillingPortal diff --git a/test/stripe/capability_test.rb b/test/stripe/capability_test.rb index 1b1c5655..698e28d1 100644 --- a/test/stripe/capability_test.rb +++ b/test/stripe/capability_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class CapabilityTest < Test::Unit::TestCase diff --git a/test/stripe/charge_test.rb b/test/stripe/charge_test.rb index 2cbcf0e6..a6e6e4fa 100644 --- a/test/stripe/charge_test.rb +++ b/test/stripe/charge_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ChargeTest < Test::Unit::TestCase diff --git a/test/stripe/checkout/session_test.rb b/test/stripe/checkout/session_test.rb index 9eace153..08d93139 100644 --- a/test/stripe/checkout/session_test.rb +++ b/test/stripe/checkout/session_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Checkout diff --git a/test/stripe/connection_manager_test.rb b/test/stripe/connection_manager_test.rb index e70468c2..1ac0d0f7 100644 --- a/test/stripe/connection_manager_test.rb +++ b/test/stripe/connection_manager_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ConnectionManagerTest < Test::Unit::TestCase diff --git a/test/stripe/country_spec_test.rb b/test/stripe/country_spec_test.rb index 2f82066d..c497f1c9 100644 --- a/test/stripe/country_spec_test.rb +++ b/test/stripe/country_spec_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class CountrySpecTest < Test::Unit::TestCase diff --git a/test/stripe/coupon_test.rb b/test/stripe/coupon_test.rb index 7e3f95e7..0fbdbd51 100644 --- a/test/stripe/coupon_test.rb +++ b/test/stripe/coupon_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class CouponTest < Test::Unit::TestCase diff --git a/test/stripe/credit_note_test.rb b/test/stripe/credit_note_test.rb index e75c2239..d1660f11 100644 --- a/test/stripe/credit_note_test.rb +++ b/test/stripe/credit_note_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class CreditNoteTest < Test::Unit::TestCase diff --git a/test/stripe/customer_balance_transaction_test.rb b/test/stripe/customer_balance_transaction_test.rb index b577c525..b6fca01b 100644 --- a/test/stripe/customer_balance_transaction_test.rb +++ b/test/stripe/customer_balance_transaction_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __FILE__) +require File.expand_path("../test_helper", __dir__) module Stripe class CustomerBalanceTransactionTest < Test::Unit::TestCase diff --git a/test/stripe/customer_card_test.rb b/test/stripe/customer_card_test.rb index 73a0710a..84c4464a 100644 --- a/test/stripe/customer_card_test.rb +++ b/test/stripe/customer_card_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class CustomerCardTest < Test::Unit::TestCase diff --git a/test/stripe/customer_test.rb b/test/stripe/customer_test.rb index 66070b69..81eacbd9 100644 --- a/test/stripe/customer_test.rb +++ b/test/stripe/customer_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class CustomerTest < Test::Unit::TestCase diff --git a/test/stripe/dispute_test.rb b/test/stripe/dispute_test.rb index 7e955efd..f772e8fe 100644 --- a/test/stripe/dispute_test.rb +++ b/test/stripe/dispute_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class DisputeTest < Test::Unit::TestCase diff --git a/test/stripe/ephemeral_key_test.rb b/test/stripe/ephemeral_key_test.rb index f0fe408b..f35d6d49 100644 --- a/test/stripe/ephemeral_key_test.rb +++ b/test/stripe/ephemeral_key_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class EphemeralKeyTest < Test::Unit::TestCase diff --git a/test/stripe/errors_test.rb b/test/stripe/errors_test.rb index 677976e1..85a611d0 100644 --- a/test/stripe/errors_test.rb +++ b/test/stripe/errors_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class StripeErrorTest < Test::Unit::TestCase diff --git a/test/stripe/exchange_rate_test.rb b/test/stripe/exchange_rate_test.rb index 865887f5..a756dc60 100644 --- a/test/stripe/exchange_rate_test.rb +++ b/test/stripe/exchange_rate_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ExchangeRateTest < Test::Unit::TestCase diff --git a/test/stripe/file_link_test.rb b/test/stripe/file_link_test.rb index 438ffad3..4ab8645a 100644 --- a/test/stripe/file_link_test.rb +++ b/test/stripe/file_link_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class FileLinkTest < Test::Unit::TestCase diff --git a/test/stripe/file_test.rb b/test/stripe/file_test.rb index 1b16f5bb..e35de1cb 100644 --- a/test/stripe/file_test.rb +++ b/test/stripe/file_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class FileTest < Test::Unit::TestCase diff --git a/test/stripe/generated_examples_test.rb b/test/stripe/generated_examples_test.rb index 2933e386..8e9daf53 100644 --- a/test/stripe/generated_examples_test.rb +++ b/test/stripe/generated_examples_test.rb @@ -1,7 +1,7 @@ # File generated from our OpenAPI spec # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class CodegennedExampleTest < Test::Unit::TestCase should "Test account links post" do diff --git a/test/stripe/identity/verification_report_test.rb b/test/stripe/identity/verification_report_test.rb index 014362aa..379bd532 100644 --- a/test/stripe/identity/verification_report_test.rb +++ b/test/stripe/identity/verification_report_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Identity diff --git a/test/stripe/identity/verification_session_test.rb b/test/stripe/identity/verification_session_test.rb index 254a7e84..2abcf0ff 100644 --- a/test/stripe/identity/verification_session_test.rb +++ b/test/stripe/identity/verification_session_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Identity diff --git a/test/stripe/invoice_item_test.rb b/test/stripe/invoice_item_test.rb index 04a3fc1b..1b79d57d 100644 --- a/test/stripe/invoice_item_test.rb +++ b/test/stripe/invoice_item_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class InvoiceItemTest < Test::Unit::TestCase diff --git a/test/stripe/invoice_line_item_test.rb b/test/stripe/invoice_line_item_test.rb index 8515dce8..0a89776b 100644 --- a/test/stripe/invoice_line_item_test.rb +++ b/test/stripe/invoice_line_item_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class InvoiceLineItemTest < Test::Unit::TestCase diff --git a/test/stripe/invoice_test.rb b/test/stripe/invoice_test.rb index 468a2cab..9a5a6d18 100644 --- a/test/stripe/invoice_test.rb +++ b/test/stripe/invoice_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class InvoiceTest < Test::Unit::TestCase diff --git a/test/stripe/issuing/authorization_test.rb b/test/stripe/issuing/authorization_test.rb index 2134a611..cc3c16d2 100644 --- a/test/stripe/issuing/authorization_test.rb +++ b/test/stripe/issuing/authorization_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Issuing diff --git a/test/stripe/issuing/card_test.rb b/test/stripe/issuing/card_test.rb index 243ac047..c1d9120e 100644 --- a/test/stripe/issuing/card_test.rb +++ b/test/stripe/issuing/card_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Issuing diff --git a/test/stripe/issuing/cardholder_test.rb b/test/stripe/issuing/cardholder_test.rb index 0ae7151d..524f6ff3 100644 --- a/test/stripe/issuing/cardholder_test.rb +++ b/test/stripe/issuing/cardholder_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Issuing diff --git a/test/stripe/issuing/dispute_test.rb b/test/stripe/issuing/dispute_test.rb index b0299afe..6a960ac2 100644 --- a/test/stripe/issuing/dispute_test.rb +++ b/test/stripe/issuing/dispute_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Issuing diff --git a/test/stripe/issuing/transaction_test.rb b/test/stripe/issuing/transaction_test.rb index dd3e0fa0..4e00ffe3 100644 --- a/test/stripe/issuing/transaction_test.rb +++ b/test/stripe/issuing/transaction_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Issuing diff --git a/test/stripe/list_object_test.rb b/test/stripe/list_object_test.rb index f65ee5ee..ccbaff0e 100644 --- a/test/stripe/list_object_test.rb +++ b/test/stripe/list_object_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ListObjectTest < Test::Unit::TestCase diff --git a/test/stripe/login_link_test.rb b/test/stripe/login_link_test.rb index 8682e261..3a5aba99 100644 --- a/test/stripe/login_link_test.rb +++ b/test/stripe/login_link_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class LoginLinkTest < Test::Unit::TestCase diff --git a/test/stripe/mandate_test.rb b/test/stripe/mandate_test.rb index 4f872478..513aa6b2 100644 --- a/test/stripe/mandate_test.rb +++ b/test/stripe/mandate_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class MandateTest < Test::Unit::TestCase diff --git a/test/stripe/multipart_encoder_test.rb b/test/stripe/multipart_encoder_test.rb index 10eb4df3..5ab469d7 100644 --- a/test/stripe/multipart_encoder_test.rb +++ b/test/stripe/multipart_encoder_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class MultipartEncoderTest < Test::Unit::TestCase diff --git a/test/stripe/oauth_test.rb b/test/stripe/oauth_test.rb index be7b4dc5..5e365446 100644 --- a/test/stripe/oauth_test.rb +++ b/test/stripe/oauth_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class OAuthTest < Test::Unit::TestCase @@ -83,7 +83,7 @@ module Stripe "grant_type" => "authorization_code", "code" => "this_is_an_authorization_code", }) - .with(headers: { "Authorization": "Bearer client_secret_override" }) + .with(headers: { Authorization: "Bearer client_secret_override" }) .to_return(body: JSON.generate(access_token: "another_access_token")) resp = OAuth.token(client_secret: "client_secret_override", diff --git a/test/stripe/payment_intent_test.rb b/test/stripe/payment_intent_test.rb index d5832334..d6e15adb 100644 --- a/test/stripe/payment_intent_test.rb +++ b/test/stripe/payment_intent_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class PaymentIntentTest < Test::Unit::TestCase diff --git a/test/stripe/payment_method_test.rb b/test/stripe/payment_method_test.rb index 04d52d31..81031883 100644 --- a/test/stripe/payment_method_test.rb +++ b/test/stripe/payment_method_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class PaymentMethodTest < Test::Unit::TestCase diff --git a/test/stripe/payout_test.rb b/test/stripe/payout_test.rb index d7c19788..83bdd2b0 100644 --- a/test/stripe/payout_test.rb +++ b/test/stripe/payout_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class PayoutTest < Test::Unit::TestCase diff --git a/test/stripe/person_test.rb b/test/stripe/person_test.rb index f3d90e5c..6be32673 100644 --- a/test/stripe/person_test.rb +++ b/test/stripe/person_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class PersonTest < Test::Unit::TestCase diff --git a/test/stripe/plan_test.rb b/test/stripe/plan_test.rb index f9d4b295..a82b9128 100644 --- a/test/stripe/plan_test.rb +++ b/test/stripe/plan_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class PlanTest < Test::Unit::TestCase diff --git a/test/stripe/preview_test.rb b/test/stripe/preview_test.rb index 211f524c..9d31e61c 100644 --- a/test/stripe/preview_test.rb +++ b/test/stripe/preview_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) class PreviewTest < Test::Unit::TestCase context "preview raw requests" do diff --git a/test/stripe/price_test.rb b/test/stripe/price_test.rb index fdaf3d4f..4b1ca95c 100644 --- a/test/stripe/price_test.rb +++ b/test/stripe/price_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class PriceTest < Test::Unit::TestCase diff --git a/test/stripe/product_test.rb b/test/stripe/product_test.rb index a5ac0948..e26b1ada 100644 --- a/test/stripe/product_test.rb +++ b/test/stripe/product_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ProductTest < Test::Unit::TestCase diff --git a/test/stripe/promotion_code_test.rb b/test/stripe/promotion_code_test.rb index b09a8d94..82e547ae 100644 --- a/test/stripe/promotion_code_test.rb +++ b/test/stripe/promotion_code_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class PromotionCodeTest < Test::Unit::TestCase diff --git a/test/stripe/quote_test.rb b/test/stripe/quote_test.rb index 164cc546..586bfe39 100644 --- a/test/stripe/quote_test.rb +++ b/test/stripe/quote_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class QuoteTest < Test::Unit::TestCase @@ -191,7 +191,7 @@ module Stripe should "raise when no ID is passed" do assert_raises ArgumentError do - Stripe::Quote.pdf do |read_body_chunk| + Stripe::Quote.pdf do |read_body_chunk| # rubocop:todo Lint/EmptyBlock end end end diff --git a/test/stripe/radar/early_fraud_warning_test.rb b/test/stripe/radar/early_fraud_warning_test.rb index 8b52a451..ee9280ed 100644 --- a/test/stripe/radar/early_fraud_warning_test.rb +++ b/test/stripe/radar/early_fraud_warning_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Radar diff --git a/test/stripe/radar/value_list_item_test.rb b/test/stripe/radar/value_list_item_test.rb index 1d29a776..5608a37c 100644 --- a/test/stripe/radar/value_list_item_test.rb +++ b/test/stripe/radar/value_list_item_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Radar diff --git a/test/stripe/radar/value_list_test.rb b/test/stripe/radar/value_list_test.rb index ce2ddd13..2270b7ff 100644 --- a/test/stripe/radar/value_list_test.rb +++ b/test/stripe/radar/value_list_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Radar diff --git a/test/stripe/raw_request_test.rb b/test/stripe/raw_request_test.rb index 78875def..182c4a33 100644 --- a/test/stripe/raw_request_test.rb +++ b/test/stripe/raw_request_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) class RawRequestTest < Test::Unit::TestCase context "raw_request" do diff --git a/test/stripe/refund_test.rb b/test/stripe/refund_test.rb index 65c72409..58a6cdae 100644 --- a/test/stripe/refund_test.rb +++ b/test/stripe/refund_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class RefundTest < Test::Unit::TestCase diff --git a/test/stripe/reporting/report_run_test.rb b/test/stripe/reporting/report_run_test.rb index 191c6cd8..7ec6b22d 100644 --- a/test/stripe/reporting/report_run_test.rb +++ b/test/stripe/reporting/report_run_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Reporting diff --git a/test/stripe/reporting/report_type_test.rb b/test/stripe/reporting/report_type_test.rb index c70169fb..1b34ec42 100644 --- a/test/stripe/reporting/report_type_test.rb +++ b/test/stripe/reporting/report_type_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Reporting diff --git a/test/stripe/request_signing_authenticator_test.rb b/test/stripe/request_signing_authenticator_test.rb index 442048db..d0edf0f1 100644 --- a/test/stripe/request_signing_authenticator_test.rb +++ b/test/stripe/request_signing_authenticator_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class RequestSigningAuthenticatorTest < Test::Unit::TestCase @@ -44,15 +44,15 @@ module Stripe authenticator.authenticate(:post, headers, '{"string":"String!"}') assert_equal("\"content-type\": application/json\n" \ - "\"content-digest\": sha-256=:HA3i38j+04ac71IzPtG1JK8o4q9sPK0fYPmJHmci5bg=:\n" \ - "\"stripe-context\": \n" \ + "\"content-digest\": sha-256=:HA3i38j+04ac71IzPtG1JK8o4q9sPK0fYPmJHmci5bg=:\n" \ + "\"stripe-context\": \n" \ "\"stripe-account\": \n" \ "\"authorization\": STRIPE-V2-SIG keyid\n" \ "\"@signature-params\": (\"content-type\" \"content-digest\" \"stripe-context\" \"stripe-account\" \"authorization\");created=123456789", signature_bases[0].force_encoding(Encoding::UTF_8)) assert_equal('sig1=("content-type" "content-digest" "stripe-context" "stripe-account" "authorization");' \ - "created=123456789", headers["Signature-Input"]) + "created=123456789", headers["Signature-Input"]) assert_equal("sha-256=:HA3i38j+04ac71IzPtG1JK8o4q9sPK0fYPmJHmci5bg=:", headers["Content-Digest"]) assert_equal("sig1=:AQIDBAU=:", headers["Signature"]) assert_equal("STRIPE-V2-SIG keyid", headers["Authorization"]) @@ -70,15 +70,15 @@ module Stripe authenticator.authenticate(:delete, headers, nil) assert_equal("\"content-type\": application/json\n" \ - "\"content-digest\": sha-256=:47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU=:\n" \ - "\"stripe-context\": \n" \ - "\"stripe-account\": \n" \ - "\"authorization\": STRIPE-V2-SIG keyid\n" \ - "\"@signature-params\": (\"content-type\" \"content-digest\" \"stripe-context\" \"stripe-account\" \"authorization\");created=123456789", + "\"content-digest\": sha-256=:47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU=:\n" \ + "\"stripe-context\": \n" \ + "\"stripe-account\": \n" \ + "\"authorization\": STRIPE-V2-SIG keyid\n" \ + "\"@signature-params\": (\"content-type\" \"content-digest\" \"stripe-context\" \"stripe-account\" \"authorization\");created=123456789", signature_bases[0].force_encoding(Encoding::UTF_8)) assert_equal('sig1=("content-type" "content-digest" "stripe-context" "stripe-account" "authorization");' \ - "created=123456789", headers["Signature-Input"]) + "created=123456789", headers["Signature-Input"]) assert_equal("sha-256=:47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU=:", headers["Content-Digest"]) assert_equal("sig1=:AQIDBAU=:", headers["Signature"]) assert_equal("STRIPE-V2-SIG keyid", headers["Authorization"]) @@ -96,13 +96,13 @@ module Stripe authenticator.authenticate(:get, headers, nil) assert_equal("\"stripe-context\": \n" \ - "\"stripe-account\": \n" \ - "\"authorization\": STRIPE-V2-SIG keyid\n" \ - "\"@signature-params\": (\"stripe-context\" \"stripe-account\" \"authorization\");created=123456789", + "\"stripe-account\": \n" \ + "\"authorization\": STRIPE-V2-SIG keyid\n" \ + "\"@signature-params\": (\"stripe-context\" \"stripe-account\" \"authorization\");created=123456789", signature_bases[0].force_encoding(Encoding::UTF_8)) assert_equal('sig1=("stripe-context" "stripe-account" "authorization");' \ - "created=123456789", headers["Signature-Input"]) + "created=123456789", headers["Signature-Input"]) assert_nil(headers["Content-Digest"]) assert_equal("sig1=:AQIDBAU=:", headers["Signature"]) assert_equal("STRIPE-V2-SIG keyid", headers["Authorization"]) diff --git a/test/stripe/reversal_test.rb b/test/stripe/reversal_test.rb index e9dd7fcd..009f9356 100644 --- a/test/stripe/reversal_test.rb +++ b/test/stripe/reversal_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ReversalTest < Test::Unit::TestCase diff --git a/test/stripe/review_test.rb b/test/stripe/review_test.rb index f2a6525d..f1d06671 100644 --- a/test/stripe/review_test.rb +++ b/test/stripe/review_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class ReviewTest < Test::Unit::TestCase diff --git a/test/stripe/search_result_object_test.rb b/test/stripe/search_result_object_test.rb index fda3a78e..9a87ff94 100644 --- a/test/stripe/search_result_object_test.rb +++ b/test/stripe/search_result_object_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class SearchResultObjectTest < Test::Unit::TestCase @@ -113,7 +113,7 @@ module Stripe object: "search_result") list.filters = { limit: 3 } stub_request(:get, "#{Stripe.api_base}/things") - .with(query: { "limit": 3, page: "next_page_token_1" }) + .with(query: { limit: 3, page: "next_page_token_1" }) .to_return(body: JSON.generate(data: [{ id: 2 }], has_more: false, object: "search_result")) next_list = list.next_search_result_page assert_equal({ limit: 3, page: "next_page_token_1" }, next_list.filters) diff --git a/test/stripe/setup_attempt_test.rb b/test/stripe/setup_attempt_test.rb index 6d22d519..bc38be4a 100644 --- a/test/stripe/setup_attempt_test.rb +++ b/test/stripe/setup_attempt_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class SetupAttemptTest < Test::Unit::TestCase diff --git a/test/stripe/setup_intent_test.rb b/test/stripe/setup_intent_test.rb index 4ce3eafc..f4830cf1 100644 --- a/test/stripe/setup_intent_test.rb +++ b/test/stripe/setup_intent_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class SetupIntentTest < Test::Unit::TestCase diff --git a/test/stripe/sigma/scheduled_query_run_test.rb b/test/stripe/sigma/scheduled_query_run_test.rb index cceceaf0..ef13f357 100644 --- a/test/stripe/sigma/scheduled_query_run_test.rb +++ b/test/stripe/sigma/scheduled_query_run_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Issuing diff --git a/test/stripe/source_test.rb b/test/stripe/source_test.rb index c1165858..944d03ce 100644 --- a/test/stripe/source_test.rb +++ b/test/stripe/source_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class SourceTest < Test::Unit::TestCase diff --git a/test/stripe/stripe_client_test.rb b/test/stripe/stripe_client_test.rb index 4b725b2e..cb400426 100644 --- a/test/stripe/stripe_client_test.rb +++ b/test/stripe/stripe_client_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class StripeClientTest < Test::Unit::TestCase @@ -449,9 +449,7 @@ module Stripe %w[execute_request execute_request_stream].each do |request_method| context "request processing for #{request_method}" do setup do - @read_body_chunk_block = if request_method == "execute_request_stream" - proc { |body_chunk| body_chunk } - end + @read_body_chunk_block = (proc { |body_chunk| body_chunk } if request_method == "execute_request_stream") end context "headers" do @@ -555,7 +553,7 @@ module Stripe data[:thread_object_id] == Thread.current.object_id && (data[:connection_manager_object_id].is_a? Numeric) && (data[:connection_object_id].is_a? Numeric) && - data[:log_timestamp] == 0.0 + data[:log_timestamp] == 0.0 # rubocop:todo Lint/FloatComparison end response_object_id = nil @@ -569,7 +567,7 @@ module Stripe data[:connection_manager_object_id] == connection_manager_data[:connection_manager_object_id] && data[:connection_object_id] == connection_manager_data[:connection_object_id] && (data[:response_object_id].is_a? Numeric) && - data[:log_timestamp] == 0.0 + data[:log_timestamp] == 0.0 # rubocop:todo Lint/FloatComparison response_object_id = data[:response_object_id] end end @@ -609,7 +607,7 @@ module Stripe data[:process_id] == Process.pid && data[:thread_object_id] == Thread.current.object_id && data[:response_object_id] == response_object_id && - data[:log_timestamp] == 0.0 + data[:log_timestamp] == 0.0 # rubocop:todo Lint/FloatComparison # Streaming requests have a different body. if request_method == "execute_request_stream" data[:body].is_a? Net::ReadAdapter @@ -856,7 +854,7 @@ module Stripe client.send(request_method, :post, "/v1/charges", &@read_body_chunk_block) end - assert_equal StripeClient::ERROR_MESSAGE_CONNECTION % Stripe.api_base + + assert_equal (StripeClient::ERROR_MESSAGE_CONNECTION % Stripe.api_base) + "\n\n(Network error: Connection refused)", e.message end @@ -1300,7 +1298,7 @@ module Stripe StripeClient.current_thread_context.active_client = :stripe_client client = StripeClient.new - client.request {} + client.request {} # rubocop:todo Lint/EmptyBlock assert_equal :stripe_client, StripeClient.current_thread_context.active_client @@ -1393,7 +1391,7 @@ module Stripe client = StripeClient.new client.request do e = assert_raises(RuntimeError) do - client.request {} + client.request {} # rubocop:todo Lint/EmptyBlock end assert_equal "calls to StripeClient#request cannot be nested within a thread", e.message @@ -1409,7 +1407,7 @@ module Stripe Stripe.proxy = "http://user:pass@localhost:8080" client = StripeClient.new - client.request {} + client.request {} # rubocop:todo Lint/EmptyBlock connection = Stripe::StripeClient.default_connection_manager.connection_for(Stripe.api_base) diff --git a/test/stripe/stripe_configuration_test.rb b/test/stripe/stripe_configuration_test.rb index fe8b8b00..01e1a792 100644 --- a/test/stripe/stripe_configuration_test.rb +++ b/test/stripe/stripe_configuration_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class StripeConfigurationTest < Test::Unit::TestCase diff --git a/test/stripe/stripe_object_test.rb b/test/stripe/stripe_object_test.rb index 47aeed25..80dde23d 100644 --- a/test/stripe/stripe_object_test.rb +++ b/test/stripe/stripe_object_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class StripeObjectTest < Test::Unit::TestCase @@ -108,7 +108,7 @@ module Stripe end should "return an instance of the same class" do - class TestObject < Stripe::StripeObject; end + class TestObject < Stripe::StripeObject; end # rubocop:todo Lint/ConstantDefinitionInBlock obj = TestObject.construct_from(id: 1) copy_obj = obj.class.send(:deep_copy, obj) @@ -349,7 +349,7 @@ module Stripe end should "#serialize_params with StripeObject that's been replaced which is `metadata`" do - class WithAdditiveObjectParam < Stripe::StripeObject + class WithAdditiveObjectParam < Stripe::StripeObject # rubocop:todo Lint/ConstantDefinitionInBlock additive_object_param :metadata end @@ -418,8 +418,8 @@ module Stripe obj.serialize_params end assert_equal "Cannot save property `customer` containing " \ - "an API resource. It doesn't appear to be persisted and is " \ - "not marked as `save_with_parent`.", e.message + "an API resource. It doesn't appear to be persisted and is " \ + "not marked as `save_with_parent`.", e.message end should "#serialize_params takes a force option" do @@ -434,7 +434,7 @@ module Stripe obj = Stripe::StripeObject.construct_from(id: "id", metadata: Stripe::StripeObject.construct_from(foo: "bar")) - # note that `force` and `dirty!` are for different things, but are + # NOTE: that `force` and `dirty!` are for different things, but are # functionally equivalent obj.dirty! diff --git a/test/stripe/stripe_response_test.rb b/test/stripe/stripe_response_test.rb index 9fa81aa1..2977326f 100644 --- a/test/stripe/stripe_response_test.rb +++ b/test/stripe/stripe_response_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class StripeResponseTest < Test::Unit::TestCase diff --git a/test/stripe/subscription_item_test.rb b/test/stripe/subscription_item_test.rb index 16f7a452..87e4ea12 100644 --- a/test/stripe/subscription_item_test.rb +++ b/test/stripe/subscription_item_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class SubscriptionItemTest < Test::Unit::TestCase diff --git a/test/stripe/subscription_schedule_test.rb b/test/stripe/subscription_schedule_test.rb index 150ea3af..a574fd43 100644 --- a/test/stripe/subscription_schedule_test.rb +++ b/test/stripe/subscription_schedule_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class SubscriptionScheduleTest < Test::Unit::TestCase diff --git a/test/stripe/subscription_test.rb b/test/stripe/subscription_test.rb index eaa8f62a..cfb525f4 100644 --- a/test/stripe/subscription_test.rb +++ b/test/stripe/subscription_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class SubscriptionTest < Test::Unit::TestCase diff --git a/test/stripe/tax_code_test.rb b/test/stripe/tax_code_test.rb index fc56e1b5..06a78c40 100644 --- a/test/stripe/tax_code_test.rb +++ b/test/stripe/tax_code_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class TaxCodeTest < Test::Unit::TestCase diff --git a/test/stripe/tax_id_test.rb b/test/stripe/tax_id_test.rb index a1f6b55e..8a84585b 100644 --- a/test/stripe/tax_id_test.rb +++ b/test/stripe/tax_id_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class TaxIdTest < Test::Unit::TestCase diff --git a/test/stripe/tax_rate_test.rb b/test/stripe/tax_rate_test.rb index a94d1b8a..59036972 100644 --- a/test/stripe/tax_rate_test.rb +++ b/test/stripe/tax_rate_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class TaxRateTest < Test::Unit::TestCase diff --git a/test/stripe/terminal/connection_token_test.rb b/test/stripe/terminal/connection_token_test.rb index a0f445d7..166defd2 100644 --- a/test/stripe/terminal/connection_token_test.rb +++ b/test/stripe/terminal/connection_token_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Terminal diff --git a/test/stripe/terminal/location_test.rb b/test/stripe/terminal/location_test.rb index 2cc8e701..50a014dc 100644 --- a/test/stripe/terminal/location_test.rb +++ b/test/stripe/terminal/location_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Terminal diff --git a/test/stripe/terminal/reader_test.rb b/test/stripe/terminal/reader_test.rb index 027804dd..1e4a7d44 100644 --- a/test/stripe/terminal/reader_test.rb +++ b/test/stripe/terminal/reader_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../../test_helper", __dir__) +require File.expand_path("../../test_helper", __dir__) module Stripe module Terminal diff --git a/test/stripe/topup_test.rb b/test/stripe/topup_test.rb index c3e87c92..7135a1d4 100644 --- a/test/stripe/topup_test.rb +++ b/test/stripe/topup_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class TopupTest < Test::Unit::TestCase diff --git a/test/stripe/transfer_test.rb b/test/stripe/transfer_test.rb index 7864cea0..e3fb068f 100644 --- a/test/stripe/transfer_test.rb +++ b/test/stripe/transfer_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class TransferTest < Test::Unit::TestCase diff --git a/test/stripe/usage_record_summary_test.rb b/test/stripe/usage_record_summary_test.rb index 8a307f7b..1cfb06ea 100644 --- a/test/stripe/usage_record_summary_test.rb +++ b/test/stripe/usage_record_summary_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class UsageRecordSummaryTest < Test::Unit::TestCase diff --git a/test/stripe/util_test.rb b/test/stripe/util_test.rb index 0da956e0..f060bda2 100644 --- a/test/stripe/util_test.rb +++ b/test/stripe/util_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class UtilTest < Test::Unit::TestCase @@ -29,7 +29,7 @@ module Stripe e: [0, 1], f: "", - # note the empty hash won't even show up in the request + # NOTE: the empty hash won't even show up in the request g: [], } assert_equal( diff --git a/test/stripe/webhook_endpoint_test.rb b/test/stripe/webhook_endpoint_test.rb index fdfab949..cb2129a3 100644 --- a/test/stripe/webhook_endpoint_test.rb +++ b/test/stripe/webhook_endpoint_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class WebhookEndpointTest < Test::Unit::TestCase diff --git a/test/stripe/webhook_test.rb b/test/stripe/webhook_test.rb index 56a8d371..98c5c7c0 100644 --- a/test/stripe/webhook_test.rb +++ b/test/stripe/webhook_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("../test_helper", __dir__) +require File.expand_path("../test_helper", __dir__) module Stripe class WebhookTest < Test::Unit::TestCase diff --git a/test/stripe_test.rb b/test/stripe_test.rb index 26a3b297..fbc60370 100644 --- a/test/stripe_test.rb +++ b/test/stripe_test.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require ::File.expand_path("test_helper", __dir__) +require File.expand_path("test_helper", __dir__) class StripeTest < Test::Unit::TestCase should "allow app_info to be configured" do diff --git a/test/test_helper.rb b/test/test_helper.rb index 76e68a15..704d2132 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -15,10 +15,10 @@ require "stringio" require "shoulda/context" require "webmock/test_unit" -PROJECT_ROOT = ::File.expand_path("../", __dir__) +PROJECT_ROOT = File.expand_path("../", __dir__) -require ::File.expand_path("test_data", __dir__) -require ::File.expand_path("stripe_mock", __dir__) +require File.expand_path("test_data", __dir__) +require File.expand_path("stripe_mock", __dir__) MOCK_MINIMUM_VERSION = "0.109.0" MOCK_PORT = Stripe::StripeMock.start @@ -36,18 +36,18 @@ begin if version.nil? abort("Couldn't find `Stripe-Mock-Version` in response from " \ - "`localhost:#{MOCK_PORT}`. Is the service running there stripe-mock?") + "`localhost:#{MOCK_PORT}`. Is the service running there stripe-mock?") end if version != "master" && Gem::Version.new(version) < Gem::Version.new(MOCK_MINIMUM_VERSION) abort("Your version of stripe-mock (#{version}) is too old. The minimum " \ - "version to run this test suite is #{MOCK_MINIMUM_VERSION}. Please " \ - "see its repository for upgrade instructions.") + "version to run this test suite is #{MOCK_MINIMUM_VERSION}. Please " \ + "see its repository for upgrade instructions.") end rescue Errno::ECONNREFUSED abort("Couldn't reach stripe-mock at `localhost:#{MOCK_PORT}`. Is " \ - "it running? Please see README for setup instructions.") + "it running? Please see README for setup instructions.") end Test::Unit.at_exit do