stripe 1.31.0 → 1.58.0

data/test/stripe/util_test.rb

@@ -3,23 +3,51 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class UtilTest < Test::Unit::TestCase
     should "#encode_parameters should prepare parameters for an HTTP request" do
-      # use array instead of hash for 1.8.7 ordering
-      params = [
-        [:a, 3],
-        [:b, "+foo?"],
-        [:c, "bar&baz"],
-        [:d, { :a => "a", :b => "b" }],
-        [:e, [0, 1]],
+      params = {
+        :a => 3,
+        :b => "+foo?",
+        :c => "bar&baz",
+        :d => { :a => "a", :b => "b" },
+        :e => [0, 1],
+        :f => "",
 
         # note the empty hash won't even show up in the request
-        [:f, []]
-      ]
+        :g => [],
+      }
       assert_equal(
-        "a=3&b=%2Bfoo%3F&c=bar%26baz&d[a]=a&d[b]=b&e[]=0&e[]=1",
+        "a=3&b=%2Bfoo%3F&c=bar%26baz&d[a]=a&d[b]=b&e[]=0&e[]=1&f=",
         Stripe::Util.encode_parameters(params)
       )
     end
 
+    should "#encode_params should throw an error on an array of maps that cannot be encoded" do
+      params = {
+        :a => [
+          { :a => 1, :b => 2 },
+          { :c => 3, :a => 4 },
+        ]
+      }
+      e = assert_raises(ArgumentError) do
+        Stripe::Util.encode_parameters(params)
+      end
+      expected = "All maps nested in an array should start with the same key " +
+        "(expected starting key 'a', got 'c')"
+      assert_equal expected, e.message
+
+      # Make sure the check is recursive by taking our original params and
+      # nesting it into yet another map and array. Should throw exactly the
+      # same error because it's still the in inner array of maps that's wrong.
+      params = {
+        :x => [
+          params
+        ]
+      }
+      e = assert_raises(ArgumentError) do
+        Stripe::Util.encode_parameters(params)
+      end
+      assert_equal expected, e.message
+    end
+
     should "#url_encode should prepare strings for HTTP" do
       assert_equal "foo",      Stripe::Util.url_encode("foo")
       assert_equal "foo",      Stripe::Util.url_encode(:foo)
@@ -36,8 +64,8 @@ module Stripe
         [:d, { :a => "a", :b => "b" }],
         [:e, [0, 1]],
         [:f, [
-          { :bar => "1", :foo => "2" },
-          { :baz => "3", :foo => "4" },
+          { :foo => "1", :ghi => "2" },
+          { :foo => "3", :bar => "4" },
         ]],
       ]
       assert_equal([
@@ -52,10 +80,10 @@ module Stripe
         # *The key here is the order*. In order to be properly interpreted as
         # an array of hashes on the server, everything from a single hash must
         # come in at once. A duplicate key in an array triggers a new element.
-        ["f[][bar]", "1"],
-        ["f[][foo]", "2"],
-        ["f[][baz]", "3"],
-        ["f[][foo]", "4"],
+        ["f[][foo]", "1"],
+        ["f[][ghi]", "2"],
+        ["f[][foo]", "3"],
+        ["f[][bar]", "4"],
       ], Stripe::Util.flatten_params(params))
     end
 
@@ -113,5 +141,9 @@ module Stripe
       obj = Util.convert_to_stripe_object([1, 2, 3], {})
       assert_equal [1, 2, 3], obj
     end
+
+    should "#array_to_hash should convert an array into a hash with integer keys" do
+      assert_equal({"0" => 1, "1" => 2, "2" => 3}, Util.array_to_hash([1, 2, 3]))
+    end
   end
 end

data/test/stripe_test.rb

@@ -13,4 +13,62 @@ class StripeTest < Test::Unit::TestCase
       $stderr = old_stderr
     end
   end
+
+  should "allow ca_bundle_path to be configured" do
+    begin
+      old = Stripe.ca_bundle_path
+      Stripe.ca_bundle_path = "path/to/ca/bundle"
+      assert_equal "path/to/ca/bundle", Stripe.ca_bundle_path
+    ensure
+      Stripe.ca_bundle_path = old
+    end
+  end
+
+  should "allow max_network_retries to be configured" do
+    begin
+      old = Stripe.max_network_retries
+      Stripe.max_network_retries = 99
+      assert_equal 99, Stripe.max_network_retries
+    ensure
+      Stripe.max_network_retries = old
+    end
+  end
+
+  should "makes requests with the Stripe-Account header" do
+    response = make_account(
+      charges_enabled: false,
+      details_submitted: false,
+      email: "test+bindings@stripe.com"
+    )
+    Stripe.stripe_account = 'acct_1234'
+
+    Stripe.expects(:execute_request).with(
+      has_entry(:headers, has_entry('Stripe-Account', 'acct_1234')),
+    ).returns(make_response(response))
+
+    Stripe.request(:post, '/v1/account', 'sk_live12334566')
+  end
+
+  context "#get_uname" do
+    should "run without failure" do
+      # Don't actually check the result because we try a variety of different
+      # strategies that will have different results depending on where this
+      # test and running. We're mostly making sure that no exception is thrown.
+      _ = Stripe.get_uname
+    end
+  end
+
+  context "#get_uname_from_system" do
+    should "run without failure" do
+      # as above, just verify that an exception is not thrown
+      _ = Stripe.get_uname_from_system
+    end
+  end
+
+  context "#get_uname_from_system_ver" do
+    should "run without failure" do
+      # as above, just verify that an exception is not thrown
+      _ = Stripe.get_uname_from_system_ver
+    end
+  end
 end