Skip to content

Commit

Permalink
Fix some parameters being sent in tests
Browse files Browse the repository at this point in the history
Fixes one parameter being sent so that the suite is compliant with a
version of stripe-mock that's checking for extra parameters.

See stripe/stripe-ruby#650 as another sample
with more context.
  • Loading branch information
brandur committed May 17, 2018
1 parent 770cfc6 commit dca4436
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 10 deletions.
22 changes: 13 additions & 9 deletions tests/api_resources/test_account.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,22 +52,26 @@ def test_is_saveable_with_additional_owners(self):
# stripe-mock does not return additional owners so we construct
account = stripe.Account.construct_from({
'id': '%s' % TEST_RESOURCE_ID,
'additional_owners': [{
'first_name': 'name',
'verification': {},
}]
'legal_entity': {
'additional_owners': [{
'first_name': 'name',
'verification': {},
}],
}
}, stripe.api_key)
owner = account.additional_owners[0]
owner = account.legal_entity.additional_owners[0]
owner.verification.document = 'file_foo'
resource = account.save()
self.assert_requested(
'post',
'/v1/accounts/%s' % TEST_RESOURCE_ID,
{
'additional_owners': {
'0': {
'verification': {
'document': 'file_foo',
'legal_entity': {
'additional_owners': {
'0': {
'verification': {
'document': 'file_foo',
},
},
},
},
Expand Down
1 change: 0 additions & 1 deletion tests/api_resources/test_subscription.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ def test_is_retrievable(self):
def test_is_creatable(self):
resource = stripe.Subscription.create(
customer='cus_123',
plan='plan'
)
self.assert_requested(
'post',
Expand Down

0 comments on commit dca4436

Please sign in to comment.