API Updates (#1056)

lib/stripe/object_types.rb

@@ -65,7 +65,6 @@ module Stripe
         LoginLink::OBJECT_NAME => LoginLink,
         Mandate::OBJECT_NAME => Mandate,
         Order::OBJECT_NAME => Order,
-        OrderReturn::OBJECT_NAME => OrderReturn,
         PaymentIntent::OBJECT_NAME => PaymentIntent,
         PaymentLink::OBJECT_NAME => PaymentLink,
         PaymentMethod::OBJECT_NAME => PaymentMethod,

lib/stripe/resources.rb

@@ -52,7 +52,6 @@ require "stripe/resources/line_item"
 require "stripe/resources/login_link"
 require "stripe/resources/mandate"
 require "stripe/resources/order"
-require "stripe/resources/order_return"
 require "stripe/resources/payment_intent"
 require "stripe/resources/payment_link"
 require "stripe/resources/payment_method"

lib/stripe/resources/financial_connections/account.rb

@@ -7,7 +7,7 @@ module Stripe
       OBJECT_NAME = "financial_connections.account"
 
       custom_method :disconnect, http_verb: :post
-      custom_method :refresh, http_verb: :post
+      custom_method :refresh_account, http_verb: :post, http_path: "refresh"
 
       def disconnect(params = {}, opts = {})
         request_stripe_object(
@@ -18,7 +18,7 @@ module Stripe
         )
       end
 
-      def refresh(params = {}, opts = {})
+      def refresh_account(params = {}, opts = {})
         request_stripe_object(
           method: :post,
           path: resource_url + "/refresh",

lib/stripe/resources/order.rb

@@ -9,22 +9,42 @@ module Stripe
 
     OBJECT_NAME = "order"
 
-    custom_method :pay, http_verb: :post
-    custom_method :return_order, http_verb: :post, http_path: "returns"
+    custom_method :cancel, http_verb: :post
+    custom_method :list_line_items, http_verb: :get, http_path: "line_items"
+    custom_method :reopen, http_verb: :post
+    custom_method :submit, http_verb: :post
 
-    def pay(params = {}, opts = {})
+    def cancel(params = {}, opts = {})
       request_stripe_object(
         method: :post,
-        path: resource_url + "/pay",
+        path: resource_url + "/cancel",
         params: params,
         opts: opts
       )
     end
 
-    def return_order(params = {}, opts = {})
+    def list_line_items(params = {}, opts = {})
+      request_stripe_object(
+        method: :get,
+        path: resource_url + "/line_items",
+        params: params,
+        opts: opts
+      )
+    end
+
+    def reopen(params = {}, opts = {})
       request_stripe_object(
         method: :post,
-        path: resource_url + "/returns",
+        path: resource_url + "/reopen",
+        params: params,
+        opts: opts
+      )
+    end
+
+    def submit(params = {}, opts = {})
+      request_stripe_object(
+        method: :post,
+        path: resource_url + "/submit",
         params: params,
         opts: opts
       )

lib/stripe/resources/order_return.rb

@@ -1,10 +0,0 @@
-# File generated from our OpenAPI spec
-# frozen_string_literal: true
-
-module Stripe
-  class OrderReturn < APIResource
-    extend Stripe::APIOperations::List
-
-    OBJECT_NAME = "order_return"
-  end
-end

test/stripe/order_return_test.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../test_helper", __dir__)
-
-module Stripe
-  class OrderReturnTest < Test::Unit::TestCase
-    should "be listable" do
-      order_returns = Stripe::OrderReturn.list
-      assert_requested :get, "#{Stripe.api_base}/v1/order_returns"
-      assert order_returns.data.is_a?(Array)
-      assert order_returns.data[0].is_a?(Stripe::OrderReturn)
-    end
-
-    should "be retrievable" do
-      order_return = Stripe::OrderReturn.retrieve("orret_123")
-      assert_requested :get,
-                       "#{Stripe.api_base}/v1/order_returns/orret_123"
-      assert order_return.is_a?(Stripe::OrderReturn)
-    end
-  end
-end

test/stripe/order_test.rb

@@ -1,82 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../test_helper", __dir__)
-
-module Stripe
-  class OrderTest < Test::Unit::TestCase
-    should "be listable" do
-      orders = Stripe::Order.list
-      assert_requested :get, "#{Stripe.api_base}/v1/orders"
-      assert orders.data.is_a?(Array)
-      assert orders.first.is_a?(Stripe::Order)
-    end
-
-    should "be retrievable" do
-      order = Stripe::Order.retrieve("or_123")
-      assert_requested :get, "#{Stripe.api_base}/v1/orders/or_123"
-      assert order.is_a?(Stripe::Order)
-    end
-
-    should "be creatable" do
-      order = Stripe::Order.create(
-        currency: "USD"
-      )
-      assert_requested :post, "#{Stripe.api_base}/v1/orders"
-      assert order.is_a?(Stripe::Order)
-    end
-
-    should "be saveable" do
-      order = Stripe::Order.retrieve("or_123")
-      order.metadata["key"] = "value"
-      order.save
-      assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}"
-    end
-
-    should "be updateable" do
-      order = Stripe::Order.update("or_123", metadata: { key: "value" })
-      assert_requested :post, "#{Stripe.api_base}/v1/orders/or_123"
-      assert order.is_a?(Stripe::Order)
-    end
-
-    context "#pay" do
-      should "pay an order" do
-        order = Stripe::Order.retrieve("or_123")
-        order = order.pay(source: "tok_123")
-        assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}/pay"
-        assert order.is_a?(Stripe::Order)
-      end
-
-      should "pay an order without additional arguments" do
-        order = Stripe::Order.retrieve("or_123")
-        order = order.pay
-        assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}/pay"
-        assert order.is_a?(Stripe::Order)
-      end
-    end
-
-    context ".pay" do
-      should "pay an order" do
-        order = Stripe::Order.pay("or_123", source: "tok_123")
-        assert_requested :post, "#{Stripe.api_base}/v1/orders/or_123/pay"
-        assert order.is_a?(Stripe::Order)
-      end
-    end
-
-    context "#return_order" do
-      should "return an order" do
-        order = Stripe::Order.retrieve("or_123")
-        order_return = order.return_order({})
-        assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}/returns"
-        assert order_return.is_a?(Stripe::OrderReturn)
-      end
-    end
-
-    context ".return_order" do
-      should "return an order" do
-        order_return = Stripe::Order.return_order("or_123")
-        assert_requested :post, "#{Stripe.api_base}/v1/orders/or_123/returns"
-        assert order_return.is_a?(Stripe::OrderReturn)
-      end
-    end
-  end
-end