mirror of
https://github.com/stripe/stripe-ruby.git
synced 2025-10-05 00:02:50 -04:00
Adds the magic `frozen_string_literal: true` comment to every file and enables a Rubocop rule to make sure that it's always going to be there going forward as well. See here for more background [1], but the basic idea is that unlike many other languages, static strings in code are mutable by default. This has since been acknowledged as not a particularly good idea, and the intention is to rectify the mistake when Ruby 3 comes out, where all string literals will be frozen. The `frozen_string_literal` magic comment was introduced in Ruby 2.3 as a way of easing the transition, and allows libraries and projects to freeze their literals in advance. I don't think this is breaking in any way: it's possible that users might've been pulling out one of are literals somehow and mutating it, but that would probably not have been useful for anything and would certainly not be recommended, so I'm quite comfortable pushing this change through as a minor version. As discussed in #641. [1] https://stackoverflow.com/a/37799399
79 lines
3.0 KiB
Ruby
79 lines
3.0 KiB
Ruby
# -*- coding: utf-8 -*-
|
|
# frozen_string_literal: true
|
|
|
|
require File.expand_path("../../test_helper", __FILE__)
|
|
|
|
module Stripe
|
|
class ApiOperationsTest < Test::Unit::TestCase
|
|
class UpdateableResource < APIResource
|
|
include Stripe::APIOperations::Save
|
|
|
|
def self.protected_fields
|
|
[:protected]
|
|
end
|
|
end
|
|
|
|
context ".update" do
|
|
should "post the correct parameters to the resource URL" do
|
|
stub_request(:post, "#{Stripe.api_base}/v1/updateableresources/id")
|
|
.with(body: { foo: "bar" })
|
|
.to_return(body: JSON.generate(foo: "bar"))
|
|
resource = UpdateableResource.update("id", foo: "bar")
|
|
assert_equal("bar", resource.foo)
|
|
end
|
|
|
|
should "error on protected fields" do
|
|
e = assert_raises do
|
|
UpdateableResource.update("id", protected: "bar")
|
|
end
|
|
assert_equal "Cannot update protected field: protected", e.message
|
|
end
|
|
end
|
|
|
|
context ".nested_resource_class_methods" do
|
|
class MainResource < APIResource
|
|
extend Stripe::APIOperations::NestedResource
|
|
nested_resource_class_methods :nested,
|
|
operations: %i[create retrieve update delete list]
|
|
end
|
|
|
|
should "define a create method" do
|
|
stub_request(:post, "#{Stripe.api_base}/v1/mainresources/id/nesteds")
|
|
.with(body: { foo: "bar" })
|
|
.to_return(body: JSON.generate(id: "nested_id", object: "nested", foo: "bar"))
|
|
nested_resource = MainResource.create_nested("id", foo: "bar")
|
|
assert_equal "bar", nested_resource.foo
|
|
end
|
|
|
|
should "define a retrieve method" do
|
|
stub_request(:get, "#{Stripe.api_base}/v1/mainresources/id/nesteds/nested_id")
|
|
.to_return(body: JSON.generate(id: "nested_id", object: "nested", foo: "bar"))
|
|
nested_resource = MainResource.retrieve_nested("id", "nested_id")
|
|
assert_equal "bar", nested_resource.foo
|
|
end
|
|
|
|
should "define an update method" do
|
|
stub_request(:post, "#{Stripe.api_base}/v1/mainresources/id/nesteds/nested_id")
|
|
.with(body: { foo: "baz" })
|
|
.to_return(body: JSON.generate(id: "nested_id", object: "nested", foo: "baz"))
|
|
nested_resource = MainResource.update_nested("id", "nested_id", foo: "baz")
|
|
assert_equal "baz", nested_resource.foo
|
|
end
|
|
|
|
should "define a delete method" do
|
|
stub_request(:delete, "#{Stripe.api_base}/v1/mainresources/id/nesteds/nested_id")
|
|
.to_return(body: JSON.generate(id: "nested_id", object: "nested", deleted: true))
|
|
nested_resource = MainResource.delete_nested("id", "nested_id")
|
|
assert_equal true, nested_resource.deleted
|
|
end
|
|
|
|
should "define a list method" do
|
|
stub_request(:get, "#{Stripe.api_base}/v1/mainresources/id/nesteds")
|
|
.to_return(body: JSON.generate(object: "list", data: []))
|
|
nested_resources = MainResource.list_nesteds("id")
|
|
assert nested_resources.data.is_a?(Array)
|
|
end
|
|
end
|
|
end
|
|
end
|