Always return the result of APIResource#refresh in APIResource.retrie…

…ve (#1473)

* Always return the result of .refresh in .retrieve

With the refactor of v13, there are now cases where `self` is not
mutated in the call to refresh and instead a new object is returned.
This change ensures that the new object is always returned by returning
the result of refresh instead.

* Update install instructions for stripe-mock

README.md

@@ -367,7 +367,7 @@ background terminal ([stripe-mock's README][stripe-mock] also contains
 instructions for installing via Homebrew and other methods):
 
 ```sh
-go get -u github.com/stripe/stripe-mock
+go install github.com/stripe/stripe-mock@latest
 stripe-mock
 ```
 

lib/stripe/api_resource.rb

@@ -110,7 +110,6 @@ def self.retrieve(id, opts = {})
       opts = Util.normalize_opts(opts)
       instance = new(id, opts)
       instance.refresh
-      instance
     end
 
     def request_stripe_object(method:, path:, params:, base_address: :api, opts: {})

test/stripe/api_resource_test.rb

@@ -919,6 +919,23 @@ def self.object_name
       end
     end
 
+    class CustomStripeObject < APIResource
+      def self.resource_url
+        "/v1/custom_stripe_object"
+      end
+    end
+
+    context "custom class extending APIResource" do
+      should "return StripeObject instance when calling retrieve" do
+        stub_request(:get, "#{Stripe.api_base}/v1/custom_stripe_object/id")
+          .to_return(body: JSON.generate({ id: "id", object: "custom_stripe_object", result: "hello" }))
+
+        custom_stripe_object = CustomStripeObject.retrieve("id")
+        assert_instance_of Stripe::StripeObject, custom_stripe_object
+        assert_equal "hello", custom_stripe_object.result
+      end
+    end
+
     @@fixtures = {} # rubocop:disable Style/ClassVars
     setup do
       if @@fixtures.empty?